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

Settle on a choice for $IPYTHONDIR #1167

Closed
fperez opened this issue Dec 17, 2011 · 4 comments · Fixed by #1693
Closed

Settle on a choice for $IPYTHONDIR #1167

fperez opened this issue Dec 17, 2011 · 4 comments · Fixed by #1693
Milestone

Comments

@fperez
Copy link
Member

fperez commented Dec 17, 2011

Our environment variable used to be called IPYTHONDIR, at some point we changed to IPYTHON_DIR, but now we have code using both. We should settle on only one, and issue a warning when we find the alternative in use. We shouldn't flat out remove the other for a long time, as I suspect by now both are used in the wild and this would be gratuitous breakage for the users.

From an informal scan of projects, IPYTHONDIR seems to be more commonly used, but we should check more carefully. We should settle on the version that we find to be in widest use, to minimize unnecessary hassles for users.

@minrk
Copy link
Member

minrk commented Dec 17, 2011

I think the IPYTHON_DIR was added in 0.11, so it's only been around for 6 months. That form is more conventional, but I'm not sure this is something worth supporting redundancy and/or breaking historical consistency. I'd be fine with dropping the new name, and returning to the old one.

@fperez
Copy link
Member Author

fperez commented Dec 17, 2011

I've been scanning external projects that use IPython, and all of them use IPYTHONDIR (esp. since most were developed in the 0.10 days). While I also prefer the _ form, I do think this is unnecessary breakage of user code. I think it's ok that we break things when we absolutely must, but this one probably isn't needed.

Question: should we at least update the docs in 0.12 to mention IPYTHONDIR? I don't want to touch the actual code this late in the game, but since the code supports both, right now it's just a matter of steering our userbase in one direction or another from the documentation. If we're going to flip the order in 0.13 and add a warning, it would be good to point people towards the right option as early as possible.

@bfroehle
Copy link
Contributor

bfroehle commented May 2, 2012

I think without the underscore is fine. It also matches Python's convention, which uses PYTHONPATH, PYTHONHOME, PYTHONSTARTUP, etc.

@fperez
Copy link
Member Author

fperez commented May 3, 2012

I'm fine with IPYTHONDIR too. If anyone takes a stab at a PR, it should have:

  • warning on detecting the _ version in use
  • visible doc update both in the configuration section and in the what's new.

@minrk minrk closed this as completed in dbeeefd May 4, 2012
mattvonrocketstein pushed a commit to mattvonrocketstein/ipython that referenced this issue Nov 3, 2014
deprecate IPYTHON_DIR in favor of IPYTHONDIR

IPYTHON_DIR was a proposed rename, whose added complexity does not outweigh any potential benefit, and is reverted.

closes ipython#1167
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 a pull request may close this issue.

3 participants