Skip to content

Moving from make.py to Makefile for sphinx-build#9712

Closed
NelleV wants to merge 11 commits into
matplotlib:masterfrom
NelleV:5798_makefile
Closed

Moving from make.py to Makefile for sphinx-build#9712
NelleV wants to merge 11 commits into
matplotlib:masterfrom
NelleV:5798_makefile

Conversation

@NelleV

@NelleV NelleV commented Nov 7, 2017

Copy link
Copy Markdown
Member

"We still use an ancient make.py that is based on what early versions of sphinx-quickstart used to generate. We should probably modernize this to use a Makefile like current versions of sphinx create."

closes #5798

@anntzer

anntzer commented Nov 7, 2017

Copy link
Copy Markdown
Contributor

I still think that #9513 is a better fix...

  • there's no need to install conda on circleci.
  • there's no need to introduce custom targets in the makefile, we can directly use the one provided by sphinx (which is also much shorter now).
  • there's no need to modify the units examples.

@dopplershift

Copy link
Copy Markdown
Contributor

Why does this PR change to use conda on circleci?

Also, I'm completely confused as to why this PR needs to change the examples while #9513 doesn't.

@efiring

efiring commented Nov 7, 2017

Copy link
Copy Markdown
Member

@NelleV, what do you see as the advantages of this over #9513? The latter has the a priori advantage of brevity.

@NelleV

NelleV commented Nov 8, 2017

Copy link
Copy Markdown
Member Author

I've closed my PR. Both solution work, and I really don't care which one gets in.

@efiring

efiring commented Nov 8, 2017

Copy link
Copy Markdown
Member

@NelleV, thank you for getting this topic rolling.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Use Makefile for sphinx build

5 participants