Skip to content

Conversation

@slipher
Copy link
Member

@slipher slipher commented Dec 16, 2025

One way of working around the linker bug from #6. Also makes the binary smaller

Use --orphan-handling=unwanted when linking nacl_helper_bootstrap which
makes the linker discard any sections not explicitly mentioned in the
linker script. This prevents the linker from producing a defective
binary with overlapping PT_LOAD headers when using GCC on certain
distros such as Ubuntu and Arch that have it configured to produce a
.note.gnu.property section
Copy link
Member

@illwieckz illwieckz left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM.

Though I don't know what to do with the BUILD.gn file we don't use.

@illwieckz illwieckz added the bug Something isn't working label Dec 17, 2025
@slipher
Copy link
Member Author

slipher commented Dec 18, 2025

Though I don't know what to do with the BUILD.gn file we don't use.

Hmm I never really paid attention to those. Apparently both build systems are still maintained upstream which is bizarre.

@slipher slipher merged commit d281e6a into master Dec 18, 2025
@slipher slipher deleted the slipher/orphan-discard branch December 18, 2025 03:10
@illwieckz
Copy link
Member

Though I don't know what to do with the BUILD.gn file we don't use.

Hmm I never really paid attention to those. Apparently both build systems are still maintained upstream which is bizarre.

I suspect that the BUILD.gn system isn't usable outside of the Chromium build system, and as it is written right now cannot be used to build a standalone nacl loader (at least I failed to achieve that when I tested them), while the SCons scripts makes it possible to build and test a standalone loader.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

bug Something isn't working

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants