Date: Wed, 15 Jan 2020 19:57:28 +0100 From: Oliver Pinter <oliver.pntr@gmail.com> To: Ben Woods <woodsb02@freebsd.org> Cc: "src-committers@freebsd.org" <src-committers@freebsd.org>, "svn-src-all@freebsd.org" <svn-src-all@freebsd.org>, "svn-src-head@freebsd.org" <svn-src-head@freebsd.org>, Ed Maste <emaste@freebsd.org> Subject: Re: svn commit: r356758 - in head/usr.sbin/bsdinstall: . scripts Message-ID: <CAPjTQNHa9Um%2B0S8zMD3eeop3YSdVti0o=DSv5%2BB=455wcrq3VA@mail.gmail.com> In-Reply-To: <202001150747.00F7lqiG071097@repo.freebsd.org> References: <202001150747.00F7lqiG071097@repo.freebsd.org>
next in thread | previous in thread | raw e-mail | index | archive | help
On Wednesday, January 15, 2020, Ben Woods <woodsb02@freebsd.org> wrote: > Author: woodsb02 (ports committer) > Date: Wed Jan 15 07:47:52 2020 > New Revision: 356758 > URL: https://svnweb.freebsd.org/changeset/base/356758 > > Log: > bsdinstall: Change "default" (first) Partitioning method to ZFS > > Reported by: Ruben Schade (during his talk at linux.conf.au) > Approved by: philip > Differential Revision: https://reviews.freebsd.org/D23173 What's the justification behind this change? Plus I miss from here the relontes tag. > > Modified: > head/usr.sbin/bsdinstall/bsdinstall.8 > head/usr.sbin/bsdinstall/scripts/auto > > Modified: head/usr.sbin/bsdinstall/bsdinstall.8 > ============================================================ > ================== > --- head/usr.sbin/bsdinstall/bsdinstall.8 Wed Jan 15 06:18:32 2020 > (r356757) > +++ head/usr.sbin/bsdinstall/bsdinstall.8 Wed Jan 15 07:47:52 2020 > (r356758) > @@ -119,7 +119,7 @@ Provides the installer's interactive guided disk parti > installations. > Defaults to UFS. > .It Cm zfsboot > -Provides an alternative ZFS-only automatic interactive disk partitioner. > +Provides a ZFS-only automatic interactive disk partitioner. > Creates a single > .Ic zpool > with separate datasets for > > Modified: head/usr.sbin/bsdinstall/scripts/auto > ============================================================ > ================== > --- head/usr.sbin/bsdinstall/scripts/auto Wed Jan 15 06:18:32 2020 > (r356757) > +++ head/usr.sbin/bsdinstall/scripts/auto Wed Jan 15 07:47:52 2020 > (r356758) > @@ -289,7 +289,7 @@ Shell \"Open a shell and partition by hand\"" > CURARCH=$( uname -m ) > case $CURARCH in > amd64|arm64|i386) # Booting ZFS Supported > - PMODES="$PMODES \"Auto (ZFS)\" \"Guided Root-on-ZFS\"" > + PMODES="\"Auto (ZFS)\" \"Guided Root-on-ZFS\" $PMODES" > ;; > *) # Booting ZFS Unspported > ;; > @@ -303,6 +303,10 @@ PARTMODE=`echo $PMODES | xargs dialog --backtitle "Fre > exec 3>&- > > case "$PARTMODE" in > +"Auto (ZFS)") # ZFS > + bsdinstall zfsboot || error "ZFS setup failed" > + bsdinstall mount || error "Failed to mount filesystem" > + ;; > "Auto (UFS)") # Guided > bsdinstall autopart || error "Partitioning error" > bsdinstall mount || error "Failed to mount filesystem" > @@ -319,10 +323,6 @@ case "$PARTMODE" in > else > bsdinstall partedit || error "Partitioning error" > fi > - bsdinstall mount || error "Failed to mount filesystem" > - ;; > -"Auto (ZFS)") # ZFS > - bsdinstall zfsboot || error "ZFS setup failed" > bsdinstall mount || error "Failed to mount filesystem" > ;; > *) > _______________________________________________ > svn-src-head@freebsd.org mailing list > https://lists.freebsd.org/mailman/listinfo/svn-src-head > To unsubscribe, send any mail to "svn-src-head-unsubscribe@freebsd.org" >
Want to link to this message? Use this URL: <https://mail-archive.FreeBSD.org/cgi/mid.cgi?CAPjTQNHa9Um%2B0S8zMD3eeop3YSdVti0o=DSv5%2BB=455wcrq3VA>