strip: Only allow one positional parameter with -o.
AcceptedPublic
Actions

Authored by markj on Jul 26 2019, 7:11 PM.

Tags

None

Referenced Files

	Unknown Object (File)
	Tue, Aug 11, 8:12 PM

	Unknown Object (File)
	Sat, Aug 8, 6:32 PM

	Unknown Object (File)
	Sat, Aug 8, 7:49 AM

	Unknown Object (File)
	Fri, Aug 7, 7:21 AM

	Unknown Object (File)
	Thu, Aug 6, 10:59 AM

	Unknown Object (File)
	Tue, Jul 28, 9:51 AM

	Unknown Object (File)
	Mon, Jul 27, 12:52 AM

	Unknown Object (File)
	Sun, Jul 26, 12:18 PM

View All 100 Files

Subscribers

jkoshy_users.sourceforge.net

Details

Reviewers

emaste
jkoshy_users.sourceforge.net

Summary

It doesn't make sense to strip multiple files into a single output file.
Moreover, this is compatible with GNU strip.

Diff Detail

Lint

No Lint Coverage

Unit

No Test Coverage

Build Status

Buildable 25523
Build 24141: arc lint + arc unit

Event Timeline

markj created this revision.Jul 26 2019, 7:11 PM

Harbormaster completed remote builds in B25523: Diff 60174.Jul 26 2019, 7:11 PM

markj added a reviewer: emaste.Jul 26 2019, 7:13 PM

Would you post the patch to elftoolchain-developers too?

This revision is now accepted and ready to land.Jul 26 2019, 7:18 PM

FYI, I've added the equivalent of these changes upstream in r3778] and r3779].

In D21080#457604, @jkoshy_users.sourceforge.net wrote:

FYI, I've added the equivalent of these changes upstream in r3778] and r3779].

Thanks! Do you have any objection to applying D21079 as well?

Revision Contents
Changeset List

Path

Size

contrib/

elftoolchain/

elfcopy/

main.c

2 lines

strip.1

3 lines

Diff 60174

View Options

contrib/elftoolchain/elfcopy/main.c