Re: [PATCH] mkfs.minix: remove die()

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

 



On Tue, Apr 12, 2011 at 12:29:07PM +0200, Sami Kerola wrote:
> On Tue, Apr 12, 2011 at 12:08, Karel Zak <kzak@xxxxxxxxxx> wrote:
> >> - Â Â die(_("%s is mounted; will not make a filesystem here!"));
> >> + Â Â errx(8, _("%s is mounted; will not make a filesystem here!"), device_name);
> >
> > ÂPlease, define and use EXIT_SOMETHING instead of the magic number '8'.
> 
> The disc-utils/mkfs.cramfs.c has
> 
> #define MKFS_OK          0     /* No errors */
> #define MKFS_ERROR       8     /* Operational error */
> #define MKFS_USAGE       16    /* Usage or syntax error */
> 
> and the return value 8 seems to be somehow special for mkfs* I wonder
> if the definition of return values should be in mkfs.h

 Good point, disk-utils/mkfs.h makes sense.

    Karel

-- 
 Karel Zak  <kzak@xxxxxxxxxx>
 http://karelzak.blogspot.com
--
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