On Wed, Jun 17, 2009 at 11:12:37AM -0500, Shawn Walker wrote:

> http://cr.opensolaris.org/~swalker/README

line 37, et al: -r is redundant with -a, and -z is probably a waste of CPU
for this case.

You talk about putting these images on optical media, but never suggest
how.

line 90: <data_source> would be "/destination" from the examples in
sections 2.1 and 2.2?

line 90: You also suddenly start using pfexec here, where none of the
previous commands have it.  I'd skip it and put some text up top saying
that it should be used, or that the commands should be run as root.  I
think there's a decent chance that these instructions will be followed on a
machine that isn't running OpenSolaris and/or doesn't have pfexec set up
for the user.

line 101: You switch to cfg_cache modification here, in between a bunch of
svccfg invocations.  It might make sense to keep them all together.

line 115: why give the example in the text body *and* the example in the
offset text on line 117, *and* have them be different?

> http://cr.opensolaris.org/~swalker/cfg_cache

line 7: since the README primarily suggests setting the depot up as an
origin server, calling it a mirror here doesn't seem right.  Same on line
28, I guess.

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

Reply via email to