Re: [PATCH 4/5] Removed the ``add'' and ``remove'' options

[olafBuddenhagen]

Hi,

On Fri, May 29, 2009 at 12:08:06AM +0300, Sergiu Ivanov wrote:

> -    case OPT_ADD:            /* --add */
> -      ulfs_mode = ULFS_MODE_ADD;
> -      break;
> -
> -    case OPT_REMOVE:         /* --remove  */
> -      ulfs_mode = ULFS_MODE_REMOVE;
> -      break;
> -

You removed the options to actually switch the mode, but the handling of
the mode is still there, as dead code... You did remove it in the next
patch, but it really belongs here.

I think in a way this illustrates a more fundamental issue though: your
command line handling changes generally seem too cosmetic. They still
leave a lot of stuff that doesn't really make sense for unionmount: stow
can't work; priorities are pointless; both the underlying directory tree
and the mountee should be writable by default, and I'm not sure it's
useful to allow overriding them at all.

Basic unionmount really just needs a very simple command line: an option
to decide whether the mountee goes on top or below the underlying tree;
and the list of argements describing the translator to mount.

-antrik-