search

jaspervdj / patat

Terminal-based presentations using Pandoc
GNU General Public License v2.0
2.42k stars 60 forks source link

test 02.lhs fails #59

Closed legrostdg closed 5 years ago

legrostdg commented 5 years ago

I tried to build 0.8.1.1 to upload it in Debian, but the test 02.lhs fails (the first putStrLn is not colored, while the second one is blue):

PATH="/usr/lib/ccache:/usr/local/sbin:/usr/local/bin:/usr/sbin:/usr/bin:/sbin:/bin:/usr/games:dist/build/patat/" bash test.sh
Testing tests/extentions0.md... OK
Testing tests/02.lhs... files differ
5c5
<     main = putStrLn "Hello World!" 
---
>     main = putStrLn "Hello World!" 
Testing tests/links.md... OK
Testing tests/comments.md... OK
Testing tests/themes.md... OK
Testing tests/slidelevel1.md... OK
Testing tests/margins.md... OK
Testing tests/01.md... OK
Testing tests/wrapping.md... OK
Testing tests/03.md... OK
Testing tests/extentions1.md... OK
Testing tests/deflist.md... OK
Testing tests/tables.md... OK
Testing tests/slidelevel2.md... OK
Testing tests/slidelevel0.md... OK
Testing tests/bolditalic.md... OK
Testing tests/syntax.md... OK
Testing tests/fragments.md... OK
Testing tests/lists.md... OK
Testing tests/headers.md... OK
Testing tests/meta.md... OK
make[1]: *** [debian/rules:34: override_dh_auto_test] Error 1
make[1]: Leaving directory '/<<PKGBUILDDIR>>'
make: *** [debian/rules:12: build] Error 2
dpkg-buildpackage: error: debian/rules build subprocess returned exit status 2
jaspervdj commented 5 years ago

@legrostdg Yes, it seems the problem is that pandoc's highlighting changed in between versions.

One solution would be to restrict the version range for the pandoc dependency, but I don't like that because then it will build in less situations.

What I will do instead is simplify the test so it works around this issue.

jaspervdj commented 5 years ago

@legrostdg I've uploaded v0.8.1.2 which should fix this issue

legrostdg commented 5 years ago

Great, thanks!

legrostdg commented 5 years ago

Could you push it to hackage, too? Thanks!

jaspervdj commented 5 years ago

@legrostdg Oops, somehow I forgot that, sorry. It's there now.