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

support -pylab flag with deprecation warning #540

Closed
wants to merge 1 commit into from

Conversation

minrk
Copy link
Member

@minrk minrk commented Jun 25, 2011

As requested on the list, the -pylab flag (one -) is allowed once again. If present, the argument is transformed to the current syntax, and a warning is displayed.

@minrk
Copy link
Member Author

minrk commented Jun 25, 2011

Pinging @fperez and @bgranger. This specifically supports -pylab only. Flags still require -- prefix. An alternative would be to accept single-- prefix for all flags. This would be more natural for flags like ipython -i for force-interactive. Leaving it as-is gives us a place to add something new in the future.

@fperez
Copy link
Member

fperez commented Jun 25, 2011

My take on this is to be conservative for now. -pylab is special because it's extremely widely documented, so we don't really want everyone's scripts, aliases and habits to break everywhere. We'll have to fine-tune our handling of command-line args after more extensive field use and feedback, so I'd suggest we stick, for now, to the bare minimum of tweaks until we can collect such information post-0.11.

@minrk
Copy link
Member Author

minrk commented Jun 25, 2011

That makes sense. I'll go ahead and merge when I am back at a computer this evening.

@ellisonbg
Copy link
Member

I am +1 on having this. I also like that this logic only applies to -pylab and not all flags/options.

@minrk minrk closed this in b6abd4d Jun 26, 2011
mattvonrocketstein pushed a commit to mattvonrocketstein/ipython that referenced this pull request Nov 3, 2014
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

3 participants