Page MenuHomeFreeBSD

jail(8): replace bsdinstall with bsdconfig
ClosedPublic

Authored by bhd on Aug 16 2018, 6:03 PM.
Tags
None
Referenced Files
F133070967: D16751.id.diff
Wed, Oct 22, 5:11 PM
F133070953: D16751.id46852.diff
Wed, Oct 22, 5:11 PM
F133070951: D16751.id46990.diff
Wed, Oct 22, 5:11 PM
F133070948: D16751.id46789.diff
Wed, Oct 22, 5:11 PM
Unknown Object (File)
Wed, Oct 22, 4:49 AM
Unknown Object (File)
Sun, Oct 19, 4:54 PM
Unknown Object (File)
Thu, Oct 16, 4:34 AM
Unknown Object (File)
Tue, Oct 14, 10:33 AM
Subscribers

Details

Summary

In r324732 sysinstall was replaced by bsdinstall. However, for post-install configuration, bsdinstall is not of much use. Point the user to bsdconfig instead.

Diff Detail

Repository
rS FreeBSD src repository - subversion
Lint
Lint Not Applicable
Unit
Tests Not Applicable

Event Timeline

0mp added inline comments.
jail/jail.8
1036 ↗(On Diff #46789)

I cannot comment on the change from bsdinstall to bsdconfig but shouldn't we use a cross reference (Xr bsdconfig 8) here instead of a path to the program?

You need to bump the .Dd at the top of the man page as this is a content change.
Cross referencing bsdconfig would indeed be helpful.

.Xr bsdconfig 8 instead of a path to the program as suggested by 0mp.

bhd marked an inline comment as done.Aug 17 2018, 5:39 PM
In D16751#356681, @bcr wrote:

You need to bump the .Dd at the top of the man page as this is a content change.
Cross referencing bsdconfig would indeed be helpful.

Thanks bcr, I will bump .Dd when ready to commit.

jail/jail.8
1036 ↗(On Diff #46789)

I totally agree on that! I've updated the patch.

Looks good, approved for commit.

This revision is now accepted and ready to land.Aug 17 2018, 8:08 PM
This revision was automatically updated to reflect the committed changes.
bhd marked an inline comment as done.