Makefile #21

Open
rking opened this Issue Oct 3, 2012 · 2 comments

Comments

Projects
None yet
3 participants

rking commented Oct 3, 2012

  1. You can prefix with "@" so the echo itself doesn't show up
  2. If you sudo make install it creates /root/.selfspy
  3. It should use the install command to set the proper permissions after copying. Currently, the permissions come from whatever umask is, and I have umask 077 for root, meaning the installed files come out unreadable to regular users.

Thanks! (And let me know if you need any clarification or input)
—☈

Collaborator

gurgeh commented Oct 23, 2012

Thanks! I will look use your advice when I give another go at packaging for Ubuntu 12.10

Owner

anarcat commented Sep 7, 2017

why is a makefile needed at all? setup.py is all that you really need here...

anarcat added a commit to anarcat/selfspy that referenced this issue Sep 7, 2017

remove Makefile (Closes: #21)
the makefile doesn't bring any advantage over the normal `python
setup.py install` instructions and just add an extra
dependencies. furthermore, it installs files in a non-standard
location that should generally be avoided.

it will also break Debian packaging, which will think this is a C
program and try to install it as such (and then in the wrong location).
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment