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
buildsys@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/buildsys
 

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