[Yum-devel] [PATCH 4/4] Changed behavior of group update to only update packages already installed.

[seth vidal]

On Wed, 2011-06-08 at 15:01 -0400, Nick Jacek wrote:
> ---
>  cli.py          |    4 ++--
>  docs/yum.8      |    8 ++++----
>  yum/__init__.py |   21 ++++++++++++++-------
>  3 files changed, 20 insertions(+), 13 deletions(-)
> 

Hi Nick, 

welcome and thanks for the patches.
 
I'm not in love with this change, though.

I know some folks use groupupdate to update the group - which means
install any pkg which is not currently installed and update the rest:

http://fedoraproject.org/wiki/YumUpgradeFaq#5._Make_sure_Fedora_is_upgraded 

as a major example of it's use.

Also making:

yum group update != yum groupupdate

is going to be a nightmare to document and explain.

since there is likely to be a good-sized change to how groups work in
general in the future - as the objects change  - postpone the behavior
changes until then and make the split out of the groups subcommand
maintain the same naming/behavior.

everything else looks great.

thanks,
-sv