--export-dynamic linker flag may be unnecessary

Bug #808338 reported by Alex Valavanis
8
This bug affects 1 person
Affects Status Importance Assigned to Milestone
Inkscape
Fix Released
Medium
Alex Valavanis
Gentoo Linux
Unknown
Unknown

Bug Description

Follow-up report to bug #707205 <Inkscape fails to build with gcc 4.6>

Inkscape is compiled with the --export-dynamic linker flag. For several years, this flag was incorrectly passed as a **compiler** option, rather than a linker option, and was ignored by GCC.[1] Inkscape appears to have worked correctly on linux throughout this period, implying that the flag is not actually needed.

Since r10071, this flag is passed explicitly to the GNU linker rather than to the compiler. However, it is possible that it is not needed at all. A discussion on the matter hasn't yielded any responses, possibly implying that no one is aware of a reason to keep the flag?[2]

It may be a good idea to try removing the flag from trunk and look out for any consequences.

[1] http://gcc.gnu.org/gcc-4.6/changes.html
[2] http://thread.gmane.org/gmane.comp.graphics.inkscape.devel/36020/focus=36358

Tags: build
Revision history for this message
su_v (suv-lp) wrote :

Setting status to 'Medium' for now - please adjust if you don't agree.

tags: added: build
Changed in inkscape:
importance: Undecided → Medium
status: New → Triaged
Revision history for this message
Emil Karlson (jekarlson) wrote :
Revision history for this message
Alex Valavanis (valavanisalex) wrote :

OK, I decided to go for it. Committed to trunk in r10439. Seems to build just fine for me with GCC 4.6... but we knew that already ;)

It's a very easy change to revert if anyone finds a reason why this flag was ever used.

Changed in inkscape:
status: Triaged → Fix Committed
assignee: nobody → Alex Valavanis (valavanisalex)
milestone: none → 0.49
assignee: Alex Valavanis (valavanisalex) → nobody
milestone: 0.49 → none
Changed in inkscape:
assignee: nobody → Alex Valavanis (valavanisalex)
milestone: none → 0.49
Changed in inkscape:
status: Fix Committed → Fix Released
To post a comment you must log in.
This report contains Public information  
Everyone can see this information.

Other bug subscribers

Remote bug watches

Bug watches keep track of this bug in other bug trackers.