On Thu, Mar 30, 2017 at 07:21:22AM -0400, Jiri B wrote:
> On Thu, Mar 30, 2017 at 12:59:00PM +0200, Bruno Flueckiger wrote:
> > I see your point with the installer. Default labels make the disks
> > indistinguishable. The following diff prints the raw infos from dmesg
> > rather than the current list of disks:
> >
> > Index: distrib/miniroot/install.sub
> > ===================================================================
> > RCS file: /cvs/src/distrib/miniroot/install.sub,v
> > retrieving revision 1.988
> > diff -u -p -r1.988 install.sub
> > --- distrib/miniroot/install.sub 13 Mar 2017 17:08:31 -0000 1.988
> > +++ distrib/miniroot/install.sub 30 Mar 2017 10:44:01 -0000
> > @@ -264,13 +264,7 @@ diskinfo() {
> > local _d
> >
> > for _d; do
> > - make_dev $_d
> > - echo -n "$_d: "
> > - disklabel -dpg $_d 2>/dev/null |
> > - sed -e '/^label: /{s,,,;s/ *$//;s/^$/<no label>/;h;d;}' \
> > - -e '/.*# total bytes: \(.*\)/{s//(\1)/;H;}' \
> > - -e '$!d;x;s/\n/ /'
> > - rm -f /dev/{r,}$_d?
> > + sed -n "/^$_d/p" /var/run/dmesg.boot
> > done
> > }
> >
>
> Your proposition is good for the installer? I doubt it.
>
> j.
AFAICT the function diskinfo() is only called once in the installer: if
you press ? a the prompt for the root disk. So my diff just changes the
output in this case, no other functionality is affected.
What causes your doubt?