Allow ELF Tool Chain elfcopy to be installed as objcopy
ClosedPublic
Actions

Authored by emaste on Jun 22 2015, 8:23 PM.

Details

Reviewers

bapt
andrew

Commits

rS286044: MK_ELFCOPY_AS_OBJCOPY should be a variable
rS286030: Allow ELF Tool Chain elfcopy to be installed as objcopy

Summary

ELF Tool Chain elfcopy is nearly a drop-in replacement for GNU objcopy (but does not currently support PE output, needed for building x86 UEFI bits). Add a make.conf knob to allow installing it as objcopy and set it by default for aarch64 only, where we don't have a native binutils.

This is WIP for comment and testing.

Diff Detail

Repository

rS FreeBSD src repository - subversion

Lint

Lint Not Applicable

Unit

Tests Not Applicable

Event Timeline

emaste updated this revision to Diff 6386.Jun 22 2015, 8:23 PM

emaste retitled this revision from to Allow ELF Tool Chain elfcopy to be installed as objcopy.

emaste updated this object.

emaste edited the test plan for this revision. (Show Details)

emaste added a reviewer: andrew.

emaste added a subscriber: freebsd-toolchain-list.

Herald added subscribers: emaste, bdrewery. · View Herald TranscriptJun 22 2015, 8:23 PM

Why not always build elfcopy and just make a hardlink objcopy if MK_ELFCOPY_AS_OBJCOPY is set?

That will make elfcopy always available to users

In D2887#56056, @bapt wrote:

Why not always build elfcopy and just make a hardlink objcopy if MK_ELFCOPY_AS_OBJCOPY is set?

That will make elfcopy always available to users

That's a possibility, although I think I'd prefer to (eventually) just install elfcopy as objcopy on all platforms, ensuring that it supports all of the options we care about. We haven't yet shipped elfcopy in a release.

bapt accepted this revision.Jun 22 2015, 9:31 PM

bapt added a reviewer: bapt.

This revision is now accepted and ready to land.Jun 22 2015, 9:31 PM

Closed by commit rS286030: Allow ELF Tool Chain elfcopy to be installed as objcopy (authored by emaste). · Explain WhyJul 29 2015, 6:45 PM

This revision was automatically updated to reflect the committed changes.

emaste added inline comments.Jul 29 2015, 9:35 PM