On Wed, 16 Jul 2008, Denys Vlasenko wrote:

> On Wednesday 16 July 2008 22:58, Robert P. J. Day wrote:
> > On Wed, 16 Jul 2008, Denys Vlasenko wrote:
> >
> > > On Wednesday 16 July 2008 20:37, Robert P. J. Day wrote:
> > > >
> > > >   from archival/cpio.c:
> > > >
> > > > ...
> > > > #if ENABLE_GETOPT_LONG && ENABLE_DESKTOP
> > > >     applet_long_options =
> > > >         "extract\0"      No_argument       "i"
> > > >         "list\0"         No_argument       "t"
> > > > #if ENABLE_FEATURE_CPIO_O
> > > >         "create\0"       No_argument       "o"
> > > >         "format\0"       Required_argument "H"
> > > > #endif
> > > >         ;
> > > > #endif
> > > > ...
> > > >
> > > >   why does defining those long options require the definition of
> > > > ENABLE_DESKTOP?  the "i" and "t" options will be defined regardless,
> > > > so shouldn't that snippet depend solely on ENABLE_GETOPT_LONG?
> > >
> > > I was lazy to add yet another option ENABLE_CPIO_GETOPT_LONG, as
> > > many other applets do.
> >
> > i don't see *any* other applet that adds that kind of applet-specific
> > option, but that's not the point i was making.
>
> # grep LONG .config
> CONFIG_GETOPT_LONG=y
> CONFIG_FEATURE_AR_LONG_FILENAMES=y
> CONFIG_FEATURE_TAR_LONG_OPTIONS=y
> CONFIG_FEATURE_ENV_LONG_OPTIONS=y
> CONFIG_FEATURE_EXPAND_LONG_OPTIONS=y
... snip ...

  ah, quite right, i was searching for the wrong regex, sorry.

rday
--

========================================================================
Robert P. J. Day
Linux Consulting, Training and Annoying Kernel Pedantry:
    Have classroom, will lecture.

http://crashcourse.ca                          Waterloo, Ontario, CANADA
========================================================================
_______________________________________________
busybox mailing list
[email protected]
http://busybox.net/cgi-bin/mailman/listinfo/busybox

Reply via email to