Page MenuHomeFreeBSD

elftoolchain: Use ${SRCTOP} for the top of the FreeBSD tree
ClosedPublic

Authored by emaste on Apr 19 2016, 1:13 PM.
Tags
None
Referenced Files
Unknown Object (File)
Fri, May 15, 2:53 AM
Unknown Object (File)
Thu, May 14, 3:38 PM
Unknown Object (File)
Sun, May 10, 7:12 AM
Unknown Object (File)
Sun, May 10, 7:07 AM
Unknown Object (File)
Mon, May 4, 5:55 AM
Unknown Object (File)
Sat, May 2, 11:04 AM
Unknown Object (File)
Wed, Apr 29, 7:29 AM
Unknown Object (File)
Wed, Apr 29, 7:25 AM
Subscribers
None

Diff Detail

Repository
rS FreeBSD src repository - subversion
Lint
Lint Not Applicable
Unit
Tests Not Applicable

Event Timeline

emaste retitled this revision from to elftoolchain: Use ${SRCTOP} for the top of the FreeBSD tree.
emaste updated this object.
emaste edited the test plan for this revision. (Show Details)
emaste added reviewers: ngie, bdrewery.
lib/libdwarf/Makefile
5 ↗(On Diff #15315)

Also renamed TOP to ELFTCDIR as used elsewhere in ELF Tool Chain build bits to avoid confusion.

ngie edited edge metadata.
ngie added inline comments.
lib/libdwarf/Makefile
5 ↗(On Diff #15315)

I think this is missing a tab.

lib/libelf/Makefile
8 ↗(On Diff #15315)

I think this is missing a tab.

83 ↗(On Diff #15315)

Isn't . implied?

This revision is now accepted and ready to land.Apr 20 2016, 12:06 AM
lib/libdwarf/Makefile
5 ↗(On Diff #15315)

Unfortunately ELFTCDIR is too long and it looks odd with a tab after, unless SRCDIR below gets indented an extra tab, and then PACKAGE= looks out of place:

PACKAGE=lib${LIB}
ELFTCDIR=       ${SRCTOP}/contrib/elftoolchain
SRCDIR=         ${ELFTCDIR}/libdwarf
lib/libelf/Makefile
83 ↗(On Diff #15315)

It's needed for < > includes relative to ..

This revision was automatically updated to reflect the committed changes.