lenovo / pylxca

PyLXCA is Python based interface for Lenovo xClarity Administrator APIs.
Apache License 2.0
14 stars 11 forks source link

"pip install pylxca" have python pointing to wrong path in lxca_shell #8

Closed tjyang closed 5 years ago

tjyang commented 5 years ago

I haven't tested on Ubuntu but for CentOS, pip install pylxca has issue. 1st line python reference pointing to non-system python.

[root@r10wk01 ~]# sudo pip install pylxca
[root@r10wk01 ~]# head  -1 /bin/lxca_shell
#!/home/prashant/.virtualenvs/python3/bin/python
[root@r10wk01 ~]# cat /etc/redhat-release
CentOS Linux release 7.6.1810 (Core)
[root@r10wk01 ~]# 
prabhosa commented 5 years ago

https://stackoverflow.com/questions/1530702/dont-touch-my-shebang

Its apparently issue with how setuptool create wheel file. I have uploaded new package in pypi which should resolve this problem.

prabhosa commented 5 years ago

code change i done to generate new wheel

diff --git a/lxca_shell b/lxca_shell index dcbcbd4..3391daf 100755 --- a/lxca_shell +++ b/lxca_shell @@ -1,4 +1,4 @@ -#!/usr/bin/env python +#!python '''

prabhosa commented 5 years ago

I am not changing this in lxca_shell script as it will make lxca_shell fail in local env

prabhosa commented 5 years ago

@tjyang Can you please check and close this bug its working for you

tjyang commented 5 years ago

I think this ticket can be closed. Following is on a manageiq hammer-3 appliance.

[root@r2wk01 ~]# pip install --upgrade pip
Collecting pip
  Downloading https://files.pythonhosted.org/packages/d8/f3/413bab4ff08e1fc4828dfc59996d721917df8e8583ea85385d51125dceff/pip-19.0.3-py2.py3-none-any.whl (1.4MB)
    100% |████████████████████████████████| 1.4MB 792kB/s
Installing collected packages: pip
  Found existing installation: pip 8.1.2
    Uninstalling pip-8.1.2:
      Successfully uninstalled pip-8.1.2
Successfully installed pip-19.0.3
[root@r2wk01 ~]# pip install pylxca
DEPRECATION: Python 2.7 will reach the end of its life on January 1st, 2020. Please upgrade your Python as Python 2.7 won't be maintained after that date. A future version of pip will drop support for Python 2.7.
Requirement already satisfied: pylxca in /usr/lib/python2.7/site-packages (2.1.1)
Requirement already satisfied: mock>=2.0.0 in /usr/lib/python2.7/site-packages (from pylxca) (2.0.0)
Requirement already satisfied: requests>=2.7.0 in /usr/lib/python2.7/site-packages (from pylxca) (2.21.0)
Requirement already satisfied: requests-toolbelt>=0.8.0 in /usr/lib/python2.7/site-packages (from pylxca) (0.8.0)
Requirement already satisfied: funcsigs>=1; python_version < "3.3" in /usr/lib/python2.7/site-packages (from mock>=2.0.0->pylxca) (1.0.2)
Requirement already satisfied: six>=1.9 in /usr/lib/python2.7/site-packages (from mock>=2.0.0->pylxca) (1.10.0)
Requirement already satisfied: pbr>=0.11 in /usr/lib/python2.7/site-packages (from mock>=2.0.0->pylxca) (5.1.1)
Requirement already satisfied: urllib3<1.25,>=1.21.1 in /usr/lib/python2.7/site-packages (from requests>=2.7.0->pylxca) (1.21.1)
Requirement already satisfied: chardet<3.1.0,>=3.0.2 in /usr/lib/python2.7/site-packages (from requests>=2.7.0->pylxca) (3.0.4)
Requirement already satisfied: idna<2.9,>=2.5 in /usr/lib/python2.7/site-packages (from requests>=2.7.0->pylxca) (2.8)
Requirement already satisfied: certifi>=2017.4.17 in /usr/lib/python2.7/site-packages (from requests>=2.7.0->pylxca) (2018.11.29)
[root@r2wk01 ~]# head  -1 /bin/lxca_shell
#!/usr/bin/python2
[root@wk01 ~]# cat /etc/redhat-release
CentOS Linux release 7.6.1810 (Core)
[root@r2wk01 ~]#