GithubHelp home page GithubHelp logo

Comments (4)

xolox avatar xolox commented on August 31, 2024

Just to be clear: You're talking about installing packages system wide, right? My intended use case for pip-accel is to bootstrap virtual environments, which is why I didn't notice this previously. Of course I'm not opposed to people using pip-accel to install system wide packages, so we can try to accommodate both use cases.

How I would like to see this fixed (if you want to create a pull request) is to detect which directories are available in the prefix that pip-accel is unpacking to and then rewrite filenames to use this directory scheme.

PS. For future reference, as pointed out by @fdv, the bug was introduced in cc06f65.

from pip-accel.

fdv avatar fdv commented on August 31, 2024

BTW, thank you for the awesome project, with a nightly generated (and packaged) cache, it reduced my deployment time from 25 to 7 minutes, AMI generation included.

from pip-accel.

xolox avatar xolox commented on August 31, 2024

Glad to hear that pip-accel is saving you time :-). I merged your pull request but changed the implementation, please refer to the pull request.


A heads up, given that 1) you seem to be using pip-accel for serious work and 2) I recently broke pip-accel for you (in cc06f65): I just pushed b766ba2 which changes the system wide installation prefix from /usr to /usr/local on Debian (derived) systems. This could be considered a big change, but I think of it as a bug fix for an oversight on my part. Here is the comment I added to explain this change (if you're interested):

On Debian derived systems only apt (dpkg) should be allowed to touch files in /usr/lib/pythonX.Y/dist-packages/ and python setup.py install knows this (see the posix_local installation scheme in /usr/lib/python2.7/sysconfig.py on Debian systems). Because pip-accel replaces python setup.py install we have to replicate this logic. Magically inferring this from the sysconfig module would be nice but Python versions before 3.2 don't even have that module (it appears to be a Debian addition in versions before 3.2).

from pip-accel.

xolox avatar xolox commented on August 31, 2024

Thanks for the feedback and pull request! I believe this problem has been resolved for a while so will close this issue. If you encounter the problem again feel free to reopen this issue or report a new one.

from pip-accel.

Related Issues (20)

Recommend Projects

  • React photo React

    A declarative, efficient, and flexible JavaScript library for building user interfaces.

  • Vue.js photo Vue.js

    🖖 Vue.js is a progressive, incrementally-adoptable JavaScript framework for building UI on the web.

  • Typescript photo Typescript

    TypeScript is a superset of JavaScript that compiles to clean JavaScript output.

  • TensorFlow photo TensorFlow

    An Open Source Machine Learning Framework for Everyone

  • Django photo Django

    The Web framework for perfectionists with deadlines.

  • D3 photo D3

    Bring data to life with SVG, Canvas and HTML. 📊📈🎉

Recommend Topics

  • javascript

    JavaScript (JS) is a lightweight interpreted programming language with first-class functions.

  • web

    Some thing interesting about web. New door for the world.

  • server

    A server is a program made to process requests and deliver data to clients.

  • Machine learning

    Machine learning is a way of modeling and interpreting data that allows a piece of software to respond intelligently.

  • Game

    Some thing interesting about game, make everyone happy.

Recommend Org

  • Facebook photo Facebook

    We are working to build community through open source technology. NB: members must have two-factor auth.

  • Microsoft photo Microsoft

    Open source projects and samples from Microsoft.

  • Google photo Google

    Google ❤️ Open Source for everyone.

  • D3 photo D3

    Data-Driven Documents codes.