search

agateau / mup

A markup previewer
Other
39 stars 3 forks source link

Installation #1

Closed Mikaela closed 9 years ago

Mikaela commented 9 years ago

Hi, I saw your post on Planet Ubuntu and it looked interesting and found these two issues from README.

First, possibly separate issue at first: there is no requirements.txt file for pip to install those Python modules.

And the second issue is that I don't think that the commands you suggest in README for installation are good.

The best way to install it is to run as root: python setup.py install --prefix=$desktop_install_dir Where $desktop_install_dir is usually /usr.

/usr should be only for things installed from apt/ and if something is installed from elsewhere, it should be in /usr/local instead which is in $PATH by default. Usually python setup.py install should install there directly without flags or without root python setup.py install --user installs to $HOME/.local which is in PYTHONPATH by default.

This won't work on Debian or Ubuntu however. On these systems, you should use: sudo python setup.py install --install-layout=deb

--install-layout=deb is not familiar flag to me, but I assume that it also installs to /usr or somewhere similar.

agateau commented 9 years ago

Regarding requirements.txt: the problem is PyQt4 is not available via pip, so it makes having a requirements.txt less useful, what do you think?

I suggest to install to the desktop install dir because if MUP is not installed that way, its .desktop file won't be found by the desktop. At least that's what I remembered: I just tried again and realized KDE Plasma found MUP even if it is installed to /usr/local. Going to fix the instructions.

agateau commented 9 years ago

Fixes are in the dev branch, do they make sense to you?

Mikaela commented 9 years ago

Regarding requirements.txt: the problem is PyQt4 is not available via pip, so it makes having a requirements.txt less useful, what do you think?

Oh, I didn't think about those missing from pypi.

Fixes are in the dev branch, do they make sense to you?

I think so. Thanks :)

agateau commented 9 years ago

Just merged the changes in.