* [edk2-devel] [edk2-platforms][PATCH v1] MinPlatformPkg: Remove usage of PYTHON_HOME @ 2023-11-18 6:36 Nate DeSimone 2023-11-18 6:36 ` [edk2-devel] [edk2-platforms][PATCH v1] PurleyOpenBoardPkg: Replace Python Interpreter Name Nate DeSimone ` (4 more replies) 0 siblings, 5 replies; 8+ messages in thread From: Nate DeSimone @ 2023-11-18 6:36 UTC (permalink / raw) To: devel; +Cc: Chasel Chiu, Liming Gao, Eric Dong Removes usage PYTHON_HOME from RebaseFspBinBaseAddress.py Cc: Chasel Chiu <chasel.chiu@intel.com> Cc: Liming Gao <gaoliming@byosoft.com.cn> Cc: Eric Dong <eric.dong@intel.com> Signed-off-by: Nate DeSimone <nathaniel.l.desimone@intel.com> --- .../MinPlatformPkg/Tools/Fsp/RebaseFspBinBaseAddress.py | 8 ++------ 1 file changed, 2 insertions(+), 6 deletions(-) diff --git a/Platform/Intel/MinPlatformPkg/Tools/Fsp/RebaseFspBinBaseAddress.py b/Platform/Intel/MinPlatformPkg/Tools/Fsp/RebaseFspBinBaseAddress.py index b7e4bcf5f9..32db9eb1c4 100644 --- a/Platform/Intel/MinPlatformPkg/Tools/Fsp/RebaseFspBinBaseAddress.py +++ b/Platform/Intel/MinPlatformPkg/Tools/Fsp/RebaseFspBinBaseAddress.py @@ -1,6 +1,6 @@ ## @ RebaseFspBinBaseAddress.py # -# Copyright (c) 2019 - 2021, Intel Corporation. All rights reserved.<BR> +# Copyright (c) 2019 - 2023, Intel Corporation. All rights reserved.<BR> # SPDX-License-Identifier: BSD-2-Clause-Patent # @@ -65,11 +65,7 @@ file.close() # Get FSP-M Size, in order to calculate the FSP-T Base. Used SplitFspBin.py script # to dump the header, and get the ImageSize in FSP-M section # -pythontool = 'python' -if 'PYTHON_HOME' in os.environ: - pythontool = os.environ['PYTHON_HOME'] + os.sep + 'python' -else: - pythontool = sys.executable +pythontool = sys.executable Process = subprocess.Popen([pythontool, splitFspBinPath, "info","-f",fspBinFilePath], stdout=subprocess.PIPE) Output = Process.communicate()[0] FsptInfo = Output.rsplit(b"FSP_M", 1); -- 2.34.1 -=-=-=-=-=-=-=-=-=-=-=- Groups.io Links: You receive all messages sent to this group. View/Reply Online (#111422): https://edk2.groups.io/g/devel/message/111422 Mute This Topic: https://groups.io/mt/102667304/7686176 Group Owner: devel+owner@edk2.groups.io Unsubscribe: https://edk2.groups.io/g/devel/unsub [rebecca@openfw.io] -=-=-=-=-=-=-=-=-=-=-=- ^ permalink raw reply related [flat|nested] 8+ messages in thread
* [edk2-devel] [edk2-platforms][PATCH v1] PurleyOpenBoardPkg: Replace Python Interpreter Name 2023-11-18 6:36 [edk2-devel] [edk2-platforms][PATCH v1] MinPlatformPkg: Remove usage of PYTHON_HOME Nate DeSimone @ 2023-11-18 6:36 ` Nate DeSimone 2023-11-28 3:20 ` Chiu, Chasel 2023-11-18 6:36 ` [edk2-devel] [edk2-platforms][PATCH v1] WhitleyOpenBoardPkg: " Nate DeSimone ` (3 subsequent siblings) 4 siblings, 1 reply; 8+ messages in thread From: Nate DeSimone @ 2023-11-18 6:36 UTC (permalink / raw) To: devel; +Cc: Chasel Chiu Removes the assumption that the python interpreter will have the name python. Cc: Chasel Chiu <chasel.chiu@intel.com> Signed-off-by: Nate DeSimone <nathaniel.l.desimone@intel.com> --- .../PurleyOpenBoardPkg/BoardMtOlympus/build_board.py | 8 ++++---- .../PurleyOpenBoardPkg/BoardTiogaPass/build_board.py | 10 +++++----- 2 files changed, 9 insertions(+), 9 deletions(-) diff --git a/Platform/Intel/PurleyOpenBoardPkg/BoardMtOlympus/build_board.py b/Platform/Intel/PurleyOpenBoardPkg/BoardMtOlympus/build_board.py index 2ba615d3e4..ffe762e289 100644 --- a/Platform/Intel/PurleyOpenBoardPkg/BoardMtOlympus/build_board.py +++ b/Platform/Intel/PurleyOpenBoardPkg/BoardMtOlympus/build_board.py @@ -1,7 +1,7 @@ # @ build_board.py # This adds additional functions to the build_bios.py # -# Copyright (c) 2019, Intel Corporation. All rights reserved.<BR> +# Copyright (c) 2019 - 2023, Intel Corporation. All rights reserved.<BR> # SPDX-License-Identifier: BSD-2-Clause-Patent # @@ -48,7 +48,7 @@ def pre_build_ex(config, functions): print("AML_FILTER= ", config.get("AML_FILTER")) # build the command with arguments - command = ["python", + command = [sys.executable, os.path.join(config["MIN_PACKAGE_TOOLS"], "AmlGenOffset", "AmlGenOffset.py"), @@ -110,7 +110,7 @@ def post_build_ex(config, functions): print("post_build_ex Error") sys.exit(1) - common_patch_command = [os.path.join(config["PYTHON_HOME"], "python"), + common_patch_command = [sys.executable, os.path.join(config["MIN_PACKAGE_TOOLS"], "PatchFv", "PatchBinFv.py"), config["TARGET"], @@ -131,7 +131,7 @@ def post_build_ex(config, functions): print("Patch Error!") sys.exit(1) - common_rebase_command = [os.path.join(config["PYTHON_HOME"], "python"), + common_rebase_command = [sys.executable, os.path.join(config["MIN_PACKAGE_TOOLS"], "PatchFv", "RebaseBinFv.py"), config["TARGET"], diff --git a/Platform/Intel/PurleyOpenBoardPkg/BoardTiogaPass/build_board.py b/Platform/Intel/PurleyOpenBoardPkg/BoardTiogaPass/build_board.py index 46fd389ef1..3b66df7115 100644 --- a/Platform/Intel/PurleyOpenBoardPkg/BoardTiogaPass/build_board.py +++ b/Platform/Intel/PurleyOpenBoardPkg/BoardTiogaPass/build_board.py @@ -1,7 +1,7 @@ # @ build_board.py # This adds additional functions to the build_bios.py # -# Copyright (c) 2018, Intel Corporation. All rights reserved.<BR> +# Copyright (c) 2018 - 2023, Intel Corporation. All rights reserved.<BR> # Copyright (c) 2021, American Megatrends International LLC.<BR> # SPDX-License-Identifier: BSD-2-Clause-Patent # @@ -49,7 +49,7 @@ def pre_build_ex(config, functions): print("AML_FILTER= ", config.get("AML_FILTER")) # build the command with arguments - command = ["python", + command = [sys.executable, os.path.join(config["MIN_PACKAGE_TOOLS"], "AmlGenOffset", "AmlGenOffset.py"), @@ -111,7 +111,7 @@ def post_build_ex(config, functions): print("post_build_ex Error") sys.exit(1) - common_patch_command = [os.path.join(config["PYTHON_HOME"], "python"), + common_patch_command = [sys.executable, os.path.join(config["MIN_PACKAGE_TOOLS"], "PatchFv", "PatchBinFv.py"), config["TARGET"], @@ -132,7 +132,7 @@ def post_build_ex(config, functions): print("Patch Error!") sys.exit(1) - common_rebase_command = [os.path.join(config["PYTHON_HOME"], "python"), + common_rebase_command = [sys.executable, os.path.join(config["MIN_PACKAGE_TOOLS"], "PatchFv", "RebaseBinFv.py"), config["TARGET"], @@ -161,7 +161,7 @@ def post_build_ex(config, functions): print("Patch Error!") sys.exit(1) - common_patchbfv_command = [os.path.join(config["PYTHON_HOME"], "python"), + common_patchbfv_command = [sys.executable, os.path.join(config["MIN_PACKAGE_TOOLS"], "PatchFv", "PatchBfv.py"), os.path.join(config["BUILD_DIR_PATH"], -- 2.34.1 -=-=-=-=-=-=-=-=-=-=-=- Groups.io Links: You receive all messages sent to this group. View/Reply Online (#111423): https://edk2.groups.io/g/devel/message/111423 Mute This Topic: https://groups.io/mt/102667305/7686176 Group Owner: devel+owner@edk2.groups.io Unsubscribe: https://edk2.groups.io/g/devel/unsub [rebecca@openfw.io] -=-=-=-=-=-=-=-=-=-=-=- ^ permalink raw reply related [flat|nested] 8+ messages in thread
* Re: [edk2-devel] [edk2-platforms][PATCH v1] PurleyOpenBoardPkg: Replace Python Interpreter Name 2023-11-18 6:36 ` [edk2-devel] [edk2-platforms][PATCH v1] PurleyOpenBoardPkg: Replace Python Interpreter Name Nate DeSimone @ 2023-11-28 3:20 ` Chiu, Chasel 0 siblings, 0 replies; 8+ messages in thread From: Chiu, Chasel @ 2023-11-28 3:20 UTC (permalink / raw) To: Desimone, Nathaniel L, devel@edk2.groups.io Reviewed-by: Chasel Chiu <chasel.chiu@intel.com> Thanks, Chasel > -----Original Message----- > From: Desimone, Nathaniel L <nathaniel.l.desimone@intel.com> > Sent: Friday, November 17, 2023 10:36 PM > To: devel@edk2.groups.io > Cc: Chiu, Chasel <chasel.chiu@intel.com> > Subject: [edk2-platforms][PATCH v1] PurleyOpenBoardPkg: Replace Python > Interpreter Name > > Removes the assumption that the python interpreter will have the name python. > > Cc: Chasel Chiu <chasel.chiu@intel.com> > Signed-off-by: Nate DeSimone <nathaniel.l.desimone@intel.com> > --- > .../PurleyOpenBoardPkg/BoardMtOlympus/build_board.py | 8 ++++---- > .../PurleyOpenBoardPkg/BoardTiogaPass/build_board.py | 10 +++++----- > 2 files changed, 9 insertions(+), 9 deletions(-) > > diff --git a/Platform/Intel/PurleyOpenBoardPkg/BoardMtOlympus/build_board.py > b/Platform/Intel/PurleyOpenBoardPkg/BoardMtOlympus/build_board.py > index 2ba615d3e4..ffe762e289 100644 > --- a/Platform/Intel/PurleyOpenBoardPkg/BoardMtOlympus/build_board.py > +++ b/Platform/Intel/PurleyOpenBoardPkg/BoardMtOlympus/build_board.py > @@ -1,7 +1,7 @@ > # @ build_board.py > # This adds additional functions to the build_bios.py # -# Copyright (c) 2019, Intel > Corporation. All rights reserved.<BR> > +# Copyright (c) 2019 - 2023, Intel Corporation. All rights > +reserved.<BR> > # SPDX-License-Identifier: BSD-2-Clause-Patent # > > @@ -48,7 +48,7 @@ def pre_build_ex(config, functions): > print("AML_FILTER= ", config.get("AML_FILTER")) > > # build the command with arguments > - command = ["python", > + command = [sys.executable, > os.path.join(config["MIN_PACKAGE_TOOLS"], > "AmlGenOffset", > "AmlGenOffset.py"), @@ -110,7 +110,7 @@ def > post_build_ex(config, functions): > print("post_build_ex Error") > sys.exit(1) > > - common_patch_command = [os.path.join(config["PYTHON_HOME"], > "python"), > + common_patch_command = [sys.executable, > os.path.join(config["MIN_PACKAGE_TOOLS"], > "PatchFv", "PatchBinFv.py"), > config["TARGET"], @@ -131,7 +131,7 @@ def > post_build_ex(config, functions): > print("Patch Error!") > sys.exit(1) > > - common_rebase_command = [os.path.join(config["PYTHON_HOME"], > "python"), > + common_rebase_command = [sys.executable, > os.path.join(config["MIN_PACKAGE_TOOLS"], > "PatchFv", "RebaseBinFv.py"), > config["TARGET"], diff --git > a/Platform/Intel/PurleyOpenBoardPkg/BoardTiogaPass/build_board.py > b/Platform/Intel/PurleyOpenBoardPkg/BoardTiogaPass/build_board.py > index 46fd389ef1..3b66df7115 100644 > --- a/Platform/Intel/PurleyOpenBoardPkg/BoardTiogaPass/build_board.py > +++ b/Platform/Intel/PurleyOpenBoardPkg/BoardTiogaPass/build_board.py > @@ -1,7 +1,7 @@ > # @ build_board.py > # This adds additional functions to the build_bios.py # -# Copyright (c) 2018, Intel > Corporation. All rights reserved.<BR> > +# Copyright (c) 2018 - 2023, Intel Corporation. All rights > +reserved.<BR> > # Copyright (c) 2021, American Megatrends International LLC.<BR> # SPDX- > License-Identifier: BSD-2-Clause-Patent # @@ -49,7 +49,7 @@ def > pre_build_ex(config, functions): > print("AML_FILTER= ", config.get("AML_FILTER")) > > # build the command with arguments > - command = ["python", > + command = [sys.executable, > os.path.join(config["MIN_PACKAGE_TOOLS"], > "AmlGenOffset", > "AmlGenOffset.py"), @@ -111,7 +111,7 @@ def > post_build_ex(config, functions): > print("post_build_ex Error") > sys.exit(1) > > - common_patch_command = [os.path.join(config["PYTHON_HOME"], > "python"), > + common_patch_command = [sys.executable, > os.path.join(config["MIN_PACKAGE_TOOLS"], > "PatchFv", "PatchBinFv.py"), > config["TARGET"], @@ -132,7 +132,7 @@ def > post_build_ex(config, functions): > print("Patch Error!") > sys.exit(1) > > - common_rebase_command = [os.path.join(config["PYTHON_HOME"], > "python"), > + common_rebase_command = [sys.executable, > os.path.join(config["MIN_PACKAGE_TOOLS"], > "PatchFv", "RebaseBinFv.py"), > config["TARGET"], @@ -161,7 +161,7 @@ def > post_build_ex(config, functions): > print("Patch Error!") > sys.exit(1) > > - common_patchbfv_command = [os.path.join(config["PYTHON_HOME"], > "python"), > + common_patchbfv_command = [sys.executable, > os.path.join(config["MIN_PACKAGE_TOOLS"], > "PatchFv", "PatchBfv.py"), > os.path.join(config["BUILD_DIR_PATH"], > -- > 2.34.1 -=-=-=-=-=-=-=-=-=-=-=- Groups.io Links: You receive all messages sent to this group. View/Reply Online (#111785): https://edk2.groups.io/g/devel/message/111785 Mute This Topic: https://groups.io/mt/102667305/7686176 Group Owner: devel+owner@edk2.groups.io Unsubscribe: https://edk2.groups.io/g/devel/unsub [rebecca@openfw.io] -=-=-=-=-=-=-=-=-=-=-=- ^ permalink raw reply [flat|nested] 8+ messages in thread
* [edk2-devel] [edk2-platforms][PATCH v1] WhitleyOpenBoardPkg: Replace Python Interpreter Name 2023-11-18 6:36 [edk2-devel] [edk2-platforms][PATCH v1] MinPlatformPkg: Remove usage of PYTHON_HOME Nate DeSimone 2023-11-18 6:36 ` [edk2-devel] [edk2-platforms][PATCH v1] PurleyOpenBoardPkg: Replace Python Interpreter Name Nate DeSimone @ 2023-11-18 6:36 ` Nate DeSimone 2023-11-28 3:19 ` Chiu, Chasel 2023-11-28 3:17 ` [edk2-devel] [edk2-platforms][PATCH v1] MinPlatformPkg: Remove usage of PYTHON_HOME Chiu, Chasel ` (2 subsequent siblings) 4 siblings, 1 reply; 8+ messages in thread From: Nate DeSimone @ 2023-11-18 6:36 UTC (permalink / raw) To: devel; +Cc: Chasel Chiu Removes the assumption that the python interpreter will have the name python. Cc: Chasel Chiu <chasel.chiu@intel.com> Signed-off-by: Nate DeSimone <nathaniel.l.desimone@intel.com --- Platform/Intel/WhitleyOpenBoardPkg/Aowanda/build_board.py | 4 ++-- .../Intel/WhitleyOpenBoardPkg/JunctionCity/build_board.py | 4 ++-- .../Intel/WhitleyOpenBoardPkg/WilsonCityRvp/build_board.py | 4 ++-- 3 files changed, 6 insertions(+), 6 deletions(-) diff --git a/Platform/Intel/WhitleyOpenBoardPkg/Aowanda/build_board.py b/Platform/Intel/WhitleyOpenBoardPkg/Aowanda/build_board.py index af6985019b..70ef4c2201 100644 --- a/Platform/Intel/WhitleyOpenBoardPkg/Aowanda/build_board.py +++ b/Platform/Intel/WhitleyOpenBoardPkg/Aowanda/build_board.py @@ -2,7 +2,7 @@ # Extensions for building Aowanda using build_bios.py # # -# Copyright (c) 2021, Intel Corporation. All rights reserved.<BR> +# Copyright (c) 2021 - 2023, Intel Corporation. All rights reserved.<BR> # Copyright (c) 2022, American Megatrends International LLC. <BR> # SPDX-License-Identifier: BSD-2-Clause-Patent # @@ -107,7 +107,7 @@ def pre_build_ex(config, functions): if not os.path.exists(out_file_dir): os.mkdir(out_file_dir) - command = ["python", + command = [sys.executable, os.path.join(config["MIN_PACKAGE_TOOLS"], "AmlGenOffset", "AmlGenOffset.py"), "-d", "--aml_filter", config["AML_FILTER"], "-o", out_file_path, diff --git a/Platform/Intel/WhitleyOpenBoardPkg/JunctionCity/build_board.py b/Platform/Intel/WhitleyOpenBoardPkg/JunctionCity/build_board.py index ccf7a8c55a..61b893686c 100644 --- a/Platform/Intel/WhitleyOpenBoardPkg/JunctionCity/build_board.py +++ b/Platform/Intel/WhitleyOpenBoardPkg/JunctionCity/build_board.py @@ -2,7 +2,7 @@ # Extensions for building JunctionCity using build_bios.py # # -# Copyright (c) 2021, Intel Corporation. All rights reserved.<BR> +# Copyright (c) 2021 - 2023, Intel Corporation. All rights reserved.<BR> # Copyright (c) 2021, American Megatrends International LLC. <BR> # SPDX-License-Identifier: BSD-2-Clause-Patent # @@ -107,7 +107,7 @@ def pre_build_ex(config, functions): if not os.path.exists(out_file_dir): os.mkdir(out_file_dir) - command = ["python", + command = [sys.executable, os.path.join(config["MIN_PACKAGE_TOOLS"], "AmlGenOffset", "AmlGenOffset.py"), "-d", "--aml_filter", config["AML_FILTER"], "-o", out_file_path, diff --git a/Platform/Intel/WhitleyOpenBoardPkg/WilsonCityRvp/build_board.py b/Platform/Intel/WhitleyOpenBoardPkg/WilsonCityRvp/build_board.py index e2d32d3e13..8d62dd43b8 100644 --- a/Platform/Intel/WhitleyOpenBoardPkg/WilsonCityRvp/build_board.py +++ b/Platform/Intel/WhitleyOpenBoardPkg/WilsonCityRvp/build_board.py @@ -1,7 +1,7 @@ # @ build_board.py # Extensions for building WilsonCityRvp using build_bios.py # -# Copyright (c) 2021, Intel Corporation. All rights reserved.<BR> +# Copyright (c) 2021 - 2023, Intel Corporation. All rights reserved.<BR> # SPDX-License-Identifier: BSD-2-Clause-Patent # @@ -105,7 +105,7 @@ def pre_build_ex(config, functions): if not os.path.exists(out_file_dir): os.mkdir(out_file_dir) - command = ["python", + command = [sys.executable, os.path.join(config["MIN_PACKAGE_TOOLS"], "AmlGenOffset", "AmlGenOffset.py"), "-d", "--aml_filter", config["AML_FILTER"], "-o", out_file_path, -- 2.34.1 -=-=-=-=-=-=-=-=-=-=-=- Groups.io Links: You receive all messages sent to this group. View/Reply Online (#111424): https://edk2.groups.io/g/devel/message/111424 Mute This Topic: https://groups.io/mt/102667306/7686176 Group Owner: devel+owner@edk2.groups.io Unsubscribe: https://edk2.groups.io/g/devel/unsub [rebecca@openfw.io] -=-=-=-=-=-=-=-=-=-=-=- ^ permalink raw reply related [flat|nested] 8+ messages in thread
* Re: [edk2-devel] [edk2-platforms][PATCH v1] WhitleyOpenBoardPkg: Replace Python Interpreter Name 2023-11-18 6:36 ` [edk2-devel] [edk2-platforms][PATCH v1] WhitleyOpenBoardPkg: " Nate DeSimone @ 2023-11-28 3:19 ` Chiu, Chasel 0 siblings, 0 replies; 8+ messages in thread From: Chiu, Chasel @ 2023-11-28 3:19 UTC (permalink / raw) To: Desimone, Nathaniel L, devel@edk2.groups.io Reviewed-by: Chasel Chiu <chasel.chiu@intel.com> Thanks, Chasel > -----Original Message----- > From: Desimone, Nathaniel L <nathaniel.l.desimone@intel.com> > Sent: Friday, November 17, 2023 10:36 PM > To: devel@edk2.groups.io > Cc: Chiu, Chasel <chasel.chiu@intel.com> > Subject: [edk2-platforms][PATCH v1] WhitleyOpenBoardPkg: Replace Python > Interpreter Name > > Removes the assumption that the python interpreter will have the name python. > > Cc: Chasel Chiu <chasel.chiu@intel.com> > Signed-off-by: Nate DeSimone <nathaniel.l.desimone@intel.com > --- > Platform/Intel/WhitleyOpenBoardPkg/Aowanda/build_board.py | 4 ++-- > .../Intel/WhitleyOpenBoardPkg/JunctionCity/build_board.py | 4 ++-- > .../Intel/WhitleyOpenBoardPkg/WilsonCityRvp/build_board.py | 4 ++-- > 3 files changed, 6 insertions(+), 6 deletions(-) > > diff --git a/Platform/Intel/WhitleyOpenBoardPkg/Aowanda/build_board.py > b/Platform/Intel/WhitleyOpenBoardPkg/Aowanda/build_board.py > index af6985019b..70ef4c2201 100644 > --- a/Platform/Intel/WhitleyOpenBoardPkg/Aowanda/build_board.py > +++ b/Platform/Intel/WhitleyOpenBoardPkg/Aowanda/build_board.py > @@ -2,7 +2,7 @@ > # Extensions for building Aowanda using build_bios.py # # -# Copyright (c) 2021, > Intel Corporation. All rights reserved.<BR> > +# Copyright (c) 2021 - 2023, Intel Corporation. All rights > +reserved.<BR> > # Copyright (c) 2022, American Megatrends International LLC. <BR> # SPDX- > License-Identifier: BSD-2-Clause-Patent # @@ -107,7 +107,7 @@ def > pre_build_ex(config, functions): > if not os.path.exists(out_file_dir): > os.mkdir(out_file_dir) > > - command = ["python", > + command = [sys.executable, > os.path.join(config["MIN_PACKAGE_TOOLS"], "AmlGenOffset", > "AmlGenOffset.py"), > "-d", "--aml_filter", config["AML_FILTER"], > "-o", out_file_path, > diff --git a/Platform/Intel/WhitleyOpenBoardPkg/JunctionCity/build_board.py > b/Platform/Intel/WhitleyOpenBoardPkg/JunctionCity/build_board.py > index ccf7a8c55a..61b893686c 100644 > --- a/Platform/Intel/WhitleyOpenBoardPkg/JunctionCity/build_board.py > +++ b/Platform/Intel/WhitleyOpenBoardPkg/JunctionCity/build_board.py > @@ -2,7 +2,7 @@ > # Extensions for building JunctionCity using build_bios.py # # -# Copyright (c) > 2021, Intel Corporation. All rights reserved.<BR> > +# Copyright (c) 2021 - 2023, Intel Corporation. All rights > +reserved.<BR> > # Copyright (c) 2021, American Megatrends International LLC. <BR> # SPDX- > License-Identifier: BSD-2-Clause-Patent # @@ -107,7 +107,7 @@ def > pre_build_ex(config, functions): > if not os.path.exists(out_file_dir): > os.mkdir(out_file_dir) > > - command = ["python", > + command = [sys.executable, > os.path.join(config["MIN_PACKAGE_TOOLS"], "AmlGenOffset", > "AmlGenOffset.py"), > "-d", "--aml_filter", config["AML_FILTER"], > "-o", out_file_path, > diff --git a/Platform/Intel/WhitleyOpenBoardPkg/WilsonCityRvp/build_board.py > b/Platform/Intel/WhitleyOpenBoardPkg/WilsonCityRvp/build_board.py > index e2d32d3e13..8d62dd43b8 100644 > --- a/Platform/Intel/WhitleyOpenBoardPkg/WilsonCityRvp/build_board.py > +++ b/Platform/Intel/WhitleyOpenBoardPkg/WilsonCityRvp/build_board.py > @@ -1,7 +1,7 @@ > # @ build_board.py > # Extensions for building WilsonCityRvp using build_bios.py # -# Copyright (c) > 2021, Intel Corporation. All rights reserved.<BR> > +# Copyright (c) 2021 - 2023, Intel Corporation. All rights > +reserved.<BR> > # SPDX-License-Identifier: BSD-2-Clause-Patent # > > @@ -105,7 +105,7 @@ def pre_build_ex(config, functions): > if not os.path.exists(out_file_dir): > os.mkdir(out_file_dir) > > - command = ["python", > + command = [sys.executable, > os.path.join(config["MIN_PACKAGE_TOOLS"], "AmlGenOffset", > "AmlGenOffset.py"), > "-d", "--aml_filter", config["AML_FILTER"], > "-o", out_file_path, > -- > 2.34.1 -=-=-=-=-=-=-=-=-=-=-=- Groups.io Links: You receive all messages sent to this group. View/Reply Online (#111784): https://edk2.groups.io/g/devel/message/111784 Mute This Topic: https://groups.io/mt/102667306/7686176 Group Owner: devel+owner@edk2.groups.io Unsubscribe: https://edk2.groups.io/g/devel/unsub [rebecca@openfw.io] -=-=-=-=-=-=-=-=-=-=-=- ^ permalink raw reply [flat|nested] 8+ messages in thread
* Re: [edk2-devel] [edk2-platforms][PATCH v1] MinPlatformPkg: Remove usage of PYTHON_HOME 2023-11-18 6:36 [edk2-devel] [edk2-platforms][PATCH v1] MinPlatformPkg: Remove usage of PYTHON_HOME Nate DeSimone 2023-11-18 6:36 ` [edk2-devel] [edk2-platforms][PATCH v1] PurleyOpenBoardPkg: Replace Python Interpreter Name Nate DeSimone 2023-11-18 6:36 ` [edk2-devel] [edk2-platforms][PATCH v1] WhitleyOpenBoardPkg: " Nate DeSimone @ 2023-11-28 3:17 ` Chiu, Chasel [not found] ` <1798A438690A80D8.26894@groups.io> [not found] ` <1798A43863B4D009.26894@groups.io> 4 siblings, 0 replies; 8+ messages in thread From: Chiu, Chasel @ 2023-11-28 3:17 UTC (permalink / raw) To: Desimone, Nathaniel L, devel@edk2.groups.io; +Cc: Gao, Liming, Dong, Eric Reviewed-by: Chasel Chiu <chasel.chiu@intel.com> Thanks, Chasel > -----Original Message----- > From: Desimone, Nathaniel L <nathaniel.l.desimone@intel.com> > Sent: Friday, November 17, 2023 10:36 PM > To: devel@edk2.groups.io > Cc: Chiu, Chasel <chasel.chiu@intel.com>; Gao, Liming > <gaoliming@byosoft.com.cn>; Dong, Eric <eric.dong@intel.com> > Subject: [edk2-platforms][PATCH v1] MinPlatformPkg: Remove usage of > PYTHON_HOME > > Removes usage PYTHON_HOME from RebaseFspBinBaseAddress.py > > Cc: Chasel Chiu <chasel.chiu@intel.com> > Cc: Liming Gao <gaoliming@byosoft.com.cn> > Cc: Eric Dong <eric.dong@intel.com> > Signed-off-by: Nate DeSimone <nathaniel.l.desimone@intel.com> > --- > .../MinPlatformPkg/Tools/Fsp/RebaseFspBinBaseAddress.py | 8 ++------ > 1 file changed, 2 insertions(+), 6 deletions(-) > > diff --git > a/Platform/Intel/MinPlatformPkg/Tools/Fsp/RebaseFspBinBaseAddress.py > b/Platform/Intel/MinPlatformPkg/Tools/Fsp/RebaseFspBinBaseAddress.py > index b7e4bcf5f9..32db9eb1c4 100644 > --- a/Platform/Intel/MinPlatformPkg/Tools/Fsp/RebaseFspBinBaseAddress.py > +++ b/Platform/Intel/MinPlatformPkg/Tools/Fsp/RebaseFspBinBaseAddress.py > @@ -1,6 +1,6 @@ > ## @ RebaseFspBinBaseAddress.py > # > -# Copyright (c) 2019 - 2021, Intel Corporation. All rights reserved.<BR> > +# Copyright (c) 2019 - 2023, Intel Corporation. All rights > +reserved.<BR> > # SPDX-License-Identifier: BSD-2-Clause-Patent # > > @@ -65,11 +65,7 @@ file.close() > # Get FSP-M Size, in order to calculate the FSP-T Base. Used SplitFspBin.py script > # to dump the header, and get the ImageSize in FSP-M section # -pythontool = > 'python' > -if 'PYTHON_HOME' in os.environ: > - pythontool = os.environ['PYTHON_HOME'] + os.sep + 'python' > -else: > - pythontool = sys.executable > +pythontool = sys.executable > Process = subprocess.Popen([pythontool, splitFspBinPath, "info","- > f",fspBinFilePath], stdout=subprocess.PIPE) Output = Process.communicate()[0] > FsptInfo = Output.rsplit(b"FSP_M", 1); > -- > 2.34.1 -=-=-=-=-=-=-=-=-=-=-=- Groups.io Links: You receive all messages sent to this group. View/Reply Online (#111782): https://edk2.groups.io/g/devel/message/111782 Mute This Topic: https://groups.io/mt/102667304/7686176 Group Owner: devel+owner@edk2.groups.io Unsubscribe: https://edk2.groups.io/g/devel/unsub [rebecca@openfw.io] -=-=-=-=-=-=-=-=-=-=-=- ^ permalink raw reply [flat|nested] 8+ messages in thread
[parent not found: <1798A438690A80D8.26894@groups.io>]
* Re: [edk2-devel] [edk2-platforms][PATCH v1] WhitleyOpenBoardPkg: Replace Python Interpreter Name [not found] ` <1798A438690A80D8.26894@groups.io> @ 2023-11-29 0:03 ` Nate DeSimone 0 siblings, 0 replies; 8+ messages in thread From: Nate DeSimone @ 2023-11-29 0:03 UTC (permalink / raw) To: devel@edk2.groups.io; +Cc: Chiu, Chasel Pushed as 7528798 -----Original Message----- From: devel@edk2.groups.io <devel@edk2.groups.io> On Behalf Of Nate DeSimone Sent: Friday, November 17, 2023 10:36 PM To: devel@edk2.groups.io Cc: Chiu, Chasel <chasel.chiu@intel.com> Subject: [edk2-devel] [edk2-platforms][PATCH v1] WhitleyOpenBoardPkg: Replace Python Interpreter Name Removes the assumption that the python interpreter will have the name python. Cc: Chasel Chiu <chasel.chiu@intel.com> Signed-off-by: Nate DeSimone <nathaniel.l.desimone@intel.com --- Platform/Intel/WhitleyOpenBoardPkg/Aowanda/build_board.py | 4 ++-- .../Intel/WhitleyOpenBoardPkg/JunctionCity/build_board.py | 4 ++-- .../Intel/WhitleyOpenBoardPkg/WilsonCityRvp/build_board.py | 4 ++-- 3 files changed, 6 insertions(+), 6 deletions(-) diff --git a/Platform/Intel/WhitleyOpenBoardPkg/Aowanda/build_board.py b/Platform/Intel/WhitleyOpenBoardPkg/Aowanda/build_board.py index af6985019b..70ef4c2201 100644 --- a/Platform/Intel/WhitleyOpenBoardPkg/Aowanda/build_board.py +++ b/Platform/Intel/WhitleyOpenBoardPkg/Aowanda/build_board.py @@ -2,7 +2,7 @@ # Extensions for building Aowanda using build_bios.py # # -# Copyright (c) 2021, Intel Corporation. All rights reserved.<BR> +# Copyright (c) 2021 - 2023, Intel Corporation. All rights +reserved.<BR> # Copyright (c) 2022, American Megatrends International LLC. <BR> # SPDX-License-Identifier: BSD-2-Clause-Patent # @@ -107,7 +107,7 @@ def pre_build_ex(config, functions): if not os.path.exists(out_file_dir): os.mkdir(out_file_dir) - command = ["python", + command = [sys.executable, os.path.join(config["MIN_PACKAGE_TOOLS"], "AmlGenOffset", "AmlGenOffset.py"), "-d", "--aml_filter", config["AML_FILTER"], "-o", out_file_path, diff --git a/Platform/Intel/WhitleyOpenBoardPkg/JunctionCity/build_board.py b/Platform/Intel/WhitleyOpenBoardPkg/JunctionCity/build_board.py index ccf7a8c55a..61b893686c 100644 --- a/Platform/Intel/WhitleyOpenBoardPkg/JunctionCity/build_board.py +++ b/Platform/Intel/WhitleyOpenBoardPkg/JunctionCity/build_board.py @@ -2,7 +2,7 @@ # Extensions for building JunctionCity using build_bios.py # # -# Copyright (c) 2021, Intel Corporation. All rights reserved.<BR> +# Copyright (c) 2021 - 2023, Intel Corporation. All rights +reserved.<BR> # Copyright (c) 2021, American Megatrends International LLC. <BR> # SPDX-License-Identifier: BSD-2-Clause-Patent # @@ -107,7 +107,7 @@ def pre_build_ex(config, functions): if not os.path.exists(out_file_dir): os.mkdir(out_file_dir) - command = ["python", + command = [sys.executable, os.path.join(config["MIN_PACKAGE_TOOLS"], "AmlGenOffset", "AmlGenOffset.py"), "-d", "--aml_filter", config["AML_FILTER"], "-o", out_file_path, diff --git a/Platform/Intel/WhitleyOpenBoardPkg/WilsonCityRvp/build_board.py b/Platform/Intel/WhitleyOpenBoardPkg/WilsonCityRvp/build_board.py index e2d32d3e13..8d62dd43b8 100644 --- a/Platform/Intel/WhitleyOpenBoardPkg/WilsonCityRvp/build_board.py +++ b/Platform/Intel/WhitleyOpenBoardPkg/WilsonCityRvp/build_board.py @@ -1,7 +1,7 @@ # @ build_board.py # Extensions for building WilsonCityRvp using build_bios.py # -# Copyright (c) 2021, Intel Corporation. All rights reserved.<BR> +# Copyright (c) 2021 - 2023, Intel Corporation. All rights +reserved.<BR> # SPDX-License-Identifier: BSD-2-Clause-Patent # @@ -105,7 +105,7 @@ def pre_build_ex(config, functions): if not os.path.exists(out_file_dir): os.mkdir(out_file_dir) - command = ["python", + command = [sys.executable, os.path.join(config["MIN_PACKAGE_TOOLS"], "AmlGenOffset", "AmlGenOffset.py"), "-d", "--aml_filter", config["AML_FILTER"], "-o", out_file_path, -- 2.34.1 -=-=-=-=-=-=-=-=-=-=-=- Groups.io Links: You receive all messages sent to this group. View/Reply Online (#111829): https://edk2.groups.io/g/devel/message/111829 Mute This Topic: https://groups.io/mt/102862612/7686176 Group Owner: devel+owner@edk2.groups.io Unsubscribe: https://edk2.groups.io/g/devel/unsub [rebecca@openfw.io] -=-=-=-=-=-=-=-=-=-=-=- ^ permalink raw reply related [flat|nested] 8+ messages in thread
[parent not found: <1798A43863B4D009.26894@groups.io>]
* Re: [edk2-devel] [edk2-platforms][PATCH v1] PurleyOpenBoardPkg: Replace Python Interpreter Name [not found] ` <1798A43863B4D009.26894@groups.io> @ 2023-11-29 0:03 ` Nate DeSimone 0 siblings, 0 replies; 8+ messages in thread From: Nate DeSimone @ 2023-11-29 0:03 UTC (permalink / raw) To: devel@edk2.groups.io; +Cc: Chiu, Chasel Pushed as 15e9fae -----Original Message----- From: devel@edk2.groups.io <devel@edk2.groups.io> On Behalf Of Nate DeSimone Sent: Friday, November 17, 2023 10:36 PM To: devel@edk2.groups.io Cc: Chiu, Chasel <chasel.chiu@intel.com> Subject: [edk2-devel] [edk2-platforms][PATCH v1] PurleyOpenBoardPkg: Replace Python Interpreter Name Removes the assumption that the python interpreter will have the name python. Cc: Chasel Chiu <chasel.chiu@intel.com> Signed-off-by: Nate DeSimone <nathaniel.l.desimone@intel.com> --- .../PurleyOpenBoardPkg/BoardMtOlympus/build_board.py | 8 ++++---- .../PurleyOpenBoardPkg/BoardTiogaPass/build_board.py | 10 +++++----- 2 files changed, 9 insertions(+), 9 deletions(-) diff --git a/Platform/Intel/PurleyOpenBoardPkg/BoardMtOlympus/build_board.py b/Platform/Intel/PurleyOpenBoardPkg/BoardMtOlympus/build_board.py index 2ba615d3e4..ffe762e289 100644 --- a/Platform/Intel/PurleyOpenBoardPkg/BoardMtOlympus/build_board.py +++ b/Platform/Intel/PurleyOpenBoardPkg/BoardMtOlympus/build_board.py @@ -1,7 +1,7 @@ # @ build_board.py # This adds additional functions to the build_bios.py # -# Copyright (c) 2019, Intel Corporation. All rights reserved.<BR> +# Copyright (c) 2019 - 2023, Intel Corporation. All rights +reserved.<BR> # SPDX-License-Identifier: BSD-2-Clause-Patent # @@ -48,7 +48,7 @@ def pre_build_ex(config, functions): print("AML_FILTER= ", config.get("AML_FILTER")) # build the command with arguments - command = ["python", + command = [sys.executable, os.path.join(config["MIN_PACKAGE_TOOLS"], "AmlGenOffset", "AmlGenOffset.py"), @@ -110,7 +110,7 @@ def post_build_ex(config, functions): print("post_build_ex Error") sys.exit(1) - common_patch_command = [os.path.join(config["PYTHON_HOME"], "python"), + common_patch_command = [sys.executable, os.path.join(config["MIN_PACKAGE_TOOLS"], "PatchFv", "PatchBinFv.py"), config["TARGET"], @@ -131,7 +131,7 @@ def post_build_ex(config, functions): print("Patch Error!") sys.exit(1) - common_rebase_command = [os.path.join(config["PYTHON_HOME"], "python"), + common_rebase_command = [sys.executable, os.path.join(config["MIN_PACKAGE_TOOLS"], "PatchFv", "RebaseBinFv.py"), config["TARGET"], diff --git a/Platform/Intel/PurleyOpenBoardPkg/BoardTiogaPass/build_board.py b/Platform/Intel/PurleyOpenBoardPkg/BoardTiogaPass/build_board.py index 46fd389ef1..3b66df7115 100644 --- a/Platform/Intel/PurleyOpenBoardPkg/BoardTiogaPass/build_board.py +++ b/Platform/Intel/PurleyOpenBoardPkg/BoardTiogaPass/build_board.py @@ -1,7 +1,7 @@ # @ build_board.py # This adds additional functions to the build_bios.py # -# Copyright (c) 2018, Intel Corporation. All rights reserved.<BR> +# Copyright (c) 2018 - 2023, Intel Corporation. All rights +reserved.<BR> # Copyright (c) 2021, American Megatrends International LLC.<BR> # SPDX-License-Identifier: BSD-2-Clause-Patent # @@ -49,7 +49,7 @@ def pre_build_ex(config, functions): print("AML_FILTER= ", config.get("AML_FILTER")) # build the command with arguments - command = ["python", + command = [sys.executable, os.path.join(config["MIN_PACKAGE_TOOLS"], "AmlGenOffset", "AmlGenOffset.py"), @@ -111,7 +111,7 @@ def post_build_ex(config, functions): print("post_build_ex Error") sys.exit(1) - common_patch_command = [os.path.join(config["PYTHON_HOME"], "python"), + common_patch_command = [sys.executable, os.path.join(config["MIN_PACKAGE_TOOLS"], "PatchFv", "PatchBinFv.py"), config["TARGET"], @@ -132,7 +132,7 @@ def post_build_ex(config, functions): print("Patch Error!") sys.exit(1) - common_rebase_command = [os.path.join(config["PYTHON_HOME"], "python"), + common_rebase_command = [sys.executable, os.path.join(config["MIN_PACKAGE_TOOLS"], "PatchFv", "RebaseBinFv.py"), config["TARGET"], @@ -161,7 +161,7 @@ def post_build_ex(config, functions): print("Patch Error!") sys.exit(1) - common_patchbfv_command = [os.path.join(config["PYTHON_HOME"], "python"), + common_patchbfv_command = [sys.executable, os.path.join(config["MIN_PACKAGE_TOOLS"], "PatchFv", "PatchBfv.py"), os.path.join(config["BUILD_DIR_PATH"], -- 2.34.1 -=-=-=-=-=-=-=-=-=-=-=- Groups.io Links: You receive all messages sent to this group. View/Reply Online (#111827): https://edk2.groups.io/g/devel/message/111827 Mute This Topic: https://groups.io/mt/102862606/7686176 Group Owner: devel+owner@edk2.groups.io Unsubscribe: https://edk2.groups.io/g/devel/unsub [rebecca@openfw.io] -=-=-=-=-=-=-=-=-=-=-=- ^ permalink raw reply related [flat|nested] 8+ messages in thread
end of thread, other threads:[~2023-11-29 0:03 UTC | newest] Thread overview: 8+ messages (download: mbox.gz follow: Atom feed -- links below jump to the message on this page -- 2023-11-18 6:36 [edk2-devel] [edk2-platforms][PATCH v1] MinPlatformPkg: Remove usage of PYTHON_HOME Nate DeSimone 2023-11-18 6:36 ` [edk2-devel] [edk2-platforms][PATCH v1] PurleyOpenBoardPkg: Replace Python Interpreter Name Nate DeSimone 2023-11-28 3:20 ` Chiu, Chasel 2023-11-18 6:36 ` [edk2-devel] [edk2-platforms][PATCH v1] WhitleyOpenBoardPkg: " Nate DeSimone 2023-11-28 3:19 ` Chiu, Chasel 2023-11-28 3:17 ` [edk2-devel] [edk2-platforms][PATCH v1] MinPlatformPkg: Remove usage of PYTHON_HOME Chiu, Chasel [not found] ` <1798A438690A80D8.26894@groups.io> 2023-11-29 0:03 ` [edk2-devel] [edk2-platforms][PATCH v1] WhitleyOpenBoardPkg: Replace Python Interpreter Name Nate DeSimone [not found] ` <1798A43863B4D009.26894@groups.io> 2023-11-29 0:03 ` [edk2-devel] [edk2-platforms][PATCH v1] PurleyOpenBoardPkg: " Nate DeSimone
This is a public inbox, see mirroring instructions for how to clone and mirror all data and code used for this inbox