dCache / oncrpc4j

Pure Java implementation of ONCRPC/SUNRPC
Other
30 stars 30 forks source link

License question #80

Closed psilly closed 5 years ago

psilly commented 5 years ago

Hi, I see that oncrpc4j says it is subject to the LGPL license, but it has a dependency on org.dcache.common:dcache-auth which contains source (such as DesiredRole.java) that is under AGPL. I believe that means oncrpc4j must also be licensed under AGPL.

Is there a way to break this dependency, so oncrpc4j can be cleanly under LGPL?

kofemann commented 5 years ago

Hi @psilly, Thanks for bringing this up! You are right, looks like we enforce mix of licenses. Though it's only a small subset of dCache code, but this have to be cleaned up anyway.

elastifile-danis commented 5 years ago

Hi @kofemann Are you going to push it to master at the near future?

kofemann commented 5 years ago

actually, I waiting for @psilly to update his PR...

psilly commented 5 years ago

Sorry, got busy.. updated the PR.

On Thu, Jun 6, 2019 at 2:03 AM Tiramisu Mokka notifications@github.com wrote:

actually, I waiting for @psilly https://github.com/psilly to update his PR...

— You are receiving this because you were mentioned. Reply to this email directly, view it on GitHub https://github.com/dCache/oncrpc4j/issues/80?email_source=notifications&email_token=ABNMJNO4472AKHFLRNSXKGDPZDAEVA5CNFSM4G4R3BD2YY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGODXCB4JQ#issuecomment-499392038, or mute the thread https://github.com/notifications/unsubscribe-auth/ABNMJNIEGTSSTVHYBVVDPI3PZDAEVANCNFSM4G4R3BDQ .

JimSermersheim commented 5 years ago

fixed keyword ordering

kofemann commented 5 years ago

fixed with 9f8fde0

elastifile-danis commented 5 years ago

@kofemann when are you planning to create a release that includes this fix?

kofemann commented 5 years ago

@elastifile-danis the -SNAPSHOT should be already available. I have to check that integration with other components of you system works as well before an official release. Does it work for you?

elastifile-danis commented 5 years ago

@kofemann Yep it will do. Thanks alot.

kofemann commented 5 years ago

@elastifile-danis The 3.1.0 is released. Thanks!