On 11 February 2013 16:41, Ryan Harkin ryan.harkin@linaro.org wrote:
On 11 February 2013 16:08, Jordan Justen jljusten@gmail.com wrote:
On Mon, Feb 11, 2013 at 12:22 AM, Ryan Harkin ryan.harkin@linaro.org wrote:
The attached patch changes the PandaBoardPkg/build.sh script RELEASE or DEBUG parameter handling.
Before this patch, build.sh can be called thus: ./build.sh ./build.sh -b DEBUG ./build.sh RELEASE
This patch simplifies the way you pass the debug/release parameter into build.sh so that build scripts can simply pass DEBUG or RELEASE, without having to create a special case to add the "-b".
In similar scripts under edk2 we force the use of -b to keep the script as a wrapper to the edk2-buildtools 'build' command. (EmulatorPkg, OvmfPkg)
Thanks for the input. I've just realised of course that my mods mean that the "target" can only realistically be "DEBUG" or "RELEASE", whereas, forcing the -b would allow the user to pass in any other (theoretical) target.
Leif (CC'd) already commented on this aspect in another discussion I had with him, but it seems I ignored him...
I'll re-work the patch
I've attached a re-worked patch. And it's simpler than the previous version, which is nice.
Now, you can call:
./build.sh ./build.sh -b DEBUG ./build.sh -b RELEASE