Re: [edk2-buildtools] [PATCH] tools_def: Change CLANG_BIN from DEF to ENV

2014-01-15 Thread Andrew Fish
Olivier, The patch looks good to me. I also need to update the X64 Xcode flags for Xcode 5 since as -ccc-host-triple x86_64-pc-win32-macho is now -target x86_64-pc-win32-macho. On OS X Mavericks /usr/bin/clang is just a redirector to the currently active tools. The xcode-select command can be

[edk2-buildtools] [PATCH] tools_def: Change CLANG_BIN from DEF to ENV

2014-01-15 Thread Olivier Martin
Andrew, Please find this patch that replaces the macro DEF(CLANG_BIN) (CLANG_BIN is a tools_def specific variable) by ENV(CLANG_BIN) (CLANG_BIN is an host environment variable that can be changed from the shell) to get Clang location. This change allows to change CLANG_BIN without editing tools_d