|
|
RE: Passing parameters to fs-dos via automount
|
|
11/26/2009 12:39 PM
post42687
|
RE: Passing parameters to fs-dos via automount
Thanks Steve & John for taking a look at this.
>The other route is to remove the comma parsing from the @file case
>(which is there solely for symmetry with the command-line behaviour)
>and switch the file to being one-entry-per-line. As you mentioned
>source, remove the strchr(cp1, ',') bit from ParseAutomountFile().
>We will consider doing this also (weighing compatability issues).
Steve mentioned this as well, and actually, I didn't see the part about
commas being separators for new entries and so initially thought that
newlines were the only delimiter. I said to myself "ah ha!" and
subsequently refused to read the rest of the line....
So...yes, I think it would be great if newlines were the only delimiter. I
think it makes sense that you could only use a newline to specify a new
entry. However, I see the problems that it could cause as well with folks
that have used commas and who's systems will subsequently break if they
upgrade.
Thanks again for considering this.
Kevin
-----Original Message-----
From: John Garvey [mailto:community-noreply@qnx.com]
Sent: November 25, 2009 09:56
To: general-filesystems
Subject: Re: Passing parameters to fs-dos via automount
Kevin Stallard wrote:
> But when I try this, it acts like it will only take one option, it seems
that commas used to separate multiple options to fs-* are used as
delimiters for higher level parsing.
Yes, sadly the getsubopt() parsing has already read ahead and tokenised
at the comma. You could provide your multiple options as generic 'dos'
defaults, eg "blk automount=+usbhd0t12:/USB:dos dos gid=90,uid=100", if
it is acceptable to apply them to all the dos automounts.
Steve will arrange to adjust the docs to refer to 'option' singular.
The other route is to remove the comma parsing from the @file case
(which is there solely for symmetry with the command-line behaviour)
and switch the file to being one-entry-per-line. As you mentioned
source, remove the strchr(cp1, ',') bit from ParseAutomountFile().
We will consider doing this also (weighing compatability issues).
_______________________________________________
General
http://community.qnx.com/sf/go/post42578
|
|
|
|
|