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

Released Ruby gem is missing LICENSE file. #292

Closed
jackorp opened this issue Mar 25, 2024 · 5 comments
Closed

Released Ruby gem is missing LICENSE file. #292

jackorp opened this issue Mar 25, 2024 · 5 comments
Assignees
Labels
🐛 bug Defect / Bug

Comments

@jackorp
Copy link
Contributor

jackorp commented Mar 25, 2024

I am working on updating a few packages in the Ruby cucumber stack in Fedora and I noticed the license file is missing from this library now, that wasn't the case with v14.

From what I understand the file should be present in the released gem as it's a copy of the software.

Not 100% sure how a proper fix should look like.

AFAICT, to build and then unpack/install the gem properly, the license has to be either on the same level with the .gemspec file in the Ruby gem or some level below. A file cannot be referenced a level up from gemspec, so doing something like s.files << "../LICENSE" is not going to work for unpacking the gem. Perhaps a Rakefile target that copies the LICENSE first and then builds the gem so that something like s.files << "./LICENSE" would then be OK.

@jackorp jackorp added the 🐛 bug Defect / Bug label Mar 25, 2024
@mpkorstanje
Copy link
Contributor

mpkorstanje commented Mar 25, 2024

Ah. That would have been my fault. I deduplicated the licenses in #195 and in other repos as well. We probably have this problem in quite a few projects then.

@jackorp
Copy link
Contributor Author

jackorp commented Jul 3, 2024

We probably have this problem in quite a few projects then.

That would be correct observation, Probably also accross languages with $LIBRARY_MANAGER (npm, rubygems, ...). https://github.com/cucumber/messages also has the same problem.

BTW the section of the MIT license: The above copyright notice and this permission notice shall be included in all copies or substantial portions of the Software. is an easy downstream fix of just pulling the file, but it is a fix I'd prefer not to do in the first place.

@mpkorstanje
Copy link
Contributor

@luke-hill anyway we can have a single licence file with Ruby?

@mpkorstanje
Copy link
Contributor

mpkorstanje commented Jul 16, 2024

Looks like fixing this in any other way but duplicating files is not going to work.

To fix:

V cucumber/gherkin-utils/LICENSE
X cucumber/gherkin-utils/java/LICENSE - not found
X cucumber/gherkin-utils/javascript/LICENSE - not found
X cucumber/query/LICENSE - differs
X cucumber/query/java/LICENSE - not found
X cucumber/query/javascript/LICENSE - not found
V cucumber/ci-environment/LICENSE
X cucumber/ci-environment/go/LICENSE - not found
X cucumber/ci-environment/java/LICENSE - not found
X cucumber/ci-environment/javascript/LICENSE - not found
X cucumber/ci-environment/python/LICENSE - not found
X cucumber/ci-environment/ruby/LICENSE - not found
V cucumber/tag-expressions/LICENSE
X cucumber/tag-expressions/go/LICENSE - not found
X cucumber/tag-expressions/java/LICENSE - not found
X cucumber/tag-expressions/javascript/LICENSE - not found
X cucumber/tag-expressions/perl/LICENSE - not found
X cucumber/tag-expressions/python/LICENSE - not found
X cucumber/tag-expressions/ruby/LICENSE - not found

#308

V cucumber/cucumber-expressions/LICENSE
X cucumber/cucumber-expressions/dotnet/LICENSE - not found
X cucumber/cucumber-expressions/go/LICENSE - not found
X cucumber/cucumber-expressions/java/LICENSE - not found
X cucumber/cucumber-expressions/javascript/LICENSE - not found
X cucumber/cucumber-expressions/python/LICENSE - not found
X cucumber/cucumber-expressions/ruby/LICENSE - not found

cucumber/gherkin#257

V cucumber/gherkin/LICENSE
X cucumber/gherkin/c/LICENSE - not found
X cucumber/gherkin/cpp/LICENSE - not found
X cucumber/gherkin/dart/LICENSE - not found
X cucumber/gherkin/dotnet/LICENSE - not found
X cucumber/gherkin/elixir/LICENSE - not found
X cucumber/gherkin/go/LICENSE - not found
X cucumber/gherkin/java/LICENSE - not found
X cucumber/gherkin/javascript/LICENSE - not found
X cucumber/gherkin/objective-c/LICENSE - not found
X cucumber/gherkin/perl/LICENSE - not found
X cucumber/gherkin/php/LICENSE - not found
X cucumber/gherkin/python/LICENSE - not found
X cucumber/gherkin/ruby/LICENSE - not found

cucumber/messages#238

V cucumber/messages/LICENSE
X cucumber/messages/cpp/LICENSE - not found
X cucumber/messages/dotnet/LICENSE - not found
X cucumber/messages/elixir/LICENSE - not found
X cucumber/messages/go/LICENSE - not found
X cucumber/messages/java/LICENSE - not found
X cucumber/messages/javascript/LICENSE - not found
X cucumber/messages/perl/LICENSE - not found
X cucumber/messages/php/LICENSE - not found
X cucumber/messages/ruby/LICENSE - not found
V cucumber/html-formatter/LICENSE
X cucumber/html-formatter/java/LICENSE - not found
X cucumber/html-formatter/javascript/LICENSE - not found
X cucumber/html-formatter/ruby/LICENSE - not found
V cucumber/compatibility-kit/LICENSE
X cucumber/compatibility-kit/javascript/LICENSE - not found
X cucumber/compatibility-kit/ruby/LICENSE - not found

@luke-hill
Copy link
Contributor

Fixed and files now present in v18 of gem

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
🐛 bug Defect / Bug
Projects
Status: Done
Development

No branches or pull requests

3 participants