[zorg] r195058 - Based on the patch proposed by Rick Foos!

[Rick Foos]

I'm a bit busy for Thanksgiving, but go ahead and make the change.

My buildslave is not critical and I'll patch it soon or take it offline.

Thanks
Rick

On 11/27/2013 12:15 PM, Galina Kistanova wrote:
> It looks like setting up C_COMPILER, CXX_COMPILER, and such locally on 
> a buildslave is the right thing to do.
> The proper change would be remove these options from the build factory.
>
> Rick, could you set up your slave accordingly, please?
>
> For now, just to keep things running and to avoid extra pressure on 
> Rick, I temporarily patch the build factory to setup DCMAKE_C_COMPILER 
> options only if we are building
> with sanitizers, and will remove this change once Rick will confirm 
> his buildslave configured.
>
> If this fine with everyone, I'll apply the workaround tonight when 
> I'll be restarting the buildmaster.
> If someone sees issues with this approach, please speak up.
>
> Thanks
>
> Galina
>
>
>
> On Fri, Nov 22, 2013 at 10:10 AM, Dmitri Gribenko <gribozavr at gmail.com 
> <mailto:gribozavr at gmail.com>> wrote:
>
>     On Fri, Nov 22, 2013 at 7:11 AM, Rick Foos <rfoos at codeaurora.org
>     <mailto:rfoos at codeaurora.org>> wrote:
>     > Maybe a cmakeOptions like options with the default like it used
>     to be
>     > (excluding C and CXX compiler), and allow the user to add options?
>
>     Maybe.  Or maybe add DCMAKE_C_COMPILER options only if we are building
>     with sanitizers?  Whichever is convenient, seems like both will work
>     for me.
>
>     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
>     <mailto:gribozavr at gmail.com>>*/
>
>


-- 
Rick Foos
Qualcomm Innovation Center, Inc. is a member of Code Aurora Forum, hosted by The Linux Foundation

-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20131127/722cac1b/attachment.html>