Hi Liming, This commit (15192) does not do what was described.
It permits '. edksetup.sh BaseTool', rather than what was described in the text - '. edksetup.sh BaseTools'. (missing a trailing 's') Regards, Leif On 27 January 2014 07:46, Gao, Liming <liming....@intel.com> wrote: > Parmeshwr: > > Your patch is good. I will help commit it. > > Signed-off-by: Gao, Liming <liming....@intel.com> > > > > Thanks > > Liming > > *From:* parmeshwr_pra...@dell.com [mailto:parmeshwr_pra...@dell.com] > *Sent:* Monday, January 27, 2014 3:00 PM > *To:* Gao, Liming > *Cc:* edk2-comm...@lists.sourceforge.net > *Subject:* RE: edk2/edksetup.sh patch to solve command line parameter > > > > Hi Liming, > > > > Please find new patch for edksetupp.sh. > > > > I have changes according to your comments. > > It can handle following cases. > > 1- Handle more than one parameter > > 2- Handle if first parameter is not "-?, -h, --help or BaseTool". > > 3- Any other thing to display error message. > > > > Please let me know with your comment. > > > > Regards > > Parmeshwr Prasad > > > > *From:* Gao, Liming [mailto:liming....@intel.com <liming....@intel.com>] > *Sent:* Friday, January 24, 2014 8:01 PM > *To:* Prasad, Parmeshwr > *Subject:* RE: edk2/edksetup.sh patch to solve command line parameter > > > > Yes. If user follows it, it should work. So, I expect the behavior is: > > 1. No parameter, edksetup.sh will set up environment. > > 2. BaseTools parameter, edksetup.sh will set up environment. > > 3. Other parameter, edksetup.sh will print help message. > > > > Thanks > > Liming > > *From:* parmeshwr_pra...@dell.com > [mailto:parmeshwr_pra...@dell.com<parmeshwr_pra...@dell.com>] > > *Sent:* Friday, January 24, 2014 7:10 PM > *To:* Gao, Liming > *Subject:* RE: edk2/edksetup.sh patch to solve command line parameter > > > > Hi Liming > > > > I got your point. I saw user manual do we give any other parameter except " > *BaseTools"* > > In parameter to edksetup.sh ? > > > > If I am not wrong than this is the point you are talking about. > > *ln -s /home/usr/BaseTools /home/usr/Edk2Workspace/Conf/BaseToolsSource* > > 4. Run "*. edksetup.sh BaseTools*" under the workspace's directory to set > > system environment, such as WORKSPACE, EDK_TOOLS_PATH etc. > > > > Regards > > Parmeshwr > > > > *From:* Prasad, Parmeshwr > *Sent:* Friday, January 24, 2014 4:26 PM > *To:* edk2-devel@lists.sourceforge.net; liming....@intel.com > > *Subject:* Re: [edk2] edk2/edksetup.sh patch to solve command line > parameter > > > > Hi Liming, > > > > See below two example- > > > > > > 1- param@param-opensource:~/Development/edk2$ source edksetup.sh -h- > > Loading previous configuration from $WORKSPACE/Conf/BuildEnv.sh > > WORKSPACE: /home/param/Development/edk2 > > EDK_TOOLS_PATH: /home/param/Development/edk2/BaseTools > > > > *In above example edksetup.sh is not able to handle "-h-" parameter it > mean it can handle only "-?, -h,--help".* > > *If we give any other parameter except above mentioned. It cannot handle. > It mean error handling is required.* > > *Even the help message is not looking good.* > > > > > > 2- param@param-opensource:~/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') > > > > This is expected behavior. > > > > If this patch is not looking good, suggest me how it can be made better. > > > > Regards > > Parmeshwr > > > > > > *From:* Gao, Liming [mailto:liming....@intel.com <liming....@intel.com>] > *Sent:* Friday, January 24, 2014 3:06 PM > *To:* Prasad, Parmeshwr > *Cc:* edk2-devel@lists.sourceforge.net > *Subject:* Re: [edk2] edk2/edksetup.sh patch to solve command line > parameter > > > > Hi, > > I have two comments. > > 1. BaseTools parameter is required to be supported for > compatibility, because this usage is mentioned in EDKII_UserManual.pdf > document. Some users have used it. In fact, ". edksetup.sh BaseTools" is > same to ". edksetup.sh". > > 2. In below script, BaseTools/BuildEnv $* can be cleanup to remove > $*, because no parameter is required. > > > > if [ -z "$WORKSPACE" ] > > then > > . BaseTools/BuildEnv $* > > else > > . $WORKSPACE/BaseTools/BuildEnv $* > > fi > > > > Thanks > > Liming > > *From:* parmeshwr_pra...@dell.com > [mailto:parmeshwr_pra...@dell.com<parmeshwr_pra...@dell.com>] > > *Sent:* Thursday, January 23, 2014 4:02 PM > *To:* edk2-comm...@lists.sourceforge.net > *Subject:* edk2/edksetup.sh patch to solve command line parameter > > > > 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 > > [image: cid:image002.png@01CE781A.38F61FE0] > > > > > ------------------------------------------------------------------------------ > 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-devel mailing list > edk2-devel@lists.sourceforge.net > https://lists.sourceforge.net/lists/listinfo/edk2-devel > >
<<image001.png>>
------------------------------------------------------------------------------ WatchGuard Dimension instantly turns raw network data into actionable security intelligence. It gives you real-time visual feedback on key security issues and trends. Skip the complicated setup - simply import a virtual appliance and go from zero to informed in seconds. http://pubads.g.doubleclick.net/gampad/clk?id=123612991&iu=/4140/ostg.clktrk
_______________________________________________ edk2-devel mailing list edk2-devel@lists.sourceforge.net https://lists.sourceforge.net/lists/listinfo/edk2-devel
