xinstall: Do not allow -l and -s together
ClosedPublic
Actions

Authored by emaste on Wed, Jun 3, 1:27 AM.

Tags

None

Referenced Files

	F159345725: D57398.diff
	Sat, Jun 13, 2:50 AM

	F159332744: D57398.id179124.diff
	Fri, Jun 12, 11:02 PM

	Unknown Object (File)
	Thu, Jun 11, 4:20 AM

	Unknown Object (File)
	Tue, Jun 9, 4:43 AM

	Unknown Object (File)
	Fri, Jun 5, 9:55 PM

	Unknown Object (File)
	Thu, Jun 4, 7:27 PM

	Unknown Object (File)
	Thu, Jun 4, 7:27 PM

	Unknown Object (File)
	Thu, Jun 4, 6:43 PM

View All 14 Files

Subscribers

imp

Details

Reviewers

des
jrtc27

Group Reviewers

build

Commits

rG4b570289f0cc: xinstall: Do not allow -l and -s together
rG521afce6a859: xinstall: Do not allow -l and -s together

Summary

Cannot strip the target if creating a link.

Diff Detail

Lint

Lint Skipped

Unit

Tests Skipped

Event Timeline

emaste requested review of this revision.Wed, Jun 3, 1:27 AM

emaste created this revision.

add test

emaste added reviewers: des, jrtc27.Wed, Jun 3, 2:28 PM

des accepted this revision.Wed, Jun 3, 3:23 PM

This revision is now accepted and ready to land.Wed, Jun 3, 3:23 PM

This should probably be documented, though.

Closed by commit rG521afce6a859: xinstall: Do not allow -l and -s together (authored by emaste). · Explain WhyWed, Jun 3, 3:33 PM

This revision was automatically updated to reflect the committed changes.

emaste added a commit: rG521afce6a859: xinstall: Do not allow -l and -s together.

Herald added a subscriber: imp. · View Herald TranscriptWed, Jun 3, 3:33 PM

emaste added a commit: rG4b570289f0cc: xinstall: Do not allow -l and -s together.Wed, Jun 10, 1:45 PM

Revision Contents
Changeset List

Path

Size

usr.bin/

xinstall/

tests/

install_test.sh

3 lines

xinstall.c

6 lines

Diff 179124

View Options

usr.bin/xinstall/tests/install_test.sh