[PATCH] Zorg: getLLDBuildFactory() now takes an optional 'env' parameter

Dmitri Gribenko gribozavr at gmail.com
Mon Dec 9 16:21:43 PST 2013


On Mon, Dec 9, 2013 at 4:14 PM, Mikael Lyngvig <mikael at lyngvig.org> wrote:
> I think the best way to fix this is that you add your desired environment
> variables to the call to getLLDBuildFactory() in
> zorg/buildbot/builders/LLDBuilder.py.  Sorry for the inconvenience, but I
> didn't realize that the former code basically was written so unfortunately
> that it had to break when somebody came along and tried to change it.

Indeed, this new way seems cleaner.  I added the variables for my
builders in r196847.

Dmitri

-- 
main(i,j){for(i=2;;i++){for(j=2;j<i;j++){if(!(i%j)){j=0;break;}}if
(j){printf("%d\n",i);}}} /*Dmitri Gribenko <gribozavr at gmail.com>*/



More information about the llvm-commits mailing list