On Mon 08 Sep 2008 at 03:28PM, Shawn Walker wrote:
> Greetings,
> 
> The following webrev contains fixes for the following issues:
>    3272 info and manifest links generated incorrectly by face.py
> 
> webrev:
> http://cr.opensolaris.org/~swalker/pkg-3272/
> 

This might be better stated as:  pfmri[len('pkg:/'):]

Or, add a comment.  The naked '5' will be hard to understand for a
future maintainer.

        -dp

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

Reply via email to