This is the mail archive of the binutils@sourceware.org mailing list for the binutils project.


Index Nav: [Date Index] [Subject Index] [Author Index] [Thread Index]
Message Nav: [Date Prev] [Date Next] [Thread Prev] [Thread Next]
Other format: [Raw text]

Re: [PATCH 2/2 v5] gold: enable new dtags by default


On Mon, Feb 4, 2013 at 5:44 PM, H.J. Lu <hjl.tools@gmail.com> wrote:
>
> This caused:
>
> http://www.sourceware.org/bugzilla/show_bug.cgi?id=15098
>
> I changed BFD linker not set new dtags with -rpath.

I don't see why that is the right fix.  Since DT_RPATH/DT_RUNPATH are
only ever set by the linker's -rpath option, it seems like the right
fix is to always use DT_RPATH and never use DT_RUNPATH.

Of course, since the only thing --new-dtags does in gold is select
DT_RUNPATH rather than DT_RPATH, this makes new--dtags completely
useless in gold.

It seems that we have made sensible-seeming decisions to wind up in an
absurd place.  It seems that we should now make --new-dtags a no-op
and drop all support for generating DT_RUNPATH.  Which makes me wonder
why DT_RUNPATH was invented in the first place.

Ian


Index Nav: [Date Index] [Subject Index] [Author Index] [Thread Index]
Message Nav: [Date Prev] [Date Next] [Thread Prev] [Thread Next]