[issue12242] distutils2 environment marker for current compiler

Changes by Éric Araujo : -- nosy: +ncoghlan resolution: -> out of date stage: -> committed/rejected status: open -> closed ___ Python tracker ___ __

[issue12242] distutils2 environment marker for current compiler

Eli Collins added the comment: Attached is a sixth revision (4e67e7205aba.diff) of my patch. This revision makes a couple of minor changes requested by Éric Araujo. -- ___ Python tracker _

[issue12242] distutils2 environment marker for current compiler

Changes by Eli Collins : Added file: http://bugs.python.org/file23740/4e67e7205aba.diff ___ Python tracker ___ ___ Python-bugs-list mailing li

[issue12242] distutils2 environment marker for current compiler

Eli Collins added the comment: Attached is a fifth revision (963a2cce068d.diff) of my patch. I was trying to avoid submitting any more revisions if possible, since this I think this patch is essentially feature-complete. However, this revision adds two things which I felt were needed: * This

[issue12242] distutils2 environment marker for current compiler

Changes by Eli Collins : Added file: http://bugs.python.org/file23274/963a2cce068d.diff ___ Python tracker ___ ___ Python-bugs-list mailing li

[issue12242] distutils2 environment marker for current compiler

Eli Collins added the comment: > What is result is i use GNU compiler by example with name > arm-linux-androideabi-gcc ? If the executable is named "arm-linux-androideabi-gcc", the code would currently call "arm-linux-androideabi-gcc --version". If that resulted in (for example) "arm-linux-

[issue12242] distutils2 environment marker for current compiler

Eli Collins added the comment: Attached is a fourth revision (ca53ff77ce6f.diff). This covers the typos identified in the last review, but I'm submitting a new patch because I ran a quick proofreading regexp, and identified a few more formatting mistakes. --

[issue12242] distutils2 environment marker for current compiler

Changes by Eli Collins : Added file: http://bugs.python.org/file23263/ca53ff77ce6f.diff ___ Python tracker ___ ___ Python-bugs-list mailing li

[issue12242] distutils2 environment marker for current compiler

Roumen Petrov added the comment: What is result is i use GNU compiler by example with name arm-linux-androideabi-gcc ? -- nosy: +rpetrov ___ Python tracker ___ ___

[issue12242] distutils2 environment marker for current compiler

Eli Collins added the comment: Attached is a diff (ba08e4a70631.diff) containing a third revision of my patch. This hopefully addresses all the issues brought up in the code review of the second revision, and changes little else. -- ___ Python trac

[issue12242] distutils2 environment marker for current compiler

Changes by Eli Collins : Added file: http://bugs.python.org/file23255/ba08e4a70631.diff ___ Python tracker ___ ___ Python-bugs-list mailing li

[issue12242] distutils2 environment marker for current compiler

Eli Collins added the comment: Attached is a diff (04610583238f.diff) containing a second revision of my implementation of this feature. It incorporates suggestions made during code review, as well as adds unittests and documentation of the new features. This patch retains the same basic beha

[issue12242] distutils2 environment marker for current compiler

Changes by Eli Collins : Added file: http://bugs.python.org/file23248/04610583238f.diff ___ Python tracker ___ ___ Python-bugs-list mailing li

[issue12242] distutils2 environment marker for current compiler

Changes by Eli Collins : -- hgrepos: +71 ___ Python tracker ___ ___ Python-bugs-list mailing list Unsubscribe: http://mail.python.org

[issue12242] distutils2 environment marker for current compiler

Éric Araujo added the comment: Thanks! > This patch was made against the hg.python.org/cpython repo. This is good. See http://wiki.python.org/moin/Distutils/Contributing > I'm not sure where I should add documentation about this new behavior. At least in Doc/library/packaging.compiler.rst and

[issue12242] distutils2 environment marker for current compiler

Eli Collins added the comment: Attached is a patch that implements this enhancement along the lines of what was last discussed. The behavior introduced in the patch is as follows: * It adds a stub method named get_compiler_version() to CCompiler, and implementations of that method for the

[issue12242] distutils2 environment marker for current compiler

Éric Araujo added the comment: > One issue is that multiple compiler patterns may match, Right, we can’t say “first matches” win if it’s unordered, and we won’t have OrderedDict in all versions supported by distutils2. Make it a list of tuples. First match wins, so people will have to write

[issue12242] distutils2 environment marker for current compiler

Eli Collins added the comment: > The question was about the meaning of a new “compiler” environment > marker. Would it be set to the empty string in Jython? Not available? > Or do you think that there is no issue, since Jython would not try to > compiler C files? Ah, my bad. I _had_ misunder

[issue12242] distutils2 environment marker for current compiler

Éric Araujo added the comment: >> What would the value be for non-C Python implementations? > I'm not really sure how this idea could have any value for those > implementations, at least for the ones that can't make use of C > extensions at all The question was about the meaning of a new “comp

[issue12242] distutils2 environment marker for current compiler

Eli Collins added the comment: > What would the value be for non-C Python implementations? I'm not really sure how this idea could have any value for those implementations, at least for the ones that can't make use of C extensions at all (I think PyPy can; so I guess that would leave Jython &

[issue12242] distutils2 environment marker for current compiler

Éric Araujo added the comment: What would the value be for non-C Python implementations? If the need for compiler-specific options is very common, we could consider either improving the compiler system or implement this request; if it’s not common, letting people use hooks would be enough. -

[issue12242] distutils2 environment marker for current compiler

New submission from Eli Collins : It would be useful to have an environment marker which matched the compiler being used when compiling a C extension. Sometimes different compilers require different options (different lib names, different flags, etc); distutils1 setup.py files which have to d