Page MenuHomeFreeBSD
Differential D42499

crunchgen: make special "keep" more useful, break it out
ClosedPublic
Actions

Authored by kevans on Nov 8 2023, 6:34 AM.
Tags
None
Referenced Files
F102906950: D42499.diff
Mon, Nov 18, 2:28 PM
Unknown Object (File)
Thu, Nov 14, 6:59 PM
Unknown Object (File)
Thu, Nov 14, 5:35 AM
Unknown Object (File)
Tue, Nov 5, 8:09 PM
Unknown Object (File)
Oct 18 2024, 10:35 AM
Unknown Object (File)
Oct 17 2024, 10:06 AM
Unknown Object (File)
Oct 16 2024, 8:31 PM
Unknown Object (File)
Oct 15 2024, 3:34 AM
View All 58 Files
Subscribers
bdrewery
emaste

Details

Reviewers
imp
adrian
Commits
rG8f2848eafa68: crunchgen: fix "keep" for an ELF world, break it out
Summary

Right now this prepends a leading underscore, but it's not clear that
it's even used. Preserve the symbol as it comes in instead to make it
more generically useful.

While we're here, break it out to bsd.crunchgen.mk for later use in
bsdbox.

Diff Detail

Repository
rG FreeBSD src repository
Lint
Lint Skipped
Unit
Tests Skipped
Build Status
Buildable 54326
Build 51216: arc lint + arc unit

Event Timeline

kevans created this revision.Nov 8 2023, 6:34 AM
Herald added subscribers: emaste, bdrewery. · View Herald TranscriptNov 8 2023, 6:34 AM
kevans requested review of this revision.Nov 8 2023, 6:34 AM
Harbormaster completed remote builds in B54326: Diff 129844.Nov 8 2023, 6:34 AM
kevans added a parent revision: D42498: bsdbox: fix the build.Nov 8 2023, 6:34 AM
kevans added a child revision: D42500: bsdbox: fix the hostapd build, hook it back up.
imp accepted this revision.Nov 8 2023, 10:49 AM
imp added inline comments.
usr.sbin/crunch/crunchgen/crunchgen.c
1131

A.out did this for symbol names. I wonder why it didn't die in FreeBSD 3.2

This revision is now accepted and ready to land.Nov 8 2023, 10:49 AM
adrian accepted this revision.Nov 8 2023, 4:02 PM
kevans added inline comments.Nov 8 2023, 4:10 PM
usr.sbin/crunch/crunchgen/crunchgen.c
1131

Oh, thanks for that context. In that case, I'm just going to drop the UPDATING completely and maybe flag it in relnotes at most.

Closed by commit rG8f2848eafa68: crunchgen: fix "keep" for an ELF world, break it out (authored by kevans). · Explain WhyNov 10 2023, 4:34 AM
This revision was automatically updated to reflect the committed changes.
kevans added a commit: rG8f2848eafa68: crunchgen: fix "keep" for an ELF world, break it out.

Revision Contents
Changeset List

PathSize
7 lines
share/
mk/
4 lines
usr.sbin/
crunch/
crunchgen/
2 lines

Diff 129844

View Options

UPDATING

Loading...
View Options

share/mk/bsd.crunchgen.mk

Loading...
View Options

usr.sbin/crunch/crunchgen/crunchgen.c

Loading...