Shawn,

This is a great change - we've been waiting for this for some time.

Please consider these comments.

1. I'm wondering if pkgrecv is an appropriate name for this anymore. Maybe pkgcopy might be better.

2. pkgrecv.1, lines 49 and 50 should be interchanged.

3. ditto on Stephen's comment about the env vars.

4. Please state in the manual page that a file:/// URI is allowed for the source repo, or at least show one example using it that way.
t_pkgrecv.py:

5. line 373: "receiveing"

Thanks.
Tom


Shawn Walker wrote:
Greetings,

The following webrev contains changes for the following bugs and RFEs:

  6342 pkgrecv should support local disk repository
  7712 pkgrecv stack traces when (very) incomplete FMRI specified
  8109 status during package download can exceed 80 characters
  8896 traceback when opening server manifest using CachedManifest
8897 pkg.publish.transaction close should not fail with 500 if depot close fails
  8898 pkg.publish.transaction open timestamp support desired

webrev:
http://cr.opensolaris.org/~swalker/pkg-6342/

Change Summary:
* changed publication to allow specifying a timestamp for open, allowing republishing with exact fmris

* fixed several tracebacks in pkgrecv when bad input was provided

* added ability to use file:// repositories as source repository to pkgrecv

* added wildcard pattern matching to pkgrecv

* added version and timestamp matching control to pkgrecv

* fixed several publication bugs and tracebacks

* added PKG_DEST, and PKG_SRC_REPO environment variables to pkgrecv

* added ability to specify a repository as the destination for a receive (republishes the package with exact same frmi, etc.)

* added recursive dependency retrieval

* added extensive set of unit tests for pkgrecv

At the moment, these changes are for this release, pending re-approval by sch.

Cheers,

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

Reply via email to