Shawn Walker wrote:
Greetings,

The following webrev contains fixes for the following issues:

   2691 ability to publish packages to local disk repository
   850 pkgsend traceback if invalid or unavailable repository url specified
   894 pkgsend should do more action verification
   1236 pkgsend open should not allow both -e and -n
   2430 pkgsend add of non-existent file causes traceback
   5088 server can still create a repo when in --readonly mode
   5294 pkgsend doesn't provide server response when operations fail
   5944 pkgsend traceback if invalid or no action type specified

Updated webrev:
http://cr.opensolaris.org/~swalker/pkg-2691-2/

diff from last webrev:
http://cr.opensolaris.org/~swalker/pkg-2691-2/v1-v2.patch

Change Summary:
* re-pylint'd and fixed all changed files

* dropped auto_create variable in src/depot.py

* fixed server/catalog to fork based on configuration information instead of executable name

* added __str__ that raises NotImplementedError if ActionError() raised and triggered

* dropped +1 on position in action errors

* moved sha hash calculation (except for gzip case) to a common function in misc.get_data_digest

* changed all code importing stringio to import cStringIO

* changed all code using seek() to use os.SEEK constants

* renamed FileActionCreationError to ActionDataError

* reworked pkg.size calculation in actions/generic into try/except structure

* reworked license file checks to try/except catch for errors

* renamed TransactionX classes to XTransaction.

* changed server/catalog to catch importerror for cherrypy and ignore to allow clients to use direct publishing facilities without depot server dependencies

* added fork_allowed parameter to SvrConfig to control whether or not catalog is allowed to fork for search indices updating instead of relying on executable name

* fixed server/catalog to use self.__log() method instead of emsg to print messages directly

* reworked repository directory creation logic to use try/except structure

* fixed depot to respond with 304 in incremental catalog updates case where no updates are available

* added back an erroneously removed comment before publish_package is called in server/transaction.py

* fixed distro-import/Makefile to setup proto area and pass new -p parameter to solaris.py

* added -p option to solaris.py to specify proto area

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

Reply via email to