sardemff7 commented on this pull request.
If you don’t go with fallback checks, better drop the `#else`s. Other comments
inside.
> @@ -35,4 +35,8 @@ markdown_la_CFLAGS += $(LIBMARKDOWN_CFLAGS)
markdown_la_LIBADD += $(LIBMARKDOWN_LIBS)
endif
+if MARKDOWN_WEBKIT2
+markdown_la_CFLAGS += -DMARKDOWN_WEBKIT2
Why not an `AC_DEFINE` in `markdown.m4` instead?
> @@ -49,13 +49,13 @@ AC_DEFUN([GP_CHECK_MARKDOWN],
GTK_VERSION=2.16
WEBKIT_VERSION=1.1.13
- GP_CHECK_GTK3([webkit_package=webkitgtk-3.0],
+ GP_CHECK_GTK3([webkit_package=webkit2gtk-4.0],
It should work, but you better not add fallback code for unsupported versions,
distributions with older packages will use older plugin too. (And people really
must stop thinking everything will always work for them without patching on
older systems, do not but the maintenance burden on you.)
--
You are receiving this because you are subscribed to this thread.
Reply to this email directly or view it on GitHub:
https://github.com/geany/geany-plugins/pull/746#pullrequestreview-122031090
