X Tutup
Skip to content

Remove support for -dbackend argv.#8148

Merged
tacaswell merged 1 commit intomatplotlib:masterfrom
anntzer:remove--dbackend-flag
Feb 26, 2017
Merged

Remove support for -dbackend argv.#8148
tacaswell merged 1 commit intomatplotlib:masterfrom
anntzer:remove--dbackend-flag

Conversation

@anntzer
Copy link
Copy Markdown
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
Copy link
Copy Markdown
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
Copy Markdown
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
Copy Markdown
Member

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

Support for setting the backend with `-dbackend` was deprecated in 1.5.
@anntzer
Copy link
Copy Markdown
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
Copy Markdown
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

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants

X Tutup