On 20/7/07 14:09, "Paul Howarth" <[EMAIL PROTECTED]> wrote:
> Matt Patterson wrote: >> The mock source makes it look like setting config_opts['macros'] simply >> prepends the string to a ~/.rpmmacros file, which I would've thought >> wouldn't cause other things to be, what, undefined or redefined... > > That's right, but config_opts['macros'] has a default value that's built > into mock, and that default value includes the definitions of those two > macros, so if you override the default then you need to define those > values or they won't be set anywhere. That'll teach me to actually read the source properly... > Maybe something like this would be better: > > config_opts['macros'] = config_opts['macros'] + """ > %%_dist bc_gems > """ Yes indeed, that makes good sense. Thanks, Matt -- Matt Patterson Senior Software Engineer, BBC Audio & Music Interactive http://www.bbc.co.uk/ This e-mail (and any attachments) is confidential and may contain personal views which are not the views of the BBC unless specifically stated. If you have received it in error, please delete it from your system. Do not use, copy or disclose the information in any way nor act in reliance on it and notify the sender immediately. Please note that the BBC monitors e-mails sent or received. Further communication will signify your consent to this. -- Fedora-buildsys-list mailing list [email protected] https://www.redhat.com/mailman/listinfo/fedora-buildsys-list
