Shawn Walker wrote:

> So just to clarify, the pkg.5 man page is wrong about the group-list
> prop then?
> 
> http://defect.opensolaris.org/bz/show_bug.cgi?id=12809
> 
> ...just asking (not this wad).

pkg(5) and pkgsend(1) are both ambiguous and uninformative, at best.
Frankly, group-list should either be a comma-separated list or the
attribute should be renamed to indicate that it's really just a single
value.

> src/modules/actions/group.py:
>   lines 92-93: why not cur_attrs.setdefault("gid", "") instead?

The test is against self.attrs, not cur_attrs, but the change is to
cur_attrs, not self.attrs.

>   line 99: so continuations of continuations can get their own
> additional 4 space indent?

Er, yeah, I guess.  :)  Seemed easier to read that way.

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

Reply via email to