Page MenuHomeFreeBSD

bsdinstall: Fail nicely in jail target
ClosedPublic

Authored by fernape on Oct 17 2023, 1:39 PM.
Tags
None
Referenced Files
Unknown Object (File)
Wed, Aug 5, 5:24 AM
Unknown Object (File)
Tue, Aug 4, 7:27 AM
Unknown Object (File)
Tue, Aug 4, 7:17 AM
Unknown Object (File)
Mon, Aug 3, 6:58 AM
Unknown Object (File)
Tue, Jul 28, 6:46 AM
Unknown Object (File)
Tue, Jul 28, 6:46 AM
Unknown Object (File)
Sat, Jul 25, 3:41 PM
Unknown Object (File)
Sun, Jul 12, 7:47 PM
Subscribers

Details

Summary

If the directory is empty, then we fail with a message regarding mkdir in
which the empty directory can't be seen because it is not quoted.

Show a nice message so the user knows what is going on.

Test Plan

Apply patch and execute:
bsdinstall jail

Diff Detail

Repository
rG FreeBSD src repository
Lint
Lint Not Applicable
Unit
Tests Not Applicable

Event Timeline

bapt added a subscriber: bapt.

small style remark, please address before pushing, other that that, lookgs good to me

usr.sbin/bsdinstall/scripts/jail
56

to be consistent with the rest of the code in this script please remove the { and }

This revision is now accepted and ready to land.Oct 18 2023, 6:58 AM
This revision was automatically updated to reflect the committed changes.

Done with the suggested change.

Thanks for the review!