Re: [PATCH] sfdisk: suppress Linux-irrelevant warnings with -L

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



On Monday 26 May 2014, Petr Uzel wrote:
> Sfdisk prints out a warning about extended partition not
> starting at a cylinder boundary. Since this is irrelevant
> for linux, the -L option should suppress this warning.

I would really like to have -L enabled by default on linux. Project name
is util-LINUX and if we even run it on linux, why being still bothered 
per default?

BTW we have some more warnings of this kind in libfdisk/src/dos.c
where we don't have option -L:

/* Same physical / logical beginning? */
if (cxt->geom.cylinders <= 1024
    && (pbc != lbc || pbh != lbh || pbs != lbs)) {
       fdisk_warnx(cxt, _("Partition %zu: different physical/logical "
               "beginnings (non-Linux?): "
               "phys=(%d, %d, %d), logical=(%d, %d, %d)"),
                partition + 1,
                pbc, pbh, pbs,
                lbc, lbh, lbs);
}


> Signed-off-by: Petr Uzel <petr.uzel@xxxxxxx>
> ---
>  disk-utils/sfdisk.c | 2 +-
>  1 file changed, 1 insertion(+), 1 deletion(-)
>
> diff --git a/disk-utils/sfdisk.c b/disk-utils/sfdisk.c
> index 3c55d5e..12162af 100644
> --- a/disk-utils/sfdisk.c
> +++ b/disk-utils/sfdisk.c
> @@ -1383,7 +1383,7 @@ extended_partition(char *dev, int fd, struct
> part_desc *ep, struct disk_desc *z) "from %lld to %lld\n"
>  		      "(For listing purposes only. "
>  		      "Do not change its contents.)"), ep->start, start);
> -	} else {
> +	} else if (!Linux) {
>  	    warnx(_("Warning: extended partition does not start at a "
>  		      "cylinder boundary.\n"
>
--
To unsubscribe from this list: send the line "unsubscribe util-linux" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at  http://vger.kernel.org/majordomo-info.html




[Index of Archives]     [Netdev]     [Ethernet Bridging]     [Linux Wireless]     [Kernel Newbies]     [Security]     [Linux for Hams]     [Netfilter]     [Bugtraq]     [Yosemite News]     [MIPS Linux]     [ARM Linux]     [Linux RAID]     [Linux Admin]     [Samba]

  Powered by Linux