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)
	Sat, Nov 15, 8:25 PM

	Unknown Object (File)
	Fri, Nov 14, 12:24 AM

	Unknown Object (File)
	Fri, Nov 14, 12:24 AM

	Unknown Object (File)
	Fri, Nov 14, 12:22 AM

	Unknown Object (File)
	Thu, Nov 6, 5:03 PM

	Unknown Object (File)
	Wed, Nov 5, 4:12 PM

	Unknown Object (File)
	Wed, Nov 5, 2:48 AM

	Unknown Object (File)
	Wed, Nov 5, 2:42 AM

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