On Sat, Jan 30, 2016 at 12:58 AM, Nick Coghlan <ncogh...@gmail.com> wrote: > I also think this version covers everything we need it to cover, so > I'm going to mark it as Active and point to this post as the > resolution :)
Awesome! Thanks everyone :-) Of course, as soon as this happened I discovered two small but substantive bugs in the text... - the text refers to 32-bit linux as "i386" (because I foolishly trusted PEP 425); it turns out that on any modern system the platform tag is "linux_i686". - there was a small bug in the is_manylinux1_compatible sample code, because I missed that distutils.util.get_platform() uses a hyphen separator instead of an underscore. (But now the sample code is running as part of the test suite for the docker image, so I know it works.) Both are fixed in the diff that I just sent to the pep editors: https://github.com/manylinux/manylinux/pull/20/files -n -- Nathaniel J. Smith -- https://vorpus.org _______________________________________________ Distutils-SIG maillist - Distutils-SIG@python.org https://mail.python.org/mailman/listinfo/distutils-sig
