On Wed 29 Apr 2009 at 12:09PM, Edward Pilatowicz wrote:
> hey dan,
> this looks good.
> i have a few general comments/questions.
> 
> - in common.ksh, when you run "pkg list" you set LC_ALL=C, but you don't
>   do this in other places when you run "pkg list"?

Fixed.

> - seems like there should be an RFE for "pkg list -o XXX"?

I thought there was.  If not I will file one.

> - should there be an option to install so that it looks at datestamps
>   when updating packages?  (so you don't need the hack in attach.)

There should be an option to image-update allowing you to impose
contraints-- or, we need freezes, or we need the SAT solver.
Essentially, we've bumped up against the boundary of what we can
do with what we have.

> - can package names have spaces in them?

Package names cannot have spaces.  We can certainly change -H to switch
to a more parseable mode.

g_valid_pkg_name = \
    
re.compile("^[A-Za-z0-9][A-Za-z0-9_\-\.\+]*(/[A-Za-z0-9][A-Za-z0-9_\-\.\+]*)*$")

        -dp

-- 
Daniel Price, Solaris Kernel Engineering    http://blogs.sun.com/dp
_______________________________________________
pkg-discuss mailing list
[email protected]
http://mail.opensolaris.org/mailman/listinfo/pkg-discuss

Reply via email to