Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

misc: api-description.yml license aligned with delta-sharing #367

Open
wants to merge 1 commit into
base: main
Choose a base branch
from

Conversation

cdaudt
Copy link

@cdaudt cdaudt commented Aug 3, 2023

In discussion with original author of
delta-sharing-protocl-api-description.yml file (R. Tyler Croy), listing this file as AGPL 3.0 was a clerical error when it was brought over from its original project.
This modification fixes the clerical error by removing the license information from the file, thus aligning it with the license of the project. This is being done at the request of the author.

Here is the exchange with @rtyler that prompted this mod: "
That's a good catch @christian Daudt , I originally authored that file as part of riverbank, a rust delta-sharing server which I had originally licensed as AGPL before it entered the delta-incubator.
When I updated to the Apache license I forgot about that little note, that's a clerical error and can be cleaned up.
"

In discussion with original author of
delta-sharing-protocl-api-description.yml file (R. Tyler Croy),
listing this file as AGPL 3.0 was a clerical error when it
was brought over from its original project.
This modification fixes the clerical error by removing the
license information from the file, thus aligning it with the
license of the project. This is being done at the request
of the author.

Here is the exchange with @rtyler that prompted this mod:
"
That's a good catch @christian Daudt , I originally authored that
file as part of riverbank, a rust delta-sharing server which I
had originally licensed as AGPL before it entered the
delta-incubator.
When I updated to the Apache license I forgot about that little note,
that's a clerical error and can be cleaned up.
"

Signed-off-by: Christian Daudt <[email protected]>
Copy link
Member

@rtyler rtyler left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This was part of my original riverbank code and was not properly re-licenses when I moved the project to ASL 2.0 as part of the introduction to delta-incubator

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants