Adding new group to comps-f19.xml.in

[Bill Nottingham]

Ravindra Kumar (ravindrakumar at vmware.com) said: 
> Hi Bill and others, 
> 
> Based on the earlier discussions in the emails, I have following diff for comps-f19.xml.in: 
> 
> --- comps-f19.xml.in 2013-04-29 17:41:10.003026000 -0700 
> +++ comps-f19.xml.in.new 2013-05-08 20:35:57.000112000 -0700 
> @@ -7,6 +7,9 @@ 
> <_description>Local X.org display server</_description> 
> <default>false</default> 
> <uservisible>false</uservisible> 
> + <grouplist> 
> + <groupid>virt-agents-x</groupid> 
> + </grouplist> 
> <packagelist> 
> <packagereq>xorg-x11-drv-ati</packagereq> 
> <packagereq>xorg-x11-drv-evdev</packagereq> 
> @@ -1150,6 +1153,9 @@ 
> <_description>Common set of utilities that extend the minimal installation.</_description> 
> <default>false</default> 
> <uservisible>false</uservisible> 
> + <grouplist> 
> + <groupid>virt-agents</groupid> 
> + </grouplist> 
> <packagelist> 
> <packagereq>acl</packagereq> 
> <packagereq>at</packagereq> 
> @@ -5633,6 +5639,26 @@ 
> </packagelist> 
> </group> 
> <group> 
> + <id>virt-agents</id> 
> + <_name>Virtualization Agents</_name> 
> + <_description>These packages enable better management of virtual machine.</_description> 
> + <default>true</default> 
> + <uservisible>true</uservisible> 
> + <packagelist> 
> + <packagereq type="default">open-vm-tools</packagereq> 
> + </packagelist> 
> + </group> 
> + <group> 
> + <id>virt-agents-x</id> 
> + <_name>Virtualization Agents</_name> 
> + <_description>These packages enable better user experience for virtual machine.</_description> 
> + <default>true</default> 
> + <uservisible>true</uservisible> 
> + <packagelist> 
> + <packagereq type="default">open-vm-tools-desktop</packagereq> 
> + </packagelist> 
> + </group> 
> + <group> 
> <id>virtualization</id> 
> <_name>Virtualization</_name> 
> <_description>These packages provide a virtualization environment.</_description> 
> 
> Could you please confirm if this is what you are expecting and how to test it? Wiki page 
> http://fedoraproject.org/wiki/How_to_use_and_edit_comps.xml_for_package_groups 
> says nothing about testing apart from running 'xsltproc'. I ran 'xsltproc' and it does not 
> say anything about a 'group' containing a 'grouplist', so I'm not sure if this will work 
> or not. 

The two added group are OK. grouplist is not something that actually is
used, though; they would need to either be added as mandatory/optional
groups to the desktop environments, or just brought in via the spins' kickstart
files.

Also, please attach patches - your  mailer appears to have mangled the
spacing. 

Bill