[BusyBox] [PATCH] fix compilation of libbb/mtab_file.c

[Arthur Othieno]

On Fri, Jul 22, 2005 at 07:49:34PM -0500, Rob Landley wrote:
> On Friday 22 July 2005 18:21, Arthur Othieno wrote:
> 
> > Hey, revision 10556 removed CONFIG_FEATURE_MTAB_FILENAME without _no_
> > mention of it. The log in question reads:
> >
> >   Rodney Radford submitted ipcs and ipcrm (system V IPC stuff).  They could
> > use some more work to shrink them down.
> >
> > Re-defining it was only out of the impression that it was removed
> > by accident.
> 
> As I mentioned in my next post: yeah, it was.  This is part of my mount 
> rewrite that leaked in by accident when I marshalled a patch through my 
> development tree.  (The mount side is more or less finished, but the umount 
> side still has a couple tangles I haven't managed to get to yet.)
 
Ok, just saw it now. Seems there's a (noticeable) lag between when messages
are replied to to when they are eventually posted to the list...

> > Now, after sifting through <busybox-cvs at busybox.net> archives,
> > I gather the original intent was to do away with this entirely:
> > http://www.busybox.net/lists/busybox-cvs/2005-May/005736.html
> >
> > In that case, this patch is more correct:
> >
> >
> > Index: libbb/mtab_file.c
> > ===================================================================
> > --- libbb/mtab_file.c (revision 10900)
> > +++ libbb/mtab_file.c (working copy)
> > @@ -26,7 +26,7 @@
> >  /* Busybox mount uses either /proc/mounts or /etc/mtab to
> >   * get the list of currently mounted filesystems */
> >  #if defined CONFIG_FEATURE_MTAB_SUPPORT
> > -const char bb_path_mtab_file[] = CONFIG_FEATURE_MTAB_FILENAME;
> > +const char bb_path_mtab_file[] = "/etc/mtab";
> >  #else
> >  const char bb_path_mtab_file[] = "/proc/mounts";
> >  #endif
> 
> Sounds good.  I'll add it to my queue...
> 
> Rob

FYI, also posted to bug #0000322 here:
http://busybox.net/bugs/view.php?id=322

	Arthur