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

flatbuffers 22.10.26 python distribution missing LICENSE.txt file #7628

Closed
zookae opened this issue Nov 8, 2022 · 2 comments · Fixed by #7692
Closed

flatbuffers 22.10.26 python distribution missing LICENSE.txt file #7628

zookae opened this issue Nov 8, 2022 · 2 comments · Fixed by #7692
Labels
need-more-info Need more info/context to make a decision

Comments

@zookae
Copy link

zookae commented Nov 8, 2022

What version of protobuf and what language are you using?
Version: 22.10.26 (applies to all versions we tested)
Language: Python

What operating system (Linux, Windows, ...) and version?
Windows 11. Linux.

What runtime / compiler are you using (e.g., python version or gcc version)
Pip install only.

What did you do?
Download the flatbuffers python package:

mkdir _tmp
pip install -t _tmp flatbuffers==22.10.26
find ./_tmp -name "*LIC*" -type f

Shows nothing.

What did you expect to see
The LICENSE.txt file included in the dist-info list.

What did you see instead?
No LICENSE.txt file.

Anything else we should know about your project / environment
We use a script to gather licenses for python packages we include when shipping our products. flatbuffers is one of the packages that does not include a license file for us to gather, requiring extra manual steps to collect. Ideally the solution would fix this for all flatbuffers versions going forward as we've encountered the same problem with many previous flatbuffers releases. Thanks!

@dbaileychess
Copy link
Collaborator

The license is specified in the publishing script: https://github.com/google/flatbuffers/blob/master/python/setup.py#L20 I'm not familiar with how python publishing works, so more information would be useful on how to fix it.

@dbaileychess dbaileychess added the need-more-info Need more info/context to make a decision label Nov 13, 2022
@zookae
Copy link
Author

zookae commented Nov 14, 2022

The license is specified in the publishing script: https://github.com/google/flatbuffers/blob/master/python/setup.py#L20 I'm not familiar with how python publishing works, so more information would be useful on how to fix it.

Happy to help! This stack overflow answer is pretty comprehensive on how to use setup.py to include the license file with a wheel: https://stackoverflow.com/a/66443941

Ideally the wheel would include the LICENSE.txt file (here). For our use case that helps us ensure we catch any changes to licenses over versions.

You should be able to verify the file was included by doing the download step indicated above.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
need-more-info Need more info/context to make a decision
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants