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

Internal miktex error when calling TEXINDEX #460

Closed
jeroen opened this issue Feb 3, 2020 · 7 comments
Closed

Internal miktex error when calling TEXINDEX #460

jeroen opened this issue Feb 3, 2020 · 7 comments
Assignees
Projects
Milestone

Comments

@jeroen
Copy link

@jeroen jeroen commented Feb 3, 2020

Hi, I maintain the Windows builds of the R programming language.

Since the last release of miktex basic-miktex-2.9.7269-x64.exe end november our build script started failing on Windows. The bash line that calls texindex immediately exists:

export TEXINDEX=C:/rtools40/usr/bin/texindex.exe 
texify --pdf --texinfo="@set UseExternalXrefs " R-FAQ.texi
ERROR: MiKTeX encountered an internal error.
ERROR: Info:
ERROR: Source: Libraries\MiKTeX\Core\Session/SessionImpl.h
ERROR: Line: 640

Sorry, but "MiKTeX Compiler Driver" did not succeed.

This did not happen with the previous release basic-miktex-2.9.7152-x64.exe.

I suspect the problem is related to texify not properly invoking texindex.exe or --texinfo but I cannot debug any futher what is going wrong. I tried calling with -V but this doesn't give any extra output.

@edocevoli edocevoli self-assigned this Feb 4, 2020
@edocevoli edocevoli added this to To do in What's next via automation Feb 4, 2020
@edocevoli edocevoli added this to the 2.9.7300 milestone Feb 4, 2020
@edocevoli

This comment has been minimized.

Copy link
Member

@edocevoli edocevoli commented Feb 4, 2020

Thank you. Since --texinfo is an unknown option, the program terminates (unfortunately with a an internal error). So:

texify --pdf --covefe="asdasd" test.texi

would also trigger the internal error. The correct behaviour would be to terminate with better error message. I will fix this.

@jeroen

This comment has been minimized.

Copy link
Author

@jeroen jeroen commented Feb 4, 2020

Thank you. When did this option become unsupported? We used this script for many years and the option is described in the documentation: https://docs.miktex.org/manual/texify.html

@edocevoli

This comment has been minimized.

Copy link
Member

@edocevoli edocevoli commented Feb 4, 2020

Since texindex is not a part of the MiKTeX distribution, someone was under the impression that texinfo support has to be removed altogether. I will see if this can be reverted.

@jeroen

This comment has been minimized.

Copy link
Author

@jeroen jeroen commented Feb 4, 2020

OK, thank you, much appreciated :)

@gwd999

This comment has been minimized.

Copy link

@gwd999 gwd999 commented Feb 4, 2020

Short note: I managed a successful build of [R] .pdf documentation with MiKTeX 2.9.7015. So this seems to be the last "functioning" version. But I am by no means a TeX/MiKTeX expert. Just letting the Pro's [ie. you two] know. I am on WINDOWS 10 Pro 64bit -> what is interesting is that executing texify in the CMD prompt throws the same ERROR (ERROR: Source: Libraries\MiKTeX\Core\Session/SessionImpl.h) even there.
@jeroen: Example code that executes when building with 2.9.7015
TEXINDEX=C:/Rtools/bin/texindex.exe texify --pdf --texinfo="@set UseExternalXrefs " R-lang.texi
... This is pdfTeX, Version 3.14159265-2.6-1.40.20 (MiKTeX 2.9.7000 64-bit)

edocevoli added a commit that referenced this issue Feb 4, 2020
@edocevoli edocevoli moved this from To do to In progress in What's next Feb 5, 2020
@jeroen

This comment has been minimized.

Copy link
Author

@jeroen jeroen commented Feb 5, 2020

Thank you, let me know if there is something that needs testing.

By the way, what was the rationale for removing texi2dvi (texindex) from miktex (about 4 years ago)? Do you no longer wish to support texi?

This was why we need the external TEXINDEX driver for texify in the first place. Previously we just used miktex's own texi2dvi which was much simpler.

@gwd999

This comment has been minimized.

Copy link

@gwd999 gwd999 commented Feb 13, 2020

R Manuals again build like a charm (under WIN) ... great work - thnx!

@edocevoli edocevoli closed this Feb 20, 2020
What's next automation moved this from In progress to Done Feb 20, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
What's next
  
Done
Linked pull requests

Successfully merging a pull request may close this issue.

None yet
3 participants
You can’t perform that action at this time.