Skip to content

Fix babel error issue - #13643

Merged
cderv merged 6 commits into
mainfrom
fix/tinytex/babel-error
Nov 5, 2025
Merged

cderv merged 6 commits into
mainfrom
fix/tinytex/babel-error

Conversation

@cderv

@cderv cderv commented Nov 4, 2025

Copy link
Copy Markdown
Member

when lang: en is passed, babel-english needs to be installed.

This follows up fix on tinytex R package too about new error handling

cause is new pandoc way to setup babel, and maybe new babel error in log

fixes #13641
fixes #13633

cderv added 2 commits November 4, 2025 15:13
when lang: en is passed, babel-english needs to be installed.

This follows up fix on tinytex R package too about new error handling

cause is new pandoc way to setup babel, and maybe new babel error in log
@posit-snyk-bot

posit-snyk-bot commented Nov 4, 2025 •

Copy link
Copy Markdown
Collaborator

✅ Snyk checks have passed. No issues have been found so far.

Status Scanner Critical High Medium Low Total (0)
✅ Licenses 0 0 0 0 0 issues
✅ Open Source Security 0 0 0 0 0 issues

💻 Catch issues earlier using the plugins for VS Code, JetBrains IDEs, Visual Studio, and Eclipse.

@gordonwoodhull

Copy link
Copy Markdown
Member

Unknown option 'spanish'.

been seeing this a lot past few days

@cderv

cderv commented Nov 4, 2025 •

Copy link
Copy Markdown
Member Author

yeah same problem that I am trying to fix. It means what I did is not enouhg 😞

I'll check it again

@cderv

cderv commented Nov 4, 2025

Copy link
Copy Markdown
Member Author

OK so in fact, I don't know if this is recent Pandoc or Babel, but the messages have changed, and our auto installation is not working anymore... this is related to tinytex also. I'll solve this all in this PR.

Updated regex to accept both backtick-quote (`Spanish') and straight-quote
('Spanish') formats for detecting missing hyphenation packages.

Added unit tests and modified smoke-all tests to skip package removal on CI.
@cderv

cderv commented Nov 4, 2025

Copy link
Copy Markdown
Member Author
  • Something has changed in babel for sure. I have adapted our cases.
  • We didn't had test for hyphenation warnings, this is added.
  • I revisited what to test on CI as uninstall package is not efficient to test, but I kept the test.

This should be good now ! 🤞

@cderv
cderv merged commit c1375a3 into main Nov 5, 2025
51 checks passed
@cderv
cderv deleted the fix/tinytex/babel-error branch November 5, 2025 20:55
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.

TinyTeX patterns require review babel-english not auto-installed when rendering PDF with lang: en

3 participants