Page MenuHomeFreeBSD

Allow boostrapping xinstall on Linux
ClosedPublic

Authored by arichardson on Nov 19 2019, 5:58 PM.
Tags
None
Referenced Files
Unknown Object (File)
Dec 25 2023, 3:39 AM
Unknown Object (File)
Dec 20 2023, 6:21 AM
Unknown Object (File)
Aug 25 2023, 11:30 PM
Unknown Object (File)
Aug 25 2023, 11:29 PM
Unknown Object (File)
Aug 25 2023, 11:27 PM
Unknown Object (File)
Aug 25 2023, 11:10 PM
Unknown Object (File)
Aug 2 2023, 6:20 PM
Unknown Object (File)
Jul 15 2023, 10:20 PM
Subscribers

Details

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

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 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.

This revision is now accepted and ready to land.Nov 20 2019, 4:14 PM
usr.bin/xinstall/xinstall.c
83 ↗(On Diff #64580)

We are as confident as we can be about Linux...

This revision was automatically updated to reflect the committed changes.