Skip to content

Navigation Menu

Sign in
Appearance settings

Search code, repositories, users, issues, pull requests...

Provide feedback

We read every piece of feedback, and take your input very seriously.

Saved searches

Use saved searches to filter your results more quickly

Appearance settings

Doc updates #9291

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

Merged
merged 7 commits into from
Oct 7, 2017
Merged
Changes from 1 commit
Commits
File filter

Filter by extension

Filter by extension

Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
Prev Previous commit
Next Next commit
DOC: tweak package manager istall wording
  • Loading branch information
tacaswell authored Oct 7, 2017
commit 87960f855a8413beef6dabdbd9defbd8969cd634
6 changes: 3 additions & 3 deletions 6 doc/faq/installing_faq.rst
Original file line number Diff line number Diff line change
Expand Up @@ -49,9 +49,9 @@ installation of the package. In order to fully remove an installed Matplotlib:
Linux Notes
===========

We recommend that you use your distribution's package manager to install
Matplotlib. This will guarantee that Matplotlib's dependencies will be
installed as well.
To install Matplotlib at the system-level we recommend that you use your
distribution's package manager to install Matplotlib. This will guarantee
Copy link
Member

Choose a reason for hiding this comment

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

Suggest drop: "...to install Matplotlib"

that Matplotlib's dependencies will be installed as well.

If, for some reason, you can not use the package manager, you may use the
manylinux wheels available on PyPI::
Copy link
Member

Choose a reason for hiding this comment

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

OK, when I read this I thought that manylinux was a typo. Put in quotes?

Copy link
Member

Choose a reason for hiding this comment

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

https://github.com/pypa/manylinux They never use quotes, I do not think we want to deviate from their notation.

Copy link
Contributor Author

Choose a reason for hiding this comment

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

It's a bit jargon-esque but I'd prefer leaving it as it is.

Copy link
Member

Choose a reason for hiding this comment

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

Not a big deal, but how about "...you may use the wheels available from manylinux, available on PyPI"

Copy link
Contributor Author

Choose a reason for hiding this comment

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

use the manylinux-tagged wheels, available on PyPI OK?

Copy link
Member

Choose a reason for hiding this comment

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

Sounds good!

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Upon further throught I just dropped the "manylinux", as they are the only kind of linux wheels on pypi anyways, and either the reader knows what it means in which case it's obvious the wheels should be manylinux-tagged, or the reader doesn't and that doesn't help him at all.

Expand Down
Morty Proxy This is a proxified and sanitized view of the page, visit original site.