On 20/7/07 14:09, "Paul Howarth" <paul(a)city-fan.org> 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.