Hi,

libvterm is broken when built with base-gcc, that does not recognise
`-Wpedantic':

> cc -O2 -pipe -Wall -Iinclude -std=c99 -Wpedantic -c src/encoding.c
> -fPIC -DPIC -o src/.libs/encoding.o
> cc1: error: unrecognized command line option "-Wpedantic"

Here is a patch that removes it. While here, update-patches(1) was
whining due to a git-styled patch, so i converted it to what cvs(1)
expect.

It then builds fine on macppc [0] and amd64.

REVISION bump is unneeded, this diff has no impact on the package.

Comments/feedback are welcome,

Charlène. 


[0] https://bin.charlenew.xyz/libvterm.log


Index: patches/patch-Makefile
===================================================================
RCS file: patches/patch-Makefile
diff -N patches/patch-Makefile
--- /dev/null   1 Jan 1970 00:00:00 -0000
+++ patches/patch-Makefile      24 Nov 2019 10:37:34 -0000
@@ -0,0 +1,16 @@
+$OpenBSD$
+
+-Wpedantic is not recognised by base-gcc
+
+Index: Makefile
+--- Makefile.orig
++++ Makefile
+@@ -8,7 +8,7 @@ ifneq ($(VERBOSE),1)
+   LIBTOOL +=--quiet
+ endif
+ 
+-override CFLAGS +=-Wall -Iinclude -std=c99 -Wpedantic
++override CFLAGS +=-Wall -Iinclude -std=c99
+ 
+ ifeq ($(shell uname),SunOS)
+   override CFLAGS +=-D__EXTENSIONS__ -D_XPG6 -D__XOPEN_OR_POSIX
Index: patches/patch-bin_vterm-ctrl_c
===================================================================
RCS file: /cvs/ports/devel/libvterm/patches/patch-bin_vterm-ctrl_c,v
retrieving revision 1.2
diff -u -p -u -p -r1.2 patch-bin_vterm-ctrl_c
--- patches/patch-bin_vterm-ctrl_c      20 Nov 2019 13:40:13
-0000   1.2 +++ patches/patch-bin_vterm-ctrl_c  24 Nov 2019
10:37:34 -0000 @@ -6,9 +6,8 @@ https://bugs.launchpad.net/libvterm/+bug
 Patch updated to apply on:
 https://github.com/neovim/libvterm/tree/nvim
 
-diff --git bin/vterm-ctrl.c bin/vterm-ctrl.c
-index ba0d61e..92a365f 100644
---- bin/vterm-ctrl.c
+Index: bin/vterm-ctrl.c
+--- bin/vterm-ctrl.c.orig
 +++ bin/vterm-ctrl.c
 @@ -79,9 +79,9 @@ static bool seticanon(bool icanon, bool echo)
    return ret;



Reply via email to