FYI: this patch, which was committed yesterday, breaks some build instructions (and possibly the uefi-build.sh command).
Effectively, they are deprecating the passing of a specific BaseTools directory to edksetup.sh. So it could/should henceforth simply be called as ". edksetup.sh", whereas we've tended to use ". edksetup.sh `pwd`/BaseTools".
---------- Forwarded message ---------- From: Gao, Liming liming.gao@intel.com Date: 27 January 2014 07:46 Subject: Re: [edk2] edk2/edksetup.sh patch to solve command line parameter To: "Parmeshwr_Prasad@Dell.com" Parmeshwr_Prasad@dell.com Cc: "edk2-commits@lists.sourceforge.net" edk2-commits@lists.sourceforge.net, "edk2-devel@lists.sourceforge.net" edk2-devel@lists.sourceforge.net
Parmeshwr:
Your patch is good. I will help commit it.
Signed-off-by: Gao, Liming liming.gao@intel.com
Thanks
Liming
*From:* Parmeshwr_Prasad@Dell.com [mailto:Parmeshwr_Prasad@Dell.com] *Sent:* Monday, January 27, 2014 3:00 PM *To:* Gao, Liming *Cc:* edk2-commits@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.gao@intel.com liming.gao@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_Prasad@Dell.com [mailto:Parmeshwr_Prasad@Dell.comParmeshwr_Prasad@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.gao@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.gao@intel.com liming.gao@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_Prasad@Dell.com [mailto:Parmeshwr_Prasad@Dell.comParmeshwr_Prasad@Dell.com]
*Sent:* Thursday, January 23, 2014 4:02 PM *To:* edk2-commits@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.cl... _______________________________________________ edk2-devel mailing list edk2-devel@lists.sourceforge.net https://lists.sourceforge.net/lists/listinfo/edk2-devel
