Bug #1861
spread 4.3.0 misses linking against libdl
Status: | Resolved | Start date: | 04/29/2014 | |
---|---|---|---|---|
Priority: | Normal | Due date: | ||
Assignee: | - | % Done: | 100% | |
Category: | - | |||
Target version: | - |
Description
libspread.so is not found by autotools mechanisms, because it uses dl_addr but doesn't link to -dl
Of course this is a bug in spread itself, but we should come up with a patch ourselves.
Associated revisions
Include a patch to link against dl
fixes #1861
History
#1 Updated by J. Wienke almost 10 years ago
- Assignee deleted (
J. Wienke)
Sorry, I can't support this. I have never used autotools.
#2 Updated by R. Haschke almost 10 years ago
- File configure-dl.patch added
- File libspread-dl.patch added
I have done the simple requested patch. Please add the patch to the packaging system now.
As I suggested, when building libspread.so the option '-ldl' was missing. I fixed that in libspread/Makefile.in (libspread-dl.patch).
Better of course, is to look for the correct dynamic loader in the configure process. This is done by configure-dl.patch. However, this also requires to run autoconf before packaging!
If you have any questions, don't hesitate to contact me.
Best, Robert
#3 Updated by J. Moringen almost 10 years ago
- We only have to patch 4.3.0, not 4.1.0
- We have to apply
configure-dl.patch
- The
spread.project
recipe has to be extended to run autoconf - If we have manually maintained Spread build jobs, these have to be extended to run autotools as well
Is this correct?
#4 Updated by J. Moringen almost 10 years ago
And thanks for the patch btw.
#5 Updated by R. Haschke almost 10 years ago
This is correct. If you have contacts to the spread team, you should file an appropriate bug there as well.
The configure scripts correctly look for libdl for building libspread-util. However, this is a static lib only.
Hence, when building libspread.so, -ldl needs to be added again.
Best, Robert
#6 Updated by J. Wienke almost 10 years ago
The debian build instructions have to be updated as well, probably.
#7 Updated by J. Wienke almost 10 years ago
- Status changed from New to Resolved
- % Done changed from 0 to 100
Applied in changeset r17.