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)
	Mon, Apr 27, 8:23 AM

	Unknown Object (File)
	Fri, Apr 17, 8:04 AM

	Unknown Object (File)
	Sat, Apr 11, 10:02 PM

	Unknown Object (File)
	Thu, Apr 9, 8:05 PM

	Unknown Object (File)
	Thu, Apr 9, 9:56 AM

	Unknown Object (File)
	Wed, Apr 8, 10:28 PM

	Unknown Object (File)
	Tue, Apr 7, 9:13 AM

	Unknown Object (File)
	Sun, Apr 5, 11:37 PM

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