Skip to content

Conversation

@jhermann
Copy link

@jhermann jhermann commented Oct 2, 2013

This bit me when packaging a virtualenv for Debian, the resulting package had symlinks pointing to the build staging directory in my home! Not sure if this breaks other use-cases and why the abspath is there, might need some kind of configuration switch then.

@blueyed
Copy link

blueyed commented Jun 11, 2014

Seems sensible.

But the patch/PR does not apply anymore.

Relevant code: https://github.com/pypa/virtualenv/blob/develop/virtualenv.py#L1549-1550

@dstufft dstufft mentioned this pull request Jan 5, 2015
5 tasks
@BrownTruck
Copy link

Hello!

As part of an effort to ease the contribution process and adopt a more standard workflow pip has switched to doing development on the master branch. However, this Pull Request was made against the develop branch so it will need to be resubmitted against master. Unfortunately, this pull request does not cleanly merge against the current master branch.

If you do nothing, this Pull Request will be automatically closed by @BrownTruck since it cannot be merged.

If this pull request is still valid, please rebase it against master (or merge master into it) and resubmit it against the master branch, closing and referencing the original Pull Request.

If you choose to rebase/merge and resubmit this Pull Request, here is an example message that you can copy and paste:

This bit me when packaging a virtualenv for Debian, the resulting package had symlinks pointing to the build staging directory in my home! Not sure if this breaks other use-cases and why the abspath is there, might need some kind of configuration switch then.

---

*This was migrated from pypa/virtualenv#473 to reparent it to the ``master`` branch. Please see original pull request for any previous discussion.*

@BrownTruck
Copy link

This Pull Request was closed because it cannot be automatically reparented to the master branch and it appears to have bit rotted.

Please feel free to re-open it or re-submit it if it is still valid and you have rebased it onto master or merged master into it.

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.

3 participants