Danek Duvall wrote:
> On Sun, Oct 12, 2008 at 07:57:44PM -0500, Shawn Walker wrote:
> 
>> Danek Duvall wrote:
>>> On Thu, Oct 09, 2008 at 06:55:34PM -0500, Shawn Walker wrote:
>>>   - line 213: what's "and arg" doing?
>> It ignores the --content-root parameter unless a value is specified, as per 
>> the man page.
> 
> So you mean that
> 
>     pkg.depotd --content-root
> 
> and
> 
>     pkg.depotd --content-root foo
> 
> are both legal commandlines?  That's syntactically ambiguous and we can't
> allow it.  Either the option takes an argument or it doesn't; it can't be
> optional.

--proxy-base already works that way because of SMF's annoying built-in 
limitations for options.

I don't consider it to be syntactically ambiguous, but however.

In this case, since I can put a sane default in the SMF xml, I'm willing 
to change it to not permit "" as an argument value.

But for --proxy-base, that's a no-go until we redo our SMF option parsing.

Cheers,
-- 
Shawn Walker
_______________________________________________
pkg-discuss mailing list
[email protected]
http://mail.opensolaris.org/mailman/listinfo/pkg-discuss

Reply via email to