Page MenuHomeFreeBSD
Differential D28130

elftcl: add -i flag to ignore unknown flags
ClosedPublic
Actions

Authored by emaste on Jan 13 2021, 3:29 AM.
Tags
None
Referenced Files
Unknown Object (File)
Tue, Oct 22, 4:11 PM
Unknown Object (File)
Oct 3 2024, 10:55 PM
Unknown Object (File)
Oct 3 2024, 5:59 PM
Unknown Object (File)
Oct 1 2024, 6:00 PM
Unknown Object (File)
Sep 30 2024, 2:15 AM
Unknown Object (File)
Sep 28 2024, 3:31 AM
Unknown Object (File)
Sep 26 2024, 8:31 PM
Unknown Object (File)
Sep 23 2024, 1:49 PM
View All 85 Files
Subscribers
None

Details

Reviewers
kib
markj
Commits
rG369a4023e671: elftcl: add -i flag to ignore unknown flags
rGf6d95a01103a: elftcl: add -i flag to ignore unknown flags
Summary

This may allow an identical elfctl invocation to be used on multiple FreeBSD versions, with features not implemented on older releases being silently ignored.

PR: 252629 (related)
MFC after: 1 week
Sponsored by: The FreeBSD Foundation

Diff Detail

Lint
Lint Skipped
Unit
Tests Skipped

Event Timeline

emaste requested review of this revision.Jan 13 2021, 3:29 AM
emaste created this revision.
emaste added a reviewer: markj.
kib added inline comments.Jan 13 2021, 4:00 AM
usr.bin/elfctl/elfctl.c
206

Add -i flag there ?

235

I suggest to print something anyway, so e.g. typos are still being caught. May be add quit mode where nothing is printed at all but I am not sure it is needed.

emaste updated this revision to Diff 82187.Jan 13 2021, 4:14 AM

leave warning in place with -i, per kib

emaste updated this revision to Diff 82188.Jan 13 2021, 4:17 AM

add -i to usage, per kib

kib accepted this revision.Jan 13 2021, 5:03 AM
This revision is now accepted and ready to land.Jan 13 2021, 5:03 AM
Closed by commit rGf6d95a01103a: elftcl: add -i flag to ignore unknown flags (authored by emaste). · Explain WhyJan 13 2021, 5:12 AM
This revision was automatically updated to reflect the committed changes.
emaste added a commit: rGf6d95a01103a: elftcl: add -i flag to ignore unknown flags.
emaste added a commit: rG369a4023e671: elftcl: add -i flag to ignore unknown flags.Jan 19 2021, 2:50 AM

Revision Contents
Changeset List

PathSize
usr.bin/
elfctl/
6 lines
12 lines

Diff 82187

View Options

usr.bin/elfctl/elfctl.1

Loading...
View Options

usr.bin/elfctl/elfctl.c

Loading...