FAQ Search Today's Posts Mark Forums Read
» Video Reviews

» Linux Archive

Linux-archive is a website aiming to archive linux email lists and to make them easily accessible for linux users/developers.

» Sponsor

» Partners

» Sponsor

Go Back   Linux Archive > Redhat > Fedora Build System

LinkBack Thread Tools
Old 03-07-2011, 07:27 PM
Ville Skyttä
Default Run yum commands without shell, as sequences instead of strings.

On 03/07/2011 10:17 PM, 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).

Posted here for comments before committing. The patch touches many
files but is not actually that scary and should be backwards compatible
[1]. I've been using this locally for a quite a while and found no
problems, and it fixes the above mentioned cases for me.

The *.cfg changes should be safe to be left out of the patch if people
think it's better to avoid potential *.rpmnew config file churn for
fairly cosmetic reasons for now. Either way, old config files should
continue to work as-is anyway (in fact I've been running this with such
old config files all the time).

[1] Except that preExistingDeps is now a list whereas it used to be a
string, but the only occurrence of its usage I'm aware of was in the
ccache plugin and that's addressed in the patch.
buildsys mailing list

Thread Tools

All times are GMT. The time now is 01:20 PM.

VBulletin, Copyright ©2000 - 2014, Jelsoft Enterprises Ltd.
Content Relevant URLs by vBSEO ©2007, Crawlability, Inc.
Copyright ©2007 - 2008, www.linux-archive.org