Page MenuHomeFreeBSD

bsdinstall: add pkgbase component selection dialog
Needs ReviewPublic

Authored by ifreund_freebsdfoundation.org on Mon, May 5, 8:49 PM.
Tags
None
Referenced Files
F117621369: D50178.diff
Tue, May 20, 11:55 PM
F117573160: D50178.id155432.diff
Tue, May 20, 7:09 AM
Unknown Object (File)
Mon, May 19, 11:55 AM
Unknown Object (File)
Mon, May 12, 2:58 PM
Unknown Object (File)
Mon, May 12, 1:17 PM
Unknown Object (File)
Mon, May 12, 12:40 PM
Unknown Object (File)
Sun, May 11, 10:14 AM
Unknown Object (File)
Thu, May 8, 7:00 PM

Details

Reviewers
emaste
kevans
Group Reviewers
pkgbase
Summary

This matches the style of the component selection dialog for traditional
tarball-based installations. The only difference is that there is
currently no ports component offered when using pkgbase.

Sponsored by: The FreeBSD Foundation

Diff Detail

Repository
rG FreeBSD src repository
Lint
Lint Skipped
Unit
Tests Skipped
Build Status
Buildable 63934
Build 60818: arc lint + arc unit

Event Timeline

Extract component selection to a separate function

After a nights sleep, I realized I had written an excessively long function

usr.sbin/bsdinstall/scripts/pkgbase.in
44

Can we add a comment with a brief description for each of these functions?

76

These should probably be "Kernel Debug Data" or so. You've followed what the dist sets do today so there's an argument for consistency, but I think that's a bug to be fixed in the dist sets (and can pick that up). Kernel (Debugging) could suggest this is a debugging kernel, an alternative to a Kernel (Standard).

I've submitted D50254 to change this for dist sets.

usr.sbin/bsdinstall/scripts/pkgbase.in
76

Indeed, this is a good opportunity to improve this terminology. I left a comment with a minor bikeshed on https://reviews.freebsd.org/D50254 and will copy whatever terminology gets decided upon there for this patch.

Hide components with no available packages in the menu