Differential D21107

Capsicumize nm(1).
ClosedPublic
Actions

Authored by markj on Jul 29 2019, 7:35 PM.

Tags

None

Referenced Files

	Unknown Object (File)
	Jan 3 2025, 8:48 AM

	Unknown Object (File)
	Jan 2 2025, 1:39 AM

	Unknown Object (File)
	Nov 26 2024, 12:52 PM

	Unknown Object (File)
	Nov 23 2024, 11:46 AM

	Unknown Object (File)
	Nov 22 2024, 4:12 PM

	Unknown Object (File)
	Nov 21 2024, 2:50 AM

	Unknown Object (File)
	Nov 11 2024, 5:32 PM

	Unknown Object (File)
	Nov 4 2024, 5:33 PM

View All 47 Files

Subscribers

jkoshy_users.sourceforge.net

Details

Reviewers

oshogbo
emaste

Group Reviewers

Commits

rS352909: Capsicumize nm(1).

Diff Detail

Lint

Lint Passed

Unit

No Test Coverage

Build Status

Buildable 26798
Build 25134: arc lint + arc unit

Event Timeline

markj created this revision.Jul 29 2019, 7:35 PM

Herald added a subscriber: jonathan. · View Herald TranscriptJul 29 2019, 7:35 PM

Harbormaster completed remote builds in B25572: Diff 60246.Jul 29 2019, 7:35 PM

markj added reviewers: capsicum, oshogbo, emaste.Jul 29 2019, 7:45 PM

emaste added subscribers: kaiw, jkoshy_users.sourceforge.net.Aug 16 2019, 4:11 PM

looks good

contrib/elftoolchain/nm/nm.c
28–30	suggest committing this (and similar changes from other ELF Tool Chain tools) independently first, and I'll upstream those immediately
2158–2161	These changes too could be upstreamed independently

Get rid of style fixes in #includes.
Update argc and argv in get_opt(). I think this is a bit neater.

Harbormaster completed remote builds in B26798: Diff 62761.Sep 30 2019, 5:05 PM

markj added inline comments.Sep 30 2019, 5:08 PM

contrib/elftoolchain/nm/nm.c
28–30	I just dropped those changes.
2158–2161	I will commit that separately and post a patch to elftoolchain-developers.

This revision was not accepted when it landed; it landed in state Needs Review.Sep 30 2019, 5:28 PM

Closed by commit rS352909: Capsicumize nm(1). (authored by markj). · Explain Why

This revision was automatically updated to reflect the committed changes.

markj added a commit: rS352909: Capsicumize nm(1)..

Herald added a subscriber: imp. · View Herald TranscriptSep 30 2019, 5:28 PM

Revision Contents
Changeset List

Path

Size

contrib/

elftoolchain/

nm/

57 lines

usr.bin/

nm/

6 lines

Diff 62761

contrib/elftoolchain/nm/nm.c

Loading...

usr.bin/nm/Makefile

Loading...