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 ->
Ł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
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
___
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
___
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
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.
--
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
___
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.
--
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)
--
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.
--
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.
--
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.
--
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
Changes by Arfrever Frehtes Taifersar Arahesis arfrever@gmail.com:
--
nosy: +Arfrever
___
Python tracker rep...@bugs.python.org
http://bugs.python.org/issue18257
___
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
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
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 .
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
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
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
___
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
___
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
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
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
24 matches
Mail list logo