On Fri, Feb 02, 2018 at 02:36:33PM +0100, Klemens Nanni wrote:
> On Wed, Jan 31, 2018 at 08:06:54PM +0000, Stuart Henderson wrote:
> > On 2018/01/31 20:54, Klemens Nanni wrote:
> > > * The Makefile no longer honors CPPFLAGS and instead consistently uses 
> > > CFLAGS for user-provided include paths
> > 
> > CPPFLAGS would usually be the better place for include paths ...
> Same goes for `-D_VAR=1' but you'll find inconsistent usage in nearly
> all projects.
> 
> Is this a blocker for you? I'd like to get this update in.
> 
Now with devel/p5-Test-Command in tree:

        All tests successful.
        Files=2, Tests=74,  1 wallclock secs ( 0.04 usr  0.06 sys +  0.20 cusr  
0.43 csys =  0.73 CPU)
        Result: PASS

Haven't heard from MAINTAINER so far since 29.01.2018.

OK?

Index: Makefile
===================================================================
RCS file: /cvs/ports/graphics/feh/Makefile,v
retrieving revision 1.88
diff -u -p -r1.88 Makefile
--- Makefile    7 Dec 2017 13:12:44 -0000       1.88
+++ Makefile    11 Feb 2018 15:30:24 -0000
@@ -2,12 +2,12 @@
 
 COMMENT=       lightweight image viewer
 
-DISTNAME=      feh-2.20
+DISTNAME=      feh-2.23.1
 EXTRACT_SUFX=  .tar.bz2
 CATEGORIES=    graphics
 FIX_EXTRACT_PERMISSIONS=Yes
 
-HOMEPAGE=      http://feh.finalrewind.org/
+HOMEPAGE=      https://feh.finalrewind.org/
 
 MAINTAINER=    David Coppa <dco...@openbsd.org>
 
@@ -24,21 +24,21 @@ LIB_DEPENDS=        graphics/imlib2 \
 
 RUN_DEPENDS+=  devel/desktop-file-utils \
                x11/gtk+3,-guic
+TEST_DEPENDS=  devel/p5-Test-Command
 
 CFLAGS+=       -I${LOCALBASE}/include \
                -I${X11BASE}/include
 LDFLAGS+=      -L${LOCALBASE}/lib -L${X11BASE}/lib
-MAKE_FLAGS=    LDFLAGS="${LDFLAGS}" man_dir="${DESTDIR}${PREFIX}/man" \
-               example_dir="${DESTDIR}${PREFIX}/share/examples/feh"
+MAKE_FLAGS=    LDFLAGS="${LDFLAGS}" \
+               man_dir="${DESTDIR}${PREFIX}/man" \
+               example_dir="${DESTDIR}${PREFIX}/share/examples/feh" \
+               exif=1
 FAKE_FLAGS=    PREFIX="${PREFIX}"
 
 USE_GMAKE=     Yes
 
-NO_TEST=       Yes
-
-post-configure:
+pre-configure:
        ${SUBST_CMD} ${WRKSRC}/examples/themes ${WRKSRC}/man/feh.pre \
-               ${WRKSRC}/src/events.c ${WRKSRC}/src/keyevents.c \
-                       ${WRKSRC}/src/options.c
+           ${WRKSRC}/src/{events,keyevents,options}.c
 
 .include <bsd.port.mk>
Index: distinfo
===================================================================
RCS file: /cvs/ports/graphics/feh/distinfo,v
retrieving revision 1.55
diff -u -p -r1.55 distinfo
--- distinfo    8 Sep 2017 08:19:20 -0000       1.55
+++ distinfo    11 Feb 2018 15:30:24 -0000
@@ -1,2 +1,2 @@
-SHA256 (feh-2.20.tar.bz2) = S1BDAgmext0JnITdfihcgNwxiXfr2kiUlZM4a8JucAs=
-SIZE (feh-2.20.tar.bz2) = 2129466
+SHA256 (feh-2.23.1.tar.bz2) = m8Fk0IY9QSAb0lOiZS3O5YBqnGqLWRi7i6CfzubX6b4=
+SIZE (feh-2.23.1.tar.bz2) = 2130373
Index: patches/patch-Makefile
===================================================================
RCS file: /cvs/ports/graphics/feh/patches/patch-Makefile,v
retrieving revision 1.6
diff -u -p -r1.6 patch-Makefile
--- patches/patch-Makefile      9 Jun 2017 10:20:22 -0000       1.6
+++ patches/patch-Makefile      11 Feb 2018 15:30:24 -0000
@@ -2,7 +2,7 @@ $OpenBSD: patch-Makefile,v 1.6 2017/06/0
 Index: Makefile
 --- Makefile.orig
 +++ Makefile
-@@ -90,6 +90,7 @@ install-examples:
+@@ -91,6 +91,7 @@ install-examples:
        @echo installing examples to ${example_dir}
        @mkdir -p ${example_dir}
        @cp examples/* ${example_dir}
Index: patches/patch-config_mk
===================================================================
RCS file: /cvs/ports/graphics/feh/patches/patch-config_mk,v
retrieving revision 1.8
diff -u -p -r1.8 patch-config_mk
--- patches/patch-config_mk     30 Aug 2016 14:44:37 -0000      1.8
+++ patches/patch-config_mk     11 Feb 2018 15:30:24 -0000
@@ -2,24 +2,19 @@ $OpenBSD: patch-config_mk,v 1.8 2016/08/
 
 zap redundant CFLAGS
 
-enable builtin EXIF support
+fix "filelist.c:253:45: error: use of undeclared identifier 'alphasort'"
 
---- config.mk.orig     Sun Aug 28 22:25:42 2016
-+++ config.mk  Tue Aug 30 16:29:18 2016
-@@ -7,7 +7,7 @@ curl ?= 1
- debug ?= 0
- help ?= 0
- xinerama ?= 1
--exif ?= 0
-+exif ?= 1
- 
- # Prefix for all installed files
- PREFIX ?= /usr/local
-@@ -32,7 +32,6 @@ scalable_icon_dir = ${icon_dir}/scalable/apps
+Index: config.mk
+--- config.mk.orig
++++ config.mk
+@@ -35,10 +35,6 @@ scalable_icon_dir = ${icon_dir}/scalable/apps
  
  # default CFLAGS
  CFLAGS ?= -g -O2
 -CFLAGS += -Wall -Wextra -pedantic
+-
+-# Settings for glibc >= 2.19 - may need to be adjusted for other systems
+-CFLAGS += -std=c11 -D_POSIX_C_SOURCE=200809L -D_XOPEN_SOURCE=700
  
  ifeq (${curl},1)
        CFLAGS += -DHAVE_LIBCURL
Index: patches/patch-man_feh_pre
===================================================================
RCS file: /cvs/ports/graphics/feh/patches/patch-man_feh_pre,v
retrieving revision 1.40
diff -u -p -r1.40 patch-man_feh_pre
--- patches/patch-man_feh_pre   8 Sep 2017 08:19:20 -0000       1.40
+++ patches/patch-man_feh_pre   11 Feb 2018 15:30:24 -0000
@@ -2,7 +2,7 @@ $OpenBSD: patch-man_feh_pre,v 1.40 2017/
 Index: man/feh.pre
 --- man/feh.pre.orig
 +++ man/feh.pre
-@@ -1073,7 +1073,7 @@ which
+@@ -1088,7 +1088,7 @@ which
  defaults to
  .Pa ~/.config/feh/ .
  If the files are not found in that directory, it will also try
@@ -11,86 +11,3 @@ Index: man/feh.pre
  .Pp
  All config files treat lines starting with a
  .Qq #
-@@ -1651,56 +1651,56 @@ Here are some examples of useful option combinations. 
- .
- .Bl -tag -width indent
- .
--.It feh /opt/images
-+.It feh ~/Pictures
- .
--Show all images in /opt/images
-+Show all images in ~/Pictures
- .
--.It feh -r /opt/images
-+.It feh -r ~/Pictures
- .
--Recursively show all images found in /opt/images and subdirectories
-+Recursively show all images found in ~/Pictures and subdirectories
- .
--.It feh -rSfilename /opt/images
-+.It feh -rSfilename ~/Pictures
- .
- Same as above, but sort by filename. By default, feh will show files in the
- order it finds them on the hard disk, which is usually somewhat random.
- .
--.It feh -t -Sfilename -E 128 -y 128 -W 1024 /opt/images
-+.It feh -t -Sfilename -E 128 -y 128 -W 1024 ~/Pictures
- .
- Show 128x128 pixel thumbnails, limit window width to 1024 pixels.
- .
--.It feh -t -Sfilename -E 128 -y 128 -W 1024 -P -C 
/usr/share/fonts/truetype/ttf-dejavu/ -e DejaVuSans/8 /opt/images
-+.It feh -t -Sfilename -E 128 -y 128 -W 1024 -P -C 
${X11BASE}/lib/X11/fonts/TTF/ -e DejaVuSans/8 ~/Pictures
- .
- Same as above, but enable thumbnail caching in ~/.thumbnails and use a smaller
- font.
- .
--.It feh -irFarial/14 -O index.jpg /opt/images
-+.It feh -irFarial/14 -O index.jpg ~/Pictures
- .
--Make an index print of /opt/images and all directories below it, using 14 
point
-+Make an index print of ~/Pictures and all directories below it, using 14 point
- Arial to write the image info under each thumbnail.  Save the image as
- index.jpg and don't display it, just exit.  Note that this even works without
- a running X server
- .
--.It feh --unloadable -r /opt/images
-+.It feh --unloadable -r ~/Pictures
- .
--Print all unloadable images in /opt/images, recursively
-+Print all unloadable images in ~/Pictures, recursively
- .
- .It feh -f by_width -S width --reverse --list \&.
- .
- Write a list of all images in the directory to by_width, sorted by width
- .Pq widest images first
- .
--.It feh -w /opt/images/holidays
-+.It feh -w ~/Pictures/holidays
- .
--Open each image in /opt/images/holidays in its own window
-+Open each image in ~/Pictures/holidays in its own window
- .
--.It feh -FD5 -Sname /opt/images/presentation
-+.It feh -FD5 -Sname ~/Pictures/presentation
- .
--Show the images in .../presentation, sorted by name, in fullscreen,
-+Show the images in ~/Pictures/presentation, sorted by name, in fullscreen,
- automatically change to the next image after 5 seconds
- .
--.It feh -rSwidth -A Qo mv %F ~/images/\&%N Qc /opt/images
-+.It feh -rSwidth -A Qo mv %F ~/images/\&%N Qc ~/Pictures
- .
--View all images in /opt/images and below, sorted by width, move an image to
-+View all images in ~/Pictures and below, sorted by width, move an image to
- ~/image/image_name when enter is pressed
- .
- .It feh --start-at ./foo.jpg \&.
-@@ -1745,8 +1745,6 @@ section.
- .
- .
- .Sh BUGS
--.
--.Pp
- .
- On systems with giflib 5.1.2,
- .Nm
Index: patches/patch-src_Makefile
===================================================================
RCS file: patches/patch-src_Makefile
diff -N patches/patch-src_Makefile
--- patches/patch-src_Makefile  8 Sep 2017 08:19:20 -0000       1.4
+++ /dev/null   1 Jan 1970 00:00:00 -0000
@@ -1,16 +0,0 @@
-$OpenBSD: patch-src_Makefile,v 1.4 2017/09/08 08:19:20 dcoppa Exp $
-
-Make graphics/feh build with clang
-
-Index: src/Makefile
---- src/Makefile.orig
-+++ src/Makefile
-@@ -53,7 +53,7 @@ help.inc: help.raw
- 
- 
- deps.mk: ${TARGETS} ${I_DSTS}
--      ${CC} ${CPPFLAGS} -MM ${TARGETS} > $@
-+      ${CC} ${CFLAGS} -MM ${TARGETS} > $@
- 
- clean:
-       rm -f feh *.o *.inc
Index: patches/patch-src_events_c
===================================================================
RCS file: /cvs/ports/graphics/feh/patches/patch-src_events_c,v
retrieving revision 1.10
diff -u -p -r1.10 patch-src_events_c
--- patches/patch-src_events_c  30 Aug 2016 14:44:37 -0000      1.10
+++ patches/patch-src_events_c  11 Feb 2018 15:30:24 -0000
@@ -1,7 +1,8 @@
 $OpenBSD: patch-src_events_c,v 1.10 2016/08/30 14:44:37 dcoppa Exp $
---- src/events.c.orig  Sun Aug 28 22:25:42 2016
-+++ src/events.c       Tue Aug 30 16:28:12 2016
-@@ -123,7 +123,7 @@ void init_buttonbindings(void)
+Index: src/events.c
+--- src/events.c.orig
++++ src/events.c
+@@ -124,7 +124,7 @@ void init_buttonbindings(void)
  
        free(confpath);
  
Index: patches/patch-src_getopt_c
===================================================================
RCS file: patches/patch-src_getopt_c
diff -N patches/patch-src_getopt_c
--- patches/patch-src_getopt_c  9 Sep 2011 09:51:43 -0000       1.2
+++ /dev/null   1 Jan 1970 00:00:00 -0000
@@ -1,14 +0,0 @@
-$OpenBSD: patch-src_getopt_c,v 1.2 2011/09/09 09:51:43 dcoppa Exp $
-
-missing header
-
---- src/getopt.c.orig  Wed Jul  7 11:40:32 2010
-+++ src/getopt.c       Wed Jul  7 11:40:47 2010
-@@ -36,6 +36,7 @@
- #endif
- 
- #include <stdio.h>
-+#include <string.h>
- 
- /* Comment out all this code if we are using the GNU C Library, and are not
-    actually compiling the library itself.  This code is part of the GNU C
Index: patches/patch-src_keyevents_c
===================================================================
RCS file: /cvs/ports/graphics/feh/patches/patch-src_keyevents_c,v
retrieving revision 1.15
diff -u -p -r1.15 patch-src_keyevents_c
--- patches/patch-src_keyevents_c       8 Sep 2017 08:19:20 -0000       1.15
+++ patches/patch-src_keyevents_c       11 Feb 2018 15:30:24 -0000
@@ -2,7 +2,7 @@ $OpenBSD: patch-src_keyevents_c,v 1.15 2
 Index: src/keyevents.c
 --- src/keyevents.c.orig
 +++ src/keyevents.c
-@@ -183,7 +183,7 @@ void init_keyevents(void) {
+@@ -225,7 +225,7 @@ void init_keyevents(void) {
  
        free(confpath);
  
Index: patches/patch-src_options_c
===================================================================
RCS file: /cvs/ports/graphics/feh/patches/patch-src_options_c,v
retrieving revision 1.13
diff -u -p -r1.13 patch-src_options_c
--- patches/patch-src_options_c 8 Sep 2017 08:19:20 -0000       1.13
+++ patches/patch-src_options_c 11 Feb 2018 15:30:24 -0000
@@ -2,7 +2,7 @@ $OpenBSD: patch-src_options_c,v 1.13 201
 Index: src/options.c
 --- src/options.c.orig
 +++ src/options.c
-@@ -156,7 +156,7 @@ static void feh_load_options_for_theme(char *theme)
+@@ -159,7 +159,7 @@ static void feh_load_options_for_theme(char *theme)
  
        free(oldrcpath);
  

Reply via email to