[issue18257] Two copies of python-config

[R. David Murray]

R. David Murray added the comment:

Since it hasn't been an issue for a few releases, I say we close it.  If there 
is some problem remaining, it probably deserves its own tracker issue anyway.

--
nosy: +r.david.murray
resolution:  -> fixed
stage: needs patch -> resolved
status: open -> closed

___
Python tracker 

___
___
Python-bugs-list mailing list
Unsubscribe: 
https://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com

[issue18257] Two copies of python-config

[Łukasz Langa]

Łukasz Langa added the comment:

Downgrading the priority, it's not a deferred blocker for sure. I think we 
could probably just close it?

--
nosy: +lukasz.langa
priority: deferred blocker -> 

___
Python tracker 

___
___
Python-bugs-list mailing list
Unsubscribe: 
https://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com

[issue18257] Two copies of python-config

[Larry Hastings]

Larry Hastings added the comment:

Can I mark this closed?  I'm tagging 3.4.0 final soon.

--

___
Python tracker rep...@bugs.python.org
http://bugs.python.org/issue18257
___
___
Python-bugs-list mailing list
Unsubscribe: 
https://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com

[issue18257] Two copies of python-config

[Larry Hastings]

Larry Hastings added the comment:

Okay, downgrading to deferred blocker.

--
priority: release blocker - deferred blocker

___
Python tracker rep...@bugs.python.org
http://bugs.python.org/issue18257
___
___
Python-bugs-list mailing list
Unsubscribe: 
http://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com

[issue18257] Two copies of python-config

[Roundup Robot]

Roundup Robot added the comment:

New changeset 7b165c7ab7ef by doko in branch 'default':
- Issue #18257: Fix readlink usage in python-config.  Install the python
http://hg.python.org/cpython/rev/7b165c7ab7ef

--
nosy: +python-dev

___
Python tracker rep...@bugs.python.org
http://bugs.python.org/issue18257
___
___
Python-bugs-list mailing list
Unsubscribe: 
http://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com

[issue18257] Two copies of python-config

[Matthias Klose]

Matthias Klose added the comment:

checked in a fix for the readlink issue, and Ronald's proposed solution for 
Darwin.  And documented for now, that we do have two versions of this script.  
This works as long as you don't cross-build for Darwin or on Darwin.

--

___
Python tracker rep...@bugs.python.org
http://bugs.python.org/issue18257
___
___
Python-bugs-list mailing list
Unsubscribe: 
http://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com

[issue18257] Two copies of python-config

[Larry Hastings]

Larry Hastings added the comment:

Thanks, Matthias!  Now I can cut 3.4a1 on time.  :)

--

___
Python tracker rep...@bugs.python.org
http://bugs.python.org/issue18257
___
___
Python-bugs-list mailing list
Unsubscribe: 
http://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com

[issue18257] Two copies of python-config

[Larry Hastings]

Larry Hastings added the comment:

Is there any resolution for this likely to happen soon?  I'm hoping to cut 
Python 3.4a1 in about two days, so one of three things is gonna happen:

1) This gets fixed.
2) This gets lowered from release blocker.
3) The release slips.

--

___
Python tracker rep...@bugs.python.org
http://bugs.python.org/issue18257
___
___
Python-bugs-list mailing list
Unsubscribe: 
http://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com

[issue18257] Two copies of python-config

[Ronald Oussoren]

Ronald Oussoren added the comment:

I won't be able to work on this before 3.4a1 is released, and that shouldn't be 
a problem.

I do think this should be fixed before the first beta though.

(That said, I'd rather not work on this at all, I didn't introduce this problem)

--

___
Python tracker rep...@bugs.python.org
http://bugs.python.org/issue18257
___
___
Python-bugs-list mailing list
Unsubscribe: 
http://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com

[issue18257] Two copies of python-config

[Ronald Oussoren]

Ronald Oussoren added the comment:

This is IMHO a release blocker: the the shell-script version of python-config 
doesn't work on a major platform (OSX), and (older) commercial unix systems.

The easiest workaround is to remove the shell script and keep using the python 
script.

--
nosy: +larry
priority: normal - release blocker

___
Python tracker rep...@bugs.python.org
http://bugs.python.org/issue18257
___
___
Python-bugs-list mailing list
Unsubscribe: 
http://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com

[issue18257] Two copies of python-config

[Larry Hastings]

Larry Hastings added the comment:

How about if the shell script detected that it was running on OS X and exited 
with an error instructing the user to use the Python script instead?

I don't agree that this is a release blocker.  Most people on OS X use the 
prebuilt binaries.

--

___
Python tracker rep...@bugs.python.org
http://bugs.python.org/issue18257
___
___
Python-bugs-list mailing list
Unsubscribe: 
http://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com

[issue18257] Two copies of python-config

[Matthias Klose]

Matthias Klose added the comment:

Proposing to remove the shell script as the first step looks wrong.  Do you 
know about a substitute for readlink on an enterpricy unix flavor like 
MacOSX?  Using ls -l and interpreting the last argument of the output comes to 
mind.

--

___
Python tracker rep...@bugs.python.org
http://bugs.python.org/issue18257
___
___
Python-bugs-list mailing list
Unsubscribe: 
http://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com

[issue18257] Two copies of python-config

[Ronald Oussoren]

Ronald Oussoren added the comment:

Why use a shell script in the first place? The shell script doesn't do 
everything the python script does (an example of this is that on OSX distutils 
and sysconfig can tweak the CFLAGS and LDFLAGS as needed based on the curent 
OSX version and the installed compiler, which is needed to be able to build 
extensions out of the box on all supported OSX releases) and doesn't work 
everywhere because it uses non-standard shell commands.

I'll provide a patch that ensures that the python version gets used on OSX, 
that will reduce the number of issues we get about this. 

Larry: python-config is shipped in the binary installers, and as it currently 
is doesn't work.  That's a problem because some software uses python-config to 
get the compiler options needed to use this python (for embedding, building 
extensions using a makefile, ...).  

I don't use python-config myself, but do try to keep the OSX version as closely 
as possible a normal unix install (another example of this: in a framework 
install, such as the binary installers, we ship a libpython.a at the expected 
location that is a symlink to the real *shared* library. That symlink isn't 
need for Python itself, but was added because some popular Linux software 
(can't remember which package) assumed that the library would be there).

Anyway, I'll push a patch for this when I'm home.  I'm slowly closing down my 
involvement with the EP sprints and will then head to the airport.

--

___
Python tracker rep...@bugs.python.org
http://bugs.python.org/issue18257
___
___
Python-bugs-list mailing list
Unsubscribe: 
http://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com

[issue18257] Two copies of python-config

[Arfrever Frehtes Taifersar Arahesis]

Changes by Arfrever Frehtes Taifersar Arahesis arfrever@gmail.com:


--
nosy: +Arfrever

___
Python tracker rep...@bugs.python.org
http://bugs.python.org/issue18257
___
___
Python-bugs-list mailing list
Unsubscribe: 
http://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com

[issue18257] Two copies of python-config

[Matthias Klose]

Matthias Klose added the comment:

 Why use a shell script in the first place?

you can't run python for a cross build, unless you tweak the interpreter.  
you can do that with a shell script only.  As suggested in the original 
issue/patch, I did propose to remove the python implementation, so a fix for 
the shell script seems to be the better approach.  Tweaking the CFLAGS in the 
shell script should be possible too.  Maybe you could describe what exactly 
needs tweaking, as I cannot find this in the python implementation either.

--

___
Python tracker rep...@bugs.python.org
http://bugs.python.org/issue18257
___
___
Python-bugs-list mailing list
Unsubscribe: 
http://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com

[issue18257] Two copies of python-config

[Matthias Klose]

Matthias Klose added the comment:

 Furthermore the entire readlink command is not present in HP-UX

The use of readlink is guarded, the use of readlink -f apparently not.

--

___
Python tracker rep...@bugs.python.org
http://bugs.python.org/issue18257
___
___
Python-bugs-list mailing list
Unsubscribe: 
http://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com

[issue18257] Two copies of python-config

[Ronald Oussoren]

Ronald Oussoren added the comment:

This should do the trick, the shell script version is replaced by the python 
version when building on OSX.

And yes, the use of readlink is guarded but that doesn't help because you then 
use a command-line flag that isn't supported on OSX:

$ readlink -h .
readlink: illegal option -- h
usage: readlink [-n] [file ...]

--
Added file: http://bugs.python.org/file30842/issue-18257.txt

___
Python tracker rep...@bugs.python.org
http://bugs.python.org/issue18257
___
___
Python-bugs-list mailing list
Unsubscribe: 
http://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com

[issue18257] Two copies of python-config

[Matthias Klose]

Matthias Klose added the comment:

the proposed patch won't work once the python implementation is removed.  Is -f 
supported on MacOSX? There seems to be a typo on your side trying -h, which 
isn't supported on Linux either.

--

___
Python tracker rep...@bugs.python.org
http://bugs.python.org/issue18257
___
___
Python-bugs-list mailing list
Unsubscribe: 
http://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com

[issue18257] Two copies of python-config

[Ronald Oussoren]

Ronald Oussoren added the comment:

Sorry about the confusion, -f isn't supported either that's why I noticed 
there is a problem.

$ readlink -f .
readlink: illegal option -- f
usage: readlink [-n] [file ...]

Again, why is does does have to be a shell script anyway? 

I really don't like the propect of having to reimplement the logic in 
_osx_support in shell code...  I've written loads of large, portable shell 
scripts in the past (when portable meant supporting a dozen or so different 
unix flavors), and that's not really an experience worth repeating :-/

--

___
Python tracker rep...@bugs.python.org
http://bugs.python.org/issue18257
___
___
Python-bugs-list mailing list
Unsubscribe: 
http://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com

[issue18257] Two copies of python-config

[Matthias Klose]

Matthias Klose added the comment:

 Again, why is does does have to be a shell script anyway? 

please see above. I explained it.

--

___
Python tracker rep...@bugs.python.org
http://bugs.python.org/issue18257
___
___
Python-bugs-list mailing list
Unsubscribe: 
http://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com

[issue18257] Two copies of python-config

[Matthias Klose]

Matthias Klose added the comment:

and see issue16235 (as mentioned in NEWS) for the complete discussion.

--

___
Python tracker rep...@bugs.python.org
http://bugs.python.org/issue18257
___
___
Python-bugs-list mailing list
Unsubscribe: 
http://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com

[issue18257] Two copies of python-config

[Ronald Oussoren]

Ronald Oussoren added the comment:

Matthias: sorry, I completely missed the message where you explained why the 
script is now a shell script. 

The customization is performed by _osx_xsupport.customize_config_vars.

What this is used for:

* We ship binaries build with a specific compiler configuration
  (IIRC Xcode 3 on OSX 10.5 for the 32-bit installers)

* That configuration does not work on all supported platforms:

  - OSX 10.3 (which is more less supported on a we'll try to fix
if you scream basis) does not support -arch flags or
-isysroot at all.
 
  - On OSX 10.4 you must use an SDK to build fat binaries (such as 
for the 32-bit binary installers).

  - Xcode 4, which must be used on more recent systems (at least 10.7
and 10.7, possibly 10.6 as well), is different. A particular worry
is that the gcc command for Xcode 4 is a gcc frontend for the 
LLVM system and that miscompiles parts of Python 3.3. Therefore we
use clang instead of gcc when detecting that compiler. In future
versions of Xcode there will likely not be a gcc compiler at all.

  - We cannot use the same SDK on all platforms, because Xcode ships
with a limited number of SDK (basicly just the SDK for the current
and previous releases). We therefore automaticly select the most
appropriate SDK at runtime.

(Different SDKs isn't as problematic as it sounds, you can build
with the OSX 10.8 SDK and deploy to OSX 10.4; except for PPC you
must use Xcode 3 for PPC support because Xcode 4 does not contain
a compiler that does that).

  - OSX 10.9 (currently in beta) also contains changes that affect 
compiler support. I won't comment on those changes right now because
I haven't fully investigated them yet, and because 10.9 is 
in a closed SDK.

I may have missed some details because I haven't looked at the code while
writing this, but this should at least explain why I was upset with a change to 
a shell script.  

The customization can be written as shell scripts, but it is more code to 
maintain in an inelegant scripting system.

--

___
Python tracker rep...@bugs.python.org
http://bugs.python.org/issue18257
___
___
Python-bugs-list mailing list
Unsubscribe: 
http://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com

[issue18257] Two copies of python-config

[Ronald Oussoren]

New submission from Ronald Oussoren:

Changeset c0370730b364 introduced a shell-script implementation of 
python-config (see issue #16235). The older python implementation is still 
present in $(srcdir)/Misc and generated by Makefile.pre.in.

--
messages: 191416
nosy: doko, ronaldoussoren
priority: normal
severity: normal
stage: needs patch
status: open
title: Two copies of python-config
type: behavior
versions: Python 3.4

___
Python tracker rep...@bugs.python.org
http://bugs.python.org/issue18257
___
___
Python-bugs-list mailing list
Unsubscribe: 
http://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com

[issue18257] Two copies of python-config

[Ronald Oussoren]

Ronald Oussoren added the comment:

Also, the shell-script version of python-config uses the -f option of readlink 
and that option is not supported on OSX 10.8 (or older versions, I haven't 
looked at the 10.9 beta yet)

Furthermore the entire readlink command is not present in HP-UX (and possibly 
other enterpricy unix flavors)

--
components: +Demos and Tools

___
Python tracker rep...@bugs.python.org
http://bugs.python.org/issue18257
___
___
Python-bugs-list mailing list
Unsubscribe: 
http://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com