-
Notifications
You must be signed in to change notification settings - Fork 118
Does not compile with markdown3 #682
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
Comments
Can you show what is at line 191 of pdfpc-4.6.0/build/src/classes/renderer/markdown.c? |
The problem is with libmarkdown3 which is not backward-compatible with libmarkdown2, see Orc/discount/issues/213. We list markdown2 specifically. |
Yes, I changed the flags format because I ran out of bits in the old version. already compiled code should work with the new libraries (I wedged a v2compat layer in) but new code needs to use the new flags structure. I apologize for it not actually being documented; I've been transitioning and between it and the previous wave of dysphoria I've been running at about 1/10th of my abilities for the last couple of years.) The tl/dr; of the new flags is that flags are now a
|
Currently compilation fails with: /build/src/classes/renderer/markdown.c:191:20: error: variable or field 'flags' declared void According to comment [1], markdown3 introduces backward incompatibility with respect to document flags. This commit adds conditionally compiled code that allows building pdfpc with both markdown versions 2 and 3. Fixes pdfpc#682 [1]: pdfpc#682 (comment)
Currently compilation fails with: /build/src/classes/renderer/markdown.c:191:20: error: variable or field 'flags' declared void According to comment [1], markdown3 introduces backward incompatibility with respect to document flags. This commit adds conditionally compiled code that allows building pdfpc with both markdown versions 2 and 3. The used cmake operator VERSION_GREATER_EQUAL requires cmake 3.7 so we bump minimal cmake version requirement. Fixes pdfpc#682 [1]: pdfpc#682 (comment)
Currently compilation fails with: /build/src/classes/renderer/markdown.c:191:20: error: variable or field 'flags' declared void According to comment [1], markdown3 introduces backward incompatibility with respect to document flags. This commit adds conditionally compiled code that allows building pdfpc with both markdown versions 2 and 3. The used cmake operator VERSION_GREATER_EQUAL requires cmake 3.7 so we bump minimal cmake version requirement. Fixes pdfpc#682 [1]: pdfpc#682 (comment)
Currently compilation fails with: /build/src/classes/renderer/markdown.c:191:20: error: variable or field 'flags' declared void According to comment [1], markdown3 introduces backward incompatibility with respect to document flags. This commit adds conditionally compiled code that allows building pdfpc with both markdown versions 2 and 3. The used cmake operator VERSION_GREATER_EQUAL requires cmake 3.7 so we bump minimal cmake version requirement. Fixes pdfpc#682 [1]: pdfpc#682 (comment)
Expected Behaviour
Project should build successfully using latest vala version.
Actual Behaviour
Compilation fails with message
see the following build log against Opensuse Tumbleweed:
https://build.opensuse.org/build/home:esokrates/openSUSE_Tumbleweed/x86_64/pdfpc/_log
Steps to reproduce
pdfpc version: 4.6.0
Used Distribution (GTK version, Vala version): Opensuse Tumbleweed, vala 0.56.5
Display Server (X11/Wayland): does not matter (build error)
HiDPI Setting (yes/no): does not matter (build error)
The text was updated successfully, but these errors were encountered: