[Yum-devel] [PATCH 2/2] Package deselect uses the same syntax as groups do. BZ 768104.

James Antill james at fedoraproject.org
Mon Dec 19 18:03:33 UTC 2011


On Mon, 2011-12-19 at 17:13 +0100, Zdeněk Pavlas wrote:
> Handle '- PKG', '-PKG1, PKG2' etc.  Package *selection*
> code does not support this syntax (yet).

 I'm very sure we don't want to be splitting single strings into lists
down in TransactionData.deselect().
 It's less obvious if we should be doing this kind of thing in
"install()" or "installPkgs()" ... my gut says the later, for the same
reasons (it feels really wrong to have a non-list input that is split
into a list), but I think that's hard to make this work correctly WRT
'@' and '-' and install() handles wildcards (then again people will
inevitably do <install "foo,/path/to/blah.rpm"> which can only work in
installPkgs()).

 As I said on IRC the "easy out" is to just shrug about "@group1,group2"
and call it a special case ... but allow stripping spaces.



More information about the Yum-devel mailing list