Re: CVS commit: src/external/gpl3/binutils/dist/ld/emultempl

[Joerg Sonnenberger]

On Mon, Sep 11, 2023 at 12:27:25PM +1000, matthew green wrote:
> "Rin Okuyama" writes:
> > Module Name:src
> > Committed By:   rin
> > Date:   Mon Sep 11 01:54:18 UTC 2023
> >
> > Modified Files:
> > src/external/gpl3/binutils/dist/ld/emultempl: aarch64elf.em armelf.em
> > elf.em
> >
> > Log Message:
> > ld: Enable --copy-dt-needed-entries by default again
> 
> thanks for fixing -lcurses.
> 
> can we put this into bsd.*.mk instead?
> 
> ie, if we want this, declare it explicitly, rather than
> reply upon patched binutils?  (this is eg, unfriendly to
> EXTERNAL_TOOLCHAIN etc.)

We (TNF) consider this the default a bug. Red Hat etc who've been
pushing for this mess don't, for reasons that don't really make sense.
Basically, the Linux toolchain folks decided that ELF should no longer
be recursive except it still is for every use case were performance
matters.

Joerg

Re: CVS commit: src/external/gpl3/binutils/dist/ld/emultempl

[Rin Okuyama]

On 2023/09/11 11:27, matthew green wrote:

"Rin Okuyama" writes:

Module Name:src
Committed By:   rin
Date:   Mon Sep 11 01:54:18 UTC 2023

Modified Files:
src/external/gpl3/binutils/dist/ld/emultempl: aarch64elf.em armelf.em
elf.em

Log Message:
ld: Enable --copy-dt-needed-entries by default again


thanks for fixing -lcurses.

can we put this into bsd.*.mk instead?

ie, if we want this, declare it explicitly, rather than
reply upon patched binutils?  (this is eg, unfriendly to
EXTERNAL_TOOLCHAIN etc.)

if it works, of course


Thanks for your comment. Yeah, I feel this fix is too much also...

I've found that FreeBSD uses ldscript to handle curses vs terminfo problem.
I think we can resolve the problem in a similar manner.

I've sent PR lib/57615 for this:
http://gnats.netbsd.org/57615

Thanks,
rin

re: CVS commit: src/external/gpl3/binutils/dist/ld/emultempl

[matthew green]

"Rin Okuyama" writes:
> Module Name:  src
> Committed By: rin
> Date: Mon Sep 11 01:54:18 UTC 2023
>
> Modified Files:
>   src/external/gpl3/binutils/dist/ld/emultempl: aarch64elf.em armelf.em
>   elf.em
>
> Log Message:
> ld: Enable --copy-dt-needed-entries by default again

thanks for fixing -lcurses.

can we put this into bsd.*.mk instead?

ie, if we want this, declare it explicitly, rather than
reply upon patched binutils?  (this is eg, unfriendly to
EXTERNAL_TOOLCHAIN etc.)

if it works, of course


.mrg.

Re: CVS commit: src/external/gpl3/binutils/dist/ld/emultempl

[Joerg Sonnenberger]

On Wed, Feb 18, 2015 at 03:05:44PM +, Joerg Sonnenberger wrote:
 Module Name:  src
 Committed By: joerg
 Date: Wed Feb 18 15:05:44 UTC 2015
 
 Modified Files:
   src/external/gpl3/binutils/dist/ld/emultempl: aarch64elf.em armelf.em
 
 Log Message:
 Apply elf32.em r1.8 to the corresponding sections of armelf.em and
 aarch64elf.em. Original commit message:
 
 Recursively add DT_NEEDED entries from shared libraries if symbols are
 used indirectly. This is more in line with the old GNU ld behavior, but
 not exactly the desired semantic.
 
 Patch from Martin Husemann.

Please note that this needs a make clean in src/external/gpl3/binutils
to be seen as the dependencies of the generated source files are
incomplete.

Joerg