Re: [PATCH v3 05/20] BaseTools: Use the python3-range functions

[Gary Lin <glin@suse.com>]

On Fri, Jun 22, 2018 at 08:53:04AM +0000, Zhu, Yonghong wrote:
> Hi Gary,
> 
> My test env is Windows. I applied Patch 16, and seems this patch only for Linux.
> 
Ah, right. The patch is Linux only, and sadly I only have Linux.
I guess the similar change may work for windows.

Could you try to modify build.bat and GenFds.bat in
BaseTools\BinWrappers\WindowsLike\ as the following?

@setlocal
@set ToolName=%~n0%
@set PYTHONPATH=%PYTHONPATH%;%BASE_TOOLS_PATH%\Source\Python
@%PYTHON_HOME%\python.exe -m %ToolName%.%ToolName% %*

I'm not sure whether it should be '-m' or '/m' though. The goal is treat
build and GenFds as modules to avoid the error you saw.

Thanks,

Gary Lin

> Best Regards,
> Zhu Yonghong
> 
> 
> -----Original Message-----
> From: Gary Lin [mailto:glin@suse.com] 
> Sent: Friday, June 22, 2018 4:46 PM
> To: Zhu, Yonghong <yonghong.zhu@intel.com>
> Cc: edk2-devel@lists.01.org; Gao, Liming <liming.gao@intel.com>
> Subject: Re: [edk2] [PATCH v3 05/20] BaseTools: Use the python3-range functions
> 
> On Fri, Jun 22, 2018 at 04:21:23PM +0800, Gary Lin wrote:
> > On Fri, Jun 22, 2018 at 05:29:21AM +0000, Zhu, Yonghong wrote:
> > > Hi Gary,
> > > 
> > > Patch 03 used some Tab characters, Patch 17 has some Trailing whitespace. You can use the BaseTools\Scripts\PatchCheck.py to check its format.
> > > 
> > Ok, will fix them in the next version.
> > 
> > > I try to build OVMF platform, it directly report error:
> > > from .BuildReport import BuildReport
> > > ValueError: Attempted relative import in non-package
> > > 
> > I didn't see this in my system. Will investigate what's wrong.
> > 
> Hi Yonghong,
> 
> The error showed when I reverted patch 16. Would you mind to check whether patch 16 was applied or not?
> 
> Thanks,
> 
> Gary Lin
> 
> > Thanks,
> > 
> > Gary Lin
> > 
> > > Best Regards,
> > > Zhu Yonghong
> > > 
> > > -----Original Message-----
> > > From: Gary Lin [mailto:glin@suse.com]
> > > Sent: Friday, June 22, 2018 10:04 AM
> > > To: Zhu, Yonghong <yonghong.zhu@intel.com>
> > > Cc: edk2-devel@lists.01.org; Gao, Liming <liming.gao@intel.com>
> > > Subject: Re: [edk2] [PATCH v3 05/20] BaseTools: Use the 
> > > python3-range functions
> > > 
> > > On Thu, Jun 21, 2018 at 12:40:29PM +0000, Zhu, Yonghong wrote:
> > > > Hi Gary,
> > > > 
> > > > Python 2.7 doesn't have the module builtins, if we apply this patch now, it will cause build failure on Python 2.7.
> > > > 
> > > Urhh, I thought it's a "built-in" module but it's actually from 
> > > python-future.
> > > 
> > > I'll revert this patch and post a new patchset.
> > > BTW, would you might review the first patch? It's just a typo fix 
> > > and I just caught it while running the futurize script, so the patch 
> > > is actually independent from this patch series.
> > > 
> > > Thanks,
> > > 
> > > Gary Lin
> > > 
> > > > Best Regards,
> > > > Zhu Yonghong
> > > > 
> > > > 
> > > > -----Original Message-----
> > > > From: Gary Lin [mailto:glin@suse.com]
> > > > Sent: Thursday, June 21, 2018 12:44 PM
> > > > To: edk2-devel@lists.01.org
> > > > Cc: Zhu, Yonghong <yonghong.zhu@intel.com>; Gao, Liming 
> > > > <liming.gao@intel.com>
> > > > Subject: [PATCH v3 05/20] BaseTools: Use the python3-range 
> > > > functions
> > > > 
> > > > Replace xrange() and range() with the newer range() function Based on "futurize -f libfuturize.fixes.fix_xrange_with_import"
> > > > 
> > > > Contributed-under: TianoCore Contribution Agreement 1.1
> > > > Cc: Yonghong Zhu <yonghong.zhu@intel.com>
> > > > Cc: Liming Gao <liming.gao@intel.com>
> > > > Signed-off-by: Gary Lin <glin@suse.com>
>