On Tue, Nov 17, 2015 at 09:36:26AM +0100, Jan Stary wrote: > On Nov 14 14:17:08, k...@openbsd.org wrote: > > Modified files: > > sbin/fdisk : disk.c fdisk.c > > > > Log message: > > Since -l is ignored if -c/-h/-s are specified, make that combination > > invalid and mention the constraint in usage(). > > Mention it in the manpage too. > > Jan > >
i agree it makes sense to document it. but there is already similar text there for -u and -i, and this makes it a little consistent. i propose using the same text. are you happy with this? jmc Index: fdisk.8 =================================================================== RCS file: /cvs/src/sbin/fdisk/fdisk.8,v retrieving revision 1.87 diff -u -r1.87 fdisk.8 --- fdisk.8 26 Oct 2015 18:05:25 -0000 1.87 +++ fdisk.8 17 Nov 2015 17:19:53 -0000 @@ -79,6 +79,12 @@ or values that .Em /boot has passed to the kernel. +.Pp +Only one of +.Fl chs +or +.Fl l +can be specified. .It Fl e Use the .Nm @@ -127,6 +133,12 @@ .It Fl l Ar blocks Specify the number of blocks in the disk, and force the MBR to be in LBA mode only. +.Pp +Only one of +.Fl chs +or +.Fl l +can be specified. .It Fl u Update MBR bootcode, preserving existing MBR partition table. The MBR bootcode extends from offset 0x000 to the start of the MBR partition