homepage

When specifying an RPATH with -rpath or -R you can use the special
tokens `$LIB' and `$ORIGIN' which the runtime linker interprets as
"normal search path" and "relative to current sofile" respectively. To
get these correctly to the gcc command line you need to specify this in
LDFLAGS as `\$$LIB' to work around escapes of both the makefile and
shell, so in the Python Makefile this will appear somewhere as (this is
on one line):
CONFIG_ARGS=	 '--prefix=/opt/example.com/python25'
'LDFLAGS=-Wl,-rpath=\$$LIB:/opt/example.com/lib,--enable-new-dtags'
This works for compiling the main python binary. But when the extension
modules get compiled distutils chokes on this. 
distutils.sysconfig.parse_makefile() does think that any value of a
variable that contains `$' in it refers to an other variable in the
makefile. It will fail to find the value and CONFIG_ARGS will not be
defined. This then fails in setup.py for the _ctypes extension:
if not '--with-system-ffi' in sysconfig.get_config_var("CONFIG_ARGS"):
 return
Where `None' is returned instead of a list by .get_config_var().
It seems that distutils.sysconfig.parse_makefile() needs to understand
more of the makefile synatax to deal with this.

The attached patch does fix this issue.
Concerning the specific example of LDFLAGS used here there is still and
issue with LDFLAGS being ignored by the buid for the shared modules, but
that is an other issue.

Hmm, the patch isn't quite right yet. When a $$ is present in the
makefile .parse_makefile() needs to return a single $. I'm not sure yet
what needs to happen with the \ for the shell escape.

The updated patch inserts the single $ when needed. I've checked this
on compiling python, stdlib extension modules and custom extension
modules and this gives the correct results in all cases.

The patch looks good, beside the fact that you didn't call super() in
setUp/tearDown.
I'll commit it asap and fix that small issue

Oh, sorry about the super() that is why the ar test failed then. Sorry,
I got a little confused by the conflicting update on that file while
working on this patch and must have merged it badly.

Hi
What's the status of this? I haven't seen a commit message regarding this.
Cheers

done in r73341 in trunk. I've backported it as well in 2.6/3.0/3.1
branches. (r73342, r73343, r73344)
Thanks !