Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Moderate changes and improvements; please review and, if appropriate, merge. #1

Open
wants to merge 7 commits into
base: master
Choose a base branch
from

Conversation

pprince
Copy link

@pprince pprince commented Jan 20, 2014

I release my changes under the same permissive license as specified in the LICENSE file, and you may feel free to incorporate them while attributing the copyright to yourself.

JSYK, I am also looking at generalizing this, to support a .env file or .env/ directory inside $VIRTUAL_ENV/. If I do that and you don't otherwise object, I will attribute the copyright on that project to myself, and list your two projects as inspiration. This is in case I want to standardize on releasing my projects under a different permissive license.

Feel free to shoot me an email,
-Paul Prince paul@littlebluetech.com

Differences in behavior from upstream version:

  * GEM_HOME will be set to VIRTUAL_ENV instead of
    VIRTUAL_ENV/lib/gems

      * As a handy side-effect of this improvement,
        we will no longer need to add anything to PATH.

  * Both the values, **and the set/unset** status, of
    all environmental variables will be preserved and
    restored properly on deactivating the virtualenv.
@pprince
Copy link
Author

pprince commented Jan 20, 2014

ppdevel branch has individual commits;
all have been squashed to the master branch as of now.

also, tested / working as of now.

@pprince
Copy link
Author

pprince commented Jan 20, 2014

** To enable for a given virtualenv, touch $VIRTUAL_ENV/.gem

** TODO: Update docs.

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.

None yet

1 participant