Skip to content

Conversation

@matttbe
Copy link

@matttbe matttbe commented Dec 31, 2011

Hello,

It's seems it's recommended to use 'distutils' python module to install python files.
Your script (install.sh) works fine but it seems Debian/Ubuntu devs prefer a cleaner method to install python files
Note that it can be interesting to also install .mo files with setup.py but I don't know if you really want to switch to setup.py but I just give you an idea ;)

Thank you for developing Devede!

@rastersoft
Copy link
Owner

Is a good idea, so if you can add the .mo files to it, I'll do the change.

@alessio
Copy link
Collaborator

alessio commented Apr 16, 2012

This is the setup.py we use in Debian and Ubuntu:

http://paste.debian.net/163380/

@rastersoft
Copy link
Owner

Hi alessio:

That setup.py has a good look, but seems to not install the .mo files...

(un)install.sh files should be no longer needed to install the application.
Renamed devede.py to devede and added a symbolic link (devede -> devede.py)
 in order to install the devede to /usr/bin/devede (and not /usr/bin/devede.py)
@matttbe
Copy link
Author

matttbe commented May 20, 2012

Hi rastersoft,

I modify setup.py to create .mo files and install it in the right directory. (I used this file as example: https://bazaar.launchpad.net/~jaap.karssenberg/zim/pyzim/view/head:/setup.py)
I also rename devede.py to devede (and create a symbolic link: devede -> devede.py) in order to easily install it in bin/devede but it's also possible to copy it (devede.py -> devede) when we call setup.py.

I'm sorry, I didn't test it a lot because I don't have a lot free time... but I hope it's correct and it's what you want to have!

I hope it will help you and thank you for maintaining this application!

Matt

@jirib
Copy link

jirib commented May 21, 2012

Please include this diff, install.sh is full of Linuxism and this makes hard times on BSD systems where's no bash shell in default install. setup.py is good way to go. Thanks

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants