Bug #1861

spread 4.3.0 misses linking against libdl

Added by R. Haschke almost 9 years ago. Updated almost 9 years ago.

Status:ResolvedStart date:04/29/2014
Priority:NormalDue 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.

configure-dl.patch Magnifier - looking for correct dynamic loader and adding it to $SHLIBS (498 Bytes) R. Haschke, 04/30/2014 12:23 AM

libspread-dl.patch Magnifier - adding -ldl when building libspread.so (837 Bytes) R. Haschke, 04/30/2014 12:23 AM

Associated revisions

Revision f8e52971
Added by J. Wienke almost 9 years ago

Include a patch to link against dl

fixes #1861

History

#1 Updated by J. Wienke almost 9 years ago

  • Assignee deleted (J. Wienke)

Sorry, I can't support this. I have never used autotools.

#2 Updated by R. Haschke almost 9 years ago

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 9 years ago

So if we go for the better solution, the summary is as follows:
  1. We only have to patch 4.3.0, not 4.1.0
  2. We have to apply configure-dl.patch
  3. The spread.project recipe has to be extended to run autoconf
  4. 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 9 years ago

And thanks for the patch btw.

#5 Updated by R. Haschke almost 9 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 9 years ago

The debian build instructions have to be updated as well, probably.

#7 Updated by J. Wienke almost 9 years ago

  • Status changed from New to Resolved
  • % Done changed from 0 to 100

Applied in changeset r17.

Also available in: Atom PDF