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)
Mar 22 2024, 10:24 PM
Unknown Object (File)
Mar 22 2024, 10:24 PM
Unknown Object (File)
Mar 22 2024, 10:24 PM
Unknown Object (File)
Mar 8 2024, 6:37 AM
Unknown Object (File)
Feb 23 2024, 10:46 PM
Unknown Object (File)
Jan 6 2024, 5:25 AM
Unknown Object (File)
Jan 6 2024, 5:25 AM
Unknown Object (File)
Jan 6 2024, 5: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.