[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [PATCH] gnu: kde: Add kdelibs.
From: |
Ludovic Courtès |
Subject: |
Re: [PATCH] gnu: kde: Add kdelibs. |
Date: |
Sun, 04 Jan 2015 22:29:59 +0100 |
User-agent: |
Gnus/5.13 (Gnus v5.13) Emacs/24.4 (gnu/linux) |
宋文武 <address@hidden> skribis:
>> Andreas Enge <address@hidden> skribis:
>>
>>> RUNPATH
>>> /gnu/store/7c30kyzagk84inhnb2nbxcmnh76xgh6c-ktouch-4.14.2/lib:/gnu/store/4k20pkxgvfc22wpcvh6xr26ma4b619ad-glibc-2.20/lib:/gnu/store/k7w7wdpvaqlkwq0ijz9149pqn3lq8a5n-gcc-4.8.3-lib/lib64:/gnu/store/k7w7wdpvaqlkwq0ijz9149pqn3lq8a5n-gcc-4.8.3-lib/lib:/gnu/store/k7w7wdpvaqlkwq0ijz9149pqn3lq8a5n-gcc-4.8.3-lib/lib/gcc/x86_64-unknown-linux-gnu/4.8.3/../../..
>>>
>>> (Thanks to Ludovic for the suggestion to use objdump!)
>>>
>>> The linker line looks like this:
>>> Linking CXX executable ktouch
>>> cd /tmp/nix-build-ktouch-4.14.2.drv-0/build/src &&
>>> /gnu/store/sr9lv3vhviy151zv315zdgi2rvq97a9r-cmake-2.8.12/bin/cmake -E
>>> cmake_link_script CMakeFiles/ktouch.dir/link.txt --verbose=1
>>> /gnu/store/13ahaqlmniysmbls5d1fcmzy59yd5xk5-gcc-4.8.3/bin/c++
>>
>> [...]
>>
>>> -o ktouch -rdynamic
>>> /gnu/store/mipmjv4mpsf3rrsk1mbhpqzlw939v6ny-kdelibs-4.14.2/lib/libkdeclarative.so.5.14.2
>>>
>>> /gnu/store/mipmjv4mpsf3rrsk1mbhpqzlw939v6ny-kdelibs-4.14.2/lib/libplasma.so.3.0.0
>>>
>>
>> [...]
>>
>>> You see lots of "...so" and no "-l...", which is apparently the cmake way of
>>> doing things.
>>
>> Our ld-wrapper is ineffective when the .so file names are passed
>> directly like this. That is, ld-wrapper adds -rpath for any -l switch,
>> but it does not add -rpath for libraries whose absolute file name is
>> specified.
>>
>> That could be easily fixed, but in core-updates.
>>
>>> But no "-Wl,-rpath" despite
>>> -DCMAKE_INSTALL_RPATH_USE_LINK_PATH=TRUE
>>> Does this mean that this option only honours paths coming from "-l", while
>>> at the same time using "-l" is unusual with cmake?
>>
>> Uh. While we can improve ld-wrapper, the best would of course be to get
>> CMake to do the right thing...
> We have not change ld-wrapper for this right?
I’ve made the change in d8491ba (core-updates), which Hydra is now
building. Please report any issues.
Ludo’.