Thanks for the preliminary review.  I'm now ready with the full review.

Please review my the complete changes for:
12347 new dependencies required for developer/opensolaris/osnet
  http://defect.opensolaris.org/bz/show_bug.cgi?id=12347
12463 need consolidation incorporations
  http://defect.opensolaris.org/bz/show_bug.cgi?id=12463

http://cr.opensolaris.org/~lianep/pkg-clone/

I've chosen to leave the 'entire' name alone for now, with the anticipation of changing it when we re-factor it as an incorporation which nests all of the consolidation incorporations. I did include consolidation package properties, though. (Hopefully I added to the namespace correctly -- I was unable to find a direct reference in the manpages or docs on how to use that namespace.)

It's a large changeset, but there are only a few really critical files. (Note that this is already based off David's 127 changes.) You can search directly for
  importer.py
  osnet-dev

Attached to the bug report is the list I used to script the consolidation additions to the importer files. You can easily choose to review that list rather than all of the individual package changes. That list comes from the consolidation names during consolidation delivery to Release Engineering. I'm hesitant to diverge the names at this point -- consolidations can do so when they're delivering IPS directly.

Thanks for your patience with this large, but hopefully straightforward review.

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

Reply via email to