Thomas Frohwein writes:

>> > I tested with misc/toilet, the only consumer.
>
> Not sure how much misc/toilet uses it without '--export <format>', but
> the bundled applications cacademo and cacafire work fine.

good catch on how to actually test it. I used --export html and was able
to view it.

>
> cacaclock complains about not finding a font, but that was already
> there in the previous version and doesn't need to hold up the update in
> my opinion.
>
>> >
>> > Feedback? OK?
>> 
>> ping. This diff has a tweak for PKGNAME substitution for updating
>> verison number in one place.
>
> ok thfr@, but a few comments for maybe future update considerations:
>
> 1. version 0.99.beta20 => 0.99.20 is in line with what the previous
>    version did, but in the future after a major/minor update it might
>    be worth considering keeping the "beta": 0.99beta20. This is based
>    on packages-specs(7): "The last part may contain an extra suffix
>    matching rc[N], alpha[N], beta[N],..."
>
> 2. There are some stray compiler flags in configure, probably this:
>    19672:CFLAGS="${CFLAGS} -g -O2 -fno-strength-reduce -fomit-frame-pointer"
>    I think the "-g -O2" could interfere with builder setting other flags.

Here is a fresh diff, incorporating removal of -g -O2 into the configure
patch. still OK?

Index: Makefile
===================================================================
RCS file: /cvs/ports/graphics/libcaca/Makefile,v
retrieving revision 1.36
diff -u -p -u -p -r1.36 Makefile
--- Makefile    26 Jan 2020 11:14:32 -0000      1.36
+++ Makefile    26 Nov 2021 10:36:05 -0000
@@ -2,11 +2,11 @@
 
 COMMENT=               color ascii art library
 
-DISTNAME=              libcaca-0.99.beta19
-PKGNAME=               libcaca-0.99.19
-SHARED_LIBS +=  caca           1.2               # .0.99
+V=                     0.99.beta20
+DISTNAME=              libcaca-${V}
+PKGNAME=               ${DISTNAME:S/beta//}
+SHARED_LIBS += caca    2.0 # .0.99
 CATEGORIES=            graphics
-REVISION=              2
 
 HOMEPAGE=              http://caca.zoy.org/wiki/libcaca
 
@@ -15,8 +15,7 @@ PERMIT_PACKAGE=       Yes
 
 WANTLIB=               c curses m z
 
-MASTER_SITES=          http://caca.zoy.org/files/libcaca/ \
-                       https://distfiles.sigtrap.nl/
+MASTER_SITES=          
https://github.com/cacalabs/libcaca/releases/download/v${V}/
 
 FLAVORS=               no_x11
 FLAVOR?=
Index: distinfo
===================================================================
RCS file: /cvs/ports/graphics/libcaca/distinfo,v
retrieving revision 1.10
diff -u -p -u -p -r1.10 distinfo
--- distinfo    7 Jun 2014 20:29:32 -0000       1.10
+++ distinfo    26 Nov 2021 10:36:05 -0000
@@ -1,2 +1,2 @@
-SHA256 (libcaca-0.99.beta19.tar.gz) = 
EotGfE7QMmTBh0BRcqToMEk0LMjML2VfU6LQ7p03cvQ=
-SIZE (libcaca-0.99.beta19.tar.gz) = 1203495
+SHA256 (libcaca-0.99.beta20.tar.gz) = 
itdLq8Y79mWwsjeNlbTaZbdJPBG9nzrGAFFwhbDErPI=
+SIZE (libcaca-0.99.beta20.tar.gz) = 1099916
Index: patches/patch-Makefile_in
===================================================================
RCS file: /cvs/ports/graphics/libcaca/patches/patch-Makefile_in,v
retrieving revision 1.5
diff -u -p -u -p -r1.5 patch-Makefile_in
--- patches/patch-Makefile_in   7 Jun 2014 20:29:32 -0000       1.5
+++ patches/patch-Makefile_in   26 Nov 2021 10:36:05 -0000
@@ -1,12 +1,13 @@
 $OpenBSD: patch-Makefile_in,v 1.5 2014/06/07 20:29:32 bcallah Exp $
---- Makefile.in.orig   Fri May 16 15:15:33 2014
-+++ Makefile.in        Sat Jun  7 14:03:02 2014
-@@ -410,7 +410,7 @@ top_build_prefix = @top_build_prefix@
- top_builddir = @top_builddir@
+Index: Makefile.in
+--- Makefile.in.orig
++++ Makefile.in
+@@ -427,7 +427,7 @@ top_builddir = @top_builddir@
  top_srcdir = @top_srcdir@
+ AM_CPPFLAGS = -I$(top_srcdir) -I$(top_srcdir)/caca -I$(top_builddir)/caca
  SUBDIRS = kernel caca src examples tools \
--          cxx caca-sharp python ruby java \
-+          cxx ruby \
-           doc test
+-          cxx caca-sharp python ruby java doc
++          cxx doc
  
- DIST_SUBDIRS = $(SUBDIRS) win32
+ DIST_SUBDIRS = $(SUBDIRS) build
+ EXTRA_DIST = NOTES COPYING.GPL COPYING.ISC COPYING.LGPL bootstrap 
caca-config.in libcaca.spec libcaca.sln
Index: patches/patch-configure
===================================================================
RCS file: /cvs/ports/graphics/libcaca/patches/patch-configure,v
retrieving revision 1.5
diff -u -p -u -p -r1.5 patch-configure
--- patches/patch-configure     7 Jun 2014 20:29:32 -0000       1.5
+++ patches/patch-configure     26 Nov 2021 10:36:05 -0000
@@ -1,16 +1,19 @@
 $OpenBSD: patch-configure,v 1.5 2014/06/07 20:29:32 bcallah Exp $
 
-1. Disable detection of pangoft2.
+1. Disable detection of ftgl. (no --disable-ftgl option.)
 
-2. Disable detection of cppunit.  (--disable-cppunit is a NOP.)
+2. Remove bash-ism =~
 
-3. Disable detection of ftgl. (no --disable-ftgl option.)
+3. Remove hardcoded -g -O2
 
---- configure.orig     Fri May 16 15:15:32 2014
-+++ configure  Sat Jun  7 14:03:02 2014
-@@ -17495,8 +17495,7 @@ if test -n "$FTGL_CFLAGS"; then
+4. Disable detection of pangoft2.
+
+Index: configure
+--- configure.orig
++++ configure
+@@ -19339,8 +19339,7 @@ if test -n "$FTGL_CFLAGS"; then
    ac_status=$?
-   $as_echo "$as_me:${as_lineno-$LINENO}: \$? = $ac_status" >&5
+   printf "%s\n" "$as_me:${as_lineno-$LINENO}: \$? = $ac_status" >&5
    test $ac_status = 0; }; then
 -  pkg_cv_FTGL_CFLAGS=`$PKG_CONFIG --cflags "ftgl >= 2.1.3" 2>/dev/null`
 -                    test "x$?" != "x0" && pkg_failed=yes
@@ -18,9 +21,9 @@ $OpenBSD: patch-configure,v 1.5 2014/06/
  else
    pkg_failed=yes
  fi
-@@ -17512,8 +17511,7 @@ if test -n "$FTGL_LIBS"; then
+@@ -19356,8 +19355,7 @@ if test -n "$FTGL_LIBS"; then
    ac_status=$?
-   $as_echo "$as_me:${as_lineno-$LINENO}: \$? = $ac_status" >&5
+   printf "%s\n" "$as_me:${as_lineno-$LINENO}: \$? = $ac_status" >&5
    test $ac_status = 0; }; then
 -  pkg_cv_FTGL_LIBS=`$PKG_CONFIG --libs "ftgl >= 2.1.3" 2>/dev/null`
 -                    test "x$?" != "x0" && pkg_failed=yes
@@ -28,21 +31,30 @@ $OpenBSD: patch-configure,v 1.5 2014/06/
  else
    pkg_failed=yes
  fi
-@@ -18672,7 +18670,7 @@ fi
- 
- # Build development tools?
+@@ -19422,7 +19420,7 @@ fi
+ done
+   CFLAGS="$save_CFLAGS"
+   if test "${ac_cv_my_have_cocoa}" = "yes"; then
+-    [[ "$target_os" =~ [0-9]+ ]] && darwin_ver="${BASH_REMATCH[0]}"
++    [[ "$target_os" = [0-9]+ ]] && darwin_ver="${BASH_REMATCH[0]}"
+     case x${target} in
+     xpowerpc*darwin*)
+       # 10.3 needed to link with X11
+@@ -19671,7 +19669,7 @@ fi
  
--pkg_failed=no
-+pkg_failed=yes
- { $as_echo "$as_me:${as_lineno-$LINENO}: checking for PANGOFT2" >&5
- $as_echo_n "checking for PANGOFT2... " >&6; }
  
-@@ -18963,7 +18961,7 @@ fi
+ # Optimizations
+-CFLAGS="${CFLAGS} -g -O2 -fno-strength-reduce -fomit-frame-pointer"
++CFLAGS="${CFLAGS} -fno-strength-reduce -fomit-frame-pointer"
+ # Code qui fait des warnings == code de porc == deux baffes dans ta gueule
+ CFLAGS="${CFLAGS} -Wall -Wpointer-arith -Wcast-align -Wcast-qual 
-Wstrict-prototypes -Wshadow -Waggregate-return -Wmissing-prototypes 
-Wnested-externs -Wsign-compare"
+ CXXFLAGS="${CXXFLAGS} -Wall -Wpointer-arith -Wcast-align -Wcast-qual -Wshadow 
-Wsign-compare"
+@@ -20696,7 +20694,7 @@ fi
  
- # Use cppunit for unit tests?
+ # Build development tools?
  
 -pkg_failed=no
 +pkg_failed=yes
- { $as_echo "$as_me:${as_lineno-$LINENO}: checking for CPPUNIT" >&5
- $as_echo_n "checking for CPPUNIT... " >&6; }
+ { printf "%s\n" "$as_me:${as_lineno-$LINENO}: checking for pangoft2" >&5
+ printf %s "checking for pangoft2... " >&6; }
  
Index: pkg/PLIST
===================================================================
RCS file: /cvs/ports/graphics/libcaca/pkg/PLIST,v
retrieving revision 1.7
diff -u -p -u -p -r1.7 PLIST
--- pkg/PLIST   22 May 2015 11:31:15 -0000      1.7
+++ pkg/PLIST   26 Nov 2021 10:36:05 -0000
@@ -11,7 +11,7 @@ include/caca.h
 include/caca0.h
 include/caca_conio.h
 include/caca_types.h
-lib/libcaca.a
+@static-lib lib/libcaca.a
 lib/libcaca.la
 @lib lib/libcaca.so.${LIBcaca_VERSION}
 lib/pkgconfig/caca.pc

Reply via email to