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

Add initial support for building using CMake #249

Open
wants to merge 3 commits into
base: master
Choose a base branch
from

Conversation

DanShaders
Copy link

@DanShaders DanShaders commented Oct 8, 2023

This adds limited support for building project using CMake and, crucially for me, support for building code as a shared library.

Feel free to close the PR if the maintenance burden for the CMake build will be too high. Honestly, I just haven't felt like troubleshooting autotools after configuring failed with some obscure error.

For some context, I want to use DVI parser from here in my attempt to speed up LaTeX live previews (in Sublime Text). The current idea is to fork latex process right before it reads \begin{document} (this part is simple and already done); use the fork to produce dvi file with a rendered preview; split it into glyphs using dvisvgm; and then rerender changed parts (probably, using this library).

CMake builds all tests into a single executable. So, naming these two
classes the same is an ODR violation, which also causes segmentation
fault on my machine when I try to run tests.

With this patch, test executable do not crash anymore. However, there
are still 13 failing tests, which require further investigation.
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.

1 participant