回复: [edk2-devel] Python2.7 is not working with the EDK2 build system


gaoliming
 

Bob:

 Yes. Python3 is the formal support. We recommend user to use Python3. But, if user meets the issue in Python2, user can still report the issue in BaseTools. Its priority may be low. For this case, it is the regression issue caused by the recent change. The patch owner is also identified. So, I suggest the patch owner to follow up and enhance his patch.

 

Thanks

Liming

发件人: devel@edk2.groups.io <devel@edk2.groups.io> 代表 Sunny Wang
发送时间: 202196 21:16
收件人: Feng, Bob C <bob.c.feng@...>; gaoliming <gaoliming@...>; devel@edk2.groups.io; Chen, Christine <yuwei.chen@...>; 'Cole Robinson' <crobinso@...>
抄送: Sami Mujawar <Sami.Mujawar@...>; Samer El-Haj-Mahmoud <Samer.El-Haj-Mahmoud@...>; 'Gerd Hoffmann' <kraxel@...>; Sunny Wang <Sunny.Wang@...>
主题: Re: [edk2-devel] Python2.7 is not working with the EDK2 build system

 

Thanks for checking this, Bob and Liming.

 

Hi Bob,

 

Yeah, I was aware of that as well. It makes sense to switch Python2 to Python3. However, Python 2 was discontinued for a while, but there seems no announcement in the mailing list or code change for cleaning up Python 2 stuff in edk2 BaseTools, which may make users feel Python 2 is still useable and cause confusion. Of course, I may miss some activities about deprecating Python 2 in edk2. If so, could you point me out? I just want to have an obvious way to inform edk2 users that Python 2 doesn’t work. If you think this email is good enough as a notification to the edk2 users, I’m fine with this.

 

Moreover, for a quick improvement, how about we add a python version check in the build script to prevent code building and remind users to use Python 3?     

 

Thanks,

Sunny

From: Feng, Bob C <bob.c.feng@...>
Sent: 06 September 2021 10:31
To: gaoliming <gaoliming@...>; devel@edk2.groups.io; Sunny Wang <Sunny.Wang@...>; Chen, Christine <yuwei.chen@...>; 'Cole Robinson' <crobinso@...>
Cc: Sami Mujawar <Sami.Mujawar@...>; Samer El-Haj-Mahmoud <Samer.El-Haj-Mahmoud@...>; 'Gerd Hoffmann' <kraxel@...>
Subject: RE: [edk2-devel] Python2.7 is not working with the EDK2 build system

 

Hi Sunny,

 

EDK II only formally supports Python 3 now because Python 2 is EOL with no support.  So I’d suggest BaseTools users switch Python2 to Python3.

 

Thanks,

Bob

 

From: gaoliming <gaoliming@...>
Sent: Monday, September 6, 2021 9:10 AM
To: devel@edk2.groups.io; Sunny.Wang@...; Feng, Bob C <bob.c.feng@...>; Chen, Christine <yuwei.chen@...>; 'Cole Robinson' <crobinso@...>
Cc: 'Sami Mujawar' <Sami.Mujawar@...>; 'Samer El-Haj-Mahmoud' <Samer.El-Haj-Mahmoud@...>; 'Gerd Hoffmann' <kraxel@...>
Subject:
回复: [edk2-devel] Python2.7 is not working with the EDK2 build system

 

Sunny:

 If Robinson has no response, I suggest to revert this change first.

 

Robinson:

 Can you give your fix plan for this regression issue?

 

Thanks

Liming

发件人: devel@edk2.groups.io <devel@edk2.groups.io> 代表 Sunny Wang
发送时间: 202194 2:09
收件人: Feng, Bob C <bob.c.feng@...>; Liming Gao <gaoliming@...>; Yuwei Chen <yuwei.chen@...>; Cole Robinson <crobinso@...>; devel@edk2.groups.io
抄送: Sunny Wang <Sunny.Wang@...>; Sami Mujawar <Sami.Mujawar@...>; Samer El-Haj-Mahmoud <Samer.El-Haj-Mahmoud@...>
主题: [edk2-devel] Python2.7 is not working with the EDK2 build system

 

Hi all,

 

I just ran into this build issue. After checking the edk2 emails, I saw some of you already ran into this build issue as well. At least, I saw two email threads below talking about this issue.  

https://edk2.groups.io/g/devel/message/80068?p=,,,20,0,0,0::recentpostdate%252Fsticky,,python,20,2,0,85296733

https://edk2.groups.io/g/devel/message/79022?p=,,,20,0,0,0::recentpostdate%2525252Fsticky,,DeprecationWarnings,20,2,0,84409128

 

The main purpose of this email is to have obvious email subject to get people’s attention, so that people won’t spend time to check this when running into this issue.

The solution at this moment is to use Python 3 instead (Use “export PYTHON_COMMAND=/usr/bin/python3” instead of “export PYTHON_COMMAND=/usr/bin/python”).   

 

Hi Robinson,

Are you working on this now? If you’re busy with other things, how about we revert your patch first?

 

Hi Bob, Liming, and Yuwei,

What do you prefer to do at this moment? Wait for Robinson? or revert the change?

 

Best Regards,

Sunny

IMPORTANT NOTICE: The contents of this email and any attachments are confidential and may also be privileged. If you are not the intended recipient, please notify the sender immediately and do not disclose the contents to any other person, use it for any purpose, or store or copy the information in any medium. Thank you.

IMPORTANT NOTICE: The contents of this email and any attachments are confidential and may also be privileged. If you are not the intended recipient, please notify the sender immediately and do not disclose the contents to any other person, use it for any purpose, or store or copy the information in any medium. Thank you.


Cole
 

On 9/6/21 9:18 PM, gaoliming wrote:
Bob:

 Yes. Python3 is the formal support. We recommend user to use Python3.
But, if user meets the issue in Python2, user can still report the issue
in BaseTools. Its priority may be low. For this case, it is the
regression issue caused by the recent change. The patch owner is also
identified. So, I suggest the patch owner to follow up and enhance his
patch.
Sorry for the delayed response, I was on paternity leave since August.

I haven't seen the actual error in this thread. Is there a clear python
error being thrown? Maybe the fix is simple but I can't

But as mentioned elsewhere, python2 has been End of Life since Jan 1
2020, over 1.5 years ago. It's going to become increasingly difficult to
keep code working on python2 and latest python3.

- Cole