Split CLI license out of LICENSE #548

Closed
opened 2021-08-04 14:25:57 -05:00 by tamird · 6 comments
tamird commented 2021-08-04 14:25:57 -05:00 (Migrated from github.com)

Hello. The grpc project currently uses a subset of this repository that doesn't include the CLI (https://github.com/grpc/grpc/tree/master/third_party/xxhash). Would it be possible to move remove the CLI's license from the main LICENSE file so that the GPL text does not appear in the grpc repo? That would make it easier for Fuchsia to import grpc.

Thank you!

Hello. The grpc project currently uses a subset of this repository that *doesn't* include the CLI (https://github.com/grpc/grpc/tree/master/third_party/xxhash). Would it be possible to move remove the CLI's license from the main LICENSE file so that the GPL text does not appear in the grpc repo? That would make it easier for Fuchsia to import grpc. Thank you!
Cyan4973 commented 2021-08-04 14:30:20 -05:00 (Migrated from github.com)

This request seems in direct conflict with this prior request.

Would love to find a solution which is fine for everybody.

This request seems in direct conflict with [this prior request](https://github.com/Cyan4973/xxHash/issues/253). Would love to find a solution which is fine for everybody.
tamird commented 2021-08-04 14:32:02 -05:00 (Migrated from github.com)

That makes sense. Looking at the prior request, it's talking about the contrary license in xxhash.c. As far as I can tell, that file is only needed by the CLI. Perhaps it can move into the CLI directory?

That makes sense. Looking at the prior request, it's talking about the contrary license in xxhash.c. As far as I can tell, that file is only needed by the CLI. Perhaps it can move into the CLI directory?
tamird commented 2021-08-04 14:33:42 -05:00 (Migrated from github.com)

Actually, the relevant file shuffling was already done in https://github.com/Cyan4973/xxHash/pull/542. Perhaps these requests are no longer in conflict?

Actually, the relevant file shuffling was already done in https://github.com/Cyan4973/xxHash/pull/542. Perhaps these requests are no longer in conflict?
tamird commented 2021-08-04 19:02:04 -05:00 (Migrated from github.com)

Related: https://github.com/grpc/grpc/pull/26879. Kind of a bummer that there isn't a better way to import just the library part of xxhash.

Related: https://github.com/grpc/grpc/pull/26879. Kind of a bummer that there isn't a better way to import just the library part of xxhash.
tamird commented 2021-08-05 20:09:13 -05:00 (Migrated from github.com)

Friendly ping! Any thoughts on this? I'd like to upgrade Fuchsia's grpc dependency and it'd be nice to sidestep potential licensing issues by separating the licenses.

Friendly ping! Any thoughts on this? I'd like to upgrade Fuchsia's grpc dependency and it'd be nice to sidestep potential licensing issues by separating the licenses.
Cyan4973 commented 2021-08-06 17:01:38 -05:00 (Migrated from github.com)

Sorry, I'm not yet clear about what's the best way forward on this topic.

Sorry, I'm not yet clear about what's the best way forward on this topic.
Sign in to join this conversation.
No milestone
No assignees
1 participant
Notifications
Due date
The due date is invalid or out of range. Please use the format "yyyy-mm-dd".

No due date set.

Dependencies

No dependencies set

Reference
gh-Cyan4973/xxHash#548
No description provided.