8000 Add texinfo build target in doc/make.py by tkf · Pull Request #986 · matplotlib/matplotlib · GitHub
[go: up one dir, main page]

Skip to content

Add texinfo build target in doc/make.py #986

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 5 commits into from
Aug 20, 2012
Merged

Conversation

tkf
Copy link
Contributor
@tkf tkf commented Jul 4, 2012

No description provided.

@WeatherGod
Copy link
Member

Need to add a entry for "build/texinfo" in the list in check_build() in order to make sure the directory exists.

@tkf
Copy link
Contributor Author
tkf commented Jul 5, 2012

Done!

@mdboom
Copy link
Member
mdboom commented Aug 3, 2012

This seems to work reasonably well. Is there a texinfo viewer that displays the images?

@tkf
Copy link
Contributor Author
tkf commented Aug 3, 2012

Emacs's info viewer displays images, in principle. It works well when I build other info documents based on sphinx. But the only figure I found in the info page is http://matplotlib.sourceforge.net/users/screenshots.html#eeg-demo. Probably there is some problem in plot directive when used with texinfo builder?

@mdboom
Copy link
Member
mdboom commented Aug 3, 2012

It's quite possible something is going wrong here.

Pardon my ignorance about this stuff, but how do I load the results of this build into emacs so I can test it?

@tkf
Copy link
Contributor Author
tkf commented Aug 3, 2012

I think the easiest way to open info file is to do:

M-x info-open-file RET PATH/TO/matplotlib/doc/build/texinfo/matplotlib.info RET

@tkf
Copy link
Contributor Author
tkf commented Aug 9, 2012

Thanks to @JonWaltman (tkf#1), plot directive works with texinfo build target.

@pelson
Copy link
Member
pelson commented Aug 19, 2012

There is no reason to not get this into 1.2.x. Any other issues before we merge?

@mdboom
Copy link
Member
mdboom commented Aug 19, 2012

This looks good to me.

mdboom added a commit that referenced this pull request Aug 20, 2012
Add texinfo build target in doc/make.py
@mdboom mdboom merged commit 4cf846d into matplotlib:master Aug 20, 2012
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.

5 participants
0