[PATCH] ELF: Properly copy and strip Solaris binaries
Alan Modra
amodra@gmail.com
Wed Jan 28 01:18:31 GMT 2026
On Tue, Jan 27, 2026 at 08:04:41PM +0100, Rainer Orth wrote:
> Hi Alan,
>
> > The following is what is left over from
> > https://sourceware.org/pipermail/binutils/2025-December/146789.html
> >
> > Please cast your eye over this HJ, it is mostly your patch. Rainer,
> > if you commit your ld_supports_emul patch soon I'll update the
> > "ld -V -m" here before committing.
>
> done now.
>
> Just a few comments and questions on the patch:
>
> * The ChangeLog should refer to PR binutils/33718 so the commit shows up
> in there
It is sufficient to put "PR 33718" in the subject. Note that you
need the space.
> * I wonder why neither the PR nor the patch refer to the extensive
> discussion of the issue in PR binutils/33684 which prompted this PR
> and patch.
Added to the patch.
> * Neither the patch names nor the filenames carry any meaning. I think
> they should be way more descriptive instead.
>
> * Besides, I wonder about dropping arbitrary objects into the testsuite.
> I suspect it would be way better to rely on native testing for issues
> like this, so we could both extend testing to all of Solaris/SPARC and
> x86, 32 and 64-bit. Besides, it would be possible to run elfdump >
> /dev/null on the objects which is an excellent first sanity check for
> all kinds of objects on Solaris.
>
> After all, we're just one PR away from clean Solaris testresults as a
> prerequisite for setting up such bots :-)
OK, I'll remove the testcase and just commit the objcopy.c part.
--
Alan Modra
More information about the Binutils
mailing list