Page MenuHomeFreeBSD

arm64: Use FULLKERNEL instead of .ALLSRC in .bin target
ClosedPublic

Authored by stevek on Apr 18 2023, 5:36 AM.
Tags
None
Referenced Files
Unknown Object (File)
Thu, Oct 17, 3:31 AM
Unknown Object (File)
Oct 2 2024, 9:07 PM
Unknown Object (File)
Sep 27 2024, 7:29 AM
Unknown Object (File)
Sep 26 2024, 11:46 AM
Unknown Object (File)
Sep 26 2024, 2:05 AM
Unknown Object (File)
Sep 22 2024, 11:34 PM
Unknown Object (File)
Sep 22 2024, 12:33 PM
Unknown Object (File)
Sep 19 2024, 9:10 PM

Details

Summary

Using .ALLSRC may get additional arguments that we may not want
and could cause the objcopy to fail.

Obtained from: Juniper Networks, Inc.

Diff Detail

Repository
rG FreeBSD src repository
Lint
Lint Passed
Unit
No Test Coverage
Build Status
Buildable 50986
Build 47877: arc lint + arc unit

Event Timeline

IMO this is fine, but also ${.ALLSRC} should only ever be ${FULLKERNEL} so not sure how you'd get this case.

IMO this is fine, but also ${.ALLSRC} should only ever be ${FULLKERNEL} so not sure how you'd get this case.

In downstream build environments, there could be additional dependencies added.

The ${NM} statement should probably also use ${FULLKERNEL}, since that is what we really want to operate on.

In downstream build environments, there could be additional dependencies added.

Ok, fair enough. It could maybe be something like ${.ALLSRC:[1]} instead but I'm fine with it either way

This revision is now accepted and ready to land.Apr 18 2023, 2:19 PM