[PATCH] Run yum commands without shell, as sequences instead of strings.

seth vidal skvidal at fedoraproject.org
Mon Mar 7 20:19:44 UTC 2011


On Mon, 2011-03-07 at 22:17 +0200, Ville Skyttä wrote:
> Fixes shell escaping issues like in '--install "perl(Foo::Bar)"'.  The
> "canonical" syntax for chroot_setup_cmd in configs is now a tuple or a
> list, but a string is still accepted for backwards compatibility (it
> gets split to list internally).

I think I'd be more inclined to make it possible to accept the
list/tuple but default to the string in the configs.

it makes it easier to understand/read for people not familiar with
python.

-sv




More information about the buildsys mailing list