Skip to content
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

[index] Use LaTeX package 'imakeidx' to allow for more indices. #3873

Merged
merged 1 commit into from Oct 17, 2020

Conversation

jensmaurer
Copy link
Member

@jensmaurer jensmaurer commented Mar 14, 2020

This doesn't consume a \write stream per index.
The disadvantage is that it needs the -shell-escape option for pdflatex to split up the single index file.

We can now easily add a separate index of exposition-only library names.

"diffpdf" says the before/after PDFs are the same.

@tkoeppe
Copy link
Contributor

tkoeppe commented Mar 17, 2020

Instead of this, could we try again to get xindy to work? What was the blocker there, @godbyk?

@godbyk
Copy link
Contributor

godbyk commented Mar 17, 2020

@tkoeppe If I remember correctly it was an issue the cross-references. Ah, here are my notes.

I reached out to the xindy folks on their mailing list and on TeX's StackExchange, but I didn't get a response.

@jensmaurer
Copy link
Member Author

@tkoeppe, my understanding is that xindy and this change are complementary. In fact, imakeidx can use xindy instead of makeindex by virtue of a configuration option. However, without imakeidx, our LaTeX index production would still be hampered by the 16 streams \write limit, regardless of whether we use makeindex or xindy for the postprocessing.

@godbyk
Copy link
Contributor

godbyk commented Mar 17, 2020

If the underlying issue is the 16 \write limit, does \usepackage{morewrites} do the trick? (See the morewrites docs for more details.)

@jensmaurer
Copy link
Member Author

jensmaurer commented Mar 18, 2020

"morewrites" is extremely slow. For comparison: "make" after "make clean" with the status quo takes

real 2m56,070s
user 2m50,371s
sys 0m1,175s

Simply adding "morewrites" into std.tex results in this (after a "make clean"):
real 15m32,945s
user 15m2,865s
sys 0m14,112s

That's a factor of five. No, thanks.

@jensmaurer jensmaurer added the needs rebase The pull request needs a git rebase to resolve merge conflicts. label Sep 17, 2020
@jensmaurer jensmaurer removed the needs rebase The pull request needs a git rebase to resolve merge conflicts. label Sep 20, 2020
@zygoloid zygoloid merged commit 47fa06e into cplusplus:master Oct 17, 2020
@jensmaurer jensmaurer deleted the c5 branch October 18, 2020 07:53
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.

None yet

4 participants