1 |
On 10/11/2012 8:50 AM, Mike Frysinger wrote: |
2 |
> On Thursday 11 October 2012 05:35:21 Gregory M. Turner wrote: |
3 |
>> On 10/10/2012 9:14 PM, Mike Frysinger wrote: |
4 |
> it's not particularly important, but on one hand, the LDFLAGS parsing logic |
5 |
> should not be in the tree ever. |
6 |
|
7 |
I've no major attachment to it. Took all of five minutes to code up |
8 |
reading the ld manpage, and as you pointed out, it's probably solving a |
9 |
non-problem. |
10 |
|
11 |
> on the other, i can stomach much smaller one- |
12 |
> off hacks like prepending -L. to LDFLAGS if the maintainers of the python |
13 |
> ebuild really really want to add it. |
14 |
|
15 |
If fixing the python builds proves too onerous then this is what I'll |
16 |
end up filing a bug for. |
17 |
|
18 |
> my [limited] understanding of the prefix compiler is that they wrap things with |
19 |
> a custom shell script to inject -L flags behind the back of the compiler at the |
20 |
> very last possible minute and point to the right place. |
21 |
|
22 |
Pretty fuzzy on this myself. Whatever binutils-config does works so |
23 |
well I've never had to look into it very deeply :) |
24 |
|
25 |
Above, however, I'm referring to the prefix-specific cpython patch-sets, |
26 |
where I was "sure" I'd seen patches to project LDFLAGS into setup.py's |
27 |
compiler invocation tables. But it seems I was mistaken. Perhaps it |
28 |
was in the main gentoo-patches after all... or maybe I just need more |
29 |
coffee... |
30 |
|
31 |
Anyhow one thing I have figured out is how things can work correctly on |
32 |
Linux wihtout -L.: on Linux, the python plugins aren't actually linked |
33 |
against libpython.so! |
34 |
|
35 |
With any luck, this explains what's going on and suggests that platforms |
36 |
linking python modules against libpython.so just need to wedge -L${S} |
37 |
somewhere in configure.in, possibly do-able via sed script (unless we |
38 |
end up having to mess with setup.py, in which case, things may get a bit |
39 |
more complicated). |
40 |
|
41 |
-gmt |