mock rfe - buildgroup

[Jeff Sheltren]

-----BEGIN PGP SIGNED MESSAGE-----
Hash: SHA1

On Jul 6, 2007, at 3:32 PM, Jesse Keating wrote:

> On Friday 06 July 2007 15:26:42 Michael E Brown wrote:
>> Looking through your patch:
>>
>> +++ b/etc/defaults.cfg
>> +config_opts['chroot_setup_cmd'] = 'groupinstall buildsys-build'
>>
>> This could be problematic for F7, F6, EPEL4/5 and others.
>>
>> I see two (three?) ways to handle this:
>>
>> 1) only add this line to -devel.
>>
>> 2) change the default in mock.py, and override F7/F6/EPEL4/EPEL5
>> configs.
>>
>> 3) add this line to -devel, and then migrate it to (2) above after a
>> period of time (perhaps when we drop F6?)
>
> Yeah, I had this fleeting thought when I was doing it, then  
> promptly forgot
> about it.  One of those days...
>
> I think 2 is probably safest, if people have locally modified their  
> configs,
> but I was also thinking we'd only make this change in rawhide, so  
> it would
> only be rawhide people who get it, not pushing it out to existing  
> Fedora/RHEL
> setups, so 3 is probably fine too.
>

I'm confused why you would say (2) is the safest.  Overriding the  
default would cause all those who use modified config files to fix  
them when mock gets updated.

My order of preference for the above options would be (1), (3), (2).   
In fact, if you go with (3), it makes sense to me to change the  
default after F7 is dropped rather than FC6.  F7 continues to use a  
buildgroups repo/RPM, correct?

- -Jeff
-----BEGIN PGP SIGNATURE-----
Version: GnuPG v1.4.7 (Darwin)

iD8DBQFGjp2kKe7MLJjUbNMRAkoyAJ9aBkgqnPX9zD8/bc2T1bcyioRvsQCgld3q
/Wf8M5r4txxBbc4IrVsYL5Q=
=EwT8
-----END PGP SIGNATURE-----