Re: [Distutils] setup.py should use if __name__ == '__main__', right?

2014-04-07 Thread Carl Meyer
On 04/06/2014 09:04 PM, Asheesh Laroia wrote: > Hi nice distutils/PyPA people, > > I had a question that probably is worth showing up in the archives. Namely: > > It seems to me like bizarre bad form for the setup.py file to execute > what amounts to a main() function at import time. > > I presu

Re: [Distutils] What is the syntax for passing conditional non-extra dependencies in setuptools?

2014-04-07 Thread Daniel Holth
I think setuptools' :-separated environment markers mechanism is great, and shouldn't need to be ratified anywhere. I also think the actual environment markers specification is stable, although I noticed the IPython wheel for example is using the deprecated .-separated rather than _-separated ident

Re: [Distutils] What is the syntax for passing conditional non-extra dependencies in setuptools?

2014-04-07 Thread Daniel Holth
OK, it does in fact work that way; empty-string-before-colon specifies a default requirement with a marker. Parses out to the following _dep_map with 'None' representing 'not an extra dependency': {None: [Requirement.parse('coding'), Requirement.parse('stuff')], 'quux': [Requirement.parse('more-

Re: [Distutils] What is the syntax for passing conditional non-extra dependencies in setuptools?

2014-04-07 Thread PJ Eby
On Mon, Apr 7, 2014 at 4:03 PM, Daniel Holth wrote: > OK, it does in fact work that way; empty-string-before-colon specifies > a default requirement with a marker. > > Parses out to the following _dep_map with 'None' representing 'not an > extra dependency': > > {None: [Requirement.parse('coding'

[Distutils] Comments on PEP 426 and 459

2014-04-07 Thread Daniel Holth
I read through the latest versions of PEP 426 and 459 "Metadata 2.0 and extensions". Here are my comments. The PEP suggests "setup.py dist_info" is a thing. Only "setup.py egg_info" works. It might make sense to refactor bdist_wheel to include a dist_info command, but if done badly it would break

Re: [Distutils] install_requires vs. requires_dist, and setup.cfg

2014-04-07 Thread Nick Coghlan
On 7 Apr 2014 00:59, "Marcus Smith" wrote: > > >> From what I understand in PEP 345, "Requires-Dist" is the name in PKG-INFO files for a distribution's dependencies on other distributions. > > > setuptools/distutils has never implemented "Requires-Dist" from PEP345 for PKG-INFO. > Otoh, wheel has,

Re: [Distutils] What is the syntax for passing conditional non-extra dependencies in setuptools?

2014-04-07 Thread Nick Coghlan
On 7 Apr 2014 15:09, "Daniel Holth" wrote: > > I think setuptools' :-separated environment markers mechanism is > great, and shouldn't need to be ratified anywhere. I also think the > actual environment markers specification is stable, although I noticed > the IPython wheel for example is using th

Re: [Distutils] Comments on PEP 426 and 459

2014-04-07 Thread Nick Coghlan
On 7 Apr 2014 23:15, "Daniel Holth" wrote: > > I read through the latest versions of PEP 426 and 459 "Metadata 2.0 > and extensions". Here are my comments. > > The PEP suggests "setup.py dist_info" is a thing. Only "setup.py > egg_info" works. It might make sense to refactor bdist_wheel to > inclu