On Fri, Mar 13 2020, Paul Irofti <p...@irofti.net> wrote:
> Hi,
>
> Thank you for your effort! I have a few comments and questions.

I got curious about this proposal, mostly because of the COMPILER line
change.

> I see they switched to Gitlab (hence the url changes) and that the
> author set it up for continous integration and thus made it compile with
> clang.
>
> Why did you bump the major. I tried browsing the history on Gitlab to
> see what changed in the API, but couldn't see much. I am sure I missed
> something.

As shown by /usr/src/lib/check_sym ("No dynamic export changes"), and
a diff of the installed include file, the shared lib version bump
appears unnecessary.

> Is the gnuplot bump necessary? Why?

Maybe Yozo found it useful for testing, but it doesn't seem useful for
the ports tree.

The updated diff below has been stripped from the items that
I mentioned.  It also updates WANTLIB, no need for COMPILER_LIBCXX since
this is a C-only port.  Builds fine on sparc64 along with
gnuplot,no_x11.  Paul, feel free to use the diff below.


Index: Makefile
===================================================================
RCS file: /cvs/ports/math/libcerf/Makefile,v
retrieving revision 1.5
diff -u -p -r1.5 Makefile
--- Makefile    12 Jul 2019 20:47:42 -0000      1.5
+++ Makefile    14 Mar 2020 00:23:15 -0000
@@ -2,26 +2,24 @@
 
 COMMENT =      implementation of complex error functions
 
-V =            1.11
-DISTNAME =     libcerf-${V}
+V =            1.13
+DISTNAME =     libcerf-v${V}
+PKGNAME =      libcerf-${V}
 EXTRACT_SUFX = .tgz
 CATEGORIES =   math
-MASTER_SITES = http://apps.jcns.fz-juelich.de/src/libcerf/
-REVISION =     0
+MASTER_SITES = https://jugit.fz-juelich.de/mlz/libcerf/-/archive/v${V}/
 
-SHARED_LIBS +=  cerf   2.0 # 1.10
+SHARED_LIBS +=  cerf   2.0 # 1.13
 
-HOMEPAGE =     http://apps.jcns.fz-juelich.de/doku/sc/libcerf
+HOMEPAGE =     https://jugit.fz-juelich.de/mlz/libcerf/
 
 MAINTAINER =   Paul Irofti <piro...@openbsd.org>
 
 # MIT
 PERMIT_PACKAGE =       Yes
 
-WANTLIB += c m ${COMPILER_LIBCXX}
+WANTLIB += m
 
 MODULES =              devel/cmake
-
-COMPILER =             ports-gcc   # imaginary constants are a GNU extension
 
 .include <bsd.port.mk>
Index: distinfo
===================================================================
RCS file: /cvs/ports/math/libcerf/distinfo,v
retrieving revision 1.3
diff -u -p -r1.3 distinfo
--- distinfo    29 Dec 2018 07:48:36 -0000      1.3
+++ distinfo    14 Mar 2020 00:23:15 -0000
@@ -1,2 +1,2 @@
-SHA256 (libcerf-1.11.tgz) = cBAcrEoNeGMyLU0Gz5XFB6nP1k/JmtGzGoQlIEz9lnI=
-SIZE (libcerf-1.11.tgz) = 60066
+SHA256 (libcerf-v1.13.tgz) = 5Gmfga+Diu9bPncgn+yOmCCk9JLVmPtaBwgAhUl2owU=
+SIZE (libcerf-v1.13.tgz) = 60732


-- 
jca | PGP : 0x1524E7EE / 5135 92C1 AD36 5293 2BDF  DDCC 0DFA 74AE 1524 E7EE

Reply via email to