Re: PR/51230 CVS commit: src/sbin/gpt

[christos%zoulas.com@localhost (Christos Zoulas)]

The following reply was made to PR bin/51230; it has been noted by GNATS.

From: christos%zoulas.com@localhost (Christos Zoulas)
To: gnats-bugs%NetBSD.org@localhost, gnats-admin%netbsd.org@localhost, netbsd-bugs%netbsd.org@localhost, 
	Hauke Fath <hf%spg.tu-darmstadt.de@localhost>
Cc: 
Subject: Re: PR/51230 CVS commit: src/sbin/gpt
Date: Thu, 9 Jun 2016 14:45:21 -0400

 On Jun 9,  5:25pm, kre%munnari.OZ.AU@localhost (Robert Elz) wrote:
 -- Subject: Re: PR/51230 CVS commit: src/sbin/gpt
 
 | The following reply was made to PR bin/51230; it has been noted by GNATS.
 | 
 | From: Robert Elz <kre%munnari.OZ.AU@localhost>
 | To: gnats-bugs%NetBSD.org@localhost
 | Cc: 
 | Subject: Re: PR/51230 CVS commit: src/sbin/gpt
 | Date: Fri, 10 Jun 2016 00:24:06 +0700
 | 
 |      Date:        Thu,  9 Jun 2016 15:35:02 +0000 (UTC)
 |      From:        Hauke Fath <hf%spg.tu-darmstadt.de@localhost>
 |      Message-ID:  <20160609153502.4E35C7AAB7%mollari.NetBSD.org@localhost>
 |  
 |    |  Thanks! But...
 |    |  
 |    |  # bin/gpt-current sd0 biosboot -a -i1
 |    |  gpt-current: unknown option -- a
 |    |  Usage: gpt-current <device> biosboot [-c bootcode] [-i index] [-L label]
 |    |  #  
 |  
 |  Believe it or not, the correct usage is still "gpt biosboot [opts] device
 |  The usage message prints it in the order you show whenver the command
 |  name is not "gpt" (which it isn't for you).   I have no idea why, I see
 |  no code to support the devide first ordering (whatever the command name is)
 |  
 |  I suspect it is an artifact of that you're seeing (though I get a
 |  different message when I try it that way.)
 |  
 |    |  and gpt.8 renders to
 |    |  
 |    |       gpt biosboot [-ac bootcode] [-i index] [-L label]
 |  
 |  Fixed.
 |  
 |  ALso, thanks for tracking down this problem, I noticed it back in Dec 2013,
 |  and worked out the fdisk (to put a regular MBR there) followed by gpt
 |  create (to convert that to a PMBR) solved the problem, and had worked out
 |  that gpt create on an uninit'd drive was not setting something properly,
 |  but I never got as far as working out what that was.
 |  
 |  I think though that Christos new -a flag should be changed (probably to -A)
 |  (througout) to avoid clashes with the "-a alignment" option that is
 |  used everywhere else (even the biosboot usage has a "[-a alignment]"
 |  option already in it .. just #ifdef'd out, the new -a would prevent that.)
 
 Good idea.
 
 christos