[prev in list] [next in list] [prev in thread] [next in thread] 

List:       busybox
Subject:    Re: [PATCH 2/2] mount: -T OTHERTAB support
From:       Isaac Dunham <ibid.ag () gmail ! com>
Date:       2015-03-17 1:04:25
Message-ID: 20150317010424.GD13750 () newbook
[Download RAW message or body]

On Mon, Mar 16, 2015 at 10:08:37PM +0100, Natanael Copa wrote:
> On Sun, 15 Mar 2015 01:07:02 +0000
> Isaac Dunham <ibid.ag@gmail.com> wrote:
> 
> > Also, I'm thinking about adding "mount -m"; this is an extension over
> > util-linux mount, which would create the mountpoint if it doesn't
> > exist.
> > I've asked on the util-linux list about whether that option is spoken
> > for, and if there's any interest in seeing it in util-linux.

> > I note that the option x-mount.mkdir is equivalent to the option I've
> > been contemplating, and might be more useful...
> 
> FWIW. I find either mount -m or support for x-mount.mkdir useful. Alpine
> Linux runs with a tmpfs root so we have added a hack to create missing
> dirs:
> 
> http://git.alpinelinux.org/cgit/aports/tree/main/alpine-baselayout/mkmntdirs.c
> 
> and a patch for openrc:
> http://git.alpinelinux.org/cgit/aports/tree/main/openrc/openrc-0.4.3-mkmntdirs.patch
> 
> It would be nice to get rid of it.

Thanks for the comment.

I've asked Karel Zak (the util-linux maintainer) about his recommendation,
and he recommended implementing x-mount.mkdir, with the comment that if
setting uid/gid for the mountpoint is needed, it could be extended to
x-mount.mkdir=mode:uid:gid

As a mount option, it can be stored in an fstab equivalent file or added
on the commandline.

> Adding this feature to mdev.conf would not be satisfactory since we
> also have optional support for udev.

Ah yes. 
I'd rather see (any mount) + (udev|mdev) than 
(util-linux mount + udev) | (any mount + mdev), and even supporting the
latter could be difficult.

I've been experimenting on adding this to Toybox, since that's less of a
tangle for getting an idea how something works.

Once I test that some, I may start working on it for Busybox.

Thanks,
Isaac Dunham
_______________________________________________
busybox mailing list
busybox@busybox.net
http://lists.busybox.net/mailman/listinfo/busybox
[prev in list] [next in list] [prev in thread] [next in thread]