<div dir="ltr"><div dir="ltr"><br></div><br><div class="gmail_quote"><div dir="ltr" class="gmail_attr">On Thu, Dec 10, 2020 at 6:42 AM Charles R Harris <<a href="mailto:charlesr.harris@gmail.com">charlesr.harris@gmail.com</a>> wrote:<br></div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex"><div dir="ltr"><div dir="ltr"><br></div><br><div class="gmail_quote"><div dir="ltr" class="gmail_attr">On Wed, Dec 9, 2020 at 5:17 PM Charles R Harris <<a href="mailto:charlesr.harris@gmail.com" target="_blank">charlesr.harris@gmail.com</a>> wrote:<br></div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex"><div dir="ltr">Hi All,<br><div><br></div><div>Currently we append appropriate platform  licenses to the LICENSE.txt file when building wheels for release. This means that there are uncommitted changes which shows up in the versioneer version as 'dirty', see the nightly files. This is unfortunate, but accurate :) There are at least two possible solutions to this problem.</div><div><ol><li>Patch versioneer to omit the dirty string, very easy to do.</li><li>Put the platform specific file in the repo or combine them in the LICENSE file.</li></ol><div>I don't recall why we did things the way we do, but there was a discussion. Patching is easy, but the second option seems preferable. In particular, folks who now build their own NumPy wheels aren't going to have the license files.</div></div></div></blockquote></div></div></blockquote><div><br></div><div>The reason for that construct is that GitHub won't recognize the license if we add vendored info. As a result, it would not only not display the license in its UI, but also it provides an API to query the license for a repo which then gives the wrong result. That in turn throws off Tidelift, which uses two sources of licensing info in its service (GitHub and <a href="http://libraries.io">libraries.io</a>) and those should match.</div><div><br></div><div>Please consider this an issue with versioneer, and choose (1)</div><div><br></div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex"><div dir="ltr"><div class="gmail_quote"><div>Note that LICENSES_bundled.txt, excluded from the sdist in MANIFEST.in, is included in the wheel in the dist-info file.</div></div></div></blockquote><div><br></div><div>Ah, that needs fixing then.</div><div><br></div><div>Cheers,<br></div><div>Ralf</div><div><br></div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex"><div dir="ltr"><div class="gmail_quote"><div><br></div><div><font face="arial, sans-serif">charris@fc [numpy.git (master)]$ ls dist/numpy-1.21.0.dev0+135.g26f8b11b6e.dist-info<br>entry_points.txt  LICENSES_bundled.txt  LICENSE.txt  METADATA  RECORD  top_level.txt  WHEEL</font><br></div><div><br></div><div>Looks like any LICENSE* files in the root directory will be included in the wheel.</div><div><br></div><div>Chuck</div><div><br></div><div><br></div></div></div>
_______________________________________________<br>
NumPy-Discussion mailing list<br>
<a href="mailto:NumPy-Discussion@python.org" target="_blank">NumPy-Discussion@python.org</a><br>
<a href="https://mail.python.org/mailman/listinfo/numpy-discussion" rel="noreferrer" target="_blank">https://mail.python.org/mailman/listinfo/numpy-discussion</a><br>
</blockquote></div></div>