Hi all, I've generated a third version of the webrev for the fix for bug #3705 et al.
You can find it at: http://cr.opensolaris.org/~richb/pkg-3705-v3/ The only difference from the previous version is that the amp package in .../src/util/distro-import/101a/common/Amp has been adjusted to depend on the new versions of the two PHP packages rather than the old ones. Here's that simple change: $ diff -c Amp.orig Amp *** Amp.orig Mon Nov 17 09:18:28 2008 --- Amp Mon Nov 17 09:20:17 2008 *************** *** 7,13 **** depend SUNWapch22m-php5 depend SUNWapch22m-security depend SUNWmysql5 ! depend SUNWphp524 ! depend SUNWphp524-mysql ! depend SUNWphp524man end package --- 7,12 ---- depend SUNWapch22m-php5 depend SUNWapch22m-security depend SUNWmysql5 ! depend SUNWphp52 ! depend SUNWphp52-mysql end package Tested by installing the "amp" package from a local repository with: $ pfexec pkg set-authority -P -O http://stard.sfbay.sun.com:23705 bug-3705 $ pfexec pkg install amp PHASE ACTIONS Removal Phase 10/10 Install Phase 9/9 Update Phase 1/1 PHASE ITEMS Reading Existing Index 9/9 Indexing Packages 1/1 $ pfexec pkg set-authority -P opensolaris.org $ pfexec pkg unset-authority bug-3705 and then checking on the package dependencies with: $ pfexec pkg contents -mr amp | egrep depend depend [EMAIL PROTECTED] type=require depend [EMAIL PROTECTED] type=require depend [EMAIL PROTECTED] type=require depend [EMAIL PROTECTED] type=require depend [EMAIL PROTECTED] type=require depend [EMAIL PROTECTED] type=require depend [EMAIL PROTECTED] type=require depend [EMAIL PROTECTED] type=require Please review. Thanks. -------- Original Message -------- Subject: Re: [pkg-discuss] Code review request for opensolaris packaging bug #3705 Date: Sun, 16 Nov 2008 21:08:32 -0800 From: Rich Burridge <[EMAIL PROTECTED]> To: pkg discuss <[email protected]>, David Comay <[EMAIL PROTECTED]>, Bart Smaalders <[EMAIL PROTECTED]> Hi all, I've generated a new webrev for the fix for bug #3705 et al. You can find it at: http://cr.opensolaris.org/~richb/pkg-3705-v2/ After discussions with David and Bart, it has the following additional changes over those in v1: * the depend lines in .../distro-import/101a/common/StorageNAS and .../distro-import/101a/common/TrustedExtensions have been sorted. * The amp and gcc-dev packages that were in .../distro-import/unbundleds/Clusters have been moved to new files called Amp and GccDev under .../distro-import/101a/common. Please review. Thanks. -------- Original Message -------- Subject: [pkg-discuss] Code review request for opensolaris packaging bug #3705 Date: Thu, 13 Nov 2008 14:22:52 -0800 From: Rich Burridge <[EMAIL PROTECTED]> To: pkg discuss <[email protected]> Hi all, I'm looking for a review of the fix for: http://defect.opensolaris.org/bz/show_bug.cgi?id=3705 Define how to specify specific versions for package cluster dependencies. which will also fix: http://defect.opensolaris.org/bz/show_bug.cgi?id=3363 Creation of a NAS Cluster http://defect.opensolaris.org/bz/show_bug.cgi?id=3364 Creation of Storage Server Cluster http://defect.opensolaris.org/bz/show_bug.cgi?id=3548 Add meta package for Trusted Extension http://defect.opensolaris.org/bz/show_bug.cgi?id=4214 language support cluster update as of 2008.11 snv101 Webrev is at: http://cr.opensolaris.org/~richb/pkg-3705-v1/ In particular, I'd like David, Bart and Danek to look it over. After discussing this with Bart and David this morning, we determined that the package clusters currently defined in: .../gate/src/util/distro-import/unbundleds/LangSupport (bug #4214) .../gate/src/util/distro-import/unbundleds/StorageNAS (bug #3363) and the new ones planned: .../gate/src/util/distro-import/unbundleds/Trusted (bug #3548) .../gate/src/util/distro-import/unbundleds/StorageServer (bug #3364) need to be moved to .../gate/src/util/distro-import/101a/common and include lines for them added to redist_cluster. I created a local repository on my OpenSolaris 2008.11 (build 101a) machine with a complete redist_cluster in it. People internal to Sun can view it at: http://stard.sfbay.sun.com:23705 I've also put a copy of errs.import generated when the redist_cluster was created, in /home/richb/pkg/bugs/3705 Thanks. _______________________________________________ pkg-discuss mailing list [email protected] http://mail.opensolaris.org/mailman/listinfo/pkg-discuss
