Opened 7 years ago

Closed 7 years ago

Last modified 5 months ago

#618 closed defect (fixed)

LINK_STATIC_LIBS="ld -r" apparently "misguided"

Reported by: epruesse Owned by: westram
Priority: critical Milestone: arb7.0
Component: Libraries (multi) Version: SVN
Keywords: Cc:

Description

ARB fails to build under Debian Sid / Ubuntu 14.10 due to changes in binutils. Apparently the problem is cause by our use of "ld -r":

Subject: ld -r link issues
Date: Mon, 13 Oct 2014 20:18:25 +0200
From: Matthias Klose <doko@debian.org>
To: 763228@bugs.debian.org, arb@packages.debian.org, 764573@bugs.debian.org

the PR ld/17647 issue is now fixed upstream, however upstream recommends not
using ld -r to create these "archives".

"This doesn't look easy to fix.  I'm very tempted to say that people
who use ld -r to package object files deserve to suffer for their
misguided actions..  The underlying problem, with the potential for
nasty exception handling bugs at runtime, has been there since
https://sourceware.org/ml/binutils/2003-02/msg00471.html"

Please address this upstream.

Suggested fix: use "ar rcs" instead.

Change History (6)

comment:1 Changed 7 years ago by epruesse

  • Owner changed from devel to epruesse
  • Status changed from new to _started

comment:2 Changed 7 years ago by westram

  • Owner changed from epruesse to westram
  • Status changed from _started to assigned

will apply my former patch (solving it with branch makefix seems too committed)

comment:3 Changed 7 years ago by westram

  • Status changed from assigned to _started

comment:4 Changed 7 years ago by westram

  • Resolution set to fixed
  • Status changed from _started to closed

comment:5 Changed 7 years ago by westram

Please note:
Updating old WCs and just calling 'make all' will fail with

ar: client.a: File format not recognized

or similar.

Fix that by calling 'make relink' (or 'make clean').

comment:6 Changed 5 months ago by westram

  • Milestone changed from arb6.1 to arb7.0

Milestone renamed

Note: See TracTickets for help on using tickets.