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

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

Tags

None

Referenced Files

	F159193906: D57398.diff
	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

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

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

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

Repository

rG FreeBSD src repository

Lint

Lint Not Applicable

Unit

Tests Not Applicable

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 179132

View Options

usr.bin/xinstall/tests/install_test.sh