Hi: You're right, but at least we won't have to worry about this until “python3.10”
· And we're comparing directory names, "Python 3.6.6" doesn't appear in the Python version we found · We find python applications using “whereis python” “/usr/bin/python3.6” “/usr/local/bin/python3.6” · Later we can compare the form of string length to add a layer of judgment, but we don't need it now. Any question, please let me know. Thanks. Best Regards Fan Zhiju From: devel@edk2.groups.io [mailto:devel@edk2.groups.io] On Behalf Of rebe...@bluestop.org Sent: Wednesday, June 26, 2019 11:48 PM To: devel@edk2.groups.io; Fan, ZhijuX <zhijux....@intel.com> Cc: Gao, Liming <liming....@intel.com>; Feng, Bob C <bob.c.f...@intel.com> Subject: Re: FW: [edk2-devel] [PATCH] BaseTools:Linux changes the way the latest version is judged On 2019-06-25 22:46, Fan, ZhijuX wrote: Hi: Python3.6 and python3.7 After the code removes some common values “python”, we compare “3.6” and “3.7” origin_version = “3.6”, python_version=”3.7” I'm going to change it to if [[ "$origin_version" < "$python_version" ]]; then That's fine for now, hopefully it'll keep working as long as it needs to. But once you're comparing for example 3.7 and 3.12: [bcran@photon ~]$ if [[ "3.7" > "3.12" ]]; then echo "wrong!"; else echo "correct!"; fi wrong! -- Rebecca Cran -=-=-=-=-=-=-=-=-=-=-=- Groups.io Links: You receive all messages sent to this group. View/Reply Online (#42990): https://edk2.groups.io/g/devel/message/42990 Mute This Topic: https://groups.io/mt/32154778/21656 Group Owner: devel+ow...@edk2.groups.io Unsubscribe: https://edk2.groups.io/g/devel/unsub [arch...@mail-archive.com] -=-=-=-=-=-=-=-=-=-=-=-