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

fix(build): Make sure i18n/ dir is actually distributed #1445

Merged
merged 1 commit into from
Jun 18, 2022

Conversation

ctrlcctrlv
Copy link
Member

Fixes this error:

! Error loading localizations eo: no localizations for this language at why.sil:2:1: in \languagetable: 0x41510d90
[1] [2]
! No localized message for book-chapter-title-pre found in locale eo at why.sil: in \fluent near 9:1: in \book:sectioningtable: 0x413e6178

#1444 is blocked on this being merged.

Fixes this error:

! Error loading localizations eo: no localizations for this language at
why.sil:2:1: in \languagetable: 0x41510d90
[1] [2]
! No localized message for book-chapter-title-pre found in locale eo at
why.sil: in \fluent near 9:1: in \book:sectioningtable: 0x413e6178
@alerque alerque added bug Software bug issue tooling Build tooling, release management, and packaging processes labels Jun 18, 2022
@alerque alerque added this to the v0.13.x milestone Jun 18, 2022
@alerque
Copy link
Member

alerque commented Jun 18, 2022

Ouch, what a goof.

Goes to show I do too many project builds using ~/projecs/sile-typesetter/sile/sile instead of my system default sile! Also we need to run regression tests against DISTDIR not the git repo directly...

@alerque alerque merged commit 61ed8e1 into sile-typesetter:master Jun 18, 2022
@alerque alerque deleted the dist_i18n branch June 18, 2022 16:34
@alerque alerque modified the milestones: v0.13.x, v0.13.1 Jun 18, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Software bug issue tooling Build tooling, release management, and packaging processes
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants