Paths

Table of Contentst

Differential D22446

Allow boostrapping xinstall on Linux
ClosedPublic
Actions

Authored by arichardson on Nov 19 2019, 5:58 PM.

Tags

None

Referenced Files

	F106071677: D22446.diff
	Tue, Dec 24, 10:16 PM

	Unknown Object (File)
	Fri, Dec 13, 5:33 PM

	Unknown Object (File)
	Sat, Nov 30, 9:03 AM

	Unknown Object (File)
	Oct 9 2024, 6:17 PM

	Unknown Object (File)
	Oct 7 2024, 12:49 PM

	Unknown Object (File)
	Oct 7 2024, 10:46 AM

	Unknown Object (File)
	Oct 7 2024, 5:03 AM

	Unknown Object (File)
	Oct 4 2024, 10:07 AM

View All 47 Files

Subscribers

Details

Reviewers

emaste
jhb
brooks
imp

Commits

rS354901: Allow boostrapping xinstall on Linux

Summary

Linux does not have st_flags so we have to avoid using it there.

Diff Detail

Repository

rS FreeBSD src repository - subversion

Lint

Lint Not Applicable

Unit

Tests Not Applicable

Event Timeline

arichardson created this revision.Nov 19 2019, 5:58 PM

Harbormaster completed remote builds in B27640: Diff 64580.Nov 19 2019, 5:58 PM

emaste added inline comments.Nov 19 2019, 7:29 PM

usr.bin/xinstall/xinstall.c
83 ↗	(On Diff #64580)	We're sufficiently confident that `UF_SETTABLE` will remain undefined on Linux? I guess it's not a big deal if this breaks and needs to be fixed, but probably worth a comment here at least.

arichardson added a subscriber: imp.Nov 19 2019, 11:19 PM

arichardson added inline comments.

usr.bin/xinstall/xinstall.c
83 ↗	(On Diff #64580)	Since this is the mask of st_flags values, I would assume support it existing implies st_flags also exists. This was suggested by @imp in D13307.

emaste accepted this revision.Nov 20 2019, 4:14 PM

This revision is now accepted and ready to land.Nov 20 2019, 4:14 PM

imp added inline comments.Nov 20 2019, 4:18 PM

usr.bin/xinstall/xinstall.c
83 ↗	(On Diff #64580)	We are as confident as we can be about Linux...

imp accepted this revision.Nov 20 2019, 4:18 PM

Closed by commit rS354901: Allow boostrapping xinstall on Linux (authored by arichardson). · Explain WhyNov 20 2019, 5:24 PM

This revision was automatically updated to reflect the committed changes.

arichardson added a commit: rS354901: Allow boostrapping xinstall on Linux.

Revision Contents
Changeset List

Path

Size

head/

usr.bin/

xinstall/

1 line

11 lines

Diff 64623

head/usr.bin/xinstall/Makefile

Loading...

head/usr.bin/xinstall/xinstall.c

Loading...