Re: [MAINTAINER UPDATE] archivers/zpaqfranz-55.15

On 2022/09/20 10:15, tux0r wrote:
> Ahoi, Peter,
>
> yes, you are right. I copied "-O3 -march=native" from the old Makefile that the previous port used, but I should probably have taken care of that earlier.
>
> Updated diff attached.
>
> tux0r.
>
> > On Montag, Sept. 19, 2022 at 10:38 PM, Peter Hessler <phessler@theapt.org (mailto:phessler@theapt.org)> wrote:
> > On 2022 Sep 19 (Mon) at 18:57:41 +0200 (+0200), tux0r wrote:
> > :For no obvious reason, the dev decided to delete the Makefile, so the port will have to take care of that as well...
> > :
> > :Looks like no OpenBSD-specifics are required this time - here's the patch.
> > :
> > :tux0r.
> >
> >
> > Please remove "-O3 -march=native" from the compiler line. We don't want
> > -O3 optimizations, and "-march=native" will generate code that won't run
> > on all supported machines.
> >
> >
> > --
> > Practical people would be more practical if they would take a little
> > more time for dreaming.
> > -- J. P. McEvoy


This diff looks better, but upstream's endian detection is not automatic
so it needs some attention on a big-endian machine.

USE_WXNEEDED doesn't give me warm fuzzy feelings about using this software.
I wonder if maybe the jit should be disabled by default (possibly adding
a jit FLAVOR).

Index: Makefile
===================================================================
RCS file: /cvs/ports/archivers/zpaqfranz/Makefile,v
retrieving revision 1.6
diff -u -p -r1.6 Makefile
--- Makefile 6 Sep 2022 07:07:39 -0000 1.6
+++ Makefile 20 Sep 2022 08:51:20 -0000
@@ -2,7 +2,7 @@ COMMENT = journaling archiver for increm

GH_ACCOUNT = fcorbelli
GH_PROJECT = zpaqfranz
-GH_TAGNAME = 55.14
+GH_TAGNAME = 55.15

CATEGORIES = archivers
MAINTAINER = tux0r <tux0r@rosaelefanten.org>
@@ -10,20 +10,17 @@ MAINTAINER = tux0r <tux0r@rosaelefanten.
# Public Domain
PERMIT_PACKAGE = yes

-FIX_CRLF_FILES = Makefile zpaqfranz.cpp
-
-COMPILER= base-clang ports-gcc base-gcc
+COMPILER = base-clang ports-gcc base-gcc

WANTLIB = ${COMPILER_LIBCXX} c m

-MAKE_FLAGS += CXX="${CXX}" \
- CXXFLAGS="${CXXFLAGS}" \
- CPPFLAGS="${CPPFLAGS}"
+USE_WXNEEDED = Yes

-CPPFLAGS += -Dunix
-USE_WXNEEDED= Yes
+do-build:
+ cd ${WRKSRC}; ${MAKE_PROGRAM} LDFLAGS=-lpthread zpaqfranz

-post-install:
+do-install:
+ ${INSTALL_PROGRAM} ${WRKSRC}/zpaqfranz ${PREFIX}/bin/
${INSTALL_MAN} ${WRKSRC}/man/zpaqfranz.1 ${PREFIX}/man/man1/

do-test:
Index: distinfo
===================================================================
RCS file: /cvs/ports/archivers/zpaqfranz/distinfo,v
retrieving revision 1.4
diff -u -p -r1.4 distinfo
--- distinfo 5 Sep 2022 20:14:47 -0000 1.4
+++ distinfo 20 Sep 2022 08:51:20 -0000
@@ -1,2 +1,2 @@
-SHA256 (zpaqfranz-55.14.tar.gz) = NQu4rTUPFK5MEOypueG2kaAU4dXUGG54vI2ajgTMF6M=
-SIZE (zpaqfranz-55.14.tar.gz) = 10588313
+SHA256 (zpaqfranz-55.15.tar.gz) = wgcWqzVjQ8S5bx0ecptcK3P0vuH8rUlXdfGprrIhbjQ=
+SIZE (zpaqfranz-55.15.tar.gz) = 3366870