pip install does not install scripts

Bug #919918 reported by Christopher Wilson
6
This bug affects 1 person
Affects Status Importance Assigned to Milestone
python-msp430-tools
Fix Released
Medium
zsquareplusc

Bug Description

Using the recommended "future proof" (http://guide.python-distribute.org/introduction.html#current-state-of-packaging) python package manager "pip" to install python-msp430-tools does not install any of the scripts in scripts/*. Using pip and virtualenv for isolated installation and package management is becoming increasingly popular, and the current Make based install system (i.e. "make install") does not respect the isolated nature of the virtualenv's sandboxed environment (instead installing to PREFIX)

I've attached a patch that adds the "scripts" option to setup(), causing "python setup.py install" to install the scripts to the currently active python's /bin directory (this is the default for distutils setup). For those using virtualenv (like me!), this allows multiple versions of these packages to be installed simultaneously in different virtualenvs on the same system (in contrast to "make install" with PREFIX and something like GNU Stow).

Basically, I'm trying to advocate the more "pythonic" approach to the script installation that works via the normal dist channels (tar.gz, pypi, etc), instead of requiring someone who installs via pip to have to manually download the tar.gz and install the scripts by hand...

Related branches

Revision history for this message
Christopher Wilson (cdwilson) wrote :

Please ignore the previous patch. I've added an updated patch that leaves the Make based install "as is" (for those that need to install to PREFIX), but installs the scripts when "python setup.py install" is run.

Revision history for this message
Christopher Wilson (cdwilson) wrote :

I pushed my branch with the fixes and added it to this bug. This is my first time using bzr, so if I did something strange, just let me know...

Changed in python-msp430-tools:
assignee: nobody → zsquareplusc (zsquareplusc)
Revision history for this message
Christopher Wilson (cdwilson) wrote :

I removed the patches since everything is now in the merge request

Changed in python-msp430-tools:
importance: Undecided → Medium
status: New → Fix Committed
Revision history for this message
zsquareplusc (zsquareplusc) wrote :

The makefile's install part was left over from the mspgcc times. I've removed that part. I leave the makefile as i still use it to trigger the documentation builds (committed).

Changed in python-msp430-tools:
status: Fix Committed → Fix Released
To post a comment you must log in.
This report contains Public information  
Everyone can see this information.

Other bug subscribers

Remote bug watches

Bug watches keep track of this bug in other bug trackers.