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)
Sep 24 2024, 11:08 AM
Unknown Object (File)
Sep 16 2024, 11:31 PM
Unknown Object (File)
Sep 16 2024, 11:49 AM
Unknown Object (File)
Sep 16 2024, 2:22 AM
Unknown Object (File)
Sep 15 2024, 3:37 PM
Unknown Object (File)
Sep 15 2024, 4:57 AM
Unknown Object (File)
Sep 15 2024, 4:57 AM
Unknown Object (File)
Sep 14 2024, 12:49 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!