Skip to content

Remove support for -dbackend argv. #8148

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.

Sign up for GitHub

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

Jump to bottom
Merged
merged 1 commit into from
Feb 26, 2017
Merged

Remove support for -dbackend argv. #8148

merged 1 commit into from
Feb 26, 2017

Conversation

anntzer
Copy link
Contributor

@anntzer anntzer commented Feb 25, 2017

Support for setting the backend with -dbackend was deprecated in 1.5 (#3710).

@QuLogic QuLogic added this to the 2.1 (next point release) milestone Feb 25, 2017
QuLogic
QuLogic requested changes Feb 25, 2017
View reviewed changes
Copy link
Member

@QuLogic QuLogic left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Changed my mind because there are still a few places that you missed:

  • doc/devel/contributing.rst
  • doc/faq/environment_variables_faq.rst
  • doc/users/prev_whats_new/whats_new_0.98.4.rst (iffy)

@anntzer anntzer force-pushed the remove--dbackend-flag branch from e0dd944 to 2f47d51 Compare February 25, 2017 21:46
@anntzer
Copy link
Contributor Author

anntzer commented Feb 25, 2017

Thanks for the quick review, fixed the points you mentioned, with the exception of the 0.98.4 whatsnew (I'd rather not touch old whatsnews).

@tacaswell
Copy link
Member

Can you start a 'removed_code' file in API changes? Other than that 👍

@anntzer
Remove support for -dbackend argv.
781872e 
Support for setting the backend with `-dbackend` was deprecated in 1.5.
@anntzer
Copy link
Contributor Author

anntzer commented Feb 26, 2017

Done.

@anntzer anntzer force-pushed the remove--dbackend-flag branch from 2f47d51 to 781872e Compare February 26, 2017 07:30
@tacaswell tacaswell merged commit a6c32f8 into matplotlib:master Feb 26, 2017
@tacaswell
Copy link
Member

Thanks!

@anntzer anntzer deleted the remove--dbackend-flag branch April 29, 2017 08:41
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@QuLogic QuLogic QuLogic approved these changes

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
v2.1
Development

Successfully merging this pull request may close these issues.
3 participants
@anntzer @tacaswell @QuLogic