-
Notifications
You must be signed in to change notification settings - Fork 524
Conversation
@aviau thoughts? |
What? What kind of conflict? The distro package is named python-influxdb. |
@aviau please see the last line [root@xy-dev influxdb-python]# rpm -ivh ~/sandbox/3parties/setuppackets/deps/influxdb-0.10.0-1.x86_64.rpm
Preparing... ########################################### [100%]
1:influxdb ########################################### [100%]
[root@xy-dev influxdb-python]# rpm -q influxdb
influxdb-0.10.0-1.x86_64
[root@xy-dev influxdb-python]# python setup.py bdist_rpm
[root@xy-dev influxdb-python]# rpm -i dist/influxdb-2.12.0-1.noarch.rpm
[root@xy-dev influxdb-python]# rpm -q influxdb
influxdb-0.10.0-1.x86_64
influxdb-2.12.0-1.noarch
|
@aviau if the name in setup.py is |
The proper naming for python libraries is python-influxdb. You generated a package with a dirty script, please just name the RPM properly. pyhton-influxdb has been in Debian for months now. |
@aviau bug I just use the
|
@aviau as you said, |
Using setup.py to generate a rpm is non-standard. If you can find a way that will rename the package name to python-influxdb without renaming the module name, I will merge. But I am in no way supporting the use of setup.py to generate a RPM. The package name is different than the module name. For example, requests's package name is python-requests but the module name is requests |
@aviau so, how to build rpm package with prefix |
Take a look at this guide. Or some other guide. This repository has nothing to do with distro packaging, so this is not the right place to ask. I happen to be the Debian maintainer for influxdb-python, but I maintain it in another repository in the Debian infrastructure. |
@aviau but why not change it to make more clear? currently the document said:
why the name in pip command is |
@aviau It is different from |
@aviau package name can be different from module name, but why not make it the same :) |
Because that is what every other package does. ALL of them. From http://pypi-ranking.info/alltime
Out of the 50 on the top list, only python-dateutil has the NONE of them have a
Because I am following standards. |
things are different, there are no other |
@aviau and |
The distro package name for those packages is still different. For example the debian package for |
@aviau if you see database, there are What I mean is that, |
@aviau If we can not use the |
Feel free to open a pull request. I am locking this issue. |
use
influxdb
will cause conflict in CentOS/RedHat yum