On 05/ 4/11 11:06 AM, Gary Pennington wrote:
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.

Cool.  That's what I wanted to hear.

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

Reply via email to