On 05/ 4/11 07:01 PM, Shawn Walker wrote:
On 05/ 4/11 10:46 AM, Glenn Lagasse wrote:
Hi,

I'd like to get a code review for a very simple fix to the zone install
script.  With the integration of 7034756 we should be installing beadm
inside of non-global zones at installation time.

src/brand/pkgcreatezone:

diff -r 39ae727e734e src/brand/pkgcreatezone
--- a/src/brand/pkgcreatezone   Wed May 04 16:32:30 2011 +0100
+++ b/src/brand/pkgcreatezone   Wed May 04 10:40:34 2011 -0700
@@ -502,7 +502,8 @@
         pkg:///compress/gzip
         pkg:///compress/zip
         pkg:///compress/unzip
-       pkg:///package/pkg"
+       pkg:///package/pkg
+       pkg:///install/beadm"

  #
  # Get some diagnostic tools, truss, dtrace, etc.

Out of curiosity, do you know why this list of packages for zones is handled through the brand scripts instead of package dependencies tagged with a zone variant?

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

Looks good to me.

Shawn: Your questions are being addressed by the zones team under a separate CR (7032753). The list of packages will be replaced with a single, well-factored group package that correctly expresses such dependency and variant requirements.

Cheers,

Gary


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

Reply via email to