Shouldn't "include common/osnet-dev" also be removed from redist_cluster?
Yup, thanks for catching that. I've removed the include.
Everything else I saw looked fine to me, though even I didn't check to be sure you got every single X package name.
I verified it against the versions you had in the webrev. However I did noticed that there are some additional packages that for completeness should be listed in the slim_install definition so I've done that (these are all dependencies for other packages currently part of the group package). The diff for that changes relative to the earlier slim_cluster update is included below. Thanks for the review. --- - Wed Jul 14 21:15:32 2010 +++ slim_cluster Wed Jul 14 21:11:32 2010 @@ -471,6 +471,7 @@ depend SUNWcsd depend archiver/gnu-tar depend audio/audio-utilities +depend benchmark/x11perf depend codec/flac depend codec/install/codeina depend codec/libtheora @@ -479,6 +480,7 @@ depend communication/im/pidgin depend communication/pda/gnome-pilot depend communication/pda/pilot-link +depend compatibility/packages/SUNWxwplt depend compress/bzip2 depend compress/gzip depend compress/p7zip @@ -507,6 +509,7 @@ depend desktop/remote-desktop/tigervnc depend desktop/system-monitor/gnome-system-monitor depend desktop/time-slider +depend desktop/window-manager/twm depend desktop/xdg/menu-editor/alacarte depend desktop/xdg/xdg-user-dirs-gtk depend desktop/xdg/xdg-utils @@ -1223,8 +1226,10 @@ depend web/curl depend web/data/firefox-bookmarks depend web/wget +depend x11/colormap-utilities depend x11/compatibility/links-svid depend x11/compatibility/links-xorg +depend x11/data/workspace-patterns depend x11/data/xbitmaps depend x11/data/xcursor-themes depend x11/demo @@ -1234,9 +1239,12 @@ depend x11/diagnostic/xscope depend x11/documentation/xorg-docs depend x11/font-utilities +depend x11/header/x11-protocols depend x11/keyboard/data-xkb depend x11/keyboard/xkb-utilities depend x11/keyboard/xkbcomp +depend x11/library/dps +depend x11/library/libdga depend x11/library/libdrm depend x11/library/libfontenc depend x11/library/libfs @@ -1258,18 +1266,24 @@ depend x11/library/libxkbfile depend x11/library/libxmu depend x11/library/libxmuu +depend x11/library/libxp depend x11/library/libxpm depend x11/library/libxrandr depend x11/library/libxrender depend x11/library/libxres depend x11/library/libxscrnsaver +depend x11/library/libxtst depend x11/library/libxv depend x11/library/libxvmc depend x11/library/libxxf86misc depend x11/library/libxxf86vm depend x11/library/mesa +depend x11/library/toolkit/libxaw4 +depend x11/library/toolkit/libxaw5 depend x11/library/toolkit/libxaw7 depend x11/library/toolkit/libxt +depend x11/library/xtrans +depend x11/network/rstart depend x11/server/xephyr depend x11/server/xorg $(sparc_ONLY)depend x11/server/xorg/driver/xorg-efb @@ -1280,17 +1294,52 @@ $(i386_ONLY)depend x11/server/xorg/driver/xorg-input-vmmouse $(sparc_ONLY)depend x11/server/xorg/driver/xorg-kfb depend x11/server/xorg/driver/xorg-video +$(i386_ONLY)depend x11/server/xorg/driver/xorg-video-apm +$(i386_ONLY)depend x11/server/xorg/driver/xorg-video-ark +depend x11/server/xorg/driver/xorg-video-ast +$(i386_ONLY)depend x11/server/xorg/driver/xorg-video-ati +$(i386_ONLY)depend x11/server/xorg/driver/xorg-video-chips +$(i386_ONLY)depend x11/server/xorg/driver/xorg-video-cirrus +$(i386_ONLY)depend x11/server/xorg/driver/xorg-video-glint +$(i386_ONLY)depend x11/server/xorg/driver/xorg-video-i128 +$(i386_ONLY)depend x11/server/xorg/driver/xorg-video-i740 +$(i386_ONLY)depend x11/server/xorg/driver/xorg-video-intel +$(i386_ONLY)depend x11/server/xorg/driver/xorg-video-mga +$(i386_ONLY)depend x11/server/xorg/driver/xorg-video-neomagic +$(i386_ONLY)depend x11/server/xorg/driver/xorg-video-nv +$(i386_ONLY)depend x11/server/xorg/driver/xorg-video-openchrome +$(i386_ONLY)depend x11/server/xorg/driver/xorg-video-radeonhd +$(i386_ONLY)depend x11/server/xorg/driver/xorg-video-rendition +$(i386_ONLY)depend x11/server/xorg/driver/xorg-video-s3 +$(i386_ONLY)depend x11/server/xorg/driver/xorg-video-s3virge +$(i386_ONLY)depend x11/server/xorg/driver/xorg-video-savage +$(i386_ONLY)depend x11/server/xorg/driver/xorg-video-siliconmotion +$(i386_ONLY)depend x11/server/xorg/driver/xorg-video-sis +$(i386_ONLY)depend x11/server/xorg/driver/xorg-video-tdfx +$(i386_ONLY)depend x11/server/xorg/driver/xorg-video-tga +$(i386_ONLY)depend x11/server/xorg/driver/xorg-video-trident +$(i386_ONLY)depend x11/server/xorg/driver/xorg-video-tseng +$(i386_ONLY)depend x11/server/xorg/driver/xorg-video-vesa +$(i386_ONLY)depend x11/server/xorg/driver/xorg-video-vmware depend x11/server/xserver-common depend x11/server/xvnc depend x11/session/sessreg +depend x11/session/smproxy +depend x11/session/winsysck depend x11/session/xauth depend x11/session/xinit +depend x11/trusted/libxtsol depend x11/x11-server-utilities +depend x11/x11-window-dump +depend x11/xcalc depend x11/xclipboard depend x11/xfd depend x11/xfontsel +depend x11/xfs/xfs-utilities depend x11/xkill +depend x11/xlock depend x11/xmag +depend x11/xvidtune end package package babel_install consolidation "solaris_re" _______________________________________________ pkg-discuss mailing list [email protected] http://mail.opensolaris.org/mailman/listinfo/pkg-discuss
