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

Eliminate no.tbl #1196

Merged
merged 1 commit into from
May 25, 2022
Merged

Eliminate no.tbl #1196

merged 1 commit into from
May 25, 2022

Conversation

bertfrees
Copy link
Member

The metadata is moved to no-no-g3.ctb.

I have already applied this patch in release 3.21.0-p1 of liblouis-nar. It was done as a work around for liblouis/liblouis-java#10.

You might argue that this is not so great because it masks a bug that we should try to fix instead. But we've already spend a lot of time trying to fix that bug. And this is a good change anyway because the *.tbl tables were always meant to be temporary.

@bertfrees bertfrees requested a review from egli March 28, 2022 16:46
@bertfrees bertfrees added the cleanup Cleanup (possibly removing features) to make the code more maintainable label Mar 28, 2022
@egli
Copy link
Member

egli commented Mar 29, 2022

You might argue that this is not so great because it masks a bug that we should try to fix instead. But we've already spend a lot of time trying to fix that bug.

What problem is this PR solving? What bug are you referring to?

Copy link
Member

@egli egli left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I don't really care in which file the metadata sits. So I'm fine with this change.

@bertfrees
Copy link
Member Author

What bug are you referring to?

liblouis/liblouis-java#10

@bertfrees bertfrees added this to the 3.22 milestone Mar 29, 2022
@egli
Copy link
Member

egli commented Mar 29, 2022

Oh, I get it. Reducing the includes avoids this bug. OK, workaround is fine with me

@bertfrees bertfrees self-assigned this May 25, 2022
The metadata is moved to no-no-g3.ctb.

see #1196
@bertfrees bertfrees merged commit ddc6187 into master May 25, 2022
@bertfrees bertfrees deleted the eliminate-no.tbl branch May 25, 2022 22:36
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
cleanup Cleanup (possibly removing features) to make the code more maintainable
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants