* [edk2-staging/EdkRepo] [PATCH] EdkRepo: Allow user to override python command
@ 2019-12-10 9:55 Nate DeSimone
2019-12-12 0:21 ` [edk2-devel] " Desimone, Ashley E
0 siblings, 1 reply; 2+ messages in thread
From: Nate DeSimone @ 2019-12-10 9:55 UTC (permalink / raw)
To: devel; +Cc: Ashley DeSimone, Puja Pandya, Bret Barkelew
The --py flag will allow the user to specify a specific version of
python removing the need for the python3 sym-link on some systems.
Cc: Ashley DeSimone <ashley.e.desimone@intel.com>
Cc: Puja Pandya <puja.pandya@intel.com>
Cc: Bret Barkelew <Bret.Barkelew@microsoft.com>
Signed-off-by: Nate DeSimone <nathaniel.l.desimone@intel.com>
---
edkrepo_installer/linux-scripts/install.py | 17 ++++++++++++++---
1 file changed, 14 insertions(+), 3 deletions(-)
diff --git a/edkrepo_installer/linux-scripts/install.py b/edkrepo_installer/linux-scripts/install.py
index a05ce6a..e7d3f91 100755
--- a/edkrepo_installer/linux-scripts/install.py
+++ b/edkrepo_installer/linux-scripts/install.py
@@ -27,6 +27,7 @@ tool_sign_on = 'Installer for edkrepo version {}\nCopyright(c) Intel Corporation
cfg_dir = '.edkrepo'
cfg_src_dir = os.path.abspath('config')
whl_src_dir = os.path.abspath('wheels')
+def_python = 'python3'
def default_run(cmd):
return subprocess.run(cmd, universal_newlines=True, stdout=subprocess.PIPE, stderr=subprocess.STDOUT, check=True)
@@ -45,6 +46,7 @@ def init_logger(verbose):
def get_args():
parser = ArgumentParser()
parser.add_argument('-l', '--local', action='store_true', default=False, help='Install edkrepo to the user directory instead of system wide')
+ parser.add_argument('-p', '--py', action='store', default=None, help='Specify the python command to use when installing')
parser.add_argument('-u', '--user', action='store', default=None, help='Specify user account to install edkrepo support on')
parser.add_argument('-v', '--verbose', action='store_true', default=False, help='Enables verbose output')
return parser.parse_args()
@@ -75,7 +77,7 @@ def get_required_wheels():
'version':wheel.attrib['Version'],
'install':True}
break
- pip_cmd = ['python3', '-m', 'pip', 'list', '--legacy']
+ pip_cmd = [def_python, '-m', 'pip', 'list', '--legacy']
try:
res = default_run(pip_cmd)
except:
@@ -117,8 +119,14 @@ def _check_version(current, expected):
return 0
def do_install():
+ global def_python
+ org_python = None
+
# Parse command line
args = get_args()
+ if args.py is not None:
+ org_python = def_python
+ def_python = args.py
# Enable logging output
log = init_logger(args.verbose)
@@ -132,6 +140,9 @@ def do_install():
except:
log.error('ERROR: Missing installer configuration file.')
return 1
+ if org_python is not None:
+ cfg['req_tools'][def_python] = cfg['req_tools'][org_python]
+ cfg['req_tools'].pop(org_python)
try:
sha_data = configparser.ConfigParser(allow_no_value=True)
sha_data.read(os.path.join(cfg_src_dir, 'sha_data.cfg'))
@@ -299,13 +310,13 @@ def do_install():
uninstall_whl = wheels_to_install[whl_name]['uninstall']
if uninstall_whl:
try:
- res = default_run(['python3', '-m', 'pip', 'uninstall', '--yes', whl_name])
+ res = default_run([def_python, '-m', 'pip', 'uninstall', '--yes', whl_name])
except:
log.info('- Failed to uninstall {}'.format(whl_name))
return 1
log.info('+ Uninstalled {}'.format(whl_name))
if install_whl:
- install_cmd = ['python3', '-m', 'pip', 'install']
+ install_cmd = [def_python, '-m', 'pip', 'install']
if args.local:
install_cmd.append('--user')
install_cmd.append(os.path.join(whl_src_dir, whl))
--
2.20.1
^ permalink raw reply related [flat|nested] 2+ messages in thread
* Re: [edk2-devel] [edk2-staging/EdkRepo] [PATCH] EdkRepo: Allow user to override python command
2019-12-10 9:55 [edk2-staging/EdkRepo] [PATCH] EdkRepo: Allow user to override python command Nate DeSimone
@ 2019-12-12 0:21 ` Desimone, Ashley E
0 siblings, 0 replies; 2+ messages in thread
From: Desimone, Ashley E @ 2019-12-12 0:21 UTC (permalink / raw)
To: devel@edk2.groups.io, Desimone, Nathaniel L; +Cc: Pandya, Puja, Bret Barkelew
Reviewed-by: Ashley DeSimone <ashley.e.desimone@intel.com>
-----Original Message-----
From: devel@edk2.groups.io [mailto:devel@edk2.groups.io] On Behalf Of Nate DeSimone
Sent: Tuesday, December 10, 2019 1:55 AM
To: devel@edk2.groups.io
Cc: Desimone, Ashley E <ashley.e.desimone@intel.com>; Pandya, Puja <puja.pandya@intel.com>; Bret Barkelew <Bret.Barkelew@microsoft.com>
Subject: [edk2-devel] [edk2-staging/EdkRepo] [PATCH] EdkRepo: Allow user to override python command
The --py flag will allow the user to specify a specific version of python removing the need for the python3 sym-link on some systems.
Cc: Ashley DeSimone <ashley.e.desimone@intel.com>
Cc: Puja Pandya <puja.pandya@intel.com>
Cc: Bret Barkelew <Bret.Barkelew@microsoft.com>
Signed-off-by: Nate DeSimone <nathaniel.l.desimone@intel.com>
---
edkrepo_installer/linux-scripts/install.py | 17 ++++++++++++++---
1 file changed, 14 insertions(+), 3 deletions(-)
diff --git a/edkrepo_installer/linux-scripts/install.py b/edkrepo_installer/linux-scripts/install.py
index a05ce6a..e7d3f91 100755
--- a/edkrepo_installer/linux-scripts/install.py
+++ b/edkrepo_installer/linux-scripts/install.py
@@ -27,6 +27,7 @@ tool_sign_on = 'Installer for edkrepo version {}\nCopyright(c) Intel Corporation cfg_dir = '.edkrepo'
cfg_src_dir = os.path.abspath('config') whl_src_dir = os.path.abspath('wheels')
+def_python = 'python3'
def default_run(cmd):
return subprocess.run(cmd, universal_newlines=True, stdout=subprocess.PIPE, stderr=subprocess.STDOUT, check=True) @@ -45,6 +46,7 @@ def init_logger(verbose):
def get_args():
parser = ArgumentParser()
parser.add_argument('-l', '--local', action='store_true', default=False, help='Install edkrepo to the user directory instead of system wide')
+ parser.add_argument('-p', '--py', action='store', default=None,
+ help='Specify the python command to use when installing')
parser.add_argument('-u', '--user', action='store', default=None, help='Specify user account to install edkrepo support on')
parser.add_argument('-v', '--verbose', action='store_true', default=False, help='Enables verbose output')
return parser.parse_args()
@@ -75,7 +77,7 @@ def get_required_wheels():
'version':wheel.attrib['Version'],
'install':True}
break
- pip_cmd = ['python3', '-m', 'pip', 'list', '--legacy']
+ pip_cmd = [def_python, '-m', 'pip', 'list', '--legacy']
try:
res = default_run(pip_cmd)
except:
@@ -117,8 +119,14 @@ def _check_version(current, expected):
return 0
def do_install():
+ global def_python
+ org_python = None
+
# Parse command line
args = get_args()
+ if args.py is not None:
+ org_python = def_python
+ def_python = args.py
# Enable logging output
log = init_logger(args.verbose)
@@ -132,6 +140,9 @@ def do_install():
except:
log.error('ERROR: Missing installer configuration file.')
return 1
+ if org_python is not None:
+ cfg['req_tools'][def_python] = cfg['req_tools'][org_python]
+ cfg['req_tools'].pop(org_python)
try:
sha_data = configparser.ConfigParser(allow_no_value=True)
sha_data.read(os.path.join(cfg_src_dir, 'sha_data.cfg')) @@ -299,13 +310,13 @@ def do_install():
uninstall_whl = wheels_to_install[whl_name]['uninstall']
if uninstall_whl:
try:
- res = default_run(['python3', '-m', 'pip', 'uninstall', '--yes', whl_name])
+ res = default_run([def_python, '-m', 'pip',
+ 'uninstall', '--yes', whl_name])
except:
log.info('- Failed to uninstall {}'.format(whl_name))
return 1
log.info('+ Uninstalled {}'.format(whl_name))
if install_whl:
- install_cmd = ['python3', '-m', 'pip', 'install']
+ install_cmd = [def_python, '-m', 'pip', 'install']
if args.local:
install_cmd.append('--user')
install_cmd.append(os.path.join(whl_src_dir, whl))
--
2.20.1
^ permalink raw reply related [flat|nested] 2+ messages in thread
end of thread, other threads:[~2019-12-12 0:21 UTC | newest]
Thread overview: 2+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2019-12-10 9:55 [edk2-staging/EdkRepo] [PATCH] EdkRepo: Allow user to override python command Nate DeSimone
2019-12-12 0:21 ` [edk2-devel] " Desimone, Ashley E
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox