[Differential] [Accepted] D1663: elfcopy: Consider program name to start after last -

2015-01-24 Thread dim (Dimitry Andric)
dim added a subscriber: dim.
dim accepted this revision.
dim added a reviewer: dim.
dim added a comment.
This revision is now accepted and ready to land.

LGTM.

REVISION DETAIL
  https://reviews.freebsd.org/D1663

To: emaste, dim
Cc: dim, freebsd-toolchain
___
freebsd-toolchain@freebsd.org mailing list
http://lists.freebsd.org/mailman/listinfo/freebsd-toolchain
To unsubscribe, send any mail to freebsd-toolchain-unsubscr...@freebsd.org


[Differential] [Updated, 55 lines] D1663: elfcopy: Select mode by the end of the program name

2015-01-24 Thread emaste (Ed Maste)
emaste retitled this revision from elfcopy: Consider program name to start 
after last - to elfcopy: Select mode by the end of the program name.
emaste updated the summary for this revision.
emaste updated this revision to Diff 3418.
emaste added a comment.

Output a warning if the program name (argv[0]) does not match any of the 
expected ones.

CHANGES SINCE LAST UPDATE
  https://reviews.freebsd.org/D1663?vs=3417id=3418

REVISION DETAIL
  https://reviews.freebsd.org/D1663

AFFECTED FILES
  elfcopy/main.c
  test/elfcopy/func.sh
  test/elfcopy/tc/strip-basename-1/strip-basename-1.sh

To: emaste, dim
Cc: dim, freebsd-toolchain
___
freebsd-toolchain@freebsd.org mailing list
http://lists.freebsd.org/mailman/listinfo/freebsd-toolchain
To unsubscribe, send any mail to freebsd-toolchain-unsubscr...@freebsd.org


[Differential] [Updated, 49 lines] D1663: elfcopy: Consider program name to start after last -

2015-01-24 Thread emaste (Ed Maste)
emaste updated this revision to Diff 3417.
emaste added a comment.
This revision now requires review to proceed.

Just compare the end of the program name with strip/mcs rather than looking for 
a dash. This is the way it's done in FreeBSD ar/ranlib already, and allows 
names like bsdstrip. Also add a test case.

CHANGES SINCE LAST UPDATE
  https://reviews.freebsd.org/D1663?vs=3416id=3417

REVISION DETAIL
  https://reviews.freebsd.org/D1663

AFFECTED FILES
  elfcopy/main.c
  test/elfcopy/func.sh
  test/elfcopy/tc/strip-basename-1/strip-basename-1.sh

To: emaste, dim
Cc: dim, freebsd-toolchain
___
freebsd-toolchain@freebsd.org mailing list
http://lists.freebsd.org/mailman/listinfo/freebsd-toolchain
To unsubscribe, send any mail to freebsd-toolchain-unsubscr...@freebsd.org