Re: update graphics/png
On Sun, Aug 12, 2018 at 05:03:03PM +0200, Christian Weisgerber wrote: > If I wanted to patch this, I'd add the _BSD_SOURCE line without > replacing the _GNU_SOURCE one. Upstreamed as follows: Index: patches/patch-contrib_libtests_pngvalid_c === RCS file: patches/patch-contrib_libtests_pngvalid_c diff -N patches/patch-contrib_libtests_pngvalid_c --- /dev/null 1 Jan 1970 00:00:00 - +++ patches/patch-contrib_libtests_pngvalid_c 13 Aug 2018 04:50:09 - @@ -0,0 +1,16 @@ +$OpenBSD$ + +Fix warning about implicit declaration of feenableexcept(3). +Accepted upstream: https://sourceforge.net/p/png-mng/mailman/message/36390857/ + +Index: contrib/libtests/pngvalid.c +--- contrib/libtests/pngvalid.c.orig contrib/libtests/pngvalid.c +@@ -21,6 +21,7 @@ + #define _POSIX_SOURCE 1 + #define _ISOC99_SOURCE 1 /* For floating point */ + #define _GNU_SOURCE 1 /* For the floating point exception extension */ ++#define _BSD_SOURCE 1 /* For the floating point exception extension */ + + #include + #include
Re: [MAINTAINER UPDATE] sysutils/py-scandir 1.8 -> 1.9.0
On Sun, 12 Aug 2018 19:17:40 -0300, "Elias M. Mariani" wrote: > Changelog: > Fix build on Python 2.7 (#107). > Fix dirent struct alignment on OpenBSD (#109). > > They pull my request to patch the package upstream. > So the following diff updates to 1.9.0 and removes the patch. > > No consumers yet of this port. > All test passing. > > Cheers. > Elias. Thanks committed! I plan to look at your other submissions (if no one beats me to it) in the next few days :)
[MAINTAINER UPDATE] sysutils/py-scandir 1.8 -> 1.9.0
Changelog: Fix build on Python 2.7 (#107). Fix dirent struct alignment on OpenBSD (#109). They pull my request to patch the package upstream. So the following diff updates to 1.9.0 and removes the patch. No consumers yet of this port. All test passing. Cheers. Elias. py-scandir-1.9.0.diff Description: Binary data
Re: [NEW] games/ezquake
On 8/12/18 4:28 PM, Klemens Nanni wrote: Port itself looks good, i cannot test the game, thouugh. The second part of DESCR explaining how to set up the game rather belongs into README imho. Thoughts on this? How about this version, which uses a 1-line patch to the Makefile to get rid of that arch dance you're doing there. I think it's a bit more elegant and obvious this way. I also don't own the game, so also can't test; but yes kn@ is right that we tend to put instructions in a pkg/README, though I wouldn't object to have the information in both places. What I do object to is that there's no mention of the fact that Quake itself is still software that you must purchase in order to use ezquake. See games/julius for example language to be put into this port. ~Brian ezquake.tgz Description: Binary data
Re: [NEW] games/ezquake
Port itself looks good, i cannot test the game, thouugh. The second part of DESCR explaining how to set up the game rather belongs into README imho. Thoughts on this?
UPDATE sysutils/borgbackup
The diff below brings borgbackup to 1.1.7, which is a maintenance
release. Changelog can be found at
https://github.com/borgbackup/borg/blob/1.1.7/docs/changes.rst#version-117-2018-08-11
'make test' runs successfully, and making and restoring a backup on
amd64 work as advertised.
There is no need any more for removing '__attribute__((optimize("Os")))'
in the pre-configure phase as this has been addressed upstream.
I intend to commit the diff below in a couple of days. Additional
testing would be appreciated, especially tests on other platforms than
amd64.
Index: Makefile
===
RCS file: /cvs/ports/sysutils/borgbackup/Makefile,v
retrieving revision 1.23
diff -u -p -r1.23 Makefile
--- Makefile2 Aug 2018 13:43:05 - 1.23
+++ Makefile12 Aug 2018 07:07:55 -
@@ -2,8 +2,7 @@
COMMENT = deduplicating backup program
-MODPY_EGG_VERSION =1.1.6
-REVISION = 0
+MODPY_EGG_VERSION =1.1.7
DISTNAME = borgbackup-${MODPY_EGG_VERSION}
CATEGORIES = sysutils
@@ -30,10 +29,6 @@ TEST_DEPENDS += ${RUN_DEPENDS} \
devel/py-test${MODPY_FLAVOR} \
devel/py-test-benchmark${MODPY_FLAVOR} \
devel/py-test-xdist${MODPY_FLAVOR}
-
-pre-configure:
- find ${WRKSRC} -type f \
- -exec perl -i -pe 's# __attribute__\(\(optimize\("Os"\)\)\)##g'
{} \;
post-install:
${INSTALL_MAN} ${WRKSRC}/docs/man/*.1 ${PREFIX}/man/man1/
Index: distinfo
===
RCS file: /cvs/ports/sysutils/borgbackup/distinfo,v
retrieving revision 1.15
diff -u -p -r1.15 distinfo
--- distinfo18 Jun 2018 15:40:16 - 1.15
+++ distinfo12 Aug 2018 07:07:55 -
@@ -1,2 +1,2 @@
-SHA256 (borgbackup-1.1.6.tar.gz) = odLkdMhdOtPVmz+CCbVUllPIiRIILqAVnSei6AyRCTA=
-SIZE (borgbackup-1.1.6.tar.gz) = 3441523
+SHA256 (borgbackup-1.1.7.tar.gz) = 97UaEy6e374crLT0eLKMrzYi15//yzab2un5LYyKf9w=
+SIZE (borgbackup-1.1.7.tar.gz) = 3446832
Re: update graphics/png
Theo Buehler: > There's one build warning: > > /usr/ports/pobj/png-1.6.35/libpng-1.6.35/contrib/libtests/pngvalid.c:11661:4: > warning: implicit declaration of function 'feenableexcept' is invalid in C99 > [-Wimplicit-function-declaration] > feenableexcept(FE_DIVBYZERO | FE_INVALID | FE_OVERFLOW); It's in contrib, so we don't care... Well, that one is used by the tests, but still. *shrug* > If a more experienced porter agrees with this, please include it with > your diff. > + #define _POSIX_SOURCE 1 > + #define _ISOC99_SOURCE 1 /* For floating point */ > +-#define _GNU_SOURCE 1 /* For the floating point exception extension */ > ++#define _BSD_SOURCE 1 /* For the floating point exception extension */ If I wanted to patch this, I'd add the _BSD_SOURCE line without replacing the _GNU_SOURCE one. -- Christian "naddy" Weisgerber na...@mips.inka.de
Re: update graphics/png
Solene Rapenne: > this patch updates libpng to 1.6.35, the new version changelog is the > following: OK I went over the diff .34 -> .35 and I don't see any ABI changes. -- Christian "naddy" Weisgerber na...@mips.inka.de
Re: UPDATE games/wesnoth
>On 2018-08-01 17:11:58, Bryan Linton wrote: >> On 2018-07-30 11:26:38, Kirill Bychkov wrote: >> > Hi! >> > This is a long standing update for wesnoth. Current version includes >> > security fixes (which were backported to 1.12.6 in our tree). >> > 1.14.x requires boost with icu support. Without it the game crashes >> > on preferences dialog and add-on list. >> > See my next mail with the patch for boost. >> > >> >> No problems here. >> >> I've been able to successfully download and install several >> add-ons, and play games on the multi-player server. >> > >I know that this update is still waiting on the boost ICU support >to go through a bulk and be thoroughly tested because of how high >up in the dependency chain boost is, but I just wanted to add an >update that I haven't experienced any problems with Wesnoth or the >boost ICU patch in the last two weeks. > >I've played several dozen (if not closer to a hundred) games, >added and used several add-ons/maps/etc., played both single and >multiplayer, and haven't encountered any bugs or issues yet. > >So if and when the boost ICU update gets committed, this Wesnoth >update works fine for me on amd64 at least. > >Many thanks again to Kirill for the update! > >-- >Bryan Hi, I'd love to test this as I do enjoy playing Wesnoth. The patch Kirill posted for 1.14.4 appears to require devel/boost 1.66.0p0 or later, however boost 1.66.0 is the latest version in the tree. I've looked but couldn't find 1.66.0 revision 0 patches. Is there an update to devel/boost for 1.66.0p0 somewhere? Thanks, Tom
[NEW] games/mvdsv 0.32
Here's a port for mvdsv 0.32 which is a Quakeworld server. It's kind of a sister port to games/ezquake except it only runs as a server for Quakeworld clients to connect to. The files distributed in this port are qwprogs.dat which is GPLv2 (under id Software's release) and a sample server.cfg file. OK? Thanks, Tom mvdsv.tgz Description: application/tar-gz
Re: pftop: fix sparc64 build
On 2018/08/12 11:47, Klemens Nanni wrote:
> On Sat, Aug 11, 2018 at 09:18:21PM -0600, lan...@openbsd.org wrote:
> > http://build-failures.rhaalovely.net//sparc64/2018-07-31/sysutils/pftop.log
>
> bpf_image.c: In function 'bpf_image':
> bpf_image.c:41: error: argument 'p' doesn't match prototype
> /usr/include/pcap.h:233: error: prototype declaration
>
> The following commit to lib/libpcap/pcap.h caused this:
>
> revision 1.20
> date: 2018/06/03 10:29:28; author: sthen; state: Exp; lines: +6 -6;
> commitid: yMmArqThiDSrYVYu;
> Change some libpcap functions which use pointers as arguments and
> returns to const pointers:
>
> - the prefix argument to pcap_perror();
> - the return value of pcap_strerror();
> - the filter expression argument to pcap_compile() and
> pcap_compile_nopcap();
> - the BPF filter program argument to bpf_image().
>
> Matches changes made earlier in tcpdump.org's version of libpcap.
> From Guy Harris, ok tb@, been through a bulk ports build.
>
> Tested on sparc64 and amd64, OK?
>
> Index: Makefile
> ===
> RCS file: /cvs/ports/sysutils/pftop/Makefile,v
> retrieving revision 1.33
> diff -u -p -r1.33 Makefile
> --- Makefile 15 Feb 2018 11:17:05 - 1.33
> +++ Makefile 12 Aug 2018 09:41:39 -
> @@ -5,7 +5,7 @@
> COMMENT= curses-based real time state and rule display for pf
>
> DISTNAME=pftop-0.7
> -REVISION=17
> +REVISION=18
> CATEGORIES= sysutils
Bump not needed for this.
OK
>
> HOMEPAGE=http://www.eee.metu.edu.tr/~canacar/pftop/
> Index: patches/patch-bpf_image_c
> ===
> RCS file: /cvs/ports/sysutils/pftop/patches/patch-bpf_image_c,v
> retrieving revision 1.1
> diff -u -p -r1.1 patch-bpf_image_c
> --- patches/patch-bpf_image_c 19 Jun 2013 03:46:55 - 1.1
> +++ patches/patch-bpf_image_c 12 Aug 2018 09:38:27 -
> @@ -2,8 +2,9 @@ $OpenBSD: patch-bpf_image_c,v 1.1 2013/0
>
> pcap-int.h no longer exists in /usr/include
>
> bpf_image.c.orig Wed Nov 7 01:34:18 2007
> -+++ bpf_image.c Thu Jun 13 15:24:21 2013
> +Index: bpf_image.c
> +--- bpf_image.c.orig
> bpf_image.c
> @@ -28,7 +28,7 @@
> #include
> #include
> @@ -13,3 +14,12 @@ pcap-int.h no longer exists in /usr/incl
>
> #ifdef HAVE_OS_PROTO_H
> #include "os-proto.h"
> +@@ -36,7 +36,7 @@
> +
> + char *
> + bpf_image(p, n)
> +-struct bpf_insn *p;
> ++const struct bpf_insn *p;
> + int n;
> + {
> + int v;
>
Re: pftop: fix sparc64 build
On Sun, Aug 12, 2018 at 12:06:33PM +0200, Theo Buehler wrote: > If I understand correctly, this a gcc vs clang problem, right? Correct. > btw. HOMEPAGE/MASTER_SITES (http://www.eee.metu.edu.tr/~canacar/) 404. Known, but I wanted to keep the fix committed without churn.
Re: pftop: fix sparc64 build
On Sun, Aug 12, 2018 at 11:47:38AM +0200, Klemens Nanni wrote:
> On Sat, Aug 11, 2018 at 09:18:21PM -0600, lan...@openbsd.org wrote:
> > http://build-failures.rhaalovely.net//sparc64/2018-07-31/sysutils/pftop.log
>
> bpf_image.c: In function 'bpf_image':
> bpf_image.c:41: error: argument 'p' doesn't match prototype
> /usr/include/pcap.h:233: error: prototype declaration
>
> The following commit to lib/libpcap/pcap.h caused this:
>
> revision 1.20
> date: 2018/06/03 10:29:28; author: sthen; state: Exp; lines: +6 -6;
> commitid: yMmArqThiDSrYVYu;
> Change some libpcap functions which use pointers as arguments and
> returns to const pointers:
>
> - the prefix argument to pcap_perror();
> - the return value of pcap_strerror();
> - the filter expression argument to pcap_compile() and
> pcap_compile_nopcap();
> - the BPF filter program argument to bpf_image().
>
> Matches changes made earlier in tcpdump.org's version of libpcap.
> From Guy Harris, ok tb@, been through a bulk ports build.
>
> Tested on sparc64 and amd64, OK?
If I understand correctly, this a gcc vs clang problem, right?
ok
btw. HOMEPAGE/MASTER_SITES (http://www.eee.metu.edu.tr/~canacar/) 404.
>
> Index: Makefile
> ===
> RCS file: /cvs/ports/sysutils/pftop/Makefile,v
> retrieving revision 1.33
> diff -u -p -r1.33 Makefile
> --- Makefile 15 Feb 2018 11:17:05 - 1.33
> +++ Makefile 12 Aug 2018 09:41:39 -
> @@ -5,7 +5,7 @@
> COMMENT= curses-based real time state and rule display for pf
>
> DISTNAME=pftop-0.7
> -REVISION=17
> +REVISION=18
> CATEGORIES= sysutils
>
> HOMEPAGE=http://www.eee.metu.edu.tr/~canacar/pftop/
> Index: patches/patch-bpf_image_c
> ===
> RCS file: /cvs/ports/sysutils/pftop/patches/patch-bpf_image_c,v
> retrieving revision 1.1
> diff -u -p -r1.1 patch-bpf_image_c
> --- patches/patch-bpf_image_c 19 Jun 2013 03:46:55 - 1.1
> +++ patches/patch-bpf_image_c 12 Aug 2018 09:38:27 -
> @@ -2,8 +2,9 @@ $OpenBSD: patch-bpf_image_c,v 1.1 2013/0
>
> pcap-int.h no longer exists in /usr/include
>
> bpf_image.c.orig Wed Nov 7 01:34:18 2007
> -+++ bpf_image.c Thu Jun 13 15:24:21 2013
> +Index: bpf_image.c
> +--- bpf_image.c.orig
> bpf_image.c
> @@ -28,7 +28,7 @@
> #include
> #include
> @@ -13,3 +14,12 @@ pcap-int.h no longer exists in /usr/incl
>
> #ifdef HAVE_OS_PROTO_H
> #include "os-proto.h"
> +@@ -36,7 +36,7 @@
> +
> + char *
> + bpf_image(p, n)
> +-struct bpf_insn *p;
> ++const struct bpf_insn *p;
> + int n;
> + {
> + int v;
>
pftop: fix sparc64 build
On Sat, Aug 11, 2018 at 09:18:21PM -0600, lan...@openbsd.org wrote:
> http://build-failures.rhaalovely.net//sparc64/2018-07-31/sysutils/pftop.log
bpf_image.c: In function 'bpf_image':
bpf_image.c:41: error: argument 'p' doesn't match prototype
/usr/include/pcap.h:233: error: prototype declaration
The following commit to lib/libpcap/pcap.h caused this:
revision 1.20
date: 2018/06/03 10:29:28; author: sthen; state: Exp; lines: +6 -6;
commitid: yMmArqThiDSrYVYu;
Change some libpcap functions which use pointers as arguments and
returns to const pointers:
- the prefix argument to pcap_perror();
- the return value of pcap_strerror();
- the filter expression argument to pcap_compile() and
pcap_compile_nopcap();
- the BPF filter program argument to bpf_image().
Matches changes made earlier in tcpdump.org's version of libpcap.
From Guy Harris, ok tb@, been through a bulk ports build.
Tested on sparc64 and amd64, OK?
Index: Makefile
===
RCS file: /cvs/ports/sysutils/pftop/Makefile,v
retrieving revision 1.33
diff -u -p -r1.33 Makefile
--- Makefile15 Feb 2018 11:17:05 - 1.33
+++ Makefile12 Aug 2018 09:41:39 -
@@ -5,7 +5,7 @@
COMMENT= curses-based real time state and rule display for pf
DISTNAME= pftop-0.7
-REVISION= 17
+REVISION= 18
CATEGORIES=sysutils
HOMEPAGE= http://www.eee.metu.edu.tr/~canacar/pftop/
Index: patches/patch-bpf_image_c
===
RCS file: /cvs/ports/sysutils/pftop/patches/patch-bpf_image_c,v
retrieving revision 1.1
diff -u -p -r1.1 patch-bpf_image_c
--- patches/patch-bpf_image_c 19 Jun 2013 03:46:55 - 1.1
+++ patches/patch-bpf_image_c 12 Aug 2018 09:38:27 -
@@ -2,8 +2,9 @@ $OpenBSD: patch-bpf_image_c,v 1.1 2013/0
pcap-int.h no longer exists in /usr/include
bpf_image.c.orig Wed Nov 7 01:34:18 2007
-+++ bpf_image.cThu Jun 13 15:24:21 2013
+Index: bpf_image.c
+--- bpf_image.c.orig
bpf_image.c
@@ -28,7 +28,7 @@
#include
#include
@@ -13,3 +14,12 @@ pcap-int.h no longer exists in /usr/incl
#ifdef HAVE_OS_PROTO_H
#include "os-proto.h"
+@@ -36,7 +36,7 @@
+
+ char *
+ bpf_image(p, n)
+- struct bpf_insn *p;
++ const struct bpf_insn *p;
+ int n;
+ {
+ int v;
Re: UPDATE games/wesnoth
On 2018-08-01 17:11:58, Bryan Linton wrote: > On 2018-07-30 11:26:38, Kirill Bychkov wrote: > > Hi! > > This is a long standing update for wesnoth. Current version includes > > security fixes (which were backported to 1.12.6 in our tree). > > 1.14.x requires boost with icu support. Without it the game crashes > > on preferences dialog and add-on list. > > See my next mail with the patch for boost. > > > > No problems here. > > I've been able to successfully download and install several > add-ons, and play games on the multi-player server. > I know that this update is still waiting on the boost ICU support to go through a bulk and be thoroughly tested because of how high up in the dependency chain boost is, but I just wanted to add an update that I haven't experienced any problems with Wesnoth or the boost ICU patch in the last two weeks. I've played several dozen (if not closer to a hundred) games, added and used several add-ons/maps/etc., played both single and multiplayer, and haven't encountered any bugs or issues yet. So if and when the boost ICU update gets committed, this Wesnoth update works fine for me on amd64 at least. Many thanks again to Kirill for the update! -- Bryan
Re: update graphics/png
On 2018/08/11 17:50, Theo Buehler wrote: > On Sat, Aug 11, 2018 at 02:58:44PM +0100, Stuart Henderson wrote: > > On 2018/08/11 00:43, Solene Rapenne wrote: > > > this patch updates libpng to 1.6.35, the new version changelog is the > > > following: > > > > > > Changes since the last public release (1.6.34): > > > > > > Restored 21 of the contrib/pngsuite/i*.png, which do not cause test > > > failures. Placed the remainder in contrib/pngsuite/interlaced/i*.png. > > > Added calls to png_set_*() transforms commonly used by browsers to > > > the fuzzer. > > > Removed some unnecessary brackets in pngrtran.c > > > Fixed miscellaneous typos (Patch by github user "luzpaz"). > > > Change "ASM C" to "C ASM" in CMakeLists.txt > > > Fixed incorrect handling of bKGD chunk in sub-8-bit files (Cosmin) > > > Added hardware optimization directories to zip and 7z distributions. > > > Fixed incorrect bitmask for options. > > > Fixed many spelling typos. > > > Make png_get_iCCP consistent with man page (allow compression-type > > > argument > > > to be NULL, bug report by Lenard Szolnoki). > > > Replaced the remaining uses of png_size_t with size_t (Cosmin) > > > Fixed the calculation of row_factor in png_check_chunk_length > > > (reported by Thuan Pham in SourceForge issue #278) > > > Added missing parentheses to a macro definition > > > (suggested by "irwir" in GitHub issue #216) > > > > > > > > > It compiles fine and the test pass 100%, maybe a bulk build is required? > > > > I've read through a diff of old and new source, I don't think a bulk > > build is necessary. OK with me though I wouldn't mind another pair of > > eyes on it as it's high in the dependency chain and the release approaches. > > FWIW: I looked through the diff as well and couldn't spot anything > worrisome. I'm also ok with this. > > There's one build warning: > > /usr/ports/pobj/png-1.6.35/libpng-1.6.35/contrib/libtests/pngvalid.c:11661:4: > warning: implicit declaration of function 'feenableexcept' is invalid in C99 > [-Wimplicit-function-declaration] > feenableexcept(FE_DIVBYZERO | FE_INVALID | FE_OVERFLOW); > ^ > > due to the fact that feenableexcept() is guarded by __BSD_VISIBLE, which > is set to 0 if _POSIX_SOURCE is defined without _BSD_SOURCE being defined. > > Patch below would fix this. Tests still pass. > > If a more experienced porter agrees with this, please include it with > your diff. I'd be OK either way on this, it was already there in 1.6.34, and I think it's basically a noop anyway. > Index: patches/patch-contrib_libtests_pngvalid_c > === > RCS file: patches/patch-contrib_libtests_pngvalid_c > diff -N patches/patch-contrib_libtests_pngvalid_c > --- /dev/null 1 Jan 1970 00:00:00 - > +++ patches/patch-contrib_libtests_pngvalid_c 11 Aug 2018 15:43:41 - > @@ -0,0 +1,16 @@ > +$OpenBSD$ > + > +Avoid implicit declaration of feenableexcept(3). > + > +Index: contrib/libtests/pngvalid.c > +--- contrib/libtests/pngvalid.c.orig > contrib/libtests/pngvalid.c > +@@ -20,7 +20,7 @@ > + > + #define _POSIX_SOURCE 1 > + #define _ISOC99_SOURCE 1 /* For floating point */ > +-#define _GNU_SOURCE 1 /* For the floating point exception extension */ > ++#define _BSD_SOURCE 1 /* For the floating point exception extension */ > + > + #include > + #include >
Re: [NEW] games/ezquake
On Sun, Aug 12, 2018 at 09:17:51AM +0100, Tom Murphy wrote: > Here's a new port for ezquake, a Quakeworld client. > The upstream maintainers are still working on an official > version 3.1 release, however, I'd like to get 3.1-rc2 in > before 6.4 is locked if possible. > > The game runs great. Connects to Quakeworld servers. A > bunch of us in Freenode's #openbsd-gaming have been using > it for months. > > When 3.1 is officially released, I'll change it to the > release tarball which is more preferred, but for now it's > pinned to a commit. > > OK? > > Thanks, > Tom We have been using a variation of this port for many months. Just tested the submitted tarball, this is OK awolk@ Regards, Adam
[NEW] games/ezquake
Here's a new port for ezquake, a Quakeworld client. The upstream maintainers are still working on an official version 3.1 release, however, I'd like to get 3.1-rc2 in before 6.4 is locked if possible. The game runs great. Connects to Quakeworld servers. A bunch of us in Freenode's #openbsd-gaming have been using it for months. When 3.1 is officially released, I'll change it to the release tarball which is more preferred, but for now it's pinned to a commit. OK? Thanks, Tom ezquake.tgz Description: application/tar-gz
