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

[Gary Lin <glin@suse.com>]

On Fri, Jun 22, 2018 at 09:24:44AM +0000, Zhu, Yonghong wrote:
> Any comment on which patch may cause this issue ? I revert all the 20 patches, OVMF could build PASS.
> 
I wonder if reverting patch 16~20 helps or not. Changing to absolute
import may be not as safe as I thought.

Thanks,

Gary Lin

> Best Regards,
> Zhu Yonghong
> 
> 
> -----Original Message-----
> From: Zhu, Yonghong 
> Sent: Friday, June 22, 2018 5:22 PM
> To: 'Gary Lin' <glin@suse.com>
> Cc: edk2-devel@lists.01.org; Gao, Liming <liming.gao@intel.com>; Zhu, Yonghong <yonghong.zhu@intel.com>
> Subject: RE: [edk2] [PATCH v3 05/20] BaseTools: Use the python3-range functions
> 
> Yes, after modify build.bat and GenFds.bat, I can avoid the previous error, but a new error:
> 
> build.py...
> d:\git\edk2_new\OvmfPkg\OvmfPkg.fdf.inc(83): error 3000: Invalid syntax/format
>         <type 'long'> near line 83, column 0: SET gUefiOvmfPkgTokenSpaceGuid.Pcd OvmfFlashNvStorageEventLogBase = gUefiOvmfPkgTokenSpaceGuid.PcdOvmfFlashNvStorag
> eVariableBase + gEfiMdeModulePkgTokenSpaceGuid.PcdFlashNvStorageVariableSize
> 
> Best Regards,
> Zhu Yonghong
> 
> 
> -----Original Message-----
> From: Gary Lin [mailto:glin@suse.com]
> Sent: Friday, June 22, 2018 5:13 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 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>
> > 
> _______________________________________________
> edk2-devel mailing list
> edk2-devel@lists.01.org
> https://lists.01.org/mailman/listinfo/edk2-devel
>