Hi All, I see there is a problem in "edksetup.sh" file. It accept one parameter "-?, -h, --help" for printing help message. If we give any other parameter to this it goes and set old environment with help message. Expected behavior should be either to print The help message or set environment. It is not able to handle any garbage parameter. I am sending patch for this problem, please review and commit to main stream.
Even help message was not clear I changes that also. Incorrect behavior : :~/Development/edk2$ source edksetup.sh ---jjdcncn Loading previous configuration from $WORKSPACE/Conf/BuildEnv.sh WORKSPACE: /home/param/Development/edk2 EDK_TOOLS_PATH: /home/param/Development/edk2/BaseTools Correct behavior: :~/Development/edk2$ source edksetup.sh -h BaseTools Usage: '. edksetup.sh' Please note: This script must be 'sourced' so the environment can be changed. (Either '. edksetup.sh' or 'source edksetup.sh') Correct behavior: :~/Development/edk2$ source edksetup.sh Loading previous configuration from $WORKSPACE/Conf/BuildEnv.sh WORKSPACE: /home/param/Development/edk2 EDK_TOOLS_PATH: /home/param/Development/edk2/BaseTools Best Regards, Parmeshwr Prasad Tel : +91-9743262018 [cid:[email protected]]
<<inline: image001.png>>
Pkg-Module: edk2 Full-commit-message Handle command line parameter "-? -h --help" and any other. Contributed-under: TianoCore Contribution Agreement 1.0 Signed-off-by: Parmeshwr Prasad <[email protected]> Reviewed-by: NA
edksetup.sh.patch
Description: edksetup.sh.patch
------------------------------------------------------------------------------ CenturyLink Cloud: The Leader in Enterprise Cloud Services. Learn Why More Businesses Are Choosing CenturyLink Cloud For Critical Workloads, Development Environments & Everything In Between. Get a Quote or Start a Free Trial Today. http://pubads.g.doubleclick.net/gampad/clk?id=119420431&iu=/4140/ostg.clktrk
_______________________________________________ edk2-commits mailing list [email protected] https://lists.sourceforge.net/lists/listinfo/edk2-commits
