search

keskad / nbnotify

Facebook, Twitter, Dobreprogramy, Ask.fm, Last.fm, RSS notifications on your Linux desktop
4 stars 0 forks source link

Installation mess… #1

Closed Kwpolska closed 12 years ago

Kwpolska commented 12 years ago

It looks like you have a install.py script AND the standard setup.py script. Please, stop. Just use setup.py. And upload it to pypi. Or else you won’t get anyone in a right mind to install it and you won’t get too much publicity.

keskad commented 12 years ago

Dnia 2012-09-10, o godz. 08:21:36 Chris Warrick notifications@github.com napisał(a):

It looks like you have a install.py script AND the standard setup.py script. Please, stop. Just use setup.py. And upload it to pypi. Or else you won’t get anyone in a right mind to install it and you won’t get too much publicity.

Reply to this email directly or view it on GitHub: https://github.com/webnull/nbnotify/issues/1

Hello, the install.py script is for creating Debian, Fedora etc. distribution packages, not to install python libs.

Can you tell me what you mean?

Kwpolska commented 12 years ago

What? Looking at the source code, it seems to be made for humans who want to install nbnotify without using their package system. The proper way to do this is to use setup.py. For everything. In your setup.py, it looks like it does not install locales and /usr/bin/nbnotify. And please upload your code to the PyPI, as everyone who uses python does.

keskad commented 12 years ago

So, if i will pack it into pypi how i will create Deb packages then?

Kwpolska commented 12 years ago

like everyone else? PyPI → sid — I guess they run ./setup.py in a chroot/with an install prefix, at leasts that what arch and gentoo do. But I am aware that debian has a weird system. You may want to take a look at this guide.

keskad commented 12 years ago

I found a good example on how to build own setup.py for Subget and nbnotify projects; https://code.google.com/p/giuspen-cherrytree/source/browse/setup.py

So i think it will be finished in this week.

keskad commented 12 years ago

Fixed.