search

Azure / azure-storage-python

Microsoft Azure Storage Library for Python
https://azure-storage.readthedocs.io
MIT License
339 stars 241 forks source link

Add LICENSE(.txt) to wheels #477

Closed xhochy closed 5 years ago

xhochy commented 6 years ago

Currently the LICENSE is only part of the repository but not contained in the released Python packages. It would be very helpful to ship it in the wheel and sdist tarballs.

xhochy commented 6 years ago

As an example, conda-forge packages that are built from the sdist currently need to sideload the license file: https://github.com/conda-forge/staged-recipes/pull/5433

zezha-msft commented 6 years ago

Hi @xhochy, thanks for reaching out!

I'll let our packaging expert @lmazuel comment on how should we proceed.

lmazuel commented 6 years ago

Wheel is not supposed to have the LICENCE file, as long as each file contains the licence in the header (which is the case). sdist could indeed, and that doesn't hurt. @zezha-msft copy the LICENCE.txt at the root of the repo into each package folder, and update the MANIFEST.in file of each package with include LICENCE.txt

zezha-msft commented 6 years ago

@lmazuel sounds good! I'll do this work as part of the next release.

zezha-msft commented 5 years ago

Fixed in 1.4.0.