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

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

 



On Tue, 2011-04-12 at 13:05 +0200, Karel Zak wrote:
> 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
> 
Oh, so I see these codes are kind of standard with mkfs programs... will
do, thanks for pointing this out.


--
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