Re: [PATCH] Add mount -P to hide pseudo filesystems

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

 



On 6 December 2015 at 12:39, David Wilson <dw@xxxxxxxxxxxxx> wrote:
> On 6 December 2015 at 10:22, Sami Kerola <kerolasa@xxxxxx> wrote:
>> On 6 December 2015 at 04:29, David Wilson <dw@xxxxxxxxxxxxx> wrote:
>>> @@ -842,6 +848,7 @@ int main(int argc, char **argv)
>>>                 { "make-rprivate", 0, 0, MOUNT_OPT_RPRIVATE },
>>>                 { "make-runbindable", 0, 0, MOUNT_OPT_RUNBINDABLE },
>>>                 { "no-canonicalize", 0, 0, 'c' },
>>> +               { "no-pseudofs", 0, 0, 'P' },
>>
>> Hi David,
>>
>> I doubt this is common enough case to assign short option.  How about
>> long-only option, see MOUNT_OPT_RUNBINDABLE for an example how to make
>> that work.
>
> I agree, although I see little value in a patch if it becomes hard to
> type. At that point copying around a wrapper script named 'mountp' is
> probably easier. :)
>
> How about some shorter spelling for --no-pseudofs, say, --real or
> --plain, or maybe it just becomes neater to extend -t. How about
> spellings of -t like:
>
> "-t [no]pseudo" -> anon_inodefs,bdev,cgroup,..,tmpfs
> "-t [no]net" -> cifs,smbfs,...,9p
> "-t [no]plain" -> nopseudo,nonet

Hi David,

Maybe you can rely gnu getops doing the right thing with '--no-p' when
typing. In scripts I recommend to use the full option. Try the
following ls(1) examples to see how the getops allows truncating long
options. I think it is unfortunate option shortening is not widely
know.

ls --h
ls --he

-- 
Sami Kerola
http://www.iki.fi/kerolasa/
--
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