Paths

Table of Contentst

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

	F108438898: D28130.id82187.diff
	Fri, Jan 24, 7:01 PM

	F108438200: D28130.id82189.diff
	Fri, Jan 24, 6:54 PM

	F108429229: D28130.id.diff
	Fri, Jan 24, 5:16 PM

	Unknown Object (File)
	Mon, Jan 20, 1:36 PM

	Unknown Object (File)
	Mon, Jan 20, 2:02 AM

	Unknown Object (File)
	Fri, Jan 17, 7:23 PM

	Unknown Object (File)
	Sun, Jan 12, 9:17 PM

	Unknown Object (File)
	Fri, Jan 3, 4:43 AM

View All 99 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

Repository

rG FreeBSD src repository

Lint

Lint Not Applicable

Unit

Tests Not Applicable

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
207	Add -i flag there ?
236	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.

leave warning in place with -i, per kib

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

Path

Size

usr.bin/

elfctl/

6 lines

13 lines

Diff 82189

usr.bin/elfctl/elfctl.1

Loading...

usr.bin/elfctl/elfctl.c

Loading...