Page MenuHomeFreeBSD
Differential D21080

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

Authored by markj on Jul 26 2019, 7:11 PM.
Tags
None
Referenced Files
F112825873: D21080.diff
Sun, Mar 23, 5:58 AM
Unknown Object (File)
Sat, Mar 8, 5:08 AM
Unknown Object (File)
Jan 26 2025, 11:10 PM
Unknown Object (File)
Jan 11 2025, 7:54 AM
Unknown Object (File)
Jan 10 2025, 6:33 PM
Unknown Object (File)
Dec 11 2024, 5:08 AM
Unknown Object (File)
Oct 3 2024, 11:47 PM
Unknown Object (File)
Sep 28 2024, 12:03 PM
View All 34 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
emaste accepted this revision.Jul 26 2019, 7:18 PM
emaste added a subscriber: jkoshy_users.sourceforge.net.

Would you post the patch to elftoolchain-developers too?

This revision is now accepted and ready to land.Jul 26 2019, 7:18 PM
jkoshy_users.sourceforge.net accepted this revision.Jul 27 2019, 3:38 PM

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

markj added a comment.Jul 27 2019, 4:36 PM
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

PathSize
contrib/
elftoolchain/
elfcopy/
2 lines
3 lines

Diff 60174

View Options

contrib/elftoolchain/elfcopy/main.c

Loading...
View Options

contrib/elftoolchain/elfcopy/strip.1

Loading...