Re: games/yquake2 8.00 -> 8.30

2024-04-30 Thread Adam Wolk
Hi folks,

On Mon, Apr 29, 2024, at 1:43 AM, Thomas Frohwein wrote:
> On Tue, Apr 02, 2024 at 01:36:11PM +0100, Tom Murphy wrote:
>
> Adam, if you're not using the port actively, would you be okay with
> passing on maintainership?
>> 

I won’t be able to test in reasonable time. Thanks for keeping the port alive, 
I’m happy to pass maintainer.

Adam



Re: Improvement to games/yquake2

2020-06-27 Thread Adam Wolk
I iterated over your diff like we discussed on #openbsd-gaming.

Added dedicated 1 (good suggestion) and introduced daemon_flags.
Those two changes combined with the +gamedir flag you discovered
allows us to drop rc_start completely.

I'm currently looking for an OK to commit this :)

Regards,
Adam

On Wed, Jun 24, 2020 at 11:12:14PM +0200, Adam Wolk wrote:
> Hi,
> 
> On Wed, Jun 24, 2020, at 10:57 AM, Tom Murphy wrote:
> > Hi,
> > 
> >   Running q2ded in a tty/tmux window, and then if that tty disappears,
> >   q2ded begins spinning the CPU taking up sometimes 97% of the CPU.
> >   The process is constantly polling for input, but if the tty is 
> >   marked 'bad' in fstat, the program can't cope.
> > 
> >   Setting 'busywait' variable to 1 allows q2ded to use nanosleep and
> >   stop spinning the cpu.
> > 
> 
> Great find.
> 
> >   I've improved the rc script to include the busywait variable and
> >   also removed the cd into the shared yquake2 directory since setting
> >   +set basedir to that path does the same thing and is cleaner.
> > 
> >   I've used the following flags for testing:
> > 
> >   q2ded_flags="+set dedicated 1 +set game holywars +exec server.cfg"
> > 
> >   It might be worth also hard coding +set dedicated 1 as well in the
> >   rc script? What do you think?
> > 
> 
> I think it makes sense for the rc.d script. If someone wants to run it in the 
> foreground
> he/she can cd to the game directory manually and execute the binary directly. 
> That
> is the reason we documented in the commit msg for not providing a q2ded 
> wrapper
> like we do for the game executable itself. 
> 
> One additional comment inline the diff.
> 
> >   Diff is attached.
> > 
> >   Thanks,
> >   Tom
> > 
> > 
> > Index: Makefile
> > -   ${rcexec} "cd ${TRUEPREFIX}/share/yquake2; ${daemon} ${daemon_flags}"
> > +   ${rcexec} "${daemon} +set gamedir ${TRUEPREFIX}/share/yquake2 +set 
> > busywait 0 ${daemon_flags}"
> 
> I think I don't like having this explicitly added to the command here. That 
> way no one
> can override those flags even for testing. I think a cleaner solution would 
> be to define
> daemon_flags and have those as default like we do for ie. postgresql. That 
> way people
> who want to override them just define their own in rc.conf.local and I think 
> it's reasonable
> to expect them to understand why they should copy the existing ones. I think 
> this
> would also allow us to remove the whole rc_start definition.
> 
> 
? q2ded_flags.diff
Index: Makefile
===
RCS file: /cvs/ports/games/yquake2/Makefile,v
retrieving revision 1.21
diff -u -p -r1.21 Makefile
--- Makefile25 Mar 2020 21:22:59 -  1.21
+++ Makefile27 Jun 2020 13:17:46 -
@@ -8,7 +8,7 @@ V=  7.43
 PKGNAME=   ${N}-${V}
 DISTNAME=  quake2-${V}
 CATEGORIES=games
-REVISION=  0
+REVISION=  1
 
 HOMEPAGE=  https://www.yamagi.org/quake2/
 MAINTAINER=Adam Wolk 
Index: pkg/q2ded.rc
===
RCS file: /cvs/ports/games/yquake2/pkg/q2ded.rc,v
retrieving revision 1.5
diff -u -p -r1.5 q2ded.rc
--- pkg/q2ded.rc11 Jan 2018 19:27:02 -  1.5
+++ pkg/q2ded.rc27 Jun 2020 13:17:46 -
@@ -3,6 +3,7 @@
 # $OpenBSD: q2ded.rc,v 1.5 2018/01/11 19:27:02 rpe Exp $
 
 daemon="${TRUEPREFIX}/share/yquake2/q2ded"
+daemon_flags="+set busywait 0 +set dedicated 1 +set gamedir 
${TRUEPREFIX}/share/yquake2"
 daemon_user="_yquake2"
 
 . /etc/rc.d/rc.subr
@@ -10,9 +11,5 @@ daemon_user="_yquake2"
 pexp="${daemon}.*"
 rc_bg=YES
 rc_reload=NO
-
-rc_start() {
-   ${rcexec} "cd ${TRUEPREFIX}/share/yquake2; ${daemon} ${daemon_flags}"
-}
 
 rc_cmd $1


Re: Improvement to games/yquake2

2020-06-24 Thread Adam Wolk
Hi,

On Wed, Jun 24, 2020, at 10:57 AM, Tom Murphy wrote:
> Hi,
> 
>   Running q2ded in a tty/tmux window, and then if that tty disappears,
>   q2ded begins spinning the CPU taking up sometimes 97% of the CPU.
>   The process is constantly polling for input, but if the tty is 
>   marked 'bad' in fstat, the program can't cope.
> 
>   Setting 'busywait' variable to 1 allows q2ded to use nanosleep and
>   stop spinning the cpu.
> 

Great find.

>   I've improved the rc script to include the busywait variable and
>   also removed the cd into the shared yquake2 directory since setting
>   +set basedir to that path does the same thing and is cleaner.
> 
>   I've used the following flags for testing:
> 
>   q2ded_flags="+set dedicated 1 +set game holywars +exec server.cfg"
> 
>   It might be worth also hard coding +set dedicated 1 as well in the
>   rc script? What do you think?
> 

I think it makes sense for the rc.d script. If someone wants to run it in the 
foreground
he/she can cd to the game directory manually and execute the binary directly. 
That
is the reason we documented in the commit msg for not providing a q2ded wrapper
like we do for the game executable itself. 

One additional comment inline the diff.

>   Diff is attached.
> 
>   Thanks,
>   Tom
> 
> 
> Index: Makefile
> ===
> RCS file: /cvs/ports/games/yquake2/Makefile,v
> retrieving revision 1.21
> diff -u -p -r1.21 Makefile
> --- Makefile  25 Mar 2020 21:22:59 -  1.21
> +++ Makefile  24 Jun 2020 08:49:24 -
> @@ -8,7 +8,7 @@ V=7.43
>  PKGNAME= ${N}-${V}
>  DISTNAME=quake2-${V}
>  CATEGORIES=  games
> -REVISION=0
> +REVISION=1
>  
>  HOMEPAGE=https://www.yamagi.org/quake2/
>  MAINTAINER=  Adam Wolk 
> Index: pkg/q2ded.rc
> ===
> RCS file: /cvs/ports/games/yquake2/pkg/q2ded.rc,v
> retrieving revision 1.5
> diff -u -p -r1.5 q2ded.rc
> --- pkg/q2ded.rc  11 Jan 2018 19:27:02 -  1.5
> +++ pkg/q2ded.rc  24 Jun 2020 08:49:24 -
> @@ -12,7 +12,7 @@ rc_bg=YES
>  rc_reload=NO
>  
>  rc_start() {
> - ${rcexec} "cd ${TRUEPREFIX}/share/yquake2; ${daemon} ${daemon_flags}"
> + ${rcexec} "${daemon} +set gamedir ${TRUEPREFIX}/share/yquake2 +set 
> busywait 0 ${daemon_flags}"

I think I don't like having this explicitly added to the command here. That way 
no one
can override those flags even for testing. I think a cleaner solution would be 
to define
daemon_flags and have those as default like we do for ie. postgresql. That way 
people
who want to override them just define their own in rc.conf.local and I think 
it's reasonable
to expect them to understand why they should copy the existing ones. I think 
this
would also allow us to remove the whole rc_start definition.

>  }
>  
>  rc_cmd $1
> 
>



Re: update games/yquake2

2020-03-24 Thread Adam Wolk
On Tue, Mar 24, 2020, at 5:09 PM, Solene Rapenne wrote:
> This updates yquake2 to last version
> 
> Changelog available here
> https://github.com/yquake2/yquake2/blob/master/CHANGELOG
> 
> I deleted some wantlib due to this line "Removed static dependencies to
> libogg, libvorbis and libz." and that port-libs-depend-check wasn't
> happy with them in WANTLIB.
> 
> PLIST has been updated with update-plist, it's mostly reordering.
> 
> Works fine for me
>

First off, thanks for hacking on this!
Like we discussed on IRC this is OK awolk@ 
but to the diff provided on https://perso.pw/f/1yKM4u5YyuwCBKGhn6RcIg.diff

Tested on amd64 with all 3 renderers, which confirms that my previous issue
detailed here https://marc.info/?l=openbsd-ports&m=153773376818361&w=2 resolved
itself with time.

I can also confirm the music files work at least in .ogg format as that is the 
one I own.

Regards,
Adam



Re: [UPDATE] net/py-snmp (4.4.6 -> 4.4.12)

2020-02-26 Thread Adam Wolk
On Thu, Feb 27, 2020, at 12:14 AM, Kurt Mosiejczuk wrote:
> On Wed, Feb 26, 2020 at 10:55:09PM +0100, Adam Wolk wrote:
> 
> > > Looks good and this is OK awolk@
> 
> > > I am however CC'ing ports as I would like another OK before
> > > committing the update.
> 
> > Here is an unquoted diff, don't know what I was thinking.
> 
> ok kmos
> 
> --Kurt
> 

Committed, thanks!



Re: [UPDATE] net/py-snmp (4.4.6 -> 4.4.12)

2020-02-26 Thread Adam Wolk
On Wed, Feb 26, 2020 at 10:44:49PM +0100, Adam Wolk wrote:
> On Tue, Feb 25, 2020 at 11:44:03PM +0100, Alex Naumov wrote:
> > Hey,
> > 
> > this patch updates py-snmp to version 4.4.12.
> > Everything looks ok, but there are no regression tests for this port.
> > 
> > Cheers,
> > Alex
> 
> Than you for the update!
> 
> The only consumer for py-snmp is security/routersploit.
> 
> I tested your diff with it and with the sample script
> from http://snmplabs.com/pysnmp/quick-start.html
> 
> napalm$ python2.7 test.py 
> 
> SNMPv2-MIB::sysDescr.0 = Linux zeus 4.8.6.5-smp #2 SMP Sun Nov 13 14:58:11 
> CDT 2016 i686
> 
> Looks good and this is OK awolk@
> 
> I am however CC'ing ports as I would like another OK before
> committing the update.
> 
> Regards,
> Adam
> 

Here is an unquoted diff, don't know what I was thinking.

? py-snmp-4.4.12.diff
Index: Makefile
===
RCS file: /cvs/ports/net/py-snmp/Makefile,v
retrieving revision 1.25
diff -u -p -u -p -r1.25 Makefile
--- Makefile12 Jul 2019 20:48:46 -  1.25
+++ Makefile25 Feb 2020 22:44:29 -
@@ -2,7 +2,7 @@
 
 COMMENT =  SNMP framework for Python
 
-MODPY_EGG_VERSION =4.4.6
+MODPY_EGG_VERSION =4.4.12
 DISTNAME = pysnmp-${MODPY_EGG_VERSION}
 PKGNAME =  ${DISTNAME:S/py/py-/}
 CATEGORIES =   net
Index: distinfo
===
RCS file: /cvs/ports/net/py-snmp/distinfo,v
retrieving revision 1.11
diff -u -p -u -p -r1.11 distinfo
--- distinfo23 Sep 2018 18:33:27 -  1.11
+++ distinfo25 Feb 2020 22:44:29 -
@@ -1,2 +1,2 @@
-SHA256 (pysnmp-4.4.6.tar.gz) = 40/6Dc5faa2r1Hj/dsPhsI4y67B2ffixeNBwT0oaxAY=
-SIZE (pysnmp-4.4.6.tar.gz) = 434508
+SHA256 (pysnmp-4.4.12.tar.gz) = DD2+8vlYysqWBx/lwZ3kPpwbBISrAqDPCLGQvO52i6k=
+SIZE (pysnmp-4.4.12.tar.gz) = 442270
Index: pkg/PLIST
===
RCS file: /cvs/ports/net/py-snmp/pkg/PLIST,v
retrieving revision 1.9
diff -u -p -u -p -r1.9 PLIST
--- pkg/PLIST   23 Sep 2018 18:33:27 -  1.9
+++ pkg/PLIST   25 Feb 2020 22:44:30 -
@@ -10,263 +10,262 @@ lib/python${MODPY_VERSION}/site-packages
 lib/python${MODPY_VERSION}/site-packages/pysnmp/__init__.py
 
${MODPY_COMMENT}lib/python${MODPY_VERSION}/site-packages/pysnmp/${MODPY_PYCACHE}/
 
lib/python${MODPY_VERSION}/site-packages/pysnmp/${MODPY_PYCACHE}__init__.${MODPY_PYC_MAGIC_TAG}pyc
-lib/python${MODPY_VERSION}/site-packages/pysnmp/${MODPY_PYCACHE}cache.${MODPY_PYC_MAGIC_TAG}pyc
-lib/python${MODPY_VERSION}/site-packages/pysnmp/${MODPY_PYCACHE}debug.${MODPY_PYC_MAGIC_TAG}pyc
-lib/python${MODPY_VERSION}/site-packages/pysnmp/${MODPY_PYCACHE}error.${MODPY_PYC_MAGIC_TAG}pyc
-lib/python${MODPY_VERSION}/site-packages/pysnmp/${MODPY_PYCACHE}nextid.${MODPY_PYC_MAGIC_TAG}pyc
 lib/python${MODPY_VERSION}/site-packages/pysnmp/cache.py
+lib/python${MODPY_VERSION}/site-packages/pysnmp/${MODPY_PYCACHE}cache.${MODPY_PYC_MAGIC_TAG}pyc
 lib/python${MODPY_VERSION}/site-packages/pysnmp/carrier/
 lib/python${MODPY_VERSION}/site-packages/pysnmp/carrier/__init__.py
 
${MODPY_COMMENT}lib/python${MODPY_VERSION}/site-packages/pysnmp/carrier/${MODPY_PYCACHE}/
 
lib/python${MODPY_VERSION}/site-packages/pysnmp/carrier/${MODPY_PYCACHE}__init__.${MODPY_PYC_MAGIC_TAG}pyc
-lib/python${MODPY_VERSION}/site-packages/pysnmp/carrier/${MODPY_PYCACHE}base.${MODPY_PYC_MAGIC_TAG}pyc
-lib/python${MODPY_VERSION}/site-packages/pysnmp/carrier/${MODPY_PYCACHE}error.${MODPY_PYC_MAGIC_TAG}pyc
-lib/python${MODPY_VERSION}/site-packages/pysnmp/carrier/${MODPY_PYCACHE}sockfix.${MODPY_PYC_MAGIC_TAG}pyc
-lib/python${MODPY_VERSION}/site-packages/pysnmp/carrier/${MODPY_PYCACHE}sockmsg.${MODPY_PYC_MAGIC_TAG}pyc
 lib/python${MODPY_VERSION}/site-packages/pysnmp/carrier/asyncio/
 lib/python${MODPY_VERSION}/site-packages/pysnmp/carrier/asyncio/__init__.py
 
${MODPY_COMMENT}lib/python${MODPY_VERSION}/site-packages/pysnmp/carrier/asyncio/${MODPY_PYCACHE}/
 
lib/python${MODPY_VERSION}/site-packages/pysnmp/carrier/asyncio/${MODPY_PYCACHE}__init__.${MODPY_PYC_MAGIC_TAG}pyc
-lib/python${MODPY_VERSION}/site-packages/pysnmp/carrier/asyncio/${MODPY_PYCACHE}base.${MODPY_PYC_MAGIC_TAG}pyc
-lib/python${MODPY_VERSION}/site-packages/pysnmp/carrier/asyncio/${MODPY_PYCACHE}dispatch.${MODPY_PYC_MAGIC_TAG}pyc
 lib/python${MODPY_VERSION}/site-packages/pysnmp/carrier/asyncio/base.py
+lib/python${MODPY_VERSION}/site-packages/pysnmp/carrier/asyncio/${MODPY_PYCACHE}base.${MODPY_PYC_MAGIC_TAG}pyc
 lib/python${MODPY_VERSION}/site-packages/pysnmp/carrier/asyncio/dgram/
 
lib/python${MODPY_VERSION}/site-packages/pysnmp/carrier/asyncio/dgram/__init__.py
 
${MODPY_COMMENT}lib/python${MODPY_VERSION}/site-packages/pysnmp/carrier/a

Re: [UPDATE] net/py-snmp (4.4.6 -> 4.4.12)

2020-02-26 Thread Adam Wolk
On Tue, Feb 25, 2020 at 11:44:03PM +0100, Alex Naumov wrote:
> Hey,
> 
> this patch updates py-snmp to version 4.4.12.
> Everything looks ok, but there are no regression tests for this port.
> 
> Cheers,
> Alex

Than you for the update!

The only consumer for py-snmp is security/routersploit.

I tested your diff with it and with the sample script
from http://snmplabs.com/pysnmp/quick-start.html

napalm$ python2.7 test.py   
  
SNMPv2-MIB::sysDescr.0 = Linux zeus 4.8.6.5-smp #2 SMP Sun Nov 13 14:58:11 CDT 
2016 i686

Looks good and this is OK awolk@

I am however CC'ing ports as I would like another OK before
committing the update.

Regards,
Adam

> ? py-snmp-4.4.12.diff
> Index: Makefile
> ===
> RCS file: /cvs/ports/net/py-snmp/Makefile,v
> retrieving revision 1.25
> diff -u -p -u -p -r1.25 Makefile
> --- Makefile  12 Jul 2019 20:48:46 -  1.25
> +++ Makefile  25 Feb 2020 22:44:29 -
> @@ -2,7 +2,7 @@
>  
>  COMMENT =SNMP framework for Python
>  
> -MODPY_EGG_VERSION =  4.4.6
> +MODPY_EGG_VERSION =  4.4.12
>  DISTNAME =   pysnmp-${MODPY_EGG_VERSION}
>  PKGNAME =${DISTNAME:S/py/py-/}
>  CATEGORIES = net
> Index: distinfo
> ===
> RCS file: /cvs/ports/net/py-snmp/distinfo,v
> retrieving revision 1.11
> diff -u -p -u -p -r1.11 distinfo
> --- distinfo  23 Sep 2018 18:33:27 -  1.11
> +++ distinfo  25 Feb 2020 22:44:29 -
> @@ -1,2 +1,2 @@
> -SHA256 (pysnmp-4.4.6.tar.gz) = 40/6Dc5faa2r1Hj/dsPhsI4y67B2ffixeNBwT0oaxAY=
> -SIZE (pysnmp-4.4.6.tar.gz) = 434508
> +SHA256 (pysnmp-4.4.12.tar.gz) = DD2+8vlYysqWBx/lwZ3kPpwbBISrAqDPCLGQvO52i6k=
> +SIZE (pysnmp-4.4.12.tar.gz) = 442270
> Index: pkg/PLIST
> ===
> RCS file: /cvs/ports/net/py-snmp/pkg/PLIST,v
> retrieving revision 1.9
> diff -u -p -u -p -r1.9 PLIST
> --- pkg/PLIST 23 Sep 2018 18:33:27 -  1.9
> +++ pkg/PLIST 25 Feb 2020 22:44:30 -
> @@ -10,263 +10,262 @@ lib/python${MODPY_VERSION}/site-packages
>  lib/python${MODPY_VERSION}/site-packages/pysnmp/__init__.py
>  
> ${MODPY_COMMENT}lib/python${MODPY_VERSION}/site-packages/pysnmp/${MODPY_PYCACHE}/
>  
> lib/python${MODPY_VERSION}/site-packages/pysnmp/${MODPY_PYCACHE}__init__.${MODPY_PYC_MAGIC_TAG}pyc
> -lib/python${MODPY_VERSION}/site-packages/pysnmp/${MODPY_PYCACHE}cache.${MODPY_PYC_MAGIC_TAG}pyc
> -lib/python${MODPY_VERSION}/site-packages/pysnmp/${MODPY_PYCACHE}debug.${MODPY_PYC_MAGIC_TAG}pyc
> -lib/python${MODPY_VERSION}/site-packages/pysnmp/${MODPY_PYCACHE}error.${MODPY_PYC_MAGIC_TAG}pyc
> -lib/python${MODPY_VERSION}/site-packages/pysnmp/${MODPY_PYCACHE}nextid.${MODPY_PYC_MAGIC_TAG}pyc
>  lib/python${MODPY_VERSION}/site-packages/pysnmp/cache.py
> +lib/python${MODPY_VERSION}/site-packages/pysnmp/${MODPY_PYCACHE}cache.${MODPY_PYC_MAGIC_TAG}pyc
>  lib/python${MODPY_VERSION}/site-packages/pysnmp/carrier/
>  lib/python${MODPY_VERSION}/site-packages/pysnmp/carrier/__init__.py
>  
> ${MODPY_COMMENT}lib/python${MODPY_VERSION}/site-packages/pysnmp/carrier/${MODPY_PYCACHE}/
>  
> lib/python${MODPY_VERSION}/site-packages/pysnmp/carrier/${MODPY_PYCACHE}__init__.${MODPY_PYC_MAGIC_TAG}pyc
> -lib/python${MODPY_VERSION}/site-packages/pysnmp/carrier/${MODPY_PYCACHE}base.${MODPY_PYC_MAGIC_TAG}pyc
> -lib/python${MODPY_VERSION}/site-packages/pysnmp/carrier/${MODPY_PYCACHE}error.${MODPY_PYC_MAGIC_TAG}pyc
> -lib/python${MODPY_VERSION}/site-packages/pysnmp/carrier/${MODPY_PYCACHE}sockfix.${MODPY_PYC_MAGIC_TAG}pyc
> -lib/python${MODPY_VERSION}/site-packages/pysnmp/carrier/${MODPY_PYCACHE}sockmsg.${MODPY_PYC_MAGIC_TAG}pyc
>  lib/python${MODPY_VERSION}/site-packages/pysnmp/carrier/asyncio/
>  lib/python${MODPY_VERSION}/site-packages/pysnmp/carrier/asyncio/__init__.py
>  
> ${MODPY_COMMENT}lib/python${MODPY_VERSION}/site-packages/pysnmp/carrier/asyncio/${MODPY_PYCACHE}/
>  
> lib/python${MODPY_VERSION}/site-packages/pysnmp/carrier/asyncio/${MODPY_PYCACHE}__init__.${MODPY_PYC_MAGIC_TAG}pyc
> -lib/python${MODPY_VERSION}/site-packages/pysnmp/carrier/asyncio/${MODPY_PYCACHE}base.${MODPY_PYC_MAGIC_TAG}pyc
> -lib/python${MODPY_VERSION}/site-packages/pysnmp/carrier/asyncio/${MODPY_PYCACHE}dispatch.${MODPY_PYC_MAGIC_TAG}pyc
>  lib/python${MODPY_VERSION}/site-packages/pysnmp/carrier/asyncio/base.py
> +lib/python${MODPY_VERSION}/site-packages/pysnmp/carrier/asyncio/${MODPY_PYCACHE}base.${MODPY_PYC_MAGIC_TAG}pyc
>  lib/python${MODPY_VERSION}/site-packages/pysnmp/carrier/asyncio/dgram/
>  
> lib/python${MODPY_VERSION}/site-packages/pysnmp/carrier/asyncio/dgram/__init__.py
>  
> ${MODPY_COMMENT}lib/python${MODPY_VERSION}/site-packages/pysnmp/carrier/asyncio/dgram/${MODPY_PYCACHE}/
>  
> lib/python${MODPY_VERSION}/site-packages/pysnmp/carrier/asyncio/dgram/${MODPY_PYCACHE}__init__.${MODPY_PYC_MAGIC_TAG}pyc
> -lib/python${MO

Re: UPDATE: www/otter-browser

2020-02-19 Thread Adam Wolk
Forgot to cc ports

- Original message -
From: Adam Wolk 
To: Rafael Sadowski , Adam Wolk 
Subject: Re: UPDATE: www/otter-browser
Date: Wednesday, February 19, 2020 9:02 AM



On Wed, Feb 19, 2020, at 8:49 AM, Rafael Sadowski wrote:
> On Wed Feb 12, 2020 at 09:37:38AM +0100, Rafael Sadowski wrote:
> > Update otter-browser to the latest stable version which supports
> > qtwebkit from https://github.com/qtwebkit/qtwebkit or qtwebengin (Which
> > is not ported) but no more qtwebkit 5.9.0.
> > 
> > Tested with upcoming x11/qt5/qtwebkit update.
> > 
> > OK?
> 
> Hi Adam,
> 
> any objections? We want to go in with the qtwebkit update.

No objections. Sorry for holding up, wanted to test past weekend but something 
popped up.

ok awolk@

> RS
> 
> > 
> > Index: Makefile
> > ===
> > RCS file: /cvs/ports/www/otter-browser/Makefile,v
> > retrieving revision 1.30
> > diff -u -p -u -p -r1.30 Makefile
> > --- Makefile 12 Jul 2019 20:50:49 - 1.30
> > +++ Makefile 12 Feb 2020 05:19:00 -
> > @@ -2,8 +2,7 @@
> > 
> > COMMENT = browser aiming to recreate classic Opera (12.x) UI using Qt5
> > 
> > -DISTNAME = otter-browser-0.9.94
> > -REVISION = 2
> > +DISTNAME = otter-browser-1.0.01
> > 
> > CATEGORIES = www
> > 
> > @@ -30,7 +29,7 @@ LIB_DEPENDS = textproc/hunspell \
> > x11/qt5/qtdeclarative \
> > x11/qt5/qtmultimedia \
> > x11/qt5/qtsvg \
> > - x11/qt5/qtwebkit \
> > + x11/qt5/qtwebkit>=5.212.0 \
> > x11/qt5/qtxmlpatterns
> > 
> > NO_TEST = Yes
> > Index: distinfo
> > ===
> > RCS file: /cvs/ports/www/otter-browser/distinfo,v
> > retrieving revision 1.17
> > diff -u -p -u -p -r1.17 distinfo
> > --- distinfo 4 Jan 2018 18:55:16 - 1.17
> > +++ distinfo 12 Feb 2020 05:19:00 -
> > @@ -1,2 +1,2 @@
> > -SHA256 (otter-browser-0.9.94.tar.bz2) = 
> > 9ylzbT4pzVwPOfLqqjdUpjK+6XJF5i7KbsdaHMa2jeM=
> > -SIZE (otter-browser-0.9.94.tar.bz2) = 3029327
> > +SHA256 (otter-browser-1.0.01.tar.bz2) = 
> > CG15d7yoIcpiSmywSaSldtuFfOhV+oujWlccoevzxrA=
> > +SIZE (otter-browser-1.0.01.tar.bz2) = 3189343
> > Index: pkg/PLIST
> > ===
> > RCS file: /cvs/ports/www/otter-browser/pkg/PLIST,v
> > retrieving revision 1.10
> > diff -u -p -u -p -r1.10 PLIST
> > --- pkg/PLIST 29 Jun 2018 22:16:22 - 1.10
> > +++ pkg/PLIST 12 Feb 2020 05:19:00 -
> > @@ -24,6 +24,7 @@ share/otter-browser/locale/otter-browser
> > share/otter-browser/locale/otter-browser_et.qm
> > share/otter-browser/locale/otter-browser_fi.qm
> > share/otter-browser/locale/otter-browser_fr.qm
> > +share/otter-browser/locale/otter-browser_gl.qm
> > share/otter-browser/locale/otter-browser_he.qm
> > share/otter-browser/locale/otter-browser_hr.qm
> > share/otter-browser/locale/otter-browser_hu.qm
> > @@ -47,6 +48,7 @@ share/otter-browser/locale/otter-browser
> > share/otter-browser/locale/otter-browser...@ijekavianlatin.qm
> > share/otter-browser/locale/otter-browser...@latin.qm
> > share/otter-browser/locale/otter-browser_sv.qm
> > +share/otter-browser/locale/otter-browser_th.qm
> > share/otter-browser/locale/otter-browser_tr.qm
> > share/otter-browser/locale/otter-browser_uk.qm
> > share/otter-browser/locale/otter-browser_yue.qm
> > 
> 


Re: [update] graphics/fotowall 0.9 -> 1.0

2020-02-03 Thread Adam Wolk
On Mon, Feb 03, 2020 at 06:43:28AM +0100, Rafael Sadowski wrote:
> On Sun Feb 02, 2020 at 11:10:31PM +0100, Adam Wolk wrote:
> > Feedback? OK's?
> 
> Works here also with OpenGL. Here are some tweaks, maybe you'd like to
> take over some:
> 
> - reorder MODULES, _DEPENDS
> - fix tabs/ spaces
> - use EXTRACT_SUFX = -RETRO.tar.bz2
> 
> OK rsadowski@
> 

Thanks, committed with your diff fully applied.

Regards,
Adam



[update] graphics/fotowall 0.9 -> 1.0

2020-02-02 Thread Adam Wolk
Hi ports@,

Attaching an update to fotowall 1.0

Notable port changes:
 - qt4 -> qt5.
 - Using github tarball instead of google code.
 - Adjusted WANTLIB/LIB_DEPENDS.
 - Added EXTRACT_CASES and pre-extract because
   the upstream tarball spills over WRKDIST
   instead of extracting to a sub directory.
 - Had to patch enricomath.h to not use the
   provided sincos(3) since we now have our own.
   This happened because I upstreamed our previous
   patch that enabled this path for us when we still
   didn't have it. I will upstream the revert also.
 - Patching out qstrcpy for strlcpy. This is because
   our x11/qt5 has patched out qstrcpy and the lib
   is not longer exposing it for external consumers.
   I saw two ways in how ports are handling it.

   1. x11/qt5 itself replaced all uses with strlcpy
   2. multimedia/phonon-backend patch qstrcpy -> strlcpy

   I opted for #1 but obviously having a close look on
   the patch is advised here.

Upstream changelog:

 https://github.com/enricoros/fotowall/releases/tag/v1.0

Tested on -current amd64.

Feedback? OK's?

Regards,
Adam Wolk
Index: Makefile
===
RCS file: /cvs/ports/graphics/fotowall/Makefile,v
retrieving revision 1.9
diff -u -p -r1.9 Makefile
--- Makefile12 Jul 2019 20:46:58 -  1.9
+++ Makefile2 Feb 2020 21:58:35 -
@@ -2,9 +2,9 @@
 
 COMMENT =  Pictures collage & creativity tool
 
-DISTNAME = Fotowall-0.9
-PKGNAME =  ${DISTNAME:L}
-REVISION = 3
+V =1.0
+DISTNAME = Fotowall-${V}-RETRO
+PKGNAME =  fotowall-${V}
 
 CATEGORIES =   graphics
 
@@ -14,18 +14,26 @@ MAINTAINER =Adam Wolk https://github.com/enricoros/fotowall/releases/download/v${V}/
 
 EXTRACT_SUFX = .tar.bz2
+# avoid WRKDIST = ${WRKDIR}
+EXTRACT_CASES= *.tar.bz2) \
+bzip2 -dc ${FULLDISTDIR}/$$archive| ${TAR} xf - -C ${WRKDIST};;
 
-MODULES =  devel/qmake x11/qt4
+MODULES =  devel/qmake x11/qt5
 RUN_DEPENDS =  devel/desktop-file-utils
+
+pre-extract:
+   @mkdir ${WRKDIST}
 
 pre-configure:
@sed -i 's,/usr/,${TRUEPREFIX}/,' ${WRKSRC}/fotowall.pro
Index: distinfo
===
RCS file: /cvs/ports/graphics/fotowall/distinfo,v
retrieving revision 1.1.1.1
diff -u -p -r1.1.1.1 distinfo
--- distinfo30 Aug 2016 15:19:49 -  1.1.1.1
+++ distinfo2 Feb 2020 21:58:35 -
@@ -1,2 +1,2 @@
-SHA256 (Fotowall-0.9.tar.bz2) = 5NDABdLLHXwJQ4v8MJjq3rwIlG5PvAZVt/yLBG3jgQ0=
-SIZE (Fotowall-0.9.tar.bz2) = 640175
+SHA256 (Fotowall-1.0-RETRO.tar.bz2) = 
u9jGEI7LrBlDEiqDmEdTgYZdmDOou6jBDw0yYqV4+58=
+SIZE (Fotowall-1.0-RETRO.tar.bz2) = 1454741
Index: patches/patch-3rdparty_enricomath_h
===
RCS file: patches/patch-3rdparty_enricomath_h
diff -N patches/patch-3rdparty_enricomath_h
--- /dev/null   1 Jan 1970 00:00:00 -
+++ patches/patch-3rdparty_enricomath_h 2 Feb 2020 21:58:35 -
@@ -0,0 +1,20 @@
+$OpenBSD$
+
+Revert 
https://github.com/enricoros/fotowall/commit/23b1bc61f1341c5ce61da1e81cc3888a3407e7f4#diff-0e3d52718f9d6fda9c5499355baf17b0
+since we now have sincos(3).
+
+Index: 3rdparty/enricomath.h
+--- 3rdparty/enricomath.h.orig
 3rdparty/enricomath.h
+@@ -286,9 +286,9 @@ class Matrix3
+ #define M_PI 3.141592653589793238462643
+ #endif // !M_PI
+ 
+-#if !defined(Q_CC_GNU) || defined(Q_OS_WIN32) || defined(Q_OS_OS2) || 
defined(Q_OS_ANDROID) || defined(Q_OS_OPENBSD)
++#if !defined(Q_CC_GNU) || defined(Q_OS_WIN32) || defined(Q_OS_OS2) || 
defined(Q_OS_ANDROID)
+ #if !defined(__MINGW32__)
+-//sincos is not defined in win32, MAC, OS/2, Android and OpenBSD
++//sincos is not defined in win32, MAC, OS/2 and Android
+ static inline void sincos(double th, double *s, double *c)
+ {
+ *s = sin(th);
Index: patches/patch-Shared_PropertyEditors_h
===
RCS file: patches/patch-Shared_PropertyEditors_h
diff -N patches/patch-Shared_PropertyEditors_h
--- /dev/null   1 Jan 1970 00:00:00 -
+++ patches/patch-Shared_PropertyEditors_h  2 Feb 2020 21:58:35 -
@@ -0,0 +1,16 @@
+$OpenBSD$
+
+We patch out qstrcpy from x11/qt5* so use strlcpy
+
+Index: Shared/PropertyEditors.h
+--- Shared/PropertyEditors.h.orig
 Shared/PropertyEditors.h
+@@ -99,7 +99,7 @@ class PE_Combo : public PE_TypeControl
+ if (nameLength < 255) { \
+ char signalName[256]; \
+ signalName[0] = '0' + QSIGNAL_CODE; \
+-qstrcpy(signalName + 1, notifySignal.methodSignature()); \
++strlcpy(signalName + 1, notifySignal.methodSignature(), 
sizeof(signalName) - 1); \
+ connect(m_target.data(), signalName, this, SLOT(slotName)); \
+ } \
+ }


[Update] PostgreSQL allow -D in rc.conf.local

2019-08-17 Thread Adam Wolk
Hi ports@,

I'm attaching a small diff to allow picking a different postgresql
cluster (the directory you create with initdb) location within
/etc/rc.conf.local. 

With the attached diff, one can now define the folder with:
postgresql_flags="-D /var/postgresql/testdata -w -l
/var/postgresql/logfile"

Changing the value after -D to point to any valid pg cluster.

The current implementation had the path hard-coded in the rc.d file with
no way to override it.

By using the existing daemon_flags and setting -D there we indeed
could start postgresql with a different data_directory as the last -D
passed to start takes priority. However since that flag was not used in
the remaining calls the rc script would fail to check, reload and stop a
postgresql instance started this way.

In order to allow setting a custom data_directory, we remove datadir
variable and set -D directly in daemon_flags then append ${daemon_flags}
to all invocations of pg_ctl ($rcexec)) in check, reload, start and stop
targets. The existing flags we use 'by default' which is -w and -l that
due to this change are now added to all invocations have the following
impact:

1. `-l` ignored in all targets except start, it sets the location of the
log file. adding it to other commands is a no-op and doesn't change
existing behavior.
2. `-w` affects the start and stop targets. No change in behavior for
the start target. For the stop target it's also a no-op as reading the
pg_ctl code - that's the default even if not specified on the command
line.

Since we got rid of ${datadir} we now have no way to check where is the
postmaster.pid that the start target attempts to remove. I investigated
this and came to the conclusion that removing the pidfile is not needed.

When starting, pg_ctl upon noticing that a postmaster.pid is already
present will issue a warning:

pg_ctl: another server might be running; trying to start server anyway

The pg_ctl code itself handles cases of a leftover pid:

https://github.com/postgres/postgres/blob/b8f2da0ac5a24f669c8d320c58646759b8fc69a5/src/bin/pg_ctl/pg_ctl.c#L583

So if the database starts up properly, the file will be overwritten but
a system administrator will be able to see a log entry indicating that a
dead pidfile was removed. Note that pg_ctl stop itself removes the
postmaster.pid so if it's there then likely the database was not stopped
properly (and might need investigation for recovery).

The removal of the postmaster.pid has been moved from post server stop
to server start 9 years ago in commit 1.3. It was present in stop since
revision 1.1 of the file.

I don't think the above changes require a port quirk. Worst case
scenario iff someone defined postgresql_flags in rc.conf.local their
database would now not start due to -D being likely not specified by the
user. However, this is not permanent (or any whatsoever) damage as the
user only needs to add -D /var/postgresql/data to his flags to regain
full functionality. 

I do think that this could warrant an entry in current.html. The second
diff addresses this.

I also noticed that the postgresql_flags line is used verbatim in the
ENVIRONMENT section of rc.d(8), should I update that manual to include
the -D that is now the new default for flags? I assume that yes, so the
third diff covers that, however the extended line wraps in the manual
now (perhaps that is undesired?).

As always I'm looking for feedback and OK's :)

Regards,
Adam
Index: Makefile
===
RCS file: /cvs/ports/databases/postgresql/Makefile,v
retrieving revision 1.255
diff -u -p -r1.255 Makefile
--- Makefile12 Aug 2019 16:40:40 -  1.255
+++ Makefile17 Aug 2019 20:42:47 -
@@ -9,6 +9,7 @@ COMMENT-pg_upgrade=Support for upgrading
 
 VERSION=   11.5
 PREV_MAJOR=10
+REVISION-server=1
 DISTNAME=  postgresql-${VERSION}
 PKGNAME-main=  postgresql-client-${VERSION}
 PKGNAME-server=postgresql-server-${VERSION}
Index: pkg/postgresql.rc
===
RCS file: /cvs/ports/databases/postgresql/pkg/postgresql.rc,v
retrieving revision 1.12
diff -u -p -r1.12 postgresql.rc
--- pkg/postgresql.rc   11 Jan 2018 19:27:01 -  1.12
+++ pkg/postgresql.rc   17 Aug 2019 20:42:47 -
@@ -2,10 +2,8 @@
 #
 # $OpenBSD: postgresql.rc,v 1.12 2018/01/11 19:27:01 rpe Exp $
 
-datadir="/var/postgresql/data"
-
 daemon="${TRUEPREFIX}/bin/pg_ctl"
-daemon_flags="-w -l /var/postgresql/logfile"
+daemon_flags="-D /var/postgresql/data -w -l /var/postgresql/logfile"
 daemon_user="_postgresql"
 
 . /etc/rc.d/rc.subr
@@ -13,21 +11,20 @@ daemon_user="_postgresql"
 rc_usercheck=NO
 
 rc_check() {
-   ${rcexec} "${daemon} -D ${datadir} status"
+   ${rcexec} "${daemon} status ${daemon_flags}"
 }
 
 rc_reload() {
-   ${rcexec} "${daemon} -D ${datadir} reload"
+   ${rcexec} "${daemon} reload ${daemon_flags}"
 }
 
 rc_start() {
-   rm -f ${datadir}/postmaster.pid
-

Re: devel/zeal 0.6.0 => 0.6.1

2019-01-05 Thread Adam Wolk
forgot to cc ports.

- Forwarded message from Adam Wolk  -

Date: Sat, 5 Jan 2019 10:47:08 +0100
From: Adam Wolk 
To: Rafael Sadowski 
Subject: Re: devel/zeal 0.6.0 => 0.6.1
User-Agent: Mutt/1.11.1 (2018-12-01)

On Sat, Jan 05, 2019 at 10:12:21AM +0100, Rafael Sadowski wrote:
> On Fri Jan 04, 2019 at 10:08:39PM +0100, Adam Wolk wrote:
> > Hi ports@,
> > 
> > attaching a trivial bump for devel/zeal
> > 
> > Feedback? OK's?
> > 
> > Regards,
> > Adam
> 
> Works fine here on amd64. Please find a diff with the following tweaks:
> 
> - User kf5 module, it includes qt5 and cmake.
> - remove qtbase as dependency. qtbase comes with x11/qt5

nice

> - add sqlite3 as dependency
> 

great catch, I should have checked ldd output

> RS
> 


OK awolk@ for your diff, want to commit it or should I do it?

Regards,
Adam

- End forwarded message -



devel/zeal 0.6.0 => 0.6.1

2019-01-04 Thread Adam Wolk
Hi ports@,

attaching a trivial bump for devel/zeal

Feedback? OK's?

Regards,
Adam
Index: Makefile
===
RCS file: /cvs/ports/devel/zeal/Makefile,v
retrieving revision 1.12
diff -u -p -r1.12 Makefile
--- Makefile29 Jun 2018 22:16:11 -  1.12
+++ Makefile4 Jan 2019 21:06:25 -
@@ -5,9 +5,8 @@ USE_WXNEEDED =  Yes
 
 COMMENT =  simple offline API documentation browser
 
-V =0.6.0
+V =0.6.1
 DISTNAME = zeal-${V}
-REVISION = 2
 
 CATEGORIES =   devel
 
Index: distinfo
===
RCS file: /cvs/ports/devel/zeal/distinfo,v
retrieving revision 1.6
diff -u -p -r1.6 distinfo
--- distinfo4 Mar 2018 14:00:36 -   1.6
+++ distinfo4 Jan 2019 21:06:25 -
@@ -1,2 +1,2 @@
-SHA256 (zeal-0.6.0.tar.xz) = 2eBFhGAbxQRh2Z41RRbkaJO/Leu+N3yxc9FUHU/glYo=
-SIZE (zeal-0.6.0.tar.xz) = 670416
+SHA256 (zeal-0.6.1.tar.xz) = +7knkjPSuCRbSd4CKK4NBEWEqK50z9UjnBbVCIsfx4s=
+SIZE (zeal-0.6.1.tar.xz) = 671432
Index: pkg/PLIST
===
RCS file: /cvs/ports/devel/zeal/pkg/PLIST,v
retrieving revision 1.3
diff -u -p -r1.3 PLIST
--- pkg/PLIST   29 Jun 2018 22:16:11 -  1.3
+++ pkg/PLIST   4 Jan 2019 21:06:25 -
@@ -1,10 +1,12 @@
 @comment $OpenBSD: PLIST,v 1.3 2018/06/29 22:16:11 espie Exp $
 @bin bin/zeal
-share/applications/zeal.desktop
+share/applications/org.zealdocs.Zeal.desktop
 share/icons/hicolor/128x128/apps/zeal.png
 share/icons/hicolor/16x16/apps/zeal.png
 share/icons/hicolor/24x24/apps/zeal.png
 share/icons/hicolor/32x32/apps/zeal.png
 share/icons/hicolor/64x64/apps/zeal.png
+share/metainfo/
+share/metainfo/org.zealdocs.Zeal.appdata.xml
 @tag update-desktop-database
 @tag gtk-update-icon-cache %D/share/icons/hicolor


Re: editors/ghostwriter 1.7.3 => 1.7.4

2019-01-02 Thread Adam Wolk
On Wed, Jan 02, 2019 at 10:59:39PM +, Stuart Henderson wrote:
> On 2019/01/02 22:28, Adam Wolk wrote:
> > Hi ports@,
> > 
> > trivial bump for editors/ghostwriter.
> 
> ok.
> 
> > Notable changes:
> > 
> > 1.  License clarification for resources (CC by SA 4)
> > https://github.com/wereturtle/ghostwriter/issues/385
> > 
> > 
> > 2. The distfile doubled in size
> > 
> > -rw-r--r--  1 mulander  wsrc   658K Aug 26 10:29 
> > /usr/ports/distfiles/ghostwriter-1.7.3.tar.gz
> > -rw-r--r--  1 mulander  wsrc   1.1M Jan  2 22:11 
> > /usr/ports/distfiles/ghostwriter-1.7.4.tar.gz
> > 
> > I investigated and found out that the reason is the inclusion of 
> > screenshots for appdata.xml
> > in the Linux resources. They are not included in the PLIST. Upstream 
> > decided to fetch them
> > over https directly from a raw github url and since they refuse to hand 
> > roll tarballs
> > we get the benefit of a bloated tarball with assets that are not needed 
> > during buildtime :/
> > 
> > bloat commit:
> > https://github.com/wereturtle/ghostwriter/commit/81b4e5259ca9554f2927cdc038bcbae778a468bd
> 
> 12/10 for laziness! bleurgh.
> 

committed, thanks!



editors/ghostwriter 1.7.3 => 1.7.4

2019-01-02 Thread Adam Wolk
Hi ports@,

trivial bump for editors/ghostwriter.

Notable changes:

1.  License clarification for resources (CC by SA 4)
https://github.com/wereturtle/ghostwriter/issues/385


2. The distfile doubled in size

-rw-r--r--  1 mulander  wsrc   658K Aug 26 10:29 
/usr/ports/distfiles/ghostwriter-1.7.3.tar.gz
-rw-r--r--  1 mulander  wsrc   1.1M Jan  2 22:11 
/usr/ports/distfiles/ghostwriter-1.7.4.tar.gz

I investigated and found out that the reason is the inclusion of screenshots 
for appdata.xml
in the Linux resources. They are not included in the PLIST. Upstream decided to 
fetch them
over https directly from a raw github url and since they refuse to hand roll 
tarballs
we get the benefit of a bloated tarball with assets that are not needed during 
buildtime :/

bloat commit:
https://github.com/wereturtle/ghostwriter/commit/81b4e5259ca9554f2927cdc038bcbae778a468bd

Upstream change log can be found here:

https://github.com/wereturtle/ghostwriter/releases/tag/v1.7.4

Feedback? OK's?

regards,
Adam
? changes.diff
? ghostwriter-1.7.4.diff
Index: Makefile
===
RCS file: /cvs/ports/editors/ghostwriter/Makefile,v
retrieving revision 1.11
diff -u -p -r1.11 Makefile
--- Makefile26 Aug 2018 09:00:14 -  1.11
+++ Makefile2 Jan 2019 21:10:29 -
@@ -7,7 +7,7 @@ COMMENT =   distraction-free Markdown edit
 
 GH_ACCOUNT =   wereturtle
 GH_PROJECT =   ghostwriter
-GH_TAGNAME =   v1.7.3
+GH_TAGNAME =   v1.7.4
 
 CATEGORIES =   editors
 
@@ -15,7 +15,7 @@ HOMEPAGE =https://wereturtle.github.io/
 
 MAINTAINER =   Adam Wolk 
 
-# GPLv3
+# GPLv3 code, CC BY-SA 4.0 resources
 PERMIT_PACKAGE_CDROM = Yes
 
 WANTLIB += ${COMPILER_LIBCXX}
Index: distinfo
===
RCS file: /cvs/ports/editors/ghostwriter/distinfo,v
retrieving revision 1.5
diff -u -p -r1.5 distinfo
--- distinfo26 Aug 2018 09:00:14 -  1.5
+++ distinfo2 Jan 2019 21:10:29 -
@@ -1,2 +1,2 @@
-SHA256 (ghostwriter-1.7.3.tar.gz) = 
NOs4VcXPfFQRKOt0DuQd5AAMkfcemiz0ZMNdd91SPHk=
-SIZE (ghostwriter-1.7.3.tar.gz) = 673328
+SHA256 (ghostwriter-1.7.4.tar.gz) = 
SYcSbnGjdVbldfvOIALFUEzxWgTBG8zmBKJ49Z7JwRo=
+SIZE (ghostwriter-1.7.4.tar.gz) = 1195162


Re: games/ezquake add multiplayer server list

2018-11-12 Thread Adam Wolk
On Wed, Oct 17, 2018 at 10:43:55AM +0200, Solene Rapenne wrote:
> Tom Murphy  wrote:
> >Just tested this and it works well. Thanks for this addition!
> > I can ping servers and gather their info and join them with ezquake.
> > 
> > Only thing wrong is a missing $OpenBSD$ RCS tag at the top of 
> > patches/patch-EX_browser_sources.c but otherwise works fine.
> > 
> >   Thanks,
> >   Tom
> 
> new diff, with the patch made from "make update-patches", I don't remember why
> but last time that command wasn't producing the patch so I made it manually.
> 
> +
>  do-install:
> - ${INSTALL_PROGRAM} ${WRKSRC}/ezquake ${PREFIX}/bin
> + ${INSTALL_PROGRAM}  ${WRKSRC}/ezquake ${PREFIX}/bin
> + ${INSTALL_DATA_DIR} ${GAMEDIR}
> + @cp ${WRKSRC}/misc/sb/*.txt ${GAMEDIR}/
> + #${INSTALL_DATA} ${WRKSRC}/misc/sb/sources.txt ${GAMEDIR}
>  

Any reason for the commented out INSTALL_DATA in do-install? It shows up
in the fake target:

===>  Faking installation for ezquake-3.1p2
/usr/local/pobj/ezquake-3.1/bin/install -c -s -m 755  
/usr/local/pobj/ezquake-3.1/ezquake 
/usr/local/pobj/ezquake-3.1/fake-amd64/usr/local/bin
   
/usr/local/pobj/ezquake-3.1/bin/install -d -m 755 
/usr/local/pobj/ezquake-3.1/fake-amd64/usr/local/share/ezquake
#/usr/local/pobj/ezquake-3.1/bin/install -c -m 644 
/usr/local/pobj/ezquake-3.1/misc/sb/sources.txt 
/usr/local/pobj/ezquake-3.1/fake-amd64/usr/local/share/ezquake

without it the port still builds & works correctly listing servers (thanks!).

This is OK awolk@ with the commented out INSTALL_DATA nuked (unless
it has a valid reason to exist).



Re: [NEW] games/mvdsv 0.32

2018-11-12 Thread Adam Wolk
On Fri, Oct 19, 2018 at 09:40:19PM +0200, Solene Rapenne wrote:
> Tom Murphy  wrote:
> > On Thu, Oct 04, 2018 at 12:11:01PM +0200, Solene Rapenne wrote:
> > > Small introduction for people reading ports@.
> > > 
> > > mvdsv is a quake world game server. It has some features like allowing to
> > > record games from every player point of view. That can be played again in 
> > > a
> > > quake client like ezquake.
> > > 
> > > ok solene@
> > 
> > Thanks Solene and Stuart for their help! Attached is a new tarball using
> > GH_TAGNAME and the correct DISTFILES settings.
> > 
> > -Tom
> 
> up
> 
> looks fine to me
> 

Tested on a recent snapshot. I was able to start a local server following the
README instructions and connect to it with games/ezquake.

OK awolk@ to import



Issues with ld.so loading in games/yquake2

2018-09-23 Thread Adam Wolk
Hi ports@

I have a WIP update of games/yquake2 taking it from 7.21 to 7.30 here
  https://gist.github.com/mulander/a749cb99590619748c141161c830dbeb

Testing revealed that the default ref_gl1 renderer results in the game
failing to start while other renderers work.

Reproduction (unfortunately requires the game data files)

$ quake2 +set vid_renderer gl1  # crash
$ quake2 +set vid_renderer gl3  # OK
$ quake2 +set vid_renderer soft # OK

I re-tested on 7.21 and all above renderers worked.

I think the problem is with ld.so behavior on OpenBSD but I'm not sure
how to debug this further and would appreciate any help. Here is how I
arrived at this conclusion:

The crash indicates that symbols from glapi are not found
https://gist.github.com/mulander/223ff586976e61707ee5581156f1369e

... snip ...
quake2:/usr/X11R6/lib/modules/dri/i965_dri.so: undefined symbol 
'_glapi_set_context'
quake2:/usr/X11R6/lib/modules/dri/i965_dri.so: undefined symbol 
'_glapi_set_dispatch'
libGL error: unable to load driver: i965_dri.so
... snip ...

I started comparing both versions with the help of upstream (caedes).
The game uses dlopen(3) to load ref_gl1.so (their renderer) which itself
links against libGL.so which in turn links against libglapi.so.

This is true for the new version:
 https://gist.github.com/mulander/f8b3d1e4e82cd0f48edf5096391ac0dd (ldd output)

and the old one
 https://gist.github.com/mulander/0762234a443c2bdb11b5629cf00b1eea (ldd output)

I grabbed a kdump from the 7.30 run

 
https://gist.githubusercontent.com/mulander/7d99e5195e723b9297e917d092a0b0ef/raw/e70183eb27cf2288df0598490532bb13db0d68f1/7.30%2520kdump

and checked if the differences between the main binary itself could have had
impact:

 https://gist.github.com/mulander/effd79c3cf7e6558cfdbc9a631f9152c (ldd output 
left one is 7.21)

I finally ran both versions with LD_DEBUG set, both seem to be loading 
libglapi.so
via the same depdency chain ref_gl1.so -> libGL.so -> libglapi.so
however the new one fails and this is where my debugging ability for this issue
fails (output[1] below). I did a final test by running the game with LD_PRELOAD

  LD_PRELOAD=/usr/X11R6/lib/libglapi.so.0.2 quake2

and with the above, the ref_gl1.so renderer works (game launches and is 
playable).

I would appreciate any help in diagnosing this issue.

Regards,
Adam

[1]
7.21:

https://gist.githubusercontent.com/mulander/7fb326be6848228ef72debe7ea5a7407/raw/30ee52e5458145ca7d9eb85f9e5985b44e9bb306/debug721
 (7.21)

... snip ...
loading: libGL.so.17.1 required by ./ref_gl1.so
 flags /usr/X11R6/lib/libGL.so.17.1 = 0x0
... snip ...
loading: libglapi.so.0.2 required by /usr/X11R6/lib/libGL.so.17.1
 flags /usr/X11R6/lib/libglapi.so.0.2 = 0x0

7.30:

... snip ...
loading: libGL.so.17.1 required by ./ref_gl1.so
 flags /usr/X11R6/lib/libGL.so.17.1 = 0x0
... snip ...
loading: libglapi.so.0.2 required by /usr/X11R6/lib/libGL.so.17.1
 flags /usr/X11R6/lib/libglapi.so.0.2 = 0x0
... snip ...

https://gist.githubusercontent.com/mulander/7fb326be6848228ef72debe7ea5a7407/raw/30ee52e5458145ca7d9eb85f9e5985b44e9bb306/debug730
 (7.30)7.30:





Re: update net/py-snmp to 4.4.6 and add py3 flavor

2018-09-23 Thread Adam Wolk
On Sun, Sep 23, 2018 at 06:31:14PM +0100, Stuart Henderson wrote:
> On 2018/09/22 23:00, Adam Wolk wrote:
> > Hi ports@,
> > 
> > I found out that our net/py-snmp library is broken, I wrongly
> > assumed it was just how security/routersploit used the lib or
> > the version it expected so waited for the 3.0 release.
> > 
> > Long story short, I started updating routersploit to 3.0 which
> > is python3 only so approached adding that flavor to py-snmp
> > while doing that I confirmed it doesn't work with the quickstart
> > examples from py-snmp homepage - see [1] for a transcript so
> > moved forward with a full update to the latest upstream version.
> > 
> > I'm attaching an update to the port with the following changes:
> > 
> > 1. consistent whitespace
> > 2. version update from 4.3.2 -> 4.4.6
> > 3. new HOMEPAGE
> > 4. project moved from sourceforge to github & pypi
> > 5. grabbing MAINTAINER
> > 6. adding py3 flavor
> > 7. removing a trailing space
> > 
> > I tested this against the quickstart examples both from python2.7
> > and python3 - both work.
> 
> The PLIST is a bit messed up (missing directories and missing ${MODPY_COMMENT}
> for the python3-only directories), here is a diff to apply on top of yours.
> If this fixes things with routersploit then OK with me (there are no other
> users in tree).
> 

Committed, thanks!

I indeed had some issues with it and forcing portcheck to be happy. Thanks
for the diff, I re-tested and it still works as expected unblocking work
on updating routersploit.

Regards,
Adam



update net/py-snmp to 4.4.6 and add py3 flavor

2018-09-22 Thread Adam Wolk
r: ['Traceback (most recent call last):\n', '  File 
"/usr/local/lib/python2.7/site-packages/pysnmp/smi/builder.py", line 301, in 
loadModule\nexec(modData, g)\n', '  File 
"/usr/local/lib/python2.7/site-packages/pysnmp/smi/mibs/SNMPv2-MIB.py", line 
26, in \nsysDescr = MibScalar((1, 3, 6, 1, 2, 1, 1, 1), 
DisplayString().subtype(subtypeSpec=ValueSizeConstraint(0,255))).setMaxAccess("readonly")\n',
 '  File "/usr/local/lib/python2.7/site-packages/pysnmp/proto/rfc1902.py", line 
201, in subtype\nself, value, implicitTag, explicitTag, subtypeSpec\n', 
'TypeError: subtype() takes at most 2 arguments (5 given)\n']
Index: Makefile
===
RCS file: /cvs/ports/net/py-snmp/Makefile,v
retrieving revision 1.23
diff -u -p -r1.23 Makefile
--- Makefile2 Jun 2018 12:01:58 -   1.23
+++ Makefile22 Sep 2018 20:45:30 -
@@ -1,34 +1,37 @@
 # $OpenBSD: Makefile,v 1.23 2018/06/02 12:01:58 jasper Exp $
 
-COMMENT=   SNMP framework for Python
+COMMENT =  SNMP framework for Python
 
-MODPY_EGG_VERSION = 4.3.2
-DISTNAME=  pysnmp-${MODPY_EGG_VERSION}
-PKGNAME=   ${DISTNAME:S/py/py-/}
-REVISION=  1
-CATEGORIES=net
+MODPY_EGG_VERSION =4.4.6
+DISTNAME = pysnmp-${MODPY_EGG_VERSION}
+PKGNAME =  ${DISTNAME:S/py/py-/}
+CATEGORIES =   net
 
-HOMEPAGE=  http://pysnmp.sourceforge.net/
+HOMEPAGE = http://snmplabs.com/
+
+MAINTAINER =   Adam Wolk 
 
 # BSD
 PERMIT_PACKAGE_CDROM = Yes
 
-MASTER_SITES=  ${MASTER_SITE_SOURCEFORGE:=pysnmp/}
-
-MODULES=   lang/python
+MODULES =  lang/python
 
-MODPY_SETUPTOOLS = Yes
+MODPY_PI = Yes
+MODPY_SETUPTOOLS = Yes
 
 BUILD_DEPENDS =${RUN_DEPENDS}
-RUN_DEPENDS =   security/py-cryptodome \
-   devel/py-asn1
+RUN_DEPENDS =  security/py-cryptodome${MODPY_FLAVOR} \
+   devel/py-asn1${MODPY_FLAVOR}
+
+DOCSDIR =  ${PREFIX}/share/doc/${MODPY_PY_PREFIX}snmp
+EXAMPLESDIR =  ${PREFIX}/share/examples/${MODPY_PY_PREFIX}snmp
 
-DOCSDIR=   ${PREFIX}/share/doc/py-snmp
-EXAMPLESDIR=   ${PREFIX}/share/examples/py-snmp
+FLAVORS =  python3
+FLAVOR ?=
 
 post-install:
${INSTALL_DATA_DIR} ${DOCSDIR} ${EXAMPLESDIR}
-   cd ${WRKSRC}/docs; tar cf - . | tar xf - -C ${DOCSDIR} 
+   cd ${WRKSRC}/docs; tar cf - . | tar xf - -C ${DOCSDIR}
cd ${WRKSRC}/examples; tar cf - . | tar xf - -C ${EXAMPLESDIR}
 
 NO_TEST =  Yes
Index: distinfo
===
RCS file: /cvs/ports/net/py-snmp/distinfo,v
retrieving revision 1.10
diff -u -p -r1.10 distinfo
--- distinfo27 Aug 2016 16:46:43 -  1.10
+++ distinfo22 Sep 2018 20:45:30 -
@@ -1,2 +1,2 @@
-SHA256 (pysnmp-4.3.2.tar.gz) = fCvYHfF6p9ygBXpo56MihKciMTCaAjfWbVuAO1wRiXc=
-SIZE (pysnmp-4.3.2.tar.gz) = 398593
+SHA256 (pysnmp-4.4.6.tar.gz) = 40/6Dc5faa2r1Hj/dsPhsI4y67B2ffixeNBwT0oaxAY=
+SIZE (pysnmp-4.4.6.tar.gz) = 434508
Index: pkg/PLIST
===
RCS file: /cvs/ports/net/py-snmp/pkg/PLIST,v
retrieving revision 1.8
diff -u -p -r1.8 PLIST
--- pkg/PLIST   2 Jan 2016 08:56:39 -   1.8
+++ pkg/PLIST   22 Sep 2018 20:45:30 -
@@ -1,5 +1,4 @@
-@comment $OpenBSD: PLIST,v 1.8 2016/01/02 08:56:39 benoit Exp $
-lib/python${MODPY_VERSION}/site-packages/pysnmp/
+@comment $OpenBSD: PLIST,v$
 
lib/python${MODPY_VERSION}/site-packages/pysnmp-${MODPY_EGG_VERSION}-py${MODPY_VERSION}.egg-info/
 
lib/python${MODPY_VERSION}/site-packages/pysnmp-${MODPY_EGG_VERSION}-py${MODPY_VERSION}.egg-info/PKG-INFO
 
lib/python${MODPY_VERSION}/site-packages/pysnmp-${MODPY_EGG_VERSION}-py${MODPY_VERSION}.egg-info/SOURCES.txt
@@ -8,380 +7,392 @@ lib/python${MODPY_VERSION}/site-packages
 
lib/python${MODPY_VERSION}/site-packages/pysnmp-${MODPY_EGG_VERSION}-py${MODPY_VERSION}.egg-info/top_level.txt
 
lib/python${MODPY_VERSION}/site-packages/pysnmp-${MODPY_EGG_VERSION}-py${MODPY_VERSION}.egg-info/zip-safe
 lib/python${MODPY_VERSION}/site-packages/pysnmp/__init__.py
-lib/python${MODPY_VERSION}/site-packages/pysnmp/__init__.pyc
+lib/python${MODPY_VERSION}/site-packages/pysnmp/${MODPY_PYCACHE}
+lib/python${MODPY_VERSION}/site-packages/pysnmp/${MODPY_PYCACHE}__init__.${MODPY_PYC_MAGIC_TAG}pyc
+lib/python${MODPY_VERSION}/site-packages/pysnmp/${MODPY_PYCACHE}cache.${MODPY_PYC_MAGIC_TAG}pyc
+lib/python${MODPY_VERSION}/site-packages/pysnmp/${MODPY_PYCACHE}debug.${MODPY_PYC_MAGIC_TAG}pyc
+lib/python${MODPY_VERSION}/site-packages/pysnmp/${MODPY_PYCACHE}error.${MODPY_PYC_MAGIC_TAG}pyc
+lib/python${MODPY_VERSION}/site-packages/pysnmp/${MODPY_PYCACHE}nextid.${MODPY_PYC_MAGIC_TAG}pyc
 lib/python${MODPY_VERSION}/site-packages/pysnmp/cache.py
-lib/python${MODPY_VERSION}/site-packages/pysnmp/cache.pyc
-lib/python${MODPY_VERSION}/site-packages/pysnmp/carrier/
 lib/python${MODPY_VERSION}/site-packages

Kill net/corebird?

2018-09-22 Thread Adam Wolk
Hi ports@,

I'm considering killing of net/corebird, due to the changes
in the Twitter API the main developer decided to cease
further development. This includes new features, api changes
and bug fixes.

The port still seems to be working eevn though the API it used
got decomissioned around mid August but that's likely pure luck.

Here is a blog post from the upstream on the issue:
 - https://www.patreon.com/posts/corebirds-future-18921328

Looking for an OK to kill it or rationale to still keep it around.

Regards,
Adam



Re: update editors/ghostwriter 1.7.0 => 1.7.3

2018-08-26 Thread Adam Wolk
On Sat, Aug 11, 2018 at 10:20:01PM +0200, Adam Wolk wrote:
> Hi ports@,
> 
> trivial bump for ghostwriter.
> 
> feedback? OK's?
> 
> Regards,
> Adam

Ping, with a diff bumped to 1.7.3
Index: Makefile
===
RCS file: /cvs/ports/editors/ghostwriter/Makefile,v
retrieving revision 1.10
diff -u -p -r1.10 Makefile
--- Makefile18 Jul 2018 11:51:15 -  1.10
+++ Makefile26 Aug 2018 08:36:09 -
@@ -7,7 +7,7 @@ COMMENT =   distraction-free Markdown edit
 
 GH_ACCOUNT =   wereturtle
 GH_PROJECT =   ghostwriter
-GH_TAGNAME =   v1.7.0
+GH_TAGNAME =   v1.7.3
 
 CATEGORIES =   editors
 
Index: distinfo
===
RCS file: /cvs/ports/editors/ghostwriter/distinfo,v
retrieving revision 1.4
diff -u -p -r1.4 distinfo
--- distinfo18 Jul 2018 11:51:15 -  1.4
+++ distinfo26 Aug 2018 08:36:09 -
@@ -1,2 +1,2 @@
-SHA256 (ghostwriter-1.7.0.tar.gz) = 
U2VqP3DlPHIecOxNLjKfsdlBwQWddsvNLiuUHfUOSX0=
-SIZE (ghostwriter-1.7.0.tar.gz) = 680767
+SHA256 (ghostwriter-1.7.3.tar.gz) = 
NOs4VcXPfFQRKOt0DuQd5AAMkfcemiz0ZMNdd91SPHk=
+SIZE (ghostwriter-1.7.3.tar.gz) = 673328
Index: pkg/PLIST
===
RCS file: /cvs/ports/editors/ghostwriter/pkg/PLIST,v
retrieving revision 1.4
diff -u -p -r1.4 PLIST
--- pkg/PLIST   29 Jun 2018 22:16:11 -  1.4
+++ pkg/PLIST   26 Aug 2018 08:36:09 -
@@ -6,7 +6,16 @@ share/appdata/ghostwriter.appdata.xml
 share/applications/ghostwriter.desktop
 share/ghostwriter/
 share/ghostwriter/translations/
+share/ghostwriter/translations/ghostwriter_cs.qm
+share/ghostwriter/translations/ghostwriter_de.qm
 share/ghostwriter/translations/ghostwriter_en.qm
+share/ghostwriter/translations/ghostwriter_es.qm
+share/ghostwriter/translations/ghostwriter_fr.qm
+share/ghostwriter/translations/ghostwriter_it.qm
+share/ghostwriter/translations/ghostwriter_ja.qm
+share/ghostwriter/translations/ghostwriter_pl.qm
+share/ghostwriter/translations/ghostwriter_pt_BR.qm
+share/ghostwriter/translations/ghostwriter_ru.qm
 share/ghostwriter/translations/ghostwriter_zh.qm
 share/icons/hicolor/128x128/apps/ghostwriter.png
 share/icons/hicolor/16x16/apps/ghostwriter.png


Re: [NEW] games/ezquake

2018-08-15 Thread Adam Wolk
On Wed, Aug 15, 2018 at 05:51:33PM +0100, Tom Murphy wrote:
> On Wed, Aug 15, 2018 at 10:53:23AM -0400, Brian Callahan wrote:
> > 
> > > > > > games/julius for example language to be put into this port.
> > > > > > 
> > > > > > ~Brian
> > > > > > 
> > > > > Hi Brian,
> > > > > 
> > > > > Thanks for this! That does fix the arch dance, and I moved the 
> > > > > instructions
> > > > > from pkg/DESCR into pkg/README.
> > > > > 
> > > > > Attached is the port with the changes requested.
> > > > > 
> > > > > Is this OK?
> > > > > 
> > > > > Thanks,
> > > > > Tom
> > > > > 
> > > > > 
> > > > Attached back is a tarball with some tweaks. I think this version is ok.
> > > > 
> > > > ~Brian
> > > > 
> > > Hi Brian,
> > > 
> > >Sorry but I'm not sure why you removed audio/speexdsp from the 
> > > Makefile.
> > > The game docs state this is a required library. (See:
> > > https://ezquake.github.io/docs/guides/voice-support.html)
> > > 
> > >Even the ezquake binary I built links to this library (in the ldd 
> > > output).
> > > 
> > > -Tom
> > > 
> > 
> > audio/speex already has audio/speexdsp linked into it (see the audio/speex
> > Makefile and double-check with make port-lib-depends-check in
> > games/ezquake).
> > 
> > ~Brian
> 
> OK I'm happy with that (and the formatting changes make sense).
> Feel free to commit if allowed/approved, thanks!
> 
> -Tom
> 

This is OK awolk@, go ahead if you want to import it Brian.

Regards,
Adam



Re: [NEW] games/ezquake

2018-08-12 Thread Adam Wolk
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



update editors/ghostwriter 1.7.0 => 1.7.2

2018-08-11 Thread Adam Wolk
Hi ports@,

trivial bump for ghostwriter.

feedback? OK's?

Regards,
Adam
Index: Makefile
===
RCS file: /cvs/ports/editors/ghostwriter/Makefile,v
retrieving revision 1.10
diff -u -p -r1.10 Makefile
--- Makefile18 Jul 2018 11:51:15 -  1.10
+++ Makefile11 Aug 2018 20:17:16 -
@@ -7,7 +7,7 @@ COMMENT =   distraction-free Markdown edit
 
 GH_ACCOUNT =   wereturtle
 GH_PROJECT =   ghostwriter
-GH_TAGNAME =   v1.7.0
+GH_TAGNAME =   v1.7.2
 
 CATEGORIES =   editors
 
Index: distinfo
===
RCS file: /cvs/ports/editors/ghostwriter/distinfo,v
retrieving revision 1.4
diff -u -p -r1.4 distinfo
--- distinfo18 Jul 2018 11:51:15 -  1.4
+++ distinfo11 Aug 2018 20:17:16 -
@@ -1,2 +1,2 @@
-SHA256 (ghostwriter-1.7.0.tar.gz) = 
U2VqP3DlPHIecOxNLjKfsdlBwQWddsvNLiuUHfUOSX0=
-SIZE (ghostwriter-1.7.0.tar.gz) = 680767
+SHA256 (ghostwriter-1.7.2.tar.gz) = 
A0NY+/Mlb7xM3ySULeOZSP/0IDmNr7IsVMgT28Jm6E4=
+SIZE (ghostwriter-1.7.2.tar.gz) = 672150
Index: pkg/PLIST
===
RCS file: /cvs/ports/editors/ghostwriter/pkg/PLIST,v
retrieving revision 1.4
diff -u -p -r1.4 PLIST
--- pkg/PLIST   29 Jun 2018 22:16:11 -  1.4
+++ pkg/PLIST   11 Aug 2018 20:17:16 -
@@ -6,7 +6,16 @@ share/appdata/ghostwriter.appdata.xml
 share/applications/ghostwriter.desktop
 share/ghostwriter/
 share/ghostwriter/translations/
+share/ghostwriter/translations/ghostwriter_cs.qm
+share/ghostwriter/translations/ghostwriter_de.qm
 share/ghostwriter/translations/ghostwriter_en.qm
+share/ghostwriter/translations/ghostwriter_es.qm
+share/ghostwriter/translations/ghostwriter_fr.qm
+share/ghostwriter/translations/ghostwriter_it.qm
+share/ghostwriter/translations/ghostwriter_ja.qm
+share/ghostwriter/translations/ghostwriter_pl.qm
+share/ghostwriter/translations/ghostwriter_pt_BR.qm
+share/ghostwriter/translations/ghostwriter_ru.qm
 share/ghostwriter/translations/ghostwriter_zh.qm
 share/icons/hicolor/128x128/apps/ghostwriter.png
 share/icons/hicolor/16x16/apps/ghostwriter.png


Re: editors/ghostwriter 1.6.2p1 => 1.7.0

2018-07-18 Thread Adam Wolk
On Mon, Jul 09, 2018 at 05:34:38PM +0200, Adam Wolk wrote:
> Hi ports@,
> 
> Trivial bump to a new release, the upstream changelog can be
> found here:
>   https://github.com/wereturtle/ghostwriter/releases/tag/v1.7.0
> 
> on our side:
>  - add Qt5SVG to WANTLIB and LIB_DEPENDS
> 
> unforutnately still using tarballs, they marked my request
> for release artifacts as 'wontfix':
>   https://github.com/wereturtle/ghostwriter/issues/328
> 
> OK?
> 
> Regards,
> Adam

ping

> ? ghostwriter.1.7.0.diff
> Index: Makefile
> ===
> RCS file: /cvs/ports/editors/ghostwriter/Makefile,v
> retrieving revision 1.9
> diff -u -p -r1.9 Makefile
> --- Makefile  29 Jun 2018 22:16:11 -  1.9
> +++ Makefile  9 Jul 2018 15:29:43 -
> @@ -7,8 +7,7 @@ COMMENT = distraction-free Markdown edit
>  
>  GH_ACCOUNT = wereturtle
>  GH_PROJECT = ghostwriter
> -GH_TAGNAME = v1.6.2
> -REVISION =   1
> +GH_TAGNAME = v1.7.0
>  
>  CATEGORIES = editors
>  
> @@ -21,7 +20,7 @@ PERMIT_PACKAGE_CDROM =  Yes
>  
>  WANTLIB += ${COMPILER_LIBCXX}
>  WANTLIB += GL Qt5Concurrent Qt5Core Qt5Gui Qt5Network Qt5PrintSupport
> -WANTLIB += Qt5WebKit Qt5WebKitWidgets Qt5Widgets c hunspell-1.6
> +WANTLIB += Qt5Svg Qt5WebKit Qt5WebKitWidgets Qt5Widgets c hunspell-1.6
>  WANTLIB += m pthread
>  
>  MODULES =devel/qmake \
> @@ -30,6 +29,7 @@ RUN_DEPENDS =   devel/desktop-file-utils 
>   x11/gtk+3,-guic
>  LIB_DEPENDS =textproc/hunspell \
>   x11/qt5/qtbase \
> + x11/qt5/qtsvg \
>   x11/qt5/qtwebkit
>  
>  NO_TEST =Yes
> Index: distinfo
> ===
> RCS file: /cvs/ports/editors/ghostwriter/distinfo,v
> retrieving revision 1.3
> diff -u -p -r1.3 distinfo
> --- distinfo  2 May 2018 19:42:03 -   1.3
> +++ distinfo  9 Jul 2018 15:29:43 -
> @@ -1,2 +1,2 @@
> -SHA256 (ghostwriter-1.6.2.tar.gz) = 
> /qWhsrLr15woFP8yxLnTvcmD+CYFZvx6HJeV2lktl8A=
> -SIZE (ghostwriter-1.6.2.tar.gz) = 678181
> +SHA256 (ghostwriter-1.7.0.tar.gz) = 
> U2VqP3DlPHIecOxNLjKfsdlBwQWddsvNLiuUHfUOSX0=
> +SIZE (ghostwriter-1.7.0.tar.gz) = 680767



editors/ghostwriter 1.6.2p1 => 1.7.0

2018-07-09 Thread Adam Wolk
Hi ports@,

Trivial bump to a new release, the upstream changelog can be
found here:
  https://github.com/wereturtle/ghostwriter/releases/tag/v1.7.0

on our side:
 - add Qt5SVG to WANTLIB and LIB_DEPENDS

unforutnately still using tarballs, they marked my request
for release artifacts as 'wontfix':
  https://github.com/wereturtle/ghostwriter/issues/328

OK?

Regards,
Adam
? ghostwriter.1.7.0.diff
Index: Makefile
===
RCS file: /cvs/ports/editors/ghostwriter/Makefile,v
retrieving revision 1.9
diff -u -p -r1.9 Makefile
--- Makefile29 Jun 2018 22:16:11 -  1.9
+++ Makefile9 Jul 2018 15:29:43 -
@@ -7,8 +7,7 @@ COMMENT =   distraction-free Markdown edit
 
 GH_ACCOUNT =   wereturtle
 GH_PROJECT =   ghostwriter
-GH_TAGNAME =   v1.6.2
-REVISION = 1
+GH_TAGNAME =   v1.7.0
 
 CATEGORIES =   editors
 
@@ -21,7 +20,7 @@ PERMIT_PACKAGE_CDROM =Yes
 
 WANTLIB += ${COMPILER_LIBCXX}
 WANTLIB += GL Qt5Concurrent Qt5Core Qt5Gui Qt5Network Qt5PrintSupport
-WANTLIB += Qt5WebKit Qt5WebKitWidgets Qt5Widgets c hunspell-1.6
+WANTLIB += Qt5Svg Qt5WebKit Qt5WebKitWidgets Qt5Widgets c hunspell-1.6
 WANTLIB += m pthread
 
 MODULES =  devel/qmake \
@@ -30,6 +29,7 @@ RUN_DEPENDS = devel/desktop-file-utils 
x11/gtk+3,-guic
 LIB_DEPENDS =  textproc/hunspell \
x11/qt5/qtbase \
+   x11/qt5/qtsvg \
x11/qt5/qtwebkit
 
 NO_TEST =  Yes
Index: distinfo
===
RCS file: /cvs/ports/editors/ghostwriter/distinfo,v
retrieving revision 1.3
diff -u -p -r1.3 distinfo
--- distinfo2 May 2018 19:42:03 -   1.3
+++ distinfo9 Jul 2018 15:29:43 -
@@ -1,2 +1,2 @@
-SHA256 (ghostwriter-1.6.2.tar.gz) = 
/qWhsrLr15woFP8yxLnTvcmD+CYFZvx6HJeV2lktl8A=
-SIZE (ghostwriter-1.6.2.tar.gz) = 678181
+SHA256 (ghostwriter-1.7.0.tar.gz) = 
U2VqP3DlPHIecOxNLjKfsdlBwQWddsvNLiuUHfUOSX0=
+SIZE (ghostwriter-1.7.0.tar.gz) = 680767


net/corebird 1.7.3 => 1.7.4

2018-07-08 Thread Adam Wolk
Hi ports@,

attaching a trivial bump for net/corebird
tasted on amd64 current.

Port changes:
 - PLIST change from a new translation
 - normal bump

Upstream changelog:
 https://github.com/baedert/corebird/releases/tag/1.7.4

Feedback, OK's?

Regards,
Adam
Index: Makefile
===
RCS file: /cvs/ports/net/corebird/Makefile,v
retrieving revision 1.24
diff -u -p -r1.24 Makefile
--- Makefile6 Jul 2018 06:04:43 -   1.24
+++ Makefile8 Jul 2018 21:20:24 -
@@ -2,10 +2,9 @@
 
 COMMENT =  native Gtk+ Twitter client
 
-V =1.7.3
+V =1.7.4
 DISTNAME = corebird-${V}
 MASTER_SITES = 
https://github.com/baedert/corebird/releases/download/${V}/
-REVISION = 4
 
 EXTRACT_SUFX = .tar.xz
 CATEGORIES =   net
Index: distinfo
===
RCS file: /cvs/ports/net/corebird/distinfo,v
retrieving revision 1.11
diff -u -p -r1.11 distinfo
--- distinfo20 Nov 2017 21:43:48 -  1.11
+++ distinfo8 Jul 2018 21:20:24 -
@@ -1,2 +1,2 @@
-SHA256 (corebird-1.7.3.tar.xz) = y/Qv3xhvXepHxdFx9qbdR6BdDnIaS/1uYbUwaeonjic=
-SIZE (corebird-1.7.3.tar.xz) = 705080
+SHA256 (corebird-1.7.4.tar.xz) = fFOjVu3tWOykIxKLKGhRB6EdP3uwoI4PwHK2DCsPAEE=
+SIZE (corebird-1.7.4.tar.xz) = 712704
Index: pkg/PLIST
===
RCS file: /cvs/ports/net/corebird/pkg/PLIST,v
retrieving revision 1.10
diff -u -p -r1.10 PLIST
--- pkg/PLIST   6 Jul 2018 06:04:43 -   1.10
+++ pkg/PLIST   8 Jul 2018 21:20:24 -
@@ -49,6 +49,7 @@ share/locale/pt_BR/LC_MESSAGES/corebird.
 share/locale/ro/LC_MESSAGES/corebird.mo
 share/locale/ru/LC_MESSAGES/corebird.mo
 share/locale/sr/LC_MESSAGES/corebird.mo
+share/locale/sr@latin/LC_MESSAGES/corebird.mo
 share/locale/tr/LC_MESSAGES/corebird.mo
 share/locale/uk_UA/
 share/locale/uk_UA/LC_MESSAGES/


[UPDATE] editors/ghostwriter 1.5p2 => 1.6.2

2018-04-27 Thread Adam Wolk
Hi ports@,

Trivial bump to a new release, the upstream changelog can be
found here:
  https://github.com/wereturtle/ghostwriter/releases

on our side:
 - drop patches/* as this contains our upstream patch
 - PLIST updated for a new translation file

this one still is a github auto tarball, I raised an issue:
  https://github.com/wereturtle/ghostwriter/issues/328
with upstream, asking for immutable downloads.

Feedback? OK's?

Regards,
Adam
Index: Makefile
===
RCS file: /cvs/ports/editors/ghostwriter/Makefile,v
retrieving revision 1.6
diff -u -p -r1.6 Makefile
--- Makefile28 Jan 2018 17:05:26 -  1.6
+++ Makefile27 Apr 2018 13:38:51 -
@@ -7,8 +7,7 @@ COMMENT =   distraction-free Markdown edit
 
 GH_ACCOUNT =   wereturtle
 GH_PROJECT =   ghostwriter
-GH_TAGNAME =   v1.5.0
-REVISION = 2
+GH_TAGNAME =   v1.6.2
 
 CATEGORIES =   editors
 
Index: distinfo
===
RCS file: /cvs/ports/editors/ghostwriter/distinfo,v
retrieving revision 1.2
diff -u -p -r1.2 distinfo
--- distinfo23 May 2017 09:54:13 -  1.2
+++ distinfo27 Apr 2018 13:38:51 -
@@ -1,2 +1,2 @@
-SHA256 (ghostwriter-1.5.0.tar.gz) = 
v9ZAJNl0H1XVh6+lZh9LXipIvsu8kklcIYb8pE+whIY=
-SIZE (ghostwriter-1.5.0.tar.gz) = 620301
+SHA256 (ghostwriter-1.6.2.tar.gz) = 
/qWhsrLr15woFP8yxLnTvcmD+CYFZvx6HJeV2lktl8A=
+SIZE (ghostwriter-1.6.2.tar.gz) = 678181
Index: patches/patch-src_spelling_dictionary_provider_hunspell_cpp
===
RCS file: patches/patch-src_spelling_dictionary_provider_hunspell_cpp
diff -N patches/patch-src_spelling_dictionary_provider_hunspell_cpp
--- patches/patch-src_spelling_dictionary_provider_hunspell_cpp 28 Jan 2018 
17:05:26 -  1.1
+++ /dev/null   1 Jan 1970 00:00:00 -
@@ -1,14 +0,0 @@
-$OpenBSD: patch-src_spelling_dictionary_provider_hunspell_cpp,v 1.1 2018/01/28 
17:05:26 awolk Exp $
-
-Index: src/spelling/dictionary_provider_hunspell.cpp
 src/spelling/dictionary_provider_hunspell.cpp.orig
-+++ src/spelling/dictionary_provider_hunspell.cpp
-@@ -301,7 +301,7 @@ DictionaryProviderHunspell::DictionaryProviderHunspell
-   xdg.append("/usr/local/share");
-   xdg.append("/usr/share");
-   }
--  QStringList subdirs = QStringList() << "/hunspell" << "/myspell/dicts" 
<< "/myspell";
-+  QStringList subdirs = QStringList() << "/hunspell" << "/myspell/dicts" 
<< "/myspell" << "/mozilla-dicts";
-   foreach (const QString& subdir, subdirs) {
-   foreach (const QString& dir, xdg) {
-   QString path = dir + subdir;
Index: pkg/PLIST
===
RCS file: /cvs/ports/editors/ghostwriter/pkg/PLIST,v
retrieving revision 1.1.1.1
diff -u -p -r1.1.1.1 PLIST
--- pkg/PLIST   17 Jan 2017 12:58:26 -  1.1.1.1
+++ pkg/PLIST   27 Apr 2018 13:38:51 -
@@ -7,6 +7,7 @@ share/applications/ghostwriter.desktop
 share/ghostwriter/
 share/ghostwriter/translations/
 share/ghostwriter/translations/ghostwriter_en.qm
+share/ghostwriter/translations/ghostwriter_zh.qm
 share/icons/hicolor/128x128/apps/ghostwriter.png
 share/icons/hicolor/16x16/apps/ghostwriter.png
 share/icons/hicolor/22x22/apps/ghostwriter.png


Re: Ghostwriter 1.5

2018-04-27 Thread Adam Wolk
On Fri, Mar 09, 2018 at 12:20:39AM -0500, Z Ero wrote:
> Hello,
> 
> I have some some questions on the port / package for the Ghostwriter
> markdown editor on amd64 / 6.2.
> 
> 1st. I installed the editor and started using it and it worked fine.
> Now it mysteriously is not translating at least some markdown into
> html.
> 
> Whereas before
> 
> "#heading"
> 
> would live preview / export as large bolded text reading "heading" now
> it just shows in the HTML preview and export as "#heading"
> 

Shouldn't this be # heading, I just tried and with the space it properly
renders in the HTML preview. Without the space I get the same thing as
you just '#heading' with no formatting in the output.

> Additionally I am now getting the following error.
> 
> "QStandardPaths: XDG_RUNTIME_DIR not set, defaulting to
> '/tmp/runtime-'"
> 
> at start up. Whereas before I was not.
> 

That's not an error, it's just information from the app. I wouldn't
worry about it.

> 2nd. does any one know how to enable spellcheck on Ghostwriter?
> 
> I get the errors
> /home//dict:C.aff
> 
> at start up.
> 

Spellchecking should work since 1.5.0p2 which is present in
OpenBSD 6.3. You just need to install the mozilla-dicts you
want and pick them from the drop down.

> I have tried deleting and re-installing ghost-writer and its
> dependencies and rebooting, etc.
> 
> It is really frustrating because the application was working fine the other 
> day.
> 
> The only thing I can think of is that some other application I
> installed after ghost writer changed the configuration of a shared
> dependancy / environmental variable but I don't know what and I have a
> limited amount of time to devote to troubleshooting right now.
> 
> Thanks
> 



Re: games/yquake2: UPDATE 7.10 -> 7.20

2018-03-13 Thread Adam Wolk
On Tue, Mar 13, 2018 at 09:47:03PM +, Tom Murphy wrote:
> Hi,
> 
>   Here's is an update for games/yquake2 (7.20 was released on 10th of March 
> 2018)
> 
> Thanks,
> Tom
> 

Thanks for the diff Tom.

This is OK awolk@, I would love a second OK to get this in
as 7.20 fixes saving the server list and customized
keybindings which was really annoying to re-do on each
launch.

Regards,
Adam



Re: [update] devel/zeal use release tarballs

2018-03-04 Thread Adam Wolk
On Sun, Mar 04, 2018 at 01:27:48PM +, Stuart Henderson wrote:
> Please use a temporary DISTFILES rename (see fs2open) to avoid changing the
> distfile under the same name, bulk builds don't handle it well.
> 
> -- 
>  Sent from a phone, apologies for poor formatting.
> 

same diff, switched to using EXTRACT_SUFX= .tar.xz to avoid DITFILES mangling.

OK?
Index: Makefile
===
RCS file: /cvs/ports/devel/zeal/Makefile,v
retrieving revision 1.9
diff -u -p -r1.9 Makefile
--- Makefile28 Feb 2018 22:52:53 -  1.9
+++ Makefile4 Mar 2018 13:55:43 -
@@ -5,9 +5,9 @@ USE_WXNEEDED =  Yes
 
 COMMENT =  simple offline API documentation browser
 
-GH_ACCOUNT =   zealdocs
-GH_PROJECT =   zeal
-GH_TAGNAME =   v0.6.0
+V =0.6.0
+DISTNAME = zeal-${V}
+REVISION = 0
 
 CATEGORIES =   devel
 
@@ -23,9 +23,9 @@ WANTLIB += Qt5WebKitWidgets Qt5Widgets Q
 WANTLIB += archive c m sqlite3 xcb xcb-keysyms
 WANTLIB += ${COMPILER_LIBCXX}
 
-WANTLIB += ICE Qt5Concurrent Qt5Core Qt5Gui Qt5Network Qt5WebKit
-WANTLIB += Qt5WebKitWidgets Qt5Widgets Qt5X11Extras SM X11 Xext
-WANTLIB += archive c m sqlite3 xcb xcb-keysyms
+MASTER_SITES = https://github.com/zealdocs/zeal/releases/download/v${V}/
+
+EXTRACT_SUFX = .tar.xz
 
 MODULES =  devel/cmake x11/qt5
 BUILD_DEPENDS =devel/kf5/extra-cmake-modules
Index: distinfo
===
RCS file: /cvs/ports/devel/zeal/distinfo,v
retrieving revision 1.5
diff -u -p -r1.5 distinfo
--- distinfo28 Feb 2018 22:52:53 -  1.5
+++ distinfo4 Mar 2018 13:55:43 -
@@ -1,2 +1,2 @@
-SHA256 (zeal-0.6.0.tar.gz) = 7zB9OtTwPHb6X8JJ+OfgTeh5qnV0vsH/i+VI28LAKXM=
-SIZE (zeal-0.6.0.tar.gz) = 1062223
+SHA256 (zeal-0.6.0.tar.xz) = 2eBFhGAbxQRh2Z41RRbkaJO/Leu+N3yxc9FUHU/glYo=
+SIZE (zeal-0.6.0.tar.xz) = 670416


[update] devel/zeal use release tarballs

2018-03-04 Thread Adam Wolk
Hi ports@,

devel/zeal upstream started doing release tarballs:
https://github.com/zealdocs/zeal/issues/897

Here is a diff to switch our port to them, additionally
I noticed that I accidentally duplicated WANTLIB during
the upgrade to 0.4 so while here drop the duplicated
lines.

Feedback? OK's?

Regards,
Adam
? zeal-0.3.1.diff
? zeal-0.4.0.diff
? zeal-0.5.0.diff
? zeal-0.6.0.diff
? zeal-0.6.0p0.diff
Index: Makefile
===
RCS file: /cvs/ports/devel/zeal/Makefile,v
retrieving revision 1.9
diff -u -p -r1.9 Makefile
--- Makefile28 Feb 2018 22:52:53 -  1.9
+++ Makefile4 Mar 2018 12:32:01 -
@@ -5,9 +5,9 @@ USE_WXNEEDED =  Yes
 
 COMMENT =  simple offline API documentation browser
 
-GH_ACCOUNT =   zealdocs
-GH_PROJECT =   zeal
-GH_TAGNAME =   v0.6.0
+V =0.6.0
+DISTNAME = zeal-${V}
+REVISION = 0
 
 CATEGORIES =   devel
 
@@ -23,9 +23,7 @@ WANTLIB += Qt5WebKitWidgets Qt5Widgets Q
 WANTLIB += archive c m sqlite3 xcb xcb-keysyms
 WANTLIB += ${COMPILER_LIBCXX}
 
-WANTLIB += ICE Qt5Concurrent Qt5Core Qt5Gui Qt5Network Qt5WebKit
-WANTLIB += Qt5WebKitWidgets Qt5Widgets Qt5X11Extras SM X11 Xext
-WANTLIB += archive c m sqlite3 xcb xcb-keysyms
+MASTER_SITES = https://github.com/zealdocs/zeal/releases/download/v${V}/
 
 MODULES =  devel/cmake x11/qt5
 BUILD_DEPENDS =devel/kf5/extra-cmake-modules
Index: distinfo
===
RCS file: /cvs/ports/devel/zeal/distinfo,v
retrieving revision 1.5
diff -u -p -r1.5 distinfo
--- distinfo28 Feb 2018 22:52:53 -  1.5
+++ distinfo4 Mar 2018 12:32:01 -
@@ -1,2 +1,2 @@
-SHA256 (zeal-0.6.0.tar.gz) = 7zB9OtTwPHb6X8JJ+OfgTeh5qnV0vsH/i+VI28LAKXM=
-SIZE (zeal-0.6.0.tar.gz) = 1062223
+SHA256 (zeal-0.6.0.tar.gz) = BPkpAOosxAmXZ0Lxt/Y5QxZIicH5HK14lRbolO6uGv8=
+SIZE (zeal-0.6.0.tar.gz) = 1060066


Re: devel/luaposix drop MAINTAINER

2018-02-27 Thread Adam Wolk
On Tue, Feb 27, 2018 at 12:32:52PM +0100, Adam Wolk wrote:
> Hi ports@,
> 
> I'm no longer using lua in my projects so can't commit to maintaining
> devel/luaposix. Here is a diff to drop MAINTAINER, OK?
> 
> ps.
> If someone wants to take over as MAINTAINER I will gladly place it
> there while removing mine - just let me know.
> 
> Regards,
> Adam
> 

... the missing diff
Index: Makefile
===
RCS file: /cvs/ports/devel/luaposix/Makefile,v
retrieving revision 1.23
diff -u -p -r1.23 Makefile
--- Makefile17 Oct 2016 21:39:23 -  1.23
+++ Makefile27 Feb 2018 11:30:12 -
@@ -4,14 +4,12 @@ COMMENT=  posix library for the lua langu
 V= 33.4.0
 DISTNAME=  luaposix-${V}
 EPOCH= 0
-REVISION=  0
+REVISION=  1
 CATEGORIES=devel
 
 GH_ACCOUNT=luaposix
 GH_PROJECT=luaposix
 GH_TAGNAME=release-v${V}
-
-MAINTAINER=Adam Wolk 
 
 # MIT
 PERMIT_PACKAGE_CDROM=  Yes


devel/luaposix drop MAINTAINER

2018-02-27 Thread Adam Wolk
Hi ports@,

I'm no longer using lua in my projects so can't commit to maintaining
devel/luaposix. Here is a diff to drop MAINTAINER, OK?

ps.
If someone wants to take over as MAINTAINER I will gladly place it
there while removing mine - just let me know.

Regards,
Adam



[UPDATE] devel/zeal 0.5.0 => 0.6.0

2018-02-25 Thread Adam Wolk
Hi ports@

Attaching a trivial bump to devel/zeal

upstream changelog:
https://github.com/zealdocs/zeal/releases/tag/v0.6.0

Feedback? OK's? 


Regards,

Adam  
Index: Makefile
===
RCS file: /cvs/ports/devel/zeal/Makefile,v
retrieving revision 1.8
diff -u -p -r1.8 Makefile
--- Makefile7 Jan 2018 17:33:25 -   1.8
+++ Makefile25 Feb 2018 23:17:56 -
@@ -7,7 +7,7 @@ COMMENT =   simple offline API documentati
 
 GH_ACCOUNT =   zealdocs
 GH_PROJECT =   zeal
-GH_TAGNAME =   v0.5.0
+GH_TAGNAME =   v0.6.0
 
 CATEGORIES =   devel
 
Index: distinfo
===
RCS file: /cvs/ports/devel/zeal/distinfo,v
retrieving revision 1.4
diff -u -p -r1.4 distinfo
--- distinfo7 Jan 2018 17:33:25 -   1.4
+++ distinfo25 Feb 2018 23:17:56 -
@@ -1,2 +1,2 @@
-SHA256 (zeal-0.5.0.tar.gz) = Pvt7G12aBfD8YKZoZXH3pLWPpvbWbxuvYI6DsQ+xKQw=
-SIZE (zeal-0.5.0.tar.gz) = 1041112
+SHA256 (zeal-0.6.0.tar.gz) = 7zB9OtTwPHb6X8JJ+OfgTeh5qnV0vsH/i+VI28LAKXM=
+SIZE (zeal-0.6.0.tar.gz) = 1062223


Re: update games/qstat 2.11 -> 2.14

2018-02-18 Thread Adam Wolk
On Sat, Feb 17, 2018 at 10:24:43PM +, Tom Murphy wrote:
> Forgot to remove DISTNAME (as per sthen@).
> Here's the diff (hopefully the last one!)
> 
> 

OK awolk@ for the latest diff.



Re: update games/qstat 2.11 -> 2.14

2018-02-17 Thread Adam Wolk
On Sat, Feb 17, 2018 at 08:51:02PM +, Stuart Henderson wrote:
> > + #include 
> > + #define strtok_ret strtok_r
> > ++#ifdef __OpenBSD__
> > ++#define VENTRILO_RAND arc4random()
> > ++#else
> > + #define VENTRILO_RAND random()
> > ++#endif
> > + #else
> > + #include 
> > + #define strtok_ret strtok_s
> > 
> *p = (((VENTRILO_RAND * 0x343fd) + 0x269ec3) >> 16) & 0x7fff;
> 
> any idea what's going on here?
> 

The code is for checking a ventrilo server, the header points
out that the alogirthm was developed by Luigi Auriemma as
a result of reverse engineering the protocol.

The implementation from the author seems to be:

http://aluigi.altervista.org/papers/ventrilo3_handshake.c

in this implementation the only random value is 

putbe(sbuff + 0x12, time(NULL),16); // rand useless number

in both implementations the code is in the section handling
the udp header.

I guess they don't really care if the number is truly random.

Perhaps we should not change that and let upstream decide
on that single patch.



Re: update games/qstat 2.11 -> 2.14

2018-02-17 Thread Adam Wolk
On Sat, Feb 17, 2018 at 07:39:37PM +, Tom Murphy wrote:
> Hi,
> 
>   The games/qstat source has moved from Sourceforge to Github and has jumped 
> serveral
> versions. The two patch files might be obsolete next version as I have a pull 
> request
> in to incorporate those changes.
> 
>   Thanks,
>   Tom

Want to grab MAINTAINER while at it?

> 
> Index: Makefile
> ===
> RCS file: /cvs/ports/games/qstat/Makefile,v
> retrieving revision 1.9
> diff -u -p -r1.9 Makefile
> --- Makefile  11 Mar 2013 11:07:40 -  1.9
> +++ Makefile  17 Feb 2018 19:38:37 -
> @@ -2,7 +2,11 @@
>  
>  COMMENT= displays the status of multi-player Internet Game servers
>  
> -DISTNAME=qstat-2.11
> +GH_ACCOUNT=  multiplay
> +GH_PROJECT=  qstat
> +GH_TAGNAME=  v2.14
> +
> +DISTNAME=qstat-2.14
>  REVISION=0

REVISION can be now removed.

With the above changes it's OK awolk@

Regards,
Adam



Re: games/yquake2 - add $@ to quake2 execution

2018-02-13 Thread Adam Wolk
On Tue, Feb 13, 2018 at 09:09:23PM +, Tom Murphy wrote:
> I noticed in /usr/local/bin/quake2 you couldn't pass command-line options
> to it (such as +connect ) because it didn't use them when
> executing. Here's a small patch to add it.
> 
> Thanks,
> Tom

This is OK awolk@ and has been discussed of list.

I'm looking for another OK to commit the change.

> 
> Index: Makefile
> ===
> RCS file: /cvs/ports/games/yquake2/Makefile,v
> retrieving revision 1.12
> diff -u -p -r1.12 Makefile
> --- Makefile  11 Jan 2018 19:27:02 -  1.12
> +++ Makefile  13 Feb 2018 20:49:07 -
> @@ -8,7 +8,7 @@ V=7.10
>  PKGNAME= ${N}-${V}
>  DISTNAME=quake2-${V}
>  CATEGORIES=  games
> -REVISION=0
> +REVISION=    1
>  
>  HOMEPAGE=http://www.yamagi.org/quake2/
>  MAINTAINER=  Adam Wolk 
> Index: files/quake2.sh
> ===
> RCS file: /cvs/ports/games/yquake2/files/quake2.sh,v
> retrieving revision 1.1
> diff -u -p -r1.1 quake2.sh
> --- files/quake2.sh   9 Nov 2017 16:20:31 -   1.1
> +++ files/quake2.sh   13 Feb 2018 20:49:07 -
> @@ -1,4 +1,4 @@
>  #!/bin/sh
>  cd ${TRUEPREFIX}/share/yquake2
> -exec ${TRUEPREFIX}/share/yquake2/quake2
> +exec ${TRUEPREFIX}/share/yquake2/quake2 $@
>  
> 



Re: [FIX] editors/ghostwriter 1.5p2 spellchecking fix

2018-01-28 Thread Adam Wolk
On Sun, Jan 28, 2018 at 05:50:15PM +0100, Landry Breuil wrote:
> On Sun, Jan 28, 2018 at 05:47:21PM +0100, Adam Wolk wrote:
> > On Sun, Jan 28, 2018 at 05:33:26PM +0100, Landry Breuil wrote:
> > > On Sun, Jan 28, 2018 at 05:20:38PM +0100, Landry Breuil wrote:
> > > > On Sun, Jan 28, 2018 at 04:48:13PM +0100, Adam Wolk wrote:
> > > > > Hi ports@,
> > > > > 
> > > > > On OpenBSD we expect hunspell using apps to look for dictionaries in
> > > > > /usr/local/share/mozilla-dicts yet ghostwriter hardcodes a list of 
> > > > > directories
> > > > > to check that doesn't include the one above - resulting in a non 
> > > > > functional
> > > > > spellchecker on our platform.
> > > > > 
> > > > > I sent a trivial pull request upstream, that adds this folder:
> > > > >   https://github.com/wereturtle/ghostwriter/pull/285
> > > > > 
> > > > > however, the current CONTRIBUTING policy upstream states:
> > > > > 
> > > > > > Pull requests are presently only accepted for translations into 
> > > > > > different
> > > > > > languages. I will be closing all other pull requests.
> > > > > source:
> > > > > https://github.com/wereturtle/ghostwriter/blob/master/CONTRIBUTING.md#pull-requests
> > > > > 
> > > > > which means, that likely this change might not get accepted soon and 
> > > > > could take
> > > > > some time until a release gets cut with this.
> > > > > 
> > > > > Hence, I would like to add that patch to our ports tree to enable 
> > > > > spellchecking
> > > > > with the version we have.
> > > > > 
> > > > > Feedback? OK's?
> > > > 
> > > > Sure, okay. It's fun that it's now advertised as a markdown editor, it
> > > > wasnt the case at all when i imported it :)
> > > 
> > > Dammit i'm mixing things with *focus*writer, disregard this comment :)
> > > 
> > 
> > but still OK landry@ to commit the patch? ;)
> 
> sure, always makes sense to fix bugs..
> 

committed, thanks!

Regards,
Adam



Re: [FIX] editors/ghostwriter 1.5p2 spellchecking fix

2018-01-28 Thread Adam Wolk
On Sun, Jan 28, 2018 at 05:33:26PM +0100, Landry Breuil wrote:
> On Sun, Jan 28, 2018 at 05:20:38PM +0100, Landry Breuil wrote:
> > On Sun, Jan 28, 2018 at 04:48:13PM +0100, Adam Wolk wrote:
> > > Hi ports@,
> > > 
> > > On OpenBSD we expect hunspell using apps to look for dictionaries in
> > > /usr/local/share/mozilla-dicts yet ghostwriter hardcodes a list of 
> > > directories
> > > to check that doesn't include the one above - resulting in a non 
> > > functional
> > > spellchecker on our platform.
> > > 
> > > I sent a trivial pull request upstream, that adds this folder:
> > >   https://github.com/wereturtle/ghostwriter/pull/285
> > > 
> > > however, the current CONTRIBUTING policy upstream states:
> > > 
> > > > Pull requests are presently only accepted for translations into 
> > > > different
> > > > languages. I will be closing all other pull requests.
> > > source:
> > > https://github.com/wereturtle/ghostwriter/blob/master/CONTRIBUTING.md#pull-requests
> > > 
> > > which means, that likely this change might not get accepted soon and 
> > > could take
> > > some time until a release gets cut with this.
> > > 
> > > Hence, I would like to add that patch to our ports tree to enable 
> > > spellchecking
> > > with the version we have.
> > > 
> > > Feedback? OK's?
> > 
> > Sure, okay. It's fun that it's now advertised as a markdown editor, it
> > wasnt the case at all when i imported it :)
> 
> Dammit i'm mixing things with *focus*writer, disregard this comment :)
> 

but still OK landry@ to commit the patch? ;)



[FIX] editors/ghostwriter 1.5p2 spellchecking fix

2018-01-28 Thread Adam Wolk
Hi ports@,

On OpenBSD we expect hunspell using apps to look for dictionaries in
/usr/local/share/mozilla-dicts yet ghostwriter hardcodes a list of directories
to check that doesn't include the one above - resulting in a non functional
spellchecker on our platform.

I sent a trivial pull request upstream, that adds this folder:
  https://github.com/wereturtle/ghostwriter/pull/285

however, the current CONTRIBUTING policy upstream states:

> Pull requests are presently only accepted for translations into different
> languages. I will be closing all other pull requests.
source:
https://github.com/wereturtle/ghostwriter/blob/master/CONTRIBUTING.md#pull-requests

which means, that likely this change might not get accepted soon and could take
some time until a release gets cut with this.

Hence, I would like to add that patch to our ports tree to enable spellchecking
with the version we have.

Feedback? OK's?

Regards,
Adam
Index: Makefile
===
RCS file: /cvs/ports/editors/ghostwriter/Makefile,v
retrieving revision 1.5
diff -u -p -r1.5 Makefile
--- Makefile30 Jul 2017 14:54:28 -  1.5
+++ Makefile28 Jan 2018 15:27:18 -
@@ -8,7 +8,7 @@ COMMENT =   distraction-free Markdown edit
 GH_ACCOUNT =   wereturtle
 GH_PROJECT =   ghostwriter
 GH_TAGNAME =   v1.5.0
-REVISION = 1
+REVISION = 2
 
 CATEGORIES =   editors
 
Index: patches/patch-src_spelling_dictionary_provider_hunspell_cpp
===
RCS file: patches/patch-src_spelling_dictionary_provider_hunspell_cpp
diff -N patches/patch-src_spelling_dictionary_provider_hunspell_cpp
--- /dev/null   1 Jan 1970 00:00:00 -
+++ patches/patch-src_spelling_dictionary_provider_hunspell_cpp 28 Jan 2018 
15:27:18 -
@@ -0,0 +1,14 @@
+$OpenBSD$
+
+Index: src/spelling/dictionary_provider_hunspell.cpp
+--- src/spelling/dictionary_provider_hunspell.cpp.orig
 src/spelling/dictionary_provider_hunspell.cpp
+@@ -301,7 +301,7 @@ DictionaryProviderHunspell::DictionaryProviderHunspell
+   xdg.append("/usr/local/share");
+   xdg.append("/usr/share");
+   }
+-  QStringList subdirs = QStringList() << "/hunspell" << "/myspell/dicts" 
<< "/myspell";
++  QStringList subdirs = QStringList() << "/hunspell" << "/myspell/dicts" 
<< "/myspell" << "/mozilla-dicts";
+   foreach (const QString& subdir, subdirs) {
+   foreach (const QString& dir, xdg) {
+   QString path = dir + subdir;


Re: update devel/py-icalendar

2018-01-09 Thread Adam Wolk
On Tue, Jan 09, 2018 at 09:28:41PM +0100, Joerg Jung wrote:
> Hi,
> 
> please find below an update to py-icalendar-4.0.0.
> Release drops support for some older python versions.
> 
> OK?
> 
> Regards,
> Joerg
> 

OK awolk@

I tested productivity/khal with this installed, but please get a
second OK from someone using the icalendar support as I only tested
with a local calendar.



Re: productivity/khal

2018-01-09 Thread Adam Wolk
On Tue, Jan 09, 2018 at 09:32:41PM +0100, Joerg Jung wrote:
> Hi,
> 
> please find below an update to khal-0.9.8.
> Changelog: http://lostpackets.de/khal/changelog.html
> 
> OK?
> 
> Regards,
> Joerg
> 

OK awolk@

I just happened to be evaluating khal for personal use when this came
in. Tested on amd64 -current with a local calendar only.

> 
> Index: Makefile
> ===
> RCS file: /cvs/ports/productivity/khal/Makefile,v
> retrieving revision 1.3
> diff -u -p -r1.3 Makefile
> --- Makefile  22 Jun 2017 20:12:08 -  1.3
> +++ Makefile  9 Jan 2018 20:29:59 -
> @@ -2,7 +2,7 @@
>  
>  COMMENT =standards based terminal calendar
>  
> -MODPY_EGG_VERSION =  0.9.6
> +MODPY_EGG_VERSION =  0.9.8
>  DISTNAME =   khal-${MODPY_EGG_VERSION}
>  
>  CATEGORIES = productivity
> Index: distinfo
> ===
> RCS file: /cvs/ports/productivity/khal/distinfo,v
> retrieving revision 1.2
> diff -u -p -r1.2 distinfo
> --- distinfo  22 Jun 2017 20:12:08 -  1.2
> +++ distinfo  9 Jan 2018 20:29:59 -
> @@ -1,2 +1,2 @@
> -SHA256 (khal-0.9.6.tar.gz) = bsfOAgJ2SDlSH0eAXhCsH1Tprfrj+C58HmetiO79Z3I=
> -SIZE (khal-0.9.6.tar.gz) = 169539
> +SHA256 (khal-0.9.8.tar.gz) = /gSeHtMjhGHBCPfvSrugn0YTD3fw4rgEGFKfbfsbna4=
> +SIZE (khal-0.9.8.tar.gz) = 173717
> 



[UPDATE] devel/zeal 0.4.0 => 0.5.0

2018-01-07 Thread Adam Wolk
Hi ports@

Attaching a trivial bump to devel/zeal

upstream changelog:
https://github.com/zealdocs/zeal/releases/tag/v0.5.0

Feedback? OK's?

Regards,
Adam
Index: Makefile
===
RCS file: /cvs/ports/devel/zeal/Makefile,v
retrieving revision 1.7
diff -u -p -r1.7 Makefile
--- Makefile22 Nov 2017 14:27:29 -  1.7
+++ Makefile7 Jan 2018 13:52:40 -
@@ -7,7 +7,7 @@ COMMENT =   simple offline API documentati
 
 GH_ACCOUNT =   zealdocs
 GH_PROJECT =   zeal
-GH_TAGNAME =   v0.4.0
+GH_TAGNAME =   v0.5.0
 
 CATEGORIES =   devel
 
Index: distinfo
===
RCS file: /cvs/ports/devel/zeal/distinfo,v
retrieving revision 1.3
diff -u -p -r1.3 distinfo
--- distinfo22 Nov 2017 14:27:29 -  1.3
+++ distinfo7 Jan 2018 13:52:40 -
@@ -1,2 +1,2 @@
-SHA256 (zeal-0.4.0.tar.gz) = H2tQAmA2kjrgz78/TrEGbuL9rwQl1MMhID4N2VBoI/0=
-SIZE (zeal-0.4.0.tar.gz) = 1049326
+SHA256 (zeal-0.5.0.tar.gz) = Pvt7G12aBfD8YKZoZXH3pLWPpvbWbxuvYI6DsQ+xKQw=
+SIZE (zeal-0.5.0.tar.gz) = 1041112


Re: NEW: games/sdl-jstest

2018-01-05 Thread Adam Wolk
On Fri, Jan 05, 2018 at 06:41:35PM -0500, Brian Callahan wrote:
> 
> On 01/05/18 17:49, Thomas Frohwein wrote:
> > Attached is a port sdl-jstest. sdl-jstest is a simple program to find out 
> > how many joysticks are detected by SDL and SDL2. It can list devices 
> > detected by SDL1/SDL2 joystick or gamecontroller API. It also has a flag to 
> > display events sent by the buttons and axes.
> > 
> > The primary purpose is for testing and troubleshooting 
> > joystick/gamecontroller devices. Helpful in working on SDL2's 
> > gamecontroller API.
> > 
> 
> This is ok for me, with the caveat that the V variable be merged into
> DISTNAME (since it's only used there). And maybe eliminate the verbosity of
> the license marker comment, GPLv2+ is good enough.
> 
> I'm also guessing this needs the SDL2-2.0.7 update? Otherwise, sdl2-jstest
> complains that SDL2 was built without force feedback.
> 
> ~Brian
> 

This is also OK awolk@

with same caveat regarding SDL2 and for anyone testing I found a quirk.

I have a zalman gaming mouse and for some reason this test app detected
it as 3 gaming pads... preventing the PS4 dual shock pad from being
discovered by this utility when plugged in.

After unplugging the mouse, and plugging the pad in - it's detected correctly
and the test & events mode works as expected.

So, unplug your fancy USB devices before testing a gaming pad/joystick.

Regards,
Adam



Re: NEW: games/freeserf

2018-01-05 Thread Adam Wolk
On Sat, Dec 16, 2017 at 04:20:54PM -0500, Brian Callahan wrote:
> 
> On 11/07/17 00:05, Brian Callahan wrote:
> > 
> > On 10/18/17 17:42, Brian Callahan wrote:
> > > Hi ports --
> > > 
> > > Continuing my quest to have every video game remake engine in the
> > > ports tree, here is games/freeserf, a Settlers 1 (Serf City) clone.
> > > 
> > > ---
> > > pkg/DESCR:
> > > FreeSerf is a faithful clone of the simulation game The Settlers 1 aka
> > > Serf City, a 4X game in similar vein to Populous and Sim City.
> > > 
> > > FreeSerf requires data files from the original game, either the DOS
> > > version or the Amiga version, to function properly.
> > > ---
> > > 
> > > I found a US copy of the DOS game on an abandonware site. Maybe not
> > > the most legal thing, but I was able to copy the data file from it
> > > to where FreeSerf expects it, and the game worked OK.
> > > 
> > > OK?
> > > 
> > > ~Brian
> > > 
> > 
> > Ping. New tarball to bring in latest upstream code.
> > 
> > ~Brian
> > 
> 
> Another ping. New tarball attached for latest upstream code. Still works
> great.
> 
> OK?
> 
> ~Brian
> 

OK awolk@ tested on amd64 -current



[UPDATE] sysutils/lsyncd 2.1.6 => 2.2.2

2017-12-26 Thread Adam Wolk
Hi ports@

While looking at the state of luaposix I found the only port that depends on it,
sysutils/lsyncd. Started to run `make test` and all hell broke loose. The test
expects to ssh in without password to localhost without marking itself as
TEST_IS_INTERACTIVE and on top of that breaks the terminal with lovely color
output.

Since I was already there, I started looking at updating this port and did just
that. So I'm attaching a diff and grabbing MAINTAINER but please note. I am not
using this software, I can keep it up to date but would love anyone using it to
grab MAINTAINER or at least test it when I post updates to the port.

Port changes:
 - patches/patch-cmake_FindLua_cmake is upstreamed so remove it
 - re apply other patches manually when needed & re-generate
 - patch the tests to not do color output
 - grab MAINTAINER
 - TEST_IS_INTERACTIVE set to yes

Upstream changes:
 https://github.com/axkibe/lsyncd/blob/master/ChangeLog

Feedback? OK's?

Regards,
Adam
Index: Makefile
===
RCS file: /cvs/ports/sysutils/lsyncd/Makefile,v
retrieving revision 1.2
diff -u -p -r1.2 Makefile
--- Makefile23 Jan 2016 19:30:38 -  1.2
+++ Makefile26 Dec 2017 19:12:08 -
@@ -3,10 +3,12 @@
 COMMENT =  synchronizes directories with remote targets upon changes
 GH_PROJECT =   lsyncd
 GH_ACCOUNT =   axkibe
-GH_COMMIT =cce7db5ebd352690917792848c2aea92c08efdd5
-DISTNAME = lsyncd-2.1.6
+GH_TAGNAME =   release-2.2.2
+DISTNAME = lsyncd-2.2.2
 CATEGORIES =   sysutils
 
+MAINTAINER =   Adam Wolk 
+
 # GPLv2
 PERMIT_PACKAGE_CDROM = Yes
 
@@ -26,6 +28,9 @@ TEST_DEPENDS =${RUN_DEPENDS} \
 
 TESTS =churn-direct churn-rsyncssh exclude-rsyncssh schedule \
churn-rsync exclude-rsync l4rsyncdata
+
+# expects to ssh without password to localhost
+TEST_IS_INTERACTIVE =  Yes
 
 WANTLIB += ${MODLUA_WANTLIB} lib/inotify/inotify c m pthread
 
Index: distinfo
===
RCS file: /cvs/ports/sysutils/lsyncd/distinfo,v
retrieving revision 1.2
diff -u -p -r1.2 distinfo
--- distinfo6 May 2017 15:10:18 -   1.2
+++ distinfo26 Dec 2017 19:12:08 -
@@ -1,2 +1,2 @@
-SHA256 (lsyncd-2.1.6-cce7db5e.tar.gz) = 
dBQeW6kH0mJOO/WJVV3V9Yd/gOfAkxeg7njD0G3jzQU=
-SIZE (lsyncd-2.1.6-cce7db5e.tar.gz) = 78989
+SHA256 (lsyncd-2.2.2.tar.gz) = C9sS9A8aUu0tjmy0ckLSlmY6QrMPONK478tmtDEp4Ak=
+SIZE (lsyncd-2.2.2.tar.gz) = 82231
Index: patches/patch-CMakeLists_txt
===
RCS file: /cvs/ports/sysutils/lsyncd/patches/patch-CMakeLists_txt,v
retrieving revision 1.1.1.1
diff -u -p -r1.1.1.1 patch-CMakeLists_txt
--- patches/patch-CMakeLists_txt23 Jan 2016 19:15:13 -  1.1.1.1
+++ patches/patch-CMakeLists_txt26 Dec 2017 19:12:08 -
@@ -2,47 +2,53 @@ $OpenBSD: patch-CMakeLists_txt,v 1.1.1.1
 
 No need to regen the manpage
 
 CMakeLists.txt.origThu Oct 15 08:29:47 2015
-+++ CMakeLists.txt Sat Jan 23 19:41:46 2016
+Index: CMakeLists.txt
+--- CMakeLists.txt.orig
 CMakeLists.txt
 @@ -6,7 +6,7 @@ set( CMAKE_MODULE_PATH ${CMAKE_MODULE_PATH} "${CMAKE_S
  
  
  # finding Lua
--find_package(Lua REQUIRED)
-+find_package(Lua ${MODLUA_VERSION})
+-find_package( Lua REQUIRED )
++find_package( Lua ${MODLUA_VERSION} )
  include_directories ( ${LUA_INCLUDE_DIR} )
  
  
-@@ -54,7 +54,7 @@ add_custom_command( OUTPUT runner.c 
- # this supposes the Lua compiler 'luac' is sitting right next to the Lua 
interpreter 'lua'
- add_custom_command( OUTPUT runner.out
- COMMAND ${CMAKE_COMMAND} -E echo "Compiling built-in runner" 
--  COMMAND ${LUA_EXECUTABLE}c -o runner.out 
${PROJECT_SOURCE_DIR}/lsyncd.lua
-+  COMMAND ${LUA_COMPILER} -o runner.out ${PROJECT_SOURCE_DIR}/lsyncd.lua
- DEPENDS ${PROJECT_SOURCE_DIR}/lsyncd.lua
- ) 
- 
-@@ -75,21 +75,13 @@ set( DEFAULT_CONFIGS
- 
- add_custom_command( OUTPUT defaults.out
- COMMAND ${CMAKE_COMMAND} -E echo "Compiling built-in default configs"
--  COMMAND ${LUA_EXECUTABLE}c -o defaults.out ${DEFAULT_CONFIGS}
-+  COMMAND ${LUA_COMPILER} -o defaults.out ${DEFAULT_CONFIGS}
- DEPENDS ${DEFAULT_CONFIGS}
- ) 
+@@ -47,7 +47,7 @@ include_directories("${PROJECT_BINARY_DIR}")
+ # also called "runner"
+ add_custom_command( OUTPUT runner.c
+   COMMAND ${CMAKE_COMMAND} -E echo "Generating built-in runner linkable"
+-  COMMAND ${LUA_EXECUTABLE} ${PROJECT_SOURCE_DIR}/bin2carray.lua 
runner.out runner runner.c
++COMMAND ${LUA_EXECUTABLE} ${PROJECT_SOURCE_DIR}/bin2carray.lua 
runner.out runner runner.c
+   DEPENDS runner.out
+ )
+ 
+@@ -62,7 +62,7 @@ add_custom_command( OUTPUT runner.out
+ #   rsync rysnc-ssh and direct
+ add_custom_command( OUTPUT defaults.c
+   COMMAND ${CMAKE_COMMAND} -E echo "Generating built-in default configs&

[UPDATE] security/routersploit 2.1.0 => 2.2.1

2017-12-26 Thread Adam Wolk
Hi ports@,

attaching a trivial bump for routersploit to satisfy the everhungry portroach
god.

Upstream changelog can be found here:
https://github.com/reverse-shell/routersploit/releases

Tested on amd64 -current by torturing my router.

Feedback? OK's?

Regards,
Adam
Index: Makefile
===
RCS file: /cvs/ports/security/routersploit/Makefile,v
retrieving revision 1.1.1.1
diff -u -p -r1.1.1.1 Makefile
--- Makefile4 Jul 2016 21:02:08 -   1.1.1.1
+++ Makefile26 Dec 2017 16:14:11 -
@@ -2,7 +2,7 @@
 
 COMMENT =  The Router Exploitation Framework
 
-MODPY_EGG_VERSION = 2.1.0
+MODPY_EGG_VERSION = 2.2.1
 GH_ACCOUNT =   reverse-shell
 GH_PROJECT =   routersploit
 GH_TAGNAME =   v${MODPY_EGG_VERSION}
Index: distinfo
===
RCS file: /cvs/ports/security/routersploit/distinfo,v
retrieving revision 1.1.1.1
diff -u -p -r1.1.1.1 distinfo
--- distinfo4 Jul 2016 21:02:08 -   1.1.1.1
+++ distinfo26 Dec 2017 16:14:11 -
@@ -1,2 +1,2 @@
-SHA256 (routersploit-2.1.0.tar.gz) = 
qWgntxh4vy/wEPZfW7zzRk55Ve0h3Q01hUhj0VRFh9E=
-SIZE (routersploit-2.1.0.tar.gz) = 46498
+SHA256 (routersploit-2.2.1.tar.gz) = 
NyaJQlAQtAt0KTOEwDx0zsI+K9s5bqYmRyVO0JUe8Xs=
+SIZE (routersploit-2.2.1.tar.gz) = 80925
Index: files/setup.py
===
RCS file: /cvs/ports/security/routersploit/files/setup.py,v
retrieving revision 1.1.1.1
diff -u -p -r1.1.1.1 setup.py
--- files/setup.py  4 Jul 2016 21:02:08 -   1.1.1.1
+++ files/setup.py  26 Dec 2017 16:14:11 -
@@ -5,7 +5,7 @@ import sys, os
 
 setup(
name = "routersploit",
-   version = "2.1.0",
+   version = "2.2.1",
description = "The Router Exploitation Framework",
author =  "Reverse Shell Security",
author_email = "off...@reverse-shell.com",
Index: pkg/PLIST
===
RCS file: /cvs/ports/security/routersploit/pkg/PLIST,v
retrieving revision 1.1.1.1
diff -u -p -r1.1.1.1 PLIST
--- pkg/PLIST   4 Jul 2016 21:02:08 -   1.1.1.1
+++ pkg/PLIST   26 Dec 2017 16:14:11 -
@@ -5,7 +5,6 @@ lib/python${MODPY_VERSION}/site-packages
 
lib/python${MODPY_VERSION}/site-packages/routersploit-${MODPY_EGG_VERSION}-py${MODPY_VERSION}.egg-info/PKG-INFO
 
lib/python${MODPY_VERSION}/site-packages/routersploit-${MODPY_EGG_VERSION}-py${MODPY_VERSION}.egg-info/SOURCES.txt
 
lib/python${MODPY_VERSION}/site-packages/routersploit-${MODPY_EGG_VERSION}-py${MODPY_VERSION}.egg-info/dependency_links.txt
-lib/python${MODPY_VERSION}/site-packages/routersploit-${MODPY_EGG_VERSION}-py${MODPY_VERSION}.egg-info/top_level.txt
 lib/python${MODPY_VERSION}/site-packages/routersploit/__init__.py
 lib/python${MODPY_VERSION}/site-packages/routersploit/__init__.pyc
 lib/python${MODPY_VERSION}/site-packages/routersploit/exceptions.py
@@ -44,10 +43,27 @@ lib/python${MODPY_VERSION}/site-packages
 
lib/python${MODPY_VERSION}/site-packages/routersploit/modules/creds/telnet_default.pyc
 lib/python${MODPY_VERSION}/site-packages/routersploit/modules/exploits/
 lib/python${MODPY_VERSION}/site-packages/routersploit/modules/exploits/2wire/
+lib/python${MODPY_VERSION}/site-packages/routersploit/modules/exploits/2wire/4011g_5012nv_path_traversal.py
+lib/python${MODPY_VERSION}/site-packages/routersploit/modules/exploits/2wire/4011g_5012nv_path_traversal.pyc
 
lib/python${MODPY_VERSION}/site-packages/routersploit/modules/exploits/2wire/__init__.py
 
lib/python${MODPY_VERSION}/site-packages/routersploit/modules/exploits/2wire/__init__.pyc
 
lib/python${MODPY_VERSION}/site-packages/routersploit/modules/exploits/2wire/gateway_auth_bypass.py
 
lib/python${MODPY_VERSION}/site-packages/routersploit/modules/exploits/2wire/gateway_auth_bypass.pyc
+lib/python${MODPY_VERSION}/site-packages/routersploit/modules/exploits/3com/
+lib/python${MODPY_VERSION}/site-packages/routersploit/modules/exploits/3com/3cradsl72_info_disclosure.py
+lib/python${MODPY_VERSION}/site-packages/routersploit/modules/exploits/3com/3cradsl72_info_disclosure.pyc
+lib/python${MODPY_VERSION}/site-packages/routersploit/modules/exploits/3com/__init__.py
+lib/python${MODPY_VERSION}/site-packages/routersploit/modules/exploits/3com/__init__.pyc
+lib/python${MODPY_VERSION}/site-packages/routersploit/modules/exploits/3com/ap8760_password_disclosure.py
+lib/python${MODPY_VERSION}/site-packages/routersploit/modules/exploits/3com/ap8760_password_disclosure.pyc
+lib/python${MODPY_VERSION}/site-packages/routersploit/modules/exploits/3com/imc_info_disclosure.py
+lib/python${MODPY_VERSION}/site-packages/routersploit/modules/exploits/3com/imc_info_disclosure.pyc
+lib/python${MODPY_VERSION}/site-packages/routersploit/modules/exploits/3com/imc_path_traversal.py
+lib/python${MODPY_VERSION}/site-packages/routersploit/modules/exploits/3com/imc_path_traversal.pyc
+lib/python${MODPY_VER

[UPDATE] www/otter-browser 0.9.12 => 0.9.93

2017-12-26 Thread Adam Wolk
Hi ports@

Attaching a bump to otter-browser 0.9.93. This is a 3rd release candidate for
the browser (previously we packaged weeklies and beta releases).

It's been a long time since I package an update (last one in January this year).
I plan to follow up with new release candidates as they appear and with the
final release when that happens. Time to unslack ;)

Port changes:
 - add x11/qt5/qtsvg to LIB_DEPENDS
 - drop X11 from WANTLIB (pointed out by port-lib-depends-check)

Upstream changelog:
https://otter-browser.org/

Easier to parse:
https://github.com/OtterBrowser/otter-browser/releases

we packaged 0.9.12 so the changes you need to read are:
 - 0.9.91 
 - 0.9.92 
 - 0.9.93 

Feedback? OK's?

Regards,
Adam
Index: Makefile
===
RCS file: /cvs/ports/www/otter-browser/Makefile,v
retrieving revision 1.25
diff -u -p -r1.25 Makefile
--- Makefile30 Jul 2017 14:56:23 -  1.25
+++ Makefile26 Dec 2017 12:17:06 -
@@ -3,8 +3,7 @@
 COMMENT =  browser aiming to recreate classic Opera (12.x) UI 
using Qt5
 
 MASTER_SITES = ${MASTER_SITE_SOURCEFORGE:=otter-browser/}
-DISTNAME = otter-browser-0.9.12
-REVISION = 2
+DISTNAME = otter-browser-0.9.93
 
 EXTRACT_SUFX = .tar.bz2
 
@@ -19,15 +18,15 @@ PERMIT_PACKAGE_CDROM =  Yes
 
 WANTLIB += ${COMPILER_LIBCXX}
 WANTLIB += c m pthread Qt5Core Qt5DBus Qt5Gui Qt5Multimedia
-WANTLIB += Qt5Network Qt5PrintSupport Qt5Qml Qt5WebKit
+WANTLIB += Qt5Network Qt5PrintSupport Qt5Qml Qt5Svg Qt5WebKit
 WANTLIB += Qt5WebKitWidgets Qt5Widgets Qt5XmlPatterns
-WANTLIB += hunspell-1.6 X11
-
+WANTLIB += hunspell-1.6
 
 RUN_DEPENDS =  devel/desktop-file-utils x11/gtk+3,-guic
 LIB_DEPENDS =  textproc/hunspell \
x11/qt5/qtdeclarative \
x11/qt5/qtmultimedia \
+   x11/qt5/qtsvg \
x11/qt5/qtwebkit \
x11/qt5/qtxmlpatterns
 
Index: distinfo
===
RCS file: /cvs/ports/www/otter-browser/distinfo,v
retrieving revision 1.16
diff -u -p -r1.16 distinfo
--- distinfo8 Jan 2017 12:13:49 -   1.16
+++ distinfo26 Dec 2017 12:17:06 -
@@ -1,2 +1,2 @@
-SHA256 (otter-browser-0.9.12.tar.bz2) = 
9mW9zDrzlufFz1iiI50BNuQ3vyLwOW/mvZeFaFa4sVc=
-SIZE (otter-browser-0.9.12.tar.bz2) = 2805058
+SHA256 (otter-browser-0.9.93.tar.bz2) = 
SfiZxMOaAsyHj3KEONubQ7RLkWwEZ/stcwZOqa73Dbo=
+SIZE (otter-browser-0.9.93.tar.bz2) = 2964103


Re: New port: witchblast (roguelite dungeon crawler)

2017-12-26 Thread Adam Wolk
On Mon, Dec 25, 2017 at 10:47:20PM +, Thomas Frohwein wrote:
> Updated tar attached.
> 
> - builds with ports-clang, too. So I set COMPILER=base-clang ports-gcc 
> ports-clang.

Is there an architecture where ports-clang would be with such compiler line?
It feels redundant based on bsd.port.mk(5) which states the first matching
element will be picked.

> - included awolk@'s patch to save "death certificate" screenshot in proper 
> location (~/.local/share/witchblast/data)- thanks!
> - fullscreen is marked as "experimental" by upstream's changelog. setting it 
> to 1 in config.dat uses fullscreen, but doesn't scale the game. Leaving this 
> alone for now.
> - changed email address to non-obfuscated and same as for dxx-rebirth
> - fonts license mentioned now
> 

You can add joystick support not working with a PS4 pad to the upstream bug
list, and not being able to pick up a dropped item until you re-enter a room
or drop something else.

> ... and tar'd the subdirectory this time...
>

yay \o/
 
> Ready for ok's?
> 

I tested with a solid run, it's OK awolk@ but I would like a second
OK and feedback regarding current COMPILER setting.



Re: New port: witchblast (roguelite dungeon crawler)

2017-12-24 Thread Adam Wolk
On Sun, Dec 24, 2017 at 11:23:44PM +0100, Adam Wolk wrote:
> On Sun, Dec 24, 2017 at 09:08:57PM +, Stuart Henderson wrote:
> > On 2017/12/24 21:42, Adam Wolk wrote:
> > > On Sun, Dec 24, 2017 at 01:09:45AM +, Thomas Frohwein wrote:
> > > > Follow up: The music of witchblast is licensed under CC BY-NC-SA. I 
> > > > grepped through ports and found games/dangerdeep and books/progit that 
> > > > also are CC BY-NC-SA.
> > > > 
> > > > Based on that I set:
> > > > 
> > > > PERMIT_PACKAGE_CDROM = Non-commercial
> > > > PERMIT_PACKAGE_FTP = Yes
> > > > 
> > > > Otherwise bump - it's a neat game with good music, worth a try.
> > > 
> > > The game is indeed amazing, great find thanks!
> > > 
> > > 1. I think it's worth to mention SIL Open Font License in the license 
> > > comment.
> > > That license file is installed along the fonts used by the game.
> > 
> > yep.
> > 
> > > 2. I found the F1 screenshot of the death certificate utility broken.
> > >  Failed to save image "rip_20171224212927.png"
> > > 
> > > That is because it saves to the current directory in 
> > > src/WitchBlastGame.cpp:
> > >  void WitchBlastGame::saveDeathScreen()
> > > 
> > > might be worth patching that so it saves to somewhere in users $HOME?
> > 
> > the most obvious place would be ~/.local/share/witchblast/data, but I don't 
> > think
> > it's needed for import.
> > 
> 
> I'm attaching a patch doing just that. I did find it cropping the output on i3
> due to the game having issue with determining it's dimensions on a tilling WM.
> When launched as a floating window it saves nicely looking death certificates.
> 
> First 3 images on the link below show a tilling WM bug, the other one shows it
> working with floating window. People running xfce and other non tilling WM's
> won't have the problem so I thikn the patch is worth it.
> 

and the missing link below...
https://imgur.com/a/9q28W



Re: New port: witchblast (roguelite dungeon crawler)

2017-12-24 Thread Adam Wolk
On Sun, Dec 24, 2017 at 09:08:57PM +, Stuart Henderson wrote:
> On 2017/12/24 21:42, Adam Wolk wrote:
> > On Sun, Dec 24, 2017 at 01:09:45AM +, Thomas Frohwein wrote:
> > > Follow up: The music of witchblast is licensed under CC BY-NC-SA. I 
> > > grepped through ports and found games/dangerdeep and books/progit that 
> > > also are CC BY-NC-SA.
> > > 
> > > Based on that I set:
> > > 
> > > PERMIT_PACKAGE_CDROM = Non-commercial
> > > PERMIT_PACKAGE_FTP = Yes
> > > 
> > > Otherwise bump - it's a neat game with good music, worth a try.
> > 
> > The game is indeed amazing, great find thanks!
> > 
> > 1. I think it's worth to mention SIL Open Font License in the license 
> > comment.
> > That license file is installed along the fonts used by the game.
> 
> yep.
> 
> > 2. I found the F1 screenshot of the death certificate utility broken.
> >  Failed to save image "rip_20171224212927.png"
> > 
> > That is because it saves to the current directory in src/WitchBlastGame.cpp:
> >  void WitchBlastGame::saveDeathScreen()
> > 
> > might be worth patching that so it saves to somewhere in users $HOME?
> 
> the most obvious place would be ~/.local/share/witchblast/data, but I don't 
> think
> it's needed for import.
> 

I'm attaching a patch doing just that. I did find it cropping the output on i3
due to the game having issue with determining it's dimensions on a tilling WM.
When launched as a floating window it saves nicely looking death certificates.

First 3 images on the link below show a tilling WM bug, the other one shows it
working with floating window. People running xfce and other non tilling WM's
won't have the problem so I thikn the patch is worth it.

> > 3. There are some issues with getting this game to use the full screen in 
> > i3.
> > It's 100% playable but does weird things on death (double display for a 
> > moment),
> > and doesn't take the whole real estate. I don't think however that this is 
> > a blocker.
> 
> +1.
> 
> > With the above changed it's OK awolk@ assuming we can get a second OK from 
> > someone
> > to import this :)
> 
> Obfuscated email addresses don't work for MAINTAINER. Other than that I'm OK
> with just the license fix to import.
> 
> Thomas, please use a subdirectory in tar files, it's easier to handle that 
> way.
> 

I'll import when Thomas submits an update with the license change & MAINTAINER 
fix.
I'm leaving it up to you Thomas whether to include or not the screenshot patch.

Regards,
Adam
$OpenBSD$

Index: src/WitchBlastGame.cpp
--- src/WitchBlastGame.cpp.orig
+++ src/WitchBlastGame.cpp
@@ -2816,7 +2816,7 @@ void WitchBlastGame::renderMessages()
 void WitchBlastGame::saveDeathScreen()
 {
   std::stringstream ss;
-  ss << "rip_";
+  ss << BASE_DATA_DIR << "data/rip_";
   time_t t = time(0);   // get time now
   struct tm * now = localtime( & t );
 


Re: New port: witchblast (roguelite dungeon crawler)

2017-12-24 Thread Adam Wolk
On Sun, Dec 24, 2017 at 01:09:45AM +, Thomas Frohwein wrote:
> Follow up: The music of witchblast is licensed under CC BY-NC-SA. I grepped 
> through ports and found games/dangerdeep and books/progit that also are CC 
> BY-NC-SA.
> 
> Based on that I set:
> 
> PERMIT_PACKAGE_CDROM = Non-commercial
> PERMIT_PACKAGE_FTP = Yes
> 
> Otherwise bump - it's a neat game with good music, worth a try.

The game is indeed amazing, great find thanks!

1. I think it's worth to mention SIL Open Font License in the license comment.
That license file is installed along the fonts used by the game.

2. I found the F1 screenshot of the death certificate utility broken.
 Failed to save image "rip_20171224212927.png"

That is because it saves to the current directory in src/WitchBlastGame.cpp:
 void WitchBlastGame::saveDeathScreen()

might be worth patching that so it saves to somewhere in users $HOME?

3. There are some issues with getting this game to use the full screen in i3.
It's 100% playable but does weird things on death (double display for a moment),
and doesn't take the whole real estate. I don't think however that this is a 
blocker.

With the above changed it's OK awolk@ assuming we can get a second OK from 
someone
to import this :)

Regards,
Adam



Re: NEW: games/dxx-rebirth

2017-12-24 Thread Adam Wolk
On Sat, Dec 23, 2017 at 01:49:18PM -0500, Brian Callahan wrote:
> 
> 
> OK. Here's the fixed port. Now you get both game engines!
> 
> OK?
> 
> ~Brian
>

Remove the trailing whitespace from line 31
with that it's OK awolk@

ps.
both dx1/dx2 force my dual monitor setup into mirror mode
which is a bit annoying but I don't think it's port related.



Re: NEW: games/returntotheroots

2017-12-24 Thread Adam Wolk
On Sun, Dec 24, 2017 at 10:01:58AM -0500, Brian Callahan wrote:
> 
> On 12/24/17 06:29, Adam Wolk wrote:
> > On Sat, Dec 23, 2017 at 08:50:11PM -0500, Brian Callahan wrote:
> > > Hi ports --
> > > 
> > > Attached is a new port, games/returntotheroots. Return to the Roots is an
> > > open source engine remake of The Settlers 2.
> > > 
> > > ---
> > > pkg/DESCR:
> > > Return To The Roots is a fan project which aims to renew the original
> > > The Settlers 2.
> > > 
> > > We aim to create new features such as a multiplayer mode via the
> > > Internet as well as support for modern hardware.
> > > 
> > > You will need an original copy of "The Settlers 2 Gold Edition" to play
> > > Return To The Roots.
> > > 
> > > The Settlers 2 is a 4X game similar to Civilization and Alpha Centauri.
> > > ---
> > > 
> > > This is taken from the head of upstream's development and rolled by me.
> > > Upstream claims that players normally use the head ("nightly builds")
> > > instead of the latest release, so it is probably best to go this route for
> > > multiplayer.
> > > 
> > > This requires a copy of The Settlers 2: Gold Edition. You can get it (on
> > > deep discount as of this writing) on GOG:
> > > https://www.gog.com/game/the_settlers_2_gold_edition
> > > 
> > > Also, slightly off-topic, I sent a port of freeserf, the first game in The
> > > Settlers series, to ports@ some time ago and heard nothing back. Would be
> > > nice to get both games in.
> > > 
> > I don't own the game files for the first one. Sorry.
> > 
> > > Works on amd64. Got nothing else that this would have a chance on.
> > > 
> > > OK?
> > > 
> > > ~Brian
> > > 
> > I tested on amd64 -current and it works fine. I have one gripe with the
> > port. The binaries are a bit of a mess:
> > 
> > fishtank$ grep bin pkg/PLIST
> > bin/RTTR/
> > @bin bin/RTTR/s-c_resample
> > @bin bin/RTTR/s25update
> > @bin bin/RTTR/sound-convert
> > bin/rttr.sh
> > @bin bin/s25client
> > @bin bin/s25edit
> > 
> > 
> > Is bin/rttr.sh even needed? It assumes bash is installed. I also never
> > saw a port dumping binaries to a new folder in /usr/local/bin.
> 
> As an experiment I deleted the bin/RTTR directory to see what would happen.
> And nothing happens. The game is happy without them.
> 
> So how about something like this (and the relevant PLIST changes)?
> # Remove unhelpful binaries and scripts from the package
> post-install:
>     @rm -rf ${PREFIX}/bin/RTTR ${PREFIX}/bin/rttr.sh
> 

I'm OK with removing rttr.sh. However RTTR directory has some specifics.

a) s25update

This port takes a long time to build, no point building that code just to
delete it. The CMake seems to have an existing option to not build
the auto updater:

https://github.com/Return-To-The-Roots/s25client/blob/master/CMakeLists.txt#L419

option(RTTR_BUILD_UPDATER "Build auto-updater. Not advised when changing the 
default paths as the updater is configured specifically for the official 
builds." ${RTTR_BUILD_UPDATER_DEF})

b) sound-convert and s-c_resample look like runtime dependencies. I found the 
sound-converter being
called on runtime when loading sound in this code snippet (comments are in 
german):

https://github.com/Return-To-The-Roots/s25client/blob/01cf1f7d586dde73ad3957eff1d9e8fcf594afae/src/Loader.cpp#L231

so I'm not sure they can just be removed with no consequences.

Apparently they have some settings for that:
 https://github.com/Return-To-The-Roots/s25client/issues/590
 https://github.com/Return-To-The-Roots/s25client/pull/591

both mention RTTR_LIBDIR & RTTR_DATADIR

> > Final gripe, the binary names are not obviously related to the port
> > name. Not sure how to handle that, rename them or mention what
> > they are used for in the readme?
> 
> I think the best way to handle this is two-pronged. First, people may do a
> quick search for the binary name, in that case it would be helpful to have
> the client binary name in the COMMENT (audio/mscore does this, for example)
> COMMENT =   open source engine remake of The Settlers 2 (s25client)
> 
> And a helpful line added to the end of pkg/DESCR:
> Comes with a game client (s25client) and map editor (s25edit).
> 

I like the readme & comment changes.

> ~Brian
> 
> > I tested by running s25client, didn't try other binaries.
> > 
> > Regards,
> > Adam
> 



Re: NEW: games/returntotheroots

2017-12-24 Thread Adam Wolk
On Sat, Dec 23, 2017 at 08:50:11PM -0500, Brian Callahan wrote:
> Hi ports --
> 
> Attached is a new port, games/returntotheroots. Return to the Roots is an
> open source engine remake of The Settlers 2.
> 
> ---
> pkg/DESCR:
> Return To The Roots is a fan project which aims to renew the original
> The Settlers 2.
> 
> We aim to create new features such as a multiplayer mode via the
> Internet as well as support for modern hardware.
> 
> You will need an original copy of "The Settlers 2 Gold Edition" to play
> Return To The Roots.
> 
> The Settlers 2 is a 4X game similar to Civilization and Alpha Centauri.
> ---
> 
> This is taken from the head of upstream's development and rolled by me.
> Upstream claims that players normally use the head ("nightly builds")
> instead of the latest release, so it is probably best to go this route for
> multiplayer.
> 
> This requires a copy of The Settlers 2: Gold Edition. You can get it (on
> deep discount as of this writing) on GOG:
> https://www.gog.com/game/the_settlers_2_gold_edition
> 
> Also, slightly off-topic, I sent a port of freeserf, the first game in The
> Settlers series, to ports@ some time ago and heard nothing back. Would be
> nice to get both games in.
> 

I don't own the game files for the first one. Sorry.

> Works on amd64. Got nothing else that this would have a chance on.
> 
> OK?
> 
> ~Brian
> 

I tested on amd64 -current and it works fine. I have one gripe with the
port. The binaries are a bit of a mess:

fishtank$ grep bin pkg/PLIST 
bin/RTTR/
@bin bin/RTTR/s-c_resample
@bin bin/RTTR/s25update
@bin bin/RTTR/sound-convert
bin/rttr.sh
@bin bin/s25client
@bin bin/s25edit


Is bin/rttr.sh even needed? It assumes bash is installed. I also never
saw a port dumping binaries to a new folder in /usr/local/bin.

Final gripe, the binary names are not obviously related to the port
name. Not sure how to handle that, rename them or mention what
they are used for in the readme?

I tested by running s25client, didn't try other binaries.

Regards,
Adam



Re: Develop OpenForum? Inspired by both what I use and also OpenBSD daily

2017-12-21 Thread Adam Wolk
On Tue, Dec 19, 2017 at 11:11:09AM -0700, cpb_p...@bennettconstruction.us wrote:
> I read about the problems with OpenBSD code reading and why it shut
> down.

I assume you are referring to #openbsd-daily code reads.

> Problems with the methods to study existing code and possibly submit
> diff's.
> 

I don't recall any problems with diffs? IRC is just a bad format for live code
reading in a group but I don't recall having any problems with diffs that
resulted from our reads on the channel.

> Right now I am using Mwforum, which doesn't seem to have any security
> problems,
> beyond the very rare trolls that actually register.
> 

I don't see how a forum would help with live code reading. When we discussed the
issue on IRC most people trended towards screen or terminal session sharing or
optionally audio/video streaming. I could see something like
http://bxr.su/OpenBSD/ working for live reads if it had a built in live chat and
allowed inline annotations that other people would see live.

> I have been using it to host LedgerSMB's forums.
> One of the really nice features it has is an addon to convert emails
> into posts.
> I have been doing this quite nicely for a good period of time. These
> emails go
> to read only boards.
> 
> The forum supports attachments, (diff's would be a good usage).
> It also supports addons to change the forum controls into other
> languages.
> I wrote the Spanish addon.
> 
> The forum is written in Perl and supports multiple databases, PostgreSQL
> preferred.
> It works in Perl or mod_perl.
> 
> The downside is that the author is no longer working on the project or
> accepting
> any diff's for the code.
> 

That doesn't sound encouraging.

> I would like help developing a port to make our own forum software that
> works with
> our httpd and also perl and mod_perl.
> 
> I would like to continue having the ability to post emails, such as
> ports@, tech@,
> misc@, etc.
> 
> Possibly also being able to send emails back for the existing mailing
> lists.
> 

I don't see the connection between that functionality and the code reads we did
on #openbsd-daily. Perhaps I misunderstand your intention?

> Personally, I would like to use PostgreSQL and Perl, although C would
> also be
> acceptable. Others would be better able to comment on the security of
> Perl vs C.
> 
> Is anyone already working on this or interested?
> I think this might be a good way to continue the code studying that was
> started.
> 
> 
> Thoughts?
> Chris Bennett
> 

If you want to discuss how to continue with the daily code reads, then
#openbsd-daily on IRC is a better place to have it.

Regarding the port of Mwforum, I personally have no need for it and would rather
not advocate anoyone hosting an unmaintained forum that isn't receiving security
updates. However I won't stop/discourage you from making a port.

Regards,
Adam



Re: [UPDATE] net/libgadu to 1.12.2

2017-12-01 Thread Adam Wolk
On Thu, Nov 23, 2017 at 02:54:05PM +0100, Frederic Cambus wrote:
> Hi ports@,
> 
> Here is a diff to update libgadu to 1.12.2.
> 
> This is a bugfix release, no need to bump anything.
> 
> Tested with net/ekg, net/pidgin, and x11/kde4/kopete without issues.
> 
> Comments? OK?
> 

I tested this on amd64 -current against the net/ekg client. Note I
no longer have an account so that was not a through test. I also
compared symbols of libgadu.so with the previous release to
confirm that a major bump is not needed. It looks good.

OK awolk@



Re: UPDATE: neovim-0.2.2

2017-12-01 Thread Adam Wolk
On Tue, Nov 28, 2017 at 09:45:09PM +, Edd Barrett wrote:
> Hi,
> 
> In the time we were porting neovim-0.2.1 there has been another release,
> so here it is.
> 
> I'll test this more thoroughly over the next couple of days.
> 
> Comments? OK?
> 
> (The py-neovim update still needs to go in too.)
> 

OK awolk@ tested both python2 & python3 with neovim 0.2.2

 33 ## Python 2 provider (optional)
 34   - INFO: Using: g:python_host_prog = "/usr/local/bin/python"
 35   - INFO: Executable: /usr/local/bin/python
 36   - INFO: Python2 version: 2.7.14
 37   - INFO: python-neovim version: 0.2.0
 38   - OK: Latest python-neovim is installed: 0.2.0
 39
 40 ## Python 3 provider (optional)
 41   - INFO: Using: g:python3_host_prog = "/usr/local/bin/python3"
 42   - INFO: Executable: /usr/local/bin/python3
 43   - INFO: Python3 version: 3.6.3
 44   - INFO: python3-neovim version: 0.2.0
 45   - OK: Latest python3-neovim is installed: 0.2.0



Re: [UPDATE] www/libwebsockets 2.3.0 => 2.4.1

2017-11-28 Thread Adam Wolk
On Mon, Nov 27, 2017 at 05:11:28PM +0100, Rafael Sadowski wrote:
> On Sat Nov 25, 2017 at 10:28:25PM +0100, Adam Wolk wrote:
> > On Sat, Nov 25, 2017 at 10:19:32PM +0100, Adam Wolk wrote:
> > > Hi ports@
> > > 
> > > bump for libwebsockets attached.
> > > 
> > > Port changes:
> > >  - LWS_WITHOUT_TEST_SERVER added and removing ssh
> > >test keys in post-install. This is needed to prevent
> > >the keys from being generated and installed. The
> > >issue has been reported upstream with a patch, the
> > >changes here are a workaround as I don't want to
> > >create the patches folder just for that reason.
> > > 
> > >https://github.com/warmcat/libwebsockets/issues/1108
> > >https://github.com/warmcat/libwebsockets/pull/1109
> > > 
> > >  - regenerated PLIST
> > >  - bump SO
> > > 
> > > Upstream changelog:
> > >  https://github.com/warmcat/libwebsockets/blob/v2.4-stable/changelog
> > > 
> > > tested on a -current amd64 snapshot compiled against a
> > > closed source koparo.com product.
> > > 
> > > Any feedback, OKs?
> > > 
> > > Regards,
> > > Adam
> > 
> > Right after sending the above I noticed there was a 2.4.1.
> > Here's a bumped diff for that patch release.
> > 
> 
> > Index: Makefile
> > ===
> > RCS file: /cvs/ports/www/libwebsockets/Makefile,v
> > retrieving revision 1.14
> > diff -u -p -r1.14 Makefile
> > --- Makefile13 Sep 2017 09:04:43 -  1.14
> > +++ Makefile25 Nov 2017 21:27:14 -
> > @@ -4,10 +4,10 @@ COMMENT = lightweight C library for fas
> >  
> >  GH_ACCOUNT =   warmcat
> >  GH_PROJECT =   libwebsockets
> > -V =2.3.0
> > +V =2.4.1
> >  GH_TAGNAME =   v$V
> 
> $V makes no sense to me
> 

good catch, updated diff with $V gone.

Index: Makefile
===
RCS file: /cvs/ports/www/libwebsockets/Makefile,v
retrieving revision 1.14
diff -u -p -r1.14 Makefile
--- Makefile13 Sep 2017 09:04:43 -  1.14
+++ Makefile28 Nov 2017 20:17:27 -
@@ -4,10 +4,9 @@ COMMENT =  lightweight C library for fas
 
 GH_ACCOUNT =   warmcat
 GH_PROJECT =   libwebsockets
-V =2.3.0
-GH_TAGNAME =   v$V
+GH_TAGNAME =   v2.4.1
 
-SHARED_LIBS =  websockets 8.0 # 11
+SHARED_LIBS =  websockets 9.0 # 12
 
 CATEGORIES =   www
 
@@ -22,8 +21,12 @@ WANTLIB +=   crypto m ssl z
 
 MODULES =  devel/cmake
 
-CONFIGURE_ARGS +=  -DLWS_WITHOUT_TESTAPPS=ON
+CONFIGURE_ARGS +=  -DLWS_WITHOUT_TESTAPPS=ON \
+   -DLWS_WITHOUT_TEST_SERVER=ON
 
 NO_TEST =  Yes
+
+post-install:
+   @rm -r ${WRKINST}/usr/local/share/libwebsockets-test-server
 
 .include 
Index: distinfo
===
RCS file: /cvs/ports/www/libwebsockets/distinfo,v
retrieving revision 1.12
diff -u -p -r1.12 distinfo
--- distinfo13 Sep 2017 09:04:43 -  1.12
+++ distinfo28 Nov 2017 20:17:27 -
@@ -1,2 +1,2 @@
-SHA256 (libwebsockets-2.3.0.tar.gz) = 
8IqCM8oYN2QLcrF5DM50HOSw/qqmtAj+KKMDy/BAj6E=
-SIZE (libwebsockets-2.3.0.tar.gz) = 3598002
+SHA256 (libwebsockets-2.4.1.tar.gz) = 
KUFL5PefarwOaq3M0JpNoPDEMeO1aR9Jas0IGuaoJAw=
+SIZE (libwebsockets-2.4.1.tar.gz) = 3773741
Index: pkg/PLIST
===
RCS file: /cvs/ports/www/libwebsockets/pkg/PLIST,v
retrieving revision 1.2
diff -u -p -r1.2 PLIST
--- pkg/PLIST   13 Sep 2017 09:04:43 -  1.2
+++ pkg/PLIST   28 Nov 2017 20:17:27 -
@@ -1,5 +1,6 @@
-@comment $OpenBSD: PLIST,v 1.2 2017/09/13 09:04:43 awolk Exp $
+@comment $OpenBSD$
 include/libwebsockets.h
+include/lws-plugin-ssh.h
 include/lws_config.h
 lib/cmake/
 lib/cmake/libwebsockets/


Re: [UPDATE] www/libwebsockets 2.3.0 => 2.4.0

2017-11-25 Thread Adam Wolk
On Sat, Nov 25, 2017 at 10:19:32PM +0100, Adam Wolk wrote:
> Hi ports@
> 
> bump for libwebsockets attached.
> 
> Port changes:
>  - LWS_WITHOUT_TEST_SERVER added and removing ssh
>test keys in post-install. This is needed to prevent
>the keys from being generated and installed. The
>issue has been reported upstream with a patch, the
>changes here are a workaround as I don't want to
>create the patches folder just for that reason.
> 
>https://github.com/warmcat/libwebsockets/issues/1108
>https://github.com/warmcat/libwebsockets/pull/1109
> 
>  - regenerated PLIST
>  - bump SO
> 
> Upstream changelog:
>  https://github.com/warmcat/libwebsockets/blob/v2.4-stable/changelog
> 
> tested on a -current amd64 snapshot compiled against a
> closed source koparo.com product.
> 
> Any feedback, OKs?
> 
> Regards,
> Adam

Right after sending the above I noticed there was a 2.4.1.
Here's a bumped diff for that patch release.

Index: Makefile
===
RCS file: /cvs/ports/www/libwebsockets/Makefile,v
retrieving revision 1.14
diff -u -p -r1.14 Makefile
--- Makefile13 Sep 2017 09:04:43 -  1.14
+++ Makefile25 Nov 2017 21:27:14 -
@@ -4,10 +4,10 @@ COMMENT = lightweight C library for fas
 
 GH_ACCOUNT =   warmcat
 GH_PROJECT =   libwebsockets
-V =2.3.0
+V =2.4.1
 GH_TAGNAME =   v$V
 
-SHARED_LIBS =  websockets 8.0 # 11
+SHARED_LIBS =  websockets 9.0 # 12
 
 CATEGORIES =   www
 
@@ -22,8 +22,12 @@ WANTLIB +=   crypto m ssl z
 
 MODULES =  devel/cmake
 
-CONFIGURE_ARGS +=  -DLWS_WITHOUT_TESTAPPS=ON
+CONFIGURE_ARGS +=  -DLWS_WITHOUT_TESTAPPS=ON \
+   -DLWS_WITHOUT_TEST_SERVER=ON
 
 NO_TEST =  Yes
+
+post-install:
+   @rm -r ${WRKINST}/usr/local/share/libwebsockets-test-server
 
 .include 
Index: distinfo
===
RCS file: /cvs/ports/www/libwebsockets/distinfo,v
retrieving revision 1.12
diff -u -p -r1.12 distinfo
--- distinfo13 Sep 2017 09:04:43 -  1.12
+++ distinfo25 Nov 2017 21:27:14 -
@@ -1,2 +1,2 @@
-SHA256 (libwebsockets-2.3.0.tar.gz) = 
8IqCM8oYN2QLcrF5DM50HOSw/qqmtAj+KKMDy/BAj6E=
-SIZE (libwebsockets-2.3.0.tar.gz) = 3598002
+SHA256 (libwebsockets-2.4.1.tar.gz) = 
KUFL5PefarwOaq3M0JpNoPDEMeO1aR9Jas0IGuaoJAw=
+SIZE (libwebsockets-2.4.1.tar.gz) = 3773741
Index: pkg/PLIST
===
RCS file: /cvs/ports/www/libwebsockets/pkg/PLIST,v
retrieving revision 1.2
diff -u -p -r1.2 PLIST
--- pkg/PLIST   13 Sep 2017 09:04:43 -  1.2
+++ pkg/PLIST   25 Nov 2017 21:27:14 -
@@ -1,5 +1,6 @@
-@comment $OpenBSD: PLIST,v 1.2 2017/09/13 09:04:43 awolk Exp $
+@comment $OpenBSD$
 include/libwebsockets.h
+include/lws-plugin-ssh.h
 include/lws_config.h
 lib/cmake/
 lib/cmake/libwebsockets/


[UPDATE] www/libwebsockets 2.3.0 => 2.4.0

2017-11-25 Thread Adam Wolk
Hi ports@

bump for libwebsockets attached.

Port changes:
 - LWS_WITHOUT_TEST_SERVER added and removing ssh
   test keys in post-install. This is needed to prevent
   the keys from being generated and installed. The
   issue has been reported upstream with a patch, the
   changes here are a workaround as I don't want to
   create the patches folder just for that reason.

   https://github.com/warmcat/libwebsockets/issues/1108
   https://github.com/warmcat/libwebsockets/pull/1109

 - regenerated PLIST
 - bump SO

Upstream changelog:
 https://github.com/warmcat/libwebsockets/blob/v2.4-stable/changelog

tested on a -current amd64 snapshot compiled against a
closed source koparo.com product.

Any feedback, OKs?

Regards,
Adam
Index: Makefile
===
RCS file: /cvs/ports/www/libwebsockets/Makefile,v
retrieving revision 1.14
diff -u -p -r1.14 Makefile
--- Makefile13 Sep 2017 09:04:43 -  1.14
+++ Makefile25 Nov 2017 21:11:40 -
@@ -4,10 +4,10 @@ COMMENT = lightweight C library for fas
 
 GH_ACCOUNT =   warmcat
 GH_PROJECT =   libwebsockets
-V =2.3.0
+V =2.4.0
 GH_TAGNAME =   v$V
 
-SHARED_LIBS =  websockets 8.0 # 11
+SHARED_LIBS =  websockets 9.0 # 12
 
 CATEGORIES =   www
 
@@ -22,8 +22,12 @@ WANTLIB +=   crypto m ssl z
 
 MODULES =  devel/cmake
 
-CONFIGURE_ARGS +=  -DLWS_WITHOUT_TESTAPPS=ON
+CONFIGURE_ARGS +=  -DLWS_WITHOUT_TESTAPPS=ON \
+   -DLWS_WITHOUT_TEST_SERVER=ON
 
 NO_TEST =  Yes
+
+post-install:
+   @rm -r ${WRKINST}/usr/local/share/libwebsockets-test-server
 
 .include 
Index: distinfo
===
RCS file: /cvs/ports/www/libwebsockets/distinfo,v
retrieving revision 1.12
diff -u -p -r1.12 distinfo
--- distinfo13 Sep 2017 09:04:43 -  1.12
+++ distinfo25 Nov 2017 21:11:40 -
@@ -1,2 +1,2 @@
-SHA256 (libwebsockets-2.3.0.tar.gz) = 
8IqCM8oYN2QLcrF5DM50HOSw/qqmtAj+KKMDy/BAj6E=
-SIZE (libwebsockets-2.3.0.tar.gz) = 3598002
+SHA256 (libwebsockets-2.4.0.tar.gz) = 
DcNVwfmmYLmGZ8xhb6TE/gjaza7/LVzJ905J6dSvLZU=
+SIZE (libwebsockets-2.4.0.tar.gz) = 3772428
Index: pkg/PLIST
===
RCS file: /cvs/ports/www/libwebsockets/pkg/PLIST,v
retrieving revision 1.2
diff -u -p -r1.2 PLIST
--- pkg/PLIST   13 Sep 2017 09:04:43 -  1.2
+++ pkg/PLIST   25 Nov 2017 21:11:40 -
@@ -1,5 +1,6 @@
-@comment $OpenBSD: PLIST,v 1.2 2017/09/13 09:04:43 awolk Exp $
+@comment $OpenBSD$
 include/libwebsockets.h
+include/lws-plugin-ssh.h
 include/lws_config.h
 lib/cmake/
 lib/cmake/libwebsockets/


[UPDATE] devel/zeal 0.3.1 => 0.4.0

2017-11-21 Thread Adam Wolk
Hi ports@

Attaching a bump to devel/zeal

port changes:
 - regen WANTLIB
 - drop Qt5Sql as devel/sqlite3 is now used directly
 - switch from qmake to cmake

upstream changelog:
https://github.com/zealdocs/zeal/releases/tag/v0.4.0

Feedback? OK's?

Regards,
Adam
Index: Makefile
===
RCS file: /cvs/ports/devel/zeal/Makefile,v
retrieving revision 1.6
diff -u -p -r1.6 Makefile
--- Makefile30 Jul 2017 14:54:28 -  1.6
+++ Makefile21 Nov 2017 16:05:24 -
@@ -7,8 +7,7 @@ COMMENT =   simple offline API documentati
 
 GH_ACCOUNT =   zealdocs
 GH_PROJECT =   zeal
-GH_TAGNAME =   v0.3.1
-REVISION = 1
+GH_TAGNAME =   v0.4.0
 
 CATEGORIES =   devel
 
@@ -19,12 +18,17 @@ MAINTAINER =    Adam Wolk 

Re: net/corebird 1.5.1 => 1.7.3

2017-11-20 Thread Adam Wolk
On Mon, Nov 20, 2017 at 10:36:17PM +0100, Adam Wolk wrote:
> On Mon, Nov 20, 2017 at 09:41:31PM +0100, Klemens Nanni wrote:
> > On Mon, Nov 20, 2017 at 08:15:13PM +0100, Adam Wolk wrote:
> > > On Wed, Nov 08, 2017 at 09:43:01PM +0100, Adam Wolk wrote:
> > > > Hi ports@
> > > > 
> > > > attaching a trivial bump for net/corebird
> > > > testedo n amd64 current.
> > > > 
> > > > Port changes:
> > > >  - dropping dependency on librest
> > > >  - dropping wantlib on soup-gnome
> > > >  - appdata changed it's dwelling place
> > > >  - setting HOME for tests and interactive
> > > >causing all tests to finally pass
> > > > 
> > > > Feedback, OK's?
> > > [...]
> > > 
> > > Corebird 1.7.3 has been recently released with support for 280 characters
> > > change on Twitter and a few other tweaks. Updated diff and upstream
> > > changelog below:
> > > [...]
> > Port builds and starts fine for me, I don't have a Twitter account to
> > actually test it, though. Still some feedback:
> > 
> > - DESCR's first sentence is missing a full stop
> > - HOMEPAGE supports HTTPS
> > - SEPARATE_BUILD=Yes can be set
> > - TEST and TEST_INTERACTIVE have inconsistent spacing around =
> > 
> 
> updated diff with your suggestions
> 
> 

the change has been committed, thanks Klemens & Rafael!



Re: net/corebird 1.5.1 => 1.7.3

2017-11-20 Thread Adam Wolk
On Mon, Nov 20, 2017 at 09:41:31PM +0100, Klemens Nanni wrote:
> On Mon, Nov 20, 2017 at 08:15:13PM +0100, Adam Wolk wrote:
> > On Wed, Nov 08, 2017 at 09:43:01PM +0100, Adam Wolk wrote:
> > > Hi ports@
> > > 
> > > attaching a trivial bump for net/corebird
> > > testedo n amd64 current.
> > > 
> > > Port changes:
> > >  - dropping dependency on librest
> > >  - dropping wantlib on soup-gnome
> > >  - appdata changed it's dwelling place
> > >  - setting HOME for tests and interactive
> > >causing all tests to finally pass
> > > 
> > > Feedback, OK's?
> > [...]
> > 
> > Corebird 1.7.3 has been recently released with support for 280 characters
> > change on Twitter and a few other tweaks. Updated diff and upstream
> > changelog below:
> > [...]
> Port builds and starts fine for me, I don't have a Twitter account to
> actually test it, though. Still some feedback:
> 
>   - DESCR's first sentence is missing a full stop
>   - HOMEPAGE supports HTTPS
>   - SEPARATE_BUILD=Yes can be set
>   - TEST and TEST_INTERACTIVE have inconsistent spacing around =
> 

updated diff with your suggestions


Index: Makefile
===
RCS file: /cvs/ports/net/corebird/Makefile,v
retrieving revision 1.17
diff -u -p -r1.17 Makefile
--- Makefile28 Oct 2017 18:31:41 -  1.17
+++ Makefile20 Nov 2017 21:18:37 -
@@ -2,15 +2,14 @@
 
 COMMENT =  native Gtk+ Twitter client
 
-V =1.5.1
+V =1.7.3
 DISTNAME = corebird-${V}
 MASTER_SITES = 
https://github.com/baedert/corebird/releases/download/${V}/
-REVISION =     2
 
 EXTRACT_SUFX = .tar.xz
 CATEGORIES =   net
 
-HOMEPAGE = http://corebird.baedert.org/
+HOMEPAGE = https://corebird.baedert.org/
 
 MAINTAINER =   Adam Wolk 
 
@@ -25,7 +24,7 @@ WANTLIB += gmodule-2.0 gobject-2.0 graph
 WANTLIB += gstreamer-1.0 gstvideo-1.0 gthread-2.0 gtk-3 harfbuzz
 WANTLIB += json-glib-1.0 lzma m orc-0.4 pango-1.0 pangocairo-1.0
 WANTLIB += pangoft2-1.0 pcre pixman-1 png pthread
-WANTLIB += rest-0.7 soup-2.4 soup-gnome-2.4 sqlite3 ${COMPILER_LIBCXX} xcb
+WANTLIB += soup-2.4 sqlite3 ${COMPILER_LIBCXX} xcb
 WANTLIB += xcb-render xcb-shm xml2 z
 
 MODULES =  devel/dconf \
@@ -40,7 +39,6 @@ LIB_DEPENDS = databases/sqlite3 \
devel/json-glib \
devel/libsoup>=2.4 \
devel/orc \
-   net/librest>=0.7.91 \
multimedia/gstreamer1/core \
multimedia/gstreamer1/plugins-base \
textproc/gspell \
@@ -52,5 +50,13 @@ RUN_DEPENDS =multimedia/gstreamer1/plu
 
 CONFIGURE_STYLE =  gnu
 USE_GMAKE =Yes
+
+TEST_ENV = HOME=${WRKDIR}/tests
+TEST_IS_INTERACTIVE =  x11
+
+SEPARATE_BUILD =   Yes
+
+pre-test:
+   mkdir -p ${WRKDIR}/tests/.config/corebird
 
 .include 
Index: distinfo
===
RCS file: /cvs/ports/net/corebird/distinfo,v
retrieving revision 1.10
diff -u -p -r1.10 distinfo
--- distinfo20 Jul 2017 23:22:15 -  1.10
+++ distinfo20 Nov 2017 21:18:37 -
@@ -1,2 +1,2 @@
-SHA256 (corebird-1.5.1.tar.xz) = 57ZBol4nnYJRygqfou8Rpd02TjaeH6VPVUWCTclt7aA=
-SIZE (corebird-1.5.1.tar.xz) = 676192
+SHA256 (corebird-1.7.3.tar.xz) = y/Qv3xhvXepHxdFx9qbdR6BdDnIaS/1uYbUwaeonjic=
+SIZE (corebird-1.7.3.tar.xz) = 705080
Index: pkg/DESCR
===
RCS file: /cvs/ports/net/corebird/pkg/DESCR,v
retrieving revision 1.1.1.1
diff -u -p -r1.1.1.1 DESCR
--- pkg/DESCR   2 Mar 2016 16:52:57 -   1.1.1.1
+++ pkg/DESCR   20 Nov 2017 21:18:37 -
@@ -1,4 +1,4 @@
-Corebird is a native Gtk+ Twitter client
+Corebird is a native Gtk+ Twitter client.
 
 Offers native support for multiple accounts, handles media (including
 uploads) and doesn't rescroll your timeline on new tweets.
Index: pkg/PLIST
===
RCS file: /cvs/ports/net/corebird/pkg/PLIST,v
retrieving revision 1.6
diff -u -p -r1.6 PLIST
--- pkg/PLIST   20 Jul 2017 23:22:15 -  1.6
+++ pkg/PLIST   20 Nov 2017 21:18:37 -
@@ -1,8 +1,6 @@
 @comment $OpenBSD: PLIST,v 1.6 2017/07/20 23:22:15 awolk Exp $
 @bin bin/corebird
 @man man/man1/corebird.1
-share/appdata/
-share/appdata/org.baedert.corebird.appdata.xml
 share/applications/org.baedert.corebird.desktop
 share/dbus-1/services/org.baedert.corebird.service
 share/glib-2.0/schemas/org.baedert.corebird.gschema.xml
@@ -63,3 +61,5 @@ share/locale/zh_TW/LC_MESSAGES/corebird.
 

Re: net/corebird 1.5.1 => 1.7.3

2017-11-20 Thread Adam Wolk
On Wed, Nov 08, 2017 at 09:43:01PM +0100, Adam Wolk wrote:
> Hi ports@
> 
> attaching a trivial bump for net/corebird
> testedo n amd64 current.
> 
> Port changes:
>  - dropping dependency on librest
>  - dropping wantlib on soup-gnome
>  - appdata changed it's dwelling place
>  - setting HOME for tests and interactive
>causing all tests to finally pass
> 
> Feedback, OK's?
> 
> Regards,
> Adam
> 
> upstream changelog:
> ---
> 
> Version 1.7.2
> 
> Changelog:
>  - Fix window cancel button label mixup
>  - Fix potential crash when composing a tweet (#779)
>  - Fix abort when trying to detect emoji support (#781)
>  - Fix spell checking
>  - Update translations
> 
> Release 1.7.1
> 
> Changelog:
>  - Fixed a small problem with the appdata file in 1.7
> 
> Version 1.7
> 
> Changelog:
>  - Hashtags and Mentions in profile desriptions are now clickable
>  - The mention-completion when composing a new tweet now fetches
>unknown users from the twitter server
>  - Videos larger than the screen size are now getting scaled down
>while playing
>  - Profiles now indicate when an account is suspended
>  - Profiles handle protected accounts better when trying to
>access followers/following users, etc.
>  - The compose window now allows tweets with just media
>attached but not text
>  - Improve the hashtag/mention/link detection when composing a new tweet
>  - The compose dialog now shows an emoji chooser. This is
>only available starting GTK+ 3.22.19 and the emojis will only be
>rendered with color if you have the needed cairo version
>installed, as well as an emoji font.
>  - Fixed a bug that led to wrong Direct Message info being
>inserted into the database
>  - Fixed a bug that resulted in broken files when downloading
>instagram images
> 
> Version 1.6 (2017-08-25)
> 
> Changelog:
>   - Dependencies: GTK+ 3.20 is now required, and librest is no longer
> a dependency
>   - Facelift for the tweet info page
>   - Lots of stability fixes
>   - The compose window now starts uploading images as soon as they
> are selected
> ---

Corebird 1.7.3 has been recently released with support for 280 characters
change on Twitter and a few other tweaks. Updated diff and upstream
changelog below:

upstream changelog
---

Version 1.7.3

Changelog:
  - Increase maximum tweet length to 280 characters
  - Increase maximum name length to 50 characters and improve
certain parts of the UI to cope better with longer names
  - Fix the emoji button not showing up in the compose window
  - Update translations

---
Index: Makefile
===
RCS file: /cvs/ports/net/corebird/Makefile,v
retrieving revision 1.17
diff -u -p -r1.17 Makefile
--- Makefile28 Oct 2017 18:31:41 -  1.17
+++ Makefile20 Nov 2017 19:14:24 -
@@ -2,10 +2,9 @@
 
 COMMENT =  native Gtk+ Twitter client
 
-V =1.5.1
+V =1.7.3
 DISTNAME = corebird-${V}
 MASTER_SITES = 
https://github.com/baedert/corebird/releases/download/${V}/
-REVISION = 2
 
 EXTRACT_SUFX = .tar.xz
 CATEGORIES =   net
@@ -25,7 +24,7 @@ WANTLIB += gmodule-2.0 gobject-2.0 graph
 WANTLIB += gstreamer-1.0 gstvideo-1.0 gthread-2.0 gtk-3 harfbuzz
 WANTLIB += json-glib-1.0 lzma m orc-0.4 pango-1.0 pangocairo-1.0
 WANTLIB += pangoft2-1.0 pcre pixman-1 png pthread
-WANTLIB += rest-0.7 soup-2.4 soup-gnome-2.4 sqlite3 ${COMPILER_LIBCXX} xcb
+WANTLIB += soup-2.4 sqlite3 ${COMPILER_LIBCXX} xcb
 WANTLIB += xcb-render xcb-shm xml2 z
 
 MODULES =  devel/dconf \
@@ -40,7 +39,6 @@ LIB_DEPENDS = databases/sqlite3 \
devel/json-glib \
devel/libsoup>=2.4 \
devel/orc \
-   net/librest>=0.7.91 \
multimedia/gstreamer1/core \
multimedia/gstreamer1/plugins-base \
textproc/gspell \
@@ -52,5 +50,11 @@ RUN_DEPENDS =multimedia/gstreamer1/plu
 
 CONFIGURE_STYLE =  gnu
 USE_GMAKE =Yes
+
+TEST_ENV   =   HOME=${WRKDIR}/tests
+TEST_IS_INTERACTIVE=   x11
+
+pre-test:
+   mkdir -p ${WRKDIR}/tests/.config/corebird
 
 .include 
Index: distinfo
===
RCS file: /cvs/ports/net/corebird/distinfo,v
retrieving revision 1.10
diff -u -p -r1.10 distinfo
--- distinfo20 Jul 2017 23:22:15 -  1.10
+++ distinfo20 Nov 2017 19:14:24 -
@@ -1,2 +1,2 @@
-SHA256 (corebird-1.5.1.tar.xz) = 57ZBol4nnYJRygqfou8Rpd02TjaeH6VPVUWCTclt7aA=
-SIZE (corebird-1.5.1.tar.xz) = 676192
+SHA256 (corebird-1.7.3.tar.xz) = y/Qv3xhvXepHxdFx9qbdR6BdDn

Re: PostgreSQL security update

2017-11-13 Thread Adam Wolk
On Mon, Nov 13, 2017 at 05:36:11PM +0100, Pierre-Emmanuel André wrote:
> On Sun, Nov 12, 2017 at 03:31:43PM +0100, Adam Wolk wrote:
> > On Fri, Nov 10, 2017 at 03:31:36PM +0100, Pierre-Emmanuel André wrote:
> > > Hi,
> > > 
> > > Small diff to update PostgreSQL to it's latest version (9.6.6)
> > > Release notes: 
> > > https://www.postgresql.org/docs/9.6/static/release-9-6-6.html
> > > (3 CVE fixed)
> > > 
> > > Comments, ok ?
> > > 
> > > Regards,
> > > 
> > > 
> > > ps: 10.1 is coming but need more tests ;)
> > 
> > This is true for 9.6.5 also but:
> > 
> > trailing whitespace in Makefile (line 23 last entry in SHARED_LIBS)
> > trailing whitespace in pkg/README-server (line 154)
> > 
> > pkg/README-server line 138 & 173 longer than 80 characters but I think
> > those two can just be let slip.
> > 
> > portcheck complains on SUBST_VARS for pkg/README-server and
> > pkg/postgresql.rc but I'm also willing to just slip those as they
> > are not a new addition.
> > 
> > all in all it's OK awolk@ for the update, I tested on amd64 snap with a
> > 3190 MB database dump of my production server and everything
> > seems to work fine. Considering this is a security patch I don't
> > mind it being committed without any of the above nitpicks being
> > addressed.
> >
> 
> Thanks for your review. I will correct them for the next release.
> Please note that this diff is valid for OpenBSD 6.2 too.
> No objections for the commit to -stable ?
> 
> Regards,
> 

unfortunately my server is still running 6.1 so I can't test against it.

I don't see anything that should break on 6.2 and have nothing against it
being committed but that said I personally only tested against -current.

Regards,
Adam



Re: PostgreSQL security update

2017-11-12 Thread Adam Wolk
On Fri, Nov 10, 2017 at 03:31:36PM +0100, Pierre-Emmanuel André wrote:
> Hi,
> 
> Small diff to update PostgreSQL to it's latest version (9.6.6)
> Release notes: https://www.postgresql.org/docs/9.6/static/release-9-6-6.html
> (3 CVE fixed)
> 
> Comments, ok ?
> 
> Regards,
> 
> 
> ps: 10.1 is coming but need more tests ;)

This is true for 9.6.5 also but:

trailing whitespace in Makefile (line 23 last entry in SHARED_LIBS)
trailing whitespace in pkg/README-server (line 154)

pkg/README-server line 138 & 173 longer than 80 characters but I think
those two can just be let slip.

portcheck complains on SUBST_VARS for pkg/README-server and
pkg/postgresql.rc but I'm also willing to just slip those as they
are not a new addition.

all in all it's OK awolk@ for the update, I tested on amd64 snap with a
3190 MB database dump of my production server and everything
seems to work fine. Considering this is a security patch I don't
mind it being committed without any of the above nitpicks being
addressed.

Regards,
Adam



Re: [UPDATE] games/yquake2 7.01 => 7.02

2017-11-09 Thread Adam Wolk
On Thu, Nov 09, 2017 at 11:04:18AM +0100, Adam Wolk wrote:
> Hi ports@
> 
> Upgrading our yquake2 from 7.01 to 7.02
> 

updated diff with the missing files folder
Index: Makefile
===
RCS file: /cvs/ports/games/yquake2/Makefile,v
retrieving revision 1.9
diff -u -p -r1.9 Makefile
--- Makefile20 Jul 2017 23:33:46 -  1.9
+++ Makefile9 Nov 2017 15:01:45 -
@@ -4,7 +4,7 @@ ONLY_FOR_ARCHS= i386 amd64 sparc64
 
 COMMENT=   Yamagi Quake II
 N= yquake2
-V= 7.01
+V= 7.02
 PKGNAME=   ${N}-${V}
 DISTNAME=  quake2-${V}
 CATEGORIES=games
@@ -31,14 +31,16 @@ MAKE_FLAGS = config WITH_SYSTEMWIDE=yes 
 do-install:
${INSTALL_DATA_DIR} ${PREFIX}/share/${N}
${INSTALL_PROGRAM} ${WRKBUILD}/release/{quake2,q2ded} \
-   ${PREFIX}/bin/
+   ${PREFIX}/share/${N}/
${INSTALL_DATA_DIR} ${PREFIX}/share/${N}/baseq2
${INSTALL_PROGRAM} ${WRKBUILD}/release/baseq2/game.so \
${PREFIX}/share/${N}/baseq2/
${INSTALL_DATA_DIR} ${PREFIX}/lib/${N}
${INSTALL_PROGRAM} ${WRKBUILD}/release/ref_gl1.so \
-   ${PREFIX}/lib/${N}/
+   ${PREFIX}/share/${N}/
${INSTALL_PROGRAM} ${WRKBUILD}/release/ref_gl3.so \
-   ${PREFIX}/lib/${N}/
+   ${PREFIX}/share/${N}/
+   ${SUBST_CMD} -m 555 -c ${FILESDIR}/quake2.sh ${WRKSRC}/quake2
+   ${INSTALL_SCRIPT} ${WRKSRC}/quake2 ${PREFIX}/bin/
 
 .include 
Index: distinfo
===
RCS file: /cvs/ports/games/yquake2/distinfo,v
retrieving revision 1.4
diff -u -p -r1.4 distinfo
--- distinfo20 Jul 2017 23:33:46 -  1.4
+++ distinfo9 Nov 2017 15:01:45 -
@@ -1,2 +1,2 @@
-SHA256 (quake2-7.01.tar.xz) = ly93hnCTs8Fi37KMqTBYXLcdG3sYPWdthlnS/JcTfvc=
-SIZE (quake2-7.01.tar.xz) = 1778212
+SHA256 (quake2-7.02.tar.xz) = qe29vbXRP1abqr4DRtymPcP/pQwk3GeViDnoCuQBwN4=
+SIZE (quake2-7.02.tar.xz) = 1787092
Index: files/quake2.sh
===
RCS file: files/quake2.sh
diff -N files/quake2.sh
--- /dev/null   1 Jan 1970 00:00:00 -
+++ files/quake2.sh 9 Nov 2017 15:01:45 -
@@ -0,0 +1,4 @@
+#!/bin/sh
+cd ${TRUEPREFIX}/share/yquake2
+exec ${TRUEPREFIX}/share/yquake2/quake2
+
Index: patches/patch-Makefile
===
RCS file: patches/patch-Makefile
diff -N patches/patch-Makefile
--- patches/patch-Makefile  20 Jul 2017 23:33:46 -  1.1
+++ /dev/null   1 Jan 1970 00:00:00 -
@@ -1,16 +0,0 @@
-$OpenBSD: patch-Makefile,v 1.1 2017/07/20 23:33:46 awolk Exp $
-
-Index: Makefile
 Makefile.orig
-+++ Makefile
-@@ -251,8 +251,8 @@ endif
- CFLAGS += -fvisibility=hidden
- LDFLAGS += -fvisibility=hidden
- 
--ifneq ($(YQ2_OSTYPE), Darwin)
--# for some reason the OSX linker doesn't support this
-+ifneq ($(YQ2_OSTYPE), $(filter $(YQ2_OSTYPE), Darwin, OpenBSD))
-+# for some reason the OSX & OpenBSD linker doesn't support this
- LDFLAGS += -Wl,--no-undefined
- endif
- 
Index: patches/patch-src_backends_generic_vid_c
===
RCS file: patches/patch-src_backends_generic_vid_c
diff -N patches/patch-src_backends_generic_vid_c
--- patches/patch-src_backends_generic_vid_c20 Jul 2017 23:33:46 -  
1.1
+++ /dev/null   1 Jan 1970 00:00:00 -
@@ -1,14 +0,0 @@
-$OpenBSD: patch-src_backends_generic_vid_c,v 1.1 2017/07/20 23:33:46 awolk Exp 
$
-
-Index: src/backends/generic/vid.c
 src/backends/generic/vid.c.orig
-+++ src/backends/generic/vid.c
-@@ -342,7 +342,7 @@ VID_LoadRefresh(void)
-   Com_Printf("- refresher initialization -\n");
- 
-   snprintf(reflib_name, sizeof(reflib_name), "ref_%s.%s", 
vid_renderer->string, lib_ext);
--  snprintf(reflib_path, sizeof(reflib_path), "%s%s", Sys_GetBinaryDir(), 
reflib_name);
-+  snprintf(reflib_path, sizeof(reflib_path), "/usr/local/lib/yquake2/%s", 
reflib_name);
- 
-   Com_Printf("LoadLibrary(%s)\n", reflib_name);
-   GetRefAPI = Sys_LoadLibrary(reflib_path, "GetRefAPI", &reflib_handle);
Index: pkg/PLIST
===
RCS file: /cvs/ports/games/yquake2/pkg/PLIST,v
retrieving revision 1.3
diff -u -p -r1.3 PLIST
--- pkg/PLIST   20 Jul 2017 23:33:46 -  1.3
+++ pkg/PLIST   9 Nov 2017 15:01:45 -
@@ -1,13 +1,13 @@
 @comment $OpenBSD: PLIST,v 1.3 2017/07/20 23:33:46 awolk Exp $
 @newgroup _yquake2:779
 @newuser _yquake2:779:_yquake2:daemon:Yamagi Quake II 
Server:/var/yquake2:/sbin/nologin
-@bin bin/q2ded
-@bin bin/quake2
-lib/yquake2/
-lib/yquake2/ref_gl1.so
-lib/yquake2/ref_gl3.so
+bin/quake2
+@bin share/yquake2/q2ded
+@bin share/yquake2/quake2
 share/doc/pkg-readmes/${FULLPKGNAME}
 share/yquake2/
+share/yqua

[UPDATE] games/yquake2 7.01 => 7.02

2017-11-09 Thread Adam Wolk
Hi ports@

Upgrading our yquake2 from 7.01 to 7.02

Port changes:
 - bump version
 - everything now lives under /usr/local/share/yquake2
   this has been asked by upstream and outlined in
   their porting guide that was created after
   I started complaining that they want relative
   paths between the binaries:

   
https://github.com/yquake2/yquake2/blob/master/stuff/packaging.md#where-you-should-put-the-executables

 - adding a new wrapper for the quake2 binary
   to cd into the port directory before executing it
   as we provide no way for a binary to get its executable
   directory
 - drop the patch that hardcoded the executable directory
   upstream accounted for us not supporting that feature
   and defaults to returning cwd (./) instead of bailing
   out with an error
 - modified the q2ded rc.d script to account for the need
   of changing the working directory before starting the
   server
 - note I am not providing a separate wrapper for q2ded
   in /usr/bin on purpose - I don't see a need for running
   it outside of the rc.d scripts for any reasons other
   than debugging and that's rare enough that people can
   just cd to the game folder themselves
 - we can drop the Makefile compilation flag diff as
   upstream took our patch

Upstream changelog:

Quake II 7.01 to 7.02:
- Fix several corner cases regarding render library loading. The game
  should now always fall back to the OpenGL 1.4 renderer if the new
  OpenGL 3.2 renderer can't be initialized. Also the game aborts if no
  useable OpenGL implementation exists.
- Refactor the search path code. This should fix several bugs with
  Quake II writing to the wrong directories or being unable to find
  some / all assets.
- Reimplement portable binaries. If called with the -portable command
  line option Quake II saves all data (configs, savegames, screenshorts
  etc.) into it's systemwide installation directory and not users home
  directory. In contrast to the old implementation on Windows stdout.txt
  contains all output, the first lines are no longer missing.
- vid_fullscreen set to 1 now keeps the desktops resolution. Set it to 2
  to change the resolution.
- Instead of a list with precalculated FOV values the video menu now
  shows a slider with possible values from 60 to 120. Horplus is now
  always enabled, set the horplus cvar to 0 to disable it.
- The game is now able to hold the requested framerate (either by the
  vsync or the gl_maxfps cvar) with an accuracy of about +/- 1% as long
  as the hardware is fast enough.  The framecounter was reimplemented
  to be much more precise.
- Fix misspredictions if an original client running on Win32 connects
to a Yamagi Quake II server running on Linux/i386.

Frags? OK's?
Index: Makefile
===
RCS file: /cvs/ports/games/yquake2/Makefile,v
retrieving revision 1.9
diff -u -p -r1.9 Makefile
--- Makefile20 Jul 2017 23:33:46 -  1.9
+++ Makefile9 Nov 2017 09:53:43 -
@@ -4,7 +4,7 @@ ONLY_FOR_ARCHS= i386 amd64 sparc64
 
 COMMENT=   Yamagi Quake II
 N= yquake2
-V= 7.01
+V= 7.02
 PKGNAME=   ${N}-${V}
 DISTNAME=  quake2-${V}
 CATEGORIES=games
@@ -31,14 +31,16 @@ MAKE_FLAGS = config WITH_SYSTEMWIDE=yes 
 do-install:
${INSTALL_DATA_DIR} ${PREFIX}/share/${N}
${INSTALL_PROGRAM} ${WRKBUILD}/release/{quake2,q2ded} \
-   ${PREFIX}/bin/
+   ${PREFIX}/share/${N}/
${INSTALL_DATA_DIR} ${PREFIX}/share/${N}/baseq2
${INSTALL_PROGRAM} ${WRKBUILD}/release/baseq2/game.so \
${PREFIX}/share/${N}/baseq2/
${INSTALL_DATA_DIR} ${PREFIX}/lib/${N}
${INSTALL_PROGRAM} ${WRKBUILD}/release/ref_gl1.so \
-   ${PREFIX}/lib/${N}/
+   ${PREFIX}/share/${N}/
${INSTALL_PROGRAM} ${WRKBUILD}/release/ref_gl3.so \
-   ${PREFIX}/lib/${N}/
+   ${PREFIX}/share/${N}/
+   ${SUBST_CMD} -m 555 -c ${FILESDIR}/quake2.sh ${WRKSRC}/quake2
+   ${INSTALL_SCRIPT} ${WRKSRC}/quake2 ${PREFIX}/bin/
 
 .include 
Index: distinfo
===
RCS file: /cvs/ports/games/yquake2/distinfo,v
retrieving revision 1.4
diff -u -p -r1.4 distinfo
--- distinfo20 Jul 2017 23:33:46 -  1.4
+++ distinfo9 Nov 2017 09:53:43 -
@@ -1,2 +1,2 @@
-SHA256 (quake2-7.01.tar.xz) = ly93hnCTs8Fi37KMqTBYXLcdG3sYPWdthlnS/JcTfvc=
-SIZE (quake2-7.01.tar.xz) = 1778212
+SHA256 (quake2-7.02.tar.xz) = qe29vbXRP1abqr4DRtymPcP/pQwk3GeViDnoCuQBwN4=
+SIZE (quake2-7.02.tar.xz) = 1787092
Index: patches/patch-Makefile
===
RCS file: patches/patch-Makefile
diff -N patches/patch-Makefile
--- patches/patch-Makefile  20 Jul 2017 23:33:46 -  1.1
+++ /dev/null   1 Jan 1970 00:00:00 -
@@ -1,16 +0,0 @@
-$OpenBSD: patch-Makefile,v 1.1 2017/07/20 23:33:46 awolk Exp $
-
-Index: Makefile
 Makefile.orig
-+++ Makefi

net/corebird 1.5.1 => 1.7.2

2017-11-08 Thread Adam Wolk
Hi ports@

attaching a trivial bump for net/corebird
testedo n amd64 current.

Port changes:
 - dropping dependency on librest
 - dropping wantlib on soup-gnome
 - appdata changed it's dwelling place
 - setting HOME for tests and interactive
   causing all tests to finally pass

Feedback, OK's?

Regards,
Adam

upstream changelog:
---

Version 1.7.2

Changelog:
 - Fix window cancel button label mixup
 - Fix potential crash when composing a tweet (#779)
 - Fix abort when trying to detect emoji support (#781)
 - Fix spell checking
 - Update translations

Release 1.7.1

Changelog:
 - Fixed a small problem with the appdata file in 1.7

Version 1.7

Changelog:
 - Hashtags and Mentions in profile desriptions are now clickable
 - The mention-completion when composing a new tweet now fetches
   unknown users from the twitter server
 - Videos larger than the screen size are now getting scaled down
   while playing
 - Profiles now indicate when an account is suspended
 - Profiles handle protected accounts better when trying to
   access followers/following users, etc.
 - The compose window now allows tweets with just media
   attached but not text
 - Improve the hashtag/mention/link detection when composing a new tweet
 - The compose dialog now shows an emoji chooser. This is
   only available starting GTK+ 3.22.19 and the emojis will only be
   rendered with color if you have the needed cairo version
   installed, as well as an emoji font.
 - Fixed a bug that led to wrong Direct Message info being
   inserted into the database
 - Fixed a bug that resulted in broken files when downloading
   instagram images

Version 1.6 (2017-08-25)

Changelog:
  - Dependencies: GTK+ 3.20 is now required, and librest is no longer
a dependency
  - Facelift for the tweet info page
  - Lots of stability fixes
  - The compose window now starts uploading images as soon as they
are selected
---
Index: Makefile
===
RCS file: /cvs/ports/net/corebird/Makefile,v
retrieving revision 1.17
diff -u -p -r1.17 Makefile
--- Makefile28 Oct 2017 18:31:41 -  1.17
+++ Makefile8 Nov 2017 20:32:44 -
@@ -2,10 +2,9 @@
 
 COMMENT =  native Gtk+ Twitter client
 
-V =1.5.1
+V =1.7.2
 DISTNAME = corebird-${V}
 MASTER_SITES = 
https://github.com/baedert/corebird/releases/download/${V}/
-REVISION = 2
 
 EXTRACT_SUFX = .tar.xz
 CATEGORIES =   net
@@ -25,7 +24,7 @@ WANTLIB += gmodule-2.0 gobject-2.0 graph
 WANTLIB += gstreamer-1.0 gstvideo-1.0 gthread-2.0 gtk-3 harfbuzz
 WANTLIB += json-glib-1.0 lzma m orc-0.4 pango-1.0 pangocairo-1.0
 WANTLIB += pangoft2-1.0 pcre pixman-1 png pthread
-WANTLIB += rest-0.7 soup-2.4 soup-gnome-2.4 sqlite3 ${COMPILER_LIBCXX} xcb
+WANTLIB += soup-2.4 sqlite3 ${COMPILER_LIBCXX} xcb
 WANTLIB += xcb-render xcb-shm xml2 z
 
 MODULES =  devel/dconf \
@@ -40,7 +39,6 @@ LIB_DEPENDS = databases/sqlite3 \
devel/json-glib \
devel/libsoup>=2.4 \
devel/orc \
-   net/librest>=0.7.91 \
multimedia/gstreamer1/core \
multimedia/gstreamer1/plugins-base \
textproc/gspell \
@@ -52,5 +50,11 @@ RUN_DEPENDS =multimedia/gstreamer1/plu
 
 CONFIGURE_STYLE =  gnu
 USE_GMAKE =Yes
+
+TEST_ENV   =   HOME=${WRKDIR}/tests
+TEST_IS_INTERACTIVE=   x11
+
+pre-test:
+   mkdir -p ${WRKDIR}/tests/.config/corebird
 
 .include 
Index: distinfo
===
RCS file: /cvs/ports/net/corebird/distinfo,v
retrieving revision 1.10
diff -u -p -r1.10 distinfo
--- distinfo20 Jul 2017 23:22:15 -  1.10
+++ distinfo8 Nov 2017 20:32:44 -
@@ -1,2 +1,2 @@
-SHA256 (corebird-1.5.1.tar.xz) = 57ZBol4nnYJRygqfou8Rpd02TjaeH6VPVUWCTclt7aA=
-SIZE (corebird-1.5.1.tar.xz) = 676192
+SHA256 (corebird-1.7.2.tar.xz) = mrGRWZ+fcALBwUN52nUquDT7QzJm2yYnnz6jSZN5grE=
+SIZE (corebird-1.7.2.tar.xz) = 704696
Index: pkg/PLIST
===
RCS file: /cvs/ports/net/corebird/pkg/PLIST,v
retrieving revision 1.6
diff -u -p -r1.6 PLIST
--- pkg/PLIST   20 Jul 2017 23:22:15 -  1.6
+++ pkg/PLIST   8 Nov 2017 20:32:44 -
@@ -1,8 +1,6 @@
 @comment $OpenBSD: PLIST,v 1.6 2017/07/20 23:22:15 awolk Exp $
 @bin bin/corebird
 @man man/man1/corebird.1
-share/appdata/
-share/appdata/org.baedert.corebird.appdata.xml
 share/applications/org.baedert.corebird.desktop
 share/dbus-1/services/org.baedert.corebird.service
 share/glib-2.0/schemas/org.baedert.corebird.gschema.xml
@@ -63,3 +61,5 @@ share/locale/zh_TW/LC_MESSAGES/corebird.
 @unexec-delete %D/bin/update-desktop-database
 @exec %D/bin/gtk-update-icon-cache -q -t %D/share/icons/hicolor
 @unexec-delete %D/bin/gtk-update-icon-cach

Re: UPDATE: games/openrct2 0.1.0 => 0.1.1

2017-09-09 Thread Adam Wolk
On Mon, Aug 28, 2017 at 10:56:05AM -0400, Brian Callahan wrote:
> Hi ports --
> 
> Attached is an update to games/openrct2. It is mostly a bugfix update.
> Upstream now offers the choice to have a static or shared libopenrct2, which
> it seems that the default cmake flags on OpenBSD gets us a static library.
> It doesn't matter much to me either way, as I can't imagine anything ever
> linking to libopenrct2 except for the openrct2 binary. The static library
> does simply the port a tiny bit on the plus side.
> 
> Works well here.
> 
> OK?
> 
> ~Brian
> 

OK awolk@

tested on amd64 -current (sept 9th) using a save from the previous
version.



[UPDATE] www/libwebsockets 2.2.2 => 2.3.0

2017-08-19 Thread Adam Wolk
Hi ports@,

trivial bump to libwebsockets.

upstream changelogs:
---
v2.3.0
==

 - ESP32 OpenSSL support for client and server

 - ESP32 4 x WLAN credential slots may be configured

 - Libevent event loop support

 - SOCKS5 proxy support

 - lws_meta protocol for websocket connection multiplexing

 - lws_vhost_destroy() added... allows dynamic removal of listening
   vhosts.  Vhosts with shared listen sockets adopt the listen socket
   automatically if the owner is destroyed.

 - IPv6 on Windows

 - Improved CGI handling suitable for general CGI scripting, eg, PHP

 - Convert even the "old style" test servers to use statically included
   plugin sources

 - LWS_WITH_STATS cmake option dumps resource usage and timing information
   every few seconds to debug log, including latency information about
   delay from asking for writeable callback to getting it

 - Large (> 2GB) files may be served

 - LWS_WITH_HTTP_PROXY Cmake option adds proxying mounts

 - Workaround for libev build by disabling -Werror on the test app

- HTTP2 support disabled since no way to serve websockets on it

+++
Code changes:

https://github.com/warmcat/libwebsockets/compare/v2.2.2...v2.3.0
---

tested on a -current amd64 snapshot compiled against a
closed source koparo.com product.

Any feedback, OKs?

Regards,
Adam
Index: Makefile
===
RCS file: /cvs/ports/www/libwebsockets/Makefile,v
retrieving revision 1.13
diff -u -p -r1.13 Makefile
--- Makefile23 Jul 2017 21:43:45 -  1.13
+++ Makefile19 Aug 2017 17:46:07 -
@@ -4,10 +4,10 @@ COMMENT = lightweight C library for fas
 
 GH_ACCOUNT =   warmcat
 GH_PROJECT =   libwebsockets
-V =2.2.2
+V =2.3.0
 GH_TAGNAME =   v$V
 
-SHARED_LIBS =  websockets 7.0 # 10
+SHARED_LIBS =  websockets 8.0 # 11
 
 CATEGORIES =   www
 
Index: distinfo
===
RCS file: /cvs/ports/www/libwebsockets/distinfo,v
retrieving revision 1.11
diff -u -p -r1.11 distinfo
--- distinfo23 Jul 2017 21:43:45 -  1.11
+++ distinfo19 Aug 2017 17:46:07 -
@@ -1,2 +1,2 @@
-SHA256 (libwebsockets-2.2.2.tar.gz) = 
44p9mB/oe5qGsfHN3R+eBi04lsSI5rKjLLcA1MlZ7Lo=
-SIZE (libwebsockets-2.2.2.tar.gz) = 3548981
+SHA256 (libwebsockets-2.3.0.tar.gz) = 
8IqCM8oYN2QLcrF5DM50HOSw/qqmtAj+KKMDy/BAj6E=
+SIZE (libwebsockets-2.3.0.tar.gz) = 3598002
Index: pkg/PLIST
===
RCS file: /cvs/ports/www/libwebsockets/pkg/PLIST,v
retrieving revision 1.1.1.1
diff -u -p -r1.1.1.1 PLIST
--- pkg/PLIST   12 Nov 2015 12:25:51 -  1.1.1.1
+++ pkg/PLIST   19 Aug 2017 17:46:07 -
@@ -1,6 +1,13 @@
 @comment $OpenBSD: PLIST,v 1.1.1.1 2015/11/12 12:25:51 jca Exp $
 include/libwebsockets.h
 include/lws_config.h
+lib/cmake/
+lib/cmake/libwebsockets/
+lib/cmake/libwebsockets/LibwebsocketsConfig.cmake
+lib/cmake/libwebsockets/LibwebsocketsConfigVersion.cmake
+lib/cmake/libwebsockets/LibwebsocketsTargets${MODCMAKE_BUILD_SUFFIX}
+lib/cmake/libwebsockets/LibwebsocketsTargets.cmake
 lib/libwebsockets.a
 @lib lib/libwebsockets.so.${LIBwebsockets_VERSION}
 lib/pkgconfig/libwebsockets.pc
+lib/pkgconfig/libwebsockets_static.pc


Re: NEW: games/corsixth

2017-07-31 Thread Adam Wolk
On Sun, Jul 30, 2017 at 05:33:32PM -0400, Brian Callahan wrote:
> Hi ports --
> 
> Attached is a new port, games/corsixth. CorsixTH is an open source clone of
> the game Theme Hospital.
> 

Marvelous!

Started off by building it on amd64 -current, snapshot from 07.31.

The built went fine, but make port-lib-depends-check complained:

$ make port-lib-depends-check

corsixth-0.60.1(games/corsixth):
Missing lib: lua5.3.5 (/usr/local/share/CorsixTH/CorsixTH) (NOT REACHABLE)
Extra:  lua5.1.5
*** Error 1 in target 'port-lib-depends-check' (ignored)
$

I had lua-5.3.4 as a dependency from nmap and lua5.1 as a dependency from
a bunch of stuff.

Installed and tried to start it, got welcomed by a black window with an error
message and the same error message presented in the terminal (I will skip
providing a screenshot):

$ corsixth
An error has occurred in CorsixTH:
./Lua/strict.lua:66: module 'lfs' not found:
no field package.preload['lfs']
no file '/usr/local/share/lua/5.3/lfs.lua'
no file '/usr/local/share/lua/5.3/lfs/init.lua'
no file '/usr/local/lib/lua/5.3/lfs.lua'
no file '/usr/local/lib/lua/5.3/lfs/init.lua'
no file './lfs.lua'
no file './lfs/init.lua'
no file '/usr/local/lib/lua/5.3/lfs.so'
no file '/usr/local/lib/lua/5.3/loadall.so'
no file './lfs.so'
stack traceback:
[C]: in upvalue 'fn'
./Lua/strict.lua:66: in function 'require'
./Lua/app.lua:23: in main chunk
[C]: in function 'persist.dofile'
./CorsixTH.lua:56: in function 'dofile'
./CorsixTH.lua:108: in main chunk
[C]: in ?

lfs is of course missing for lua5.3. Make install added it for 5.1.

I proceeded to remove nmap because who needs network scanning when
you can play Theme Hospital instead?

That makes port depends check really happy:
$ make port-lib-depends-check
$

Now the binary runs with a different error, probably missing files.
Here I agree with sthen@ that adding a pkg-readme of where to put
the files is a nice touch that most game ports already have (ie. for
ioquake3, yaquake2 etc). I was wrong and didn't grab the message
but not able to reproduce it yet.

After a few tries it showed me a window with the game and a picker
to locate the game files. I tried to full screen it in i3 but the
game segfaulted (without a core file). I can reproduce that by
starting the game again and hitting Mod-f for fullscreen. It
also happens when sending it to another screen.

I just start it now in a fresh window to have it fullscreen
and proceed to pick the game data folder from GOG.com.

The game starts, music and sound is present, the cutscene plays.
Everything seems to work perfectly gameplay wise.

Long story short, I think this needs perhaps a hard cap on the
lua version and the suggestions sthen@ left before. I wouldn't
worry too much on i3 resize segfault - that can be handled with
upstream post import.

Regards,
Adam

> ---
> pkg/DESCR:
> CorsixTH is a reimplementation of the 1997 Bullfrog business sim Theme
> Hospital. As well as faithfully recreating the original, CorsixTH adds
> support for modern operating systems, high resolutions, and much more.
> 
> CorsixTH requires game assets from Theme Hospital to work properly.
> ---
> 
> You will have to obtain a copy of Theme Hospital to play this (the GoG
> download worked for me using innoextract to extract it, like with OpenRCT2).
> 
> This requires a C++11 compiler. In theory, CorsixTH should work on any
> platform even those that don't have clang as their base compiler (I don't
> have the hardware to test, sadly). I don't think it's appropriate to do
> ONLY_FOR_ARCHS=${CLANG_ARCHS} unless the theory doesn't work out in
> practice. What's the best way to do this?
> COMPILER = clang gcc
> MODCLANG_ARCHS = 
> MODGCC4_ARCHS =  ${MODCLANG_ARCHS}>
> Does this make sense?
> 
> Comments/OKs welcome, port attached. Works well on amd64.
> 
> ~Brian
> 




[UPDATE] www/libwebsockets 2.0.2 => 2.2.2

2017-07-23 Thread Adam Wolk
Hi ports@,

trivial bump to libwebsockets.

upstream changelogs:
---
https://github.com/warmcat/libwebsockets/blob/v2.2-stable/changelog
https://github.com/warmcat/libwebsockets/releases/tag/v2.2.1
https://github.com/warmcat/libwebsockets/releases/tag/v2.2.2 (no notes, diff 
below):
https://github.com/warmcat/libwebsockets/compare/v2.2.1...v2.2.2
---

tested on a -current amd64 snapshot compiled against a
closed source koparo.com product.

Any feedback, OKs?

Regards,
Adam
Index: Makefile
===
RCS file: /cvs/ports/www/libwebsockets/Makefile,v
retrieving revision 1.12
diff -u -p -r1.12 Makefile
--- Makefile22 Jun 2016 17:53:14 -  1.12
+++ Makefile23 Jul 2017 14:45:59 -
@@ -4,10 +4,10 @@ COMMENT = lightweight C library for fas
 
 GH_ACCOUNT =   warmcat
 GH_PROJECT =   libwebsockets
-V =2.0.2
+V =2.2.2
 GH_TAGNAME =   v$V
 
-SHARED_LIBS =  websockets 6.0 # 8
+SHARED_LIBS =  websockets 7.0 # 10
 
 CATEGORIES =   www
 
Index: distinfo
===
RCS file: /cvs/ports/www/libwebsockets/distinfo,v
retrieving revision 1.10
diff -u -p -r1.10 distinfo
--- distinfo22 Jun 2016 17:53:14 -  1.10
+++ distinfo23 Jul 2017 14:45:59 -
@@ -1,2 +1,2 @@
-SHA256 (libwebsockets-2.0.2.tar.gz) = 
Q4ZWBN69BmhqxNjQeDl2xOEN1RnM1clOG1OHjsYXilk=
-SIZE (libwebsockets-2.0.2.tar.gz) = 2998706
+SHA256 (libwebsockets-2.2.2.tar.gz) = 
44p9mB/oe5qGsfHN3R+eBi04lsSI5rKjLLcA1MlZ7Lo=
+SIZE (libwebsockets-2.2.2.tar.gz) = 3548981


[UPDATE] games/yquake2 6.00 => 7.01

2017-07-20 Thread Adam Wolk
Hi ports@,

Upgrading our yquake2 from 6.00 to 7.01

Port changes:
 - bump version
 - install new ref_gl1 / ref_gl3 shared libraries into
   /usr/local/lib/yquake2
 - patch the Makefile to not pass -Wl,--no-undefined to
   the linker, change already merged upstream:
 https://github.com/yquake2/yquake2/pull/213
 - patch backends/generic/vid.c to dlopen libraries
   from /usr/local/lib/yquake2 instead of relying
   on Sys_GetBinaryDir() that thinks the OS can tell
   it the executable path

Regarding the .so changes, please see the following 2 github links
 - https://github.com/yquake2/yquake2/pull/213
 - https://github.com/yquake2/yquake2/issues/214
especially the second one. In short, it's possible that upstream will
add a way to define /usr/local/lib/yquake2 as a compile time option
which would make this patch obsolete. Note that I did not move the game.so
file on purpose, as upstream mentioned in 213 that mods expect it to be
in that specific spot.

If you want to test the new renderer just execute the game with:
 $ quake2 +vid_renderer gl3 

after first run it will be added to your config, or you can add it
yourself by adding:

 set vid_renderer "gl3"

to:

/home/mulander/.yq2/baseq2/config.cfg

Upstream changelog:

Quake II 7.00 to 7.01:
- Fix build of GL3 for platforms without SSE.
- Fix Jennel Jaquays name in credits and quit screen.
- Make Quake II high DPI aware on Window Vista and above.
- Fix some problems with loading dependend librarys on Windows.

Quake II 6.00 to 7.00:
- Remove the broken multitexturing render path from the OpenGL 1.4
  renderer. It was switched off by default in 6.00.
- Reimplement the support for shared renderer libraries. Please note
  the this is an incompatible implementation with an custom API. The
  original renderer libraries will not work!
- Implement an OpenGL 3.2 renderer. This renderer has the same look
  and feel as the old OpenGL 1.4 renderer but makes heavy use of
  modern OpenGL and GPU features. An OpenGL 3.2 capable GPU (Intel
  starting with Ivy Bridge on Windows or Sandy Bridge on Linux, Nvidia
  staring with G80 and AMD starting with R600 / HD2000) is required.
- Fix OpenAL compatibility with modern openal-soft versions.
- Several fixes and optimizations to OpenAL, implement support for
  doppler effects. (by xorw)

Frags? OK's?

Index: Makefile
===
RCS file: /cvs/ports/games/yquake2/Makefile,v
retrieving revision 1.8
diff -u -p -r1.8 Makefile
--- Makefile27 May 2017 19:57:06 -  1.8
+++ Makefile20 Jul 2017 20:44:12 -
@@ -4,10 +4,9 @@ ONLY_FOR_ARCHS=i386 amd64 sparc64
 
 COMMENT=   Yamagi Quake II
 N= yquake2
-V= 6.00
+V= 7.01
 PKGNAME=   ${N}-${V}
 DISTNAME=  quake2-${V}
-REVISION=  2
 CATEGORIES=games
 
 HOMEPAGE=  http://www.yamagi.org/quake2/
@@ -36,5 +35,10 @@ do-install:
${INSTALL_DATA_DIR} ${PREFIX}/share/${N}/baseq2
${INSTALL_PROGRAM} ${WRKBUILD}/release/baseq2/game.so \
${PREFIX}/share/${N}/baseq2/
+   ${INSTALL_DATA_DIR} ${PREFIX}/lib/${N}
+   ${INSTALL_PROGRAM} ${WRKBUILD}/release/ref_gl1.so \
+   ${PREFIX}/lib/${N}/
+   ${INSTALL_PROGRAM} ${WRKBUILD}/release/ref_gl3.so \
+   ${PREFIX}/lib/${N}/
 
 .include 
Index: distinfo
===
RCS file: /cvs/ports/games/yquake2/distinfo,v
retrieving revision 1.3
diff -u -p -r1.3 distinfo
--- distinfo5 Dec 2016 21:03:33 -   1.3
+++ distinfo20 Jul 2017 20:44:12 -
@@ -1,2 +1,2 @@
-SHA256 (quake2-6.00.tar.xz) = N7a7eiHUOxf2bvkIsXKjuRi5Z/r7pZwT7ssUfXXEXUw=
-SIZE (quake2-6.00.tar.xz) = 1705640
+SHA256 (quake2-7.01.tar.xz) = ly93hnCTs8Fi37KMqTBYXLcdG3sYPWdthlnS/JcTfvc=
+SIZE (quake2-7.01.tar.xz) = 1778212
Index: patches/patch-Makefile
===
RCS file: patches/patch-Makefile
diff -N patches/patch-Makefile
--- /dev/null   1 Jan 1970 00:00:00 -
+++ patches/patch-Makefile  20 Jul 2017 20:44:12 -
@@ -0,0 +1,16 @@
+$OpenBSD$
+
+Index: Makefile
+--- Makefile.orig
 Makefile
+@@ -251,8 +251,8 @@ endif
+ CFLAGS += -fvisibility=hidden
+ LDFLAGS += -fvisibility=hidden
+ 
+-ifneq ($(YQ2_OSTYPE), Darwin)
+-# for some reason the OSX linker doesn't support this
++ifneq ($(YQ2_OSTYPE), $(filter $(YQ2_OSTYPE), Darwin, OpenBSD))
++# for some reason the OSX & OpenBSD linker doesn't support this
+ LDFLAGS += -Wl,--no-undefined
+ endif
+ 
Index: patches/patch-src_backends_generic_vid_c
===
RCS file: patches/patch-src_backends_generic_vid_c
diff -N patches/patch-src_backends_generic_vid_c
--- /dev/null   1 Jan 1970 00:00:00 -
+++ patches/patch-src_backends_generic_vid_c20 Jul 2017 20:44:12 -
@@ -0,0 +1,14 @@
+$OpenBSD$
+
+Index: src/backends/generic/vid.c
+--- src/backends/generic/vid.c.orig
 src

[UPDATE] net/corebird 1.5 => 1.5.1

2017-07-19 Thread Adam Wolk
Hi ports@

attaching a trivial bump for net/corebird
tested on amd64 current.

upstream changelog:
---
Changelog:
 - Stability improvements
---

actual diff:
https://github.com/baedert/corebird/compare/1.5...1.5.1

ports wise we just bump the version and add a new translation.

Feedback, OK's?

Regards,
Adam
Index: Makefile
===
RCS file: /cvs/ports/net/corebird/Makefile,v
retrieving revision 1.13
diff -u -p -r1.13 Makefile
--- Makefile16 Jul 2017 19:18:57 -  1.13
+++ Makefile19 Jul 2017 20:45:50 -
@@ -2,7 +2,7 @@
 
 COMMENT =  native Gtk+ Twitter client
 
-V =1.5
+V =1.5.1
 DISTNAME = corebird-${V}
 MASTER_SITES = 
https://github.com/baedert/corebird/releases/download/${V}/
 
Index: distinfo
===
RCS file: /cvs/ports/net/corebird/distinfo,v
retrieving revision 1.9
diff -u -p -r1.9 distinfo
--- distinfo28 May 2017 08:15:35 -  1.9
+++ distinfo19 Jul 2017 20:45:50 -
@@ -1,2 +1,2 @@
-SHA256 (corebird-1.5.tar.xz) = ZBRmg4PAjzpLu4PaYexGQ+590EcPYLrCbx5ldx8D9+E=
-SIZE (corebird-1.5.tar.xz) = 668064
+SHA256 (corebird-1.5.1.tar.xz) = 57ZBol4nnYJRygqfou8Rpd02TjaeH6VPVUWCTclt7aA=
+SIZE (corebird-1.5.1.tar.xz) = 676192
Index: pkg/PLIST
===
RCS file: /cvs/ports/net/corebird/pkg/PLIST,v
retrieving revision 1.5
diff -u -p -r1.5 PLIST
--- pkg/PLIST   28 May 2017 08:15:35 -  1.5
+++ pkg/PLIST   19 Jul 2017 20:45:50 -
@@ -20,6 +20,7 @@ share/locale/ca/LC_MESSAGES/corebird.mo
 share/locale/ca@valencia/LC_MESSAGES/corebird.mo
 share/locale/de/LC_MESSAGES/corebird.mo
 share/locale/en_AU/LC_MESSAGES/corebird.mo
+share/locale/eo/LC_MESSAGES/corebird.mo
 share/locale/es/LC_MESSAGES/corebird.mo
 share/locale/es_419/
 share/locale/es_419/LC_MESSAGES/


Re: [NEW] sysutils/bdfresize

2017-06-13 Thread Adam Wolk
On Thu, Jun 08, 2017 at 11:48:54PM +0200, Frederic Cambus wrote:
> On Wed, May 31, 2017 at 12:48:13PM +0200, Frederic Cambus wrote:
> 
> > > Here is a new port: sysutils/bdfresize
> > 
> > Improved tarball attached, which also avoids picking local getopt
> > headers.
> > 
> > Comments? OK to import?
> 
> Ping. Anyone?
> 

OK awolk@ tested on amd64 -current



Re: irssi on stable

2017-06-07 Thread adam . wolk
On Tue, Jun 06, 2017 at 07:08:11PM -0400, Daniel Jakots wrote:
> Hi,
> 
> https://irssi.org/security/irssi_sa_2017_06.txt
> 
> I backported the patch to stable. I don't use irssi so it's not tested.
> 

Just tested on 6.1 stable. Everything seems to work correctly, including 
irssi-icb.

OK awolk@ (assuming you get a go ahead from people responsible for the 6.1 
branch)



Re: [update] security/keybase to 1.0.21

2017-05-28 Thread Adam Wolk
On Sun, May 28, 2017 at 08:11:14AM -0600, Aaron Bieber wrote:
> Hola,
> 
> Here is a trivial update to bring keybase to the latest version. OK?
> 
> I have been using it without issue for a few weeks now (amd64) with no
> issues.
> 
> Cheers,
> Aaron
> 

OK awolk@ if you can decrypt :)

-BEGIN PGP MESSAGE-
Comment: https://keybase.io/download
Version: Keybase Go 1.0.21 (openbsd)

wcFMA1Md1R9Yd11NARAAZ/X0myJWUAEA6+cTbjUX87DWJFPkpnvRu5CiSYgYj5Ke
LoXTzupg3WOFHCqtC4zD235rs7xkthn5VUL+i/om7rwfDMjY9nk8WrzlMbbJUbgF
uSX2MCzeC54tIn7CJqgl2O78NmlvRb3L/z9A3QjLmRrUO3RBsW4C66pE9D2DWuW7
pWhHC1cluB1CtggRtDHzJPN4AtH8YrdK/aYJ1vn72VL4Q6ublpSJtKZVm+GrDKSZ
0IIYjl+MjA9JGNHLDQA2/vqmH5+p1fbpNeL8rCC/VDSDVnbDF91gWV8vx5NtD6KH
MiKuR8P1B+veA8PqvLCmgbOoZ50DjX5hcJfM8t7kzT1mvffJPpJTwwtkJte6WV80
b0vo/cCijGcLED3oXeh95iEN4aRuBFzT3NV6tUyakASU5GedMaZ+d9jxjo7q+mpc
bjt8jZpdWZlxgD2f3yYd+7ArgAjfsH38neMbBG6xyEPCFNprqcm3RufYLYt2jiam
4zIUFtihef24u31rL+oGofIbsnszH2f1bVtopxrYarCPkawcyNfbJ/G2zpAaqtPr
R6XeaV9rjJ+K1uq4yN4DTm011yX9eVZnwT0OrP7kDPzMpQSqQavoIOQ75SfJIrZN
XCsnzJA/nLpMdDTvEZAIeZBOObgkVI4+qB0EIeL+OQAcgOXFIB5tBeKmo8IDqmbB
wUwDIK2t4N1IsQkBEAARzkY4QLUnPmiDDPH51PdkDSOgOX4iCF/ZWGqW8JQdUFLZ
HHBbj43+kPW8E79oifbBmDwLLhcFPWRveSrIGfif0uLXrcmIumDpXJhI8NhCF79S
HQcfaj5GBL32wk7WDdlpkMIN9/KMURNyLyfHSizrowepQW4EoonWFDHmcdrqsMBf
H9TpCHhqhWjY/gaL3UAkj5LltaW6dY0/bRFN7LHwn4GeeIYTbMsq5tWZ3hjBJfYn
7zhcX88etkJLhB+U6cL63wB3PFNbRysspYiGWBpGTdkc3++uDBOKwwICzTnlwFep
sj7/5QembLWC4Y+j8uEkYfgpSU5ajS2pVTUVtCutR/jtLoUV6ODp0WfJvqKsZ7Mq
8xcokhRfLByTho9PK5+tN2pnD5Kgd54A0LqD2Hom/QUmDGlFNWnb99sYcOOA0WlY
QrcOHXgRqIeJf1iDMYKPz8tUDACIhJx331vfVmrW74EzmDA+4FZiJ7gaESxEMbpH
SrYjyYzmn1yp3gh4nYUpwp+IIoZvgE7zyMZmrqS+l3keVjvehdrpJ955ztWezOeN
H5iTs8mfe0M8Wl6CbcX//NcXet1IGdk2wERW0kW9I7LZPYQN85V+KIisCC0hPqgi
404A0foWpYNQYkORJ28PNniLjmmFsIuEghd2R4YMOiSIsIp+DiKx9RJ/vUnCkdLg
AeQ2YaLvcspS7vas6XI7Yyjo4UuP4LLgaOHSV+CC4jCw7aPgNeOQiFkw0m65feDQ
4B3gWOTw88UjoCFUKp27fsk5Aur04qASEcfhgdgA
=DTI5
-END PGP MESSAGE-



[UPDATE] editors/ghostwriter 1.4.2 => 1.5

2017-05-22 Thread Adam Wolk
Hi ports@,

Attaching a trivial bump for editors/ghostwriter

port changes:
 - no need to @sed VERSION as upstream stopped shelling out to git

upstream changelog:
---
New Features

* HUD windows can now be closed using the Esc key when they have focus.
* A new option has been added to highlight double spaces at the end of a line.
* Github-style task lists can now be made using the * and + bullet point
  characters, in addition to the - character.
* Support for ConTeXt and wkhtmltopdf when exporting to PDF with Pandoc has been
  added.
* Auto-matching characters for selected text now respects the existing 
preferences
  for the matching done while typing without text selected.
* Typing a single quote (') will no longer result in a second quote being
  automatically inserted after the cursor if the cursor is positioned in the
  middle of a word.
* Most settings have been moved into a new Preferences dialog to accommodate the
  ever increasing number of application options.
* Font size can now be increased by pressing Ctrl+=or decreased by pressing
  Ctrl+-. It can also be changed by using Ctrl and the mouse wheel.
* When exporting to other file formats, the output file will be opened after
  export with its default application.
* E-books created using Pandoc will now also be generated with a table of
  contents.
* When passing in a file path via the command line, a new file will be created
  with that path if it does not already exist.
* When typing a * character, ghostwriter would auto-insert a second * character 
in
  anticipation of emphasized text. The auto-inserted second * will now be 
removed
  if a space is typed to accommodate a bullet list instead.
* Rudimentary support for HiDPI has been added. See notes below.
* Various performance enhancements and tweaks have been made under the hood.

Fixes

* Issue #128: Hyphenated words are now considered one word for spell checking.
* Issue #131: ghostwriter can now be built from its source tarball without 
needing
  git to be installed.
* Issue #132: Linux desktop file now specifies that ghostwriter can open only 
one
  file from the command line rather than multiple files.
* Issue #170: Ctrl+M now opens the HTML Preview instead of Ctrl+W. This is to
  avoid a conflict with closing windows in Linux.
* Issue #219: Copy and paste no longer makes the word count jump.
* Sundry minor fixes have been added.
---

Looking for feedback and OK's.

Regards,
Adam
? ghostwriter.1.5.diff
Index: Makefile
===
RCS file: /cvs/ports/editors/ghostwriter/Makefile,v
retrieving revision 1.2
diff -u -p -r1.2 Makefile
--- Makefile12 Apr 2017 18:12:33 -  1.2
+++ Makefile22 May 2017 22:33:35 -
@@ -7,8 +7,7 @@ COMMENT =   distraction-free Markdown edit
 
 GH_ACCOUNT =   wereturtle
 GH_PROJECT =   ghostwriter
-GH_TAGNAME =   v1.4.2
-REVISION = 0
+GH_TAGNAME =   v1.5.0
 
 CATEGORIES =   editors
 
@@ -34,8 +33,6 @@ LIB_DEPENDS = textproc/hunspell \
 NO_TEST =  Yes
 
 pre-configure:
-   @sed -i 's,VERSION = ..system.*,VERSION =,' \
-   ${WRKSRC}/ghostwriter.pro
@sed -i 's,/share/man/man1,/man/man1,' \
${WRKSRC}/ghostwriter.pro
 
Index: distinfo
===
RCS file: /cvs/ports/editors/ghostwriter/distinfo,v
retrieving revision 1.1.1.1
diff -u -p -r1.1.1.1 distinfo
--- distinfo17 Jan 2017 12:58:26 -  1.1.1.1
+++ distinfo22 May 2017 22:33:35 -
@@ -1,2 +1,2 @@
-SHA256 (ghostwriter-1.4.2.tar.gz) = 
d1dg0ExaeUw4EM/+nR2qcu4SPHfMMEH8FaIFJQOJlDg=
-SIZE (ghostwriter-1.4.2.tar.gz) = 589182
+SHA256 (ghostwriter-1.5.0.tar.gz) = 
v9ZAJNl0H1XVh6+lZh9LXipIvsu8kklcIYb8pE+whIY=
+SIZE (ghostwriter-1.5.0.tar.gz) = 620301


[UPDATE] net/corebird 1.4.2 => 1.5

2017-05-22 Thread Adam Wolk
Hi ports@

attaching a trivial bump for net/corebird
tested on amd64 current with gcc & clang.

upstream changelog:
---
Changelog:
  - Media attached to tweets can be downloaded using Right Click
and selecting "save as"
  - Profiles use the profile background color set in the Twitter
settings if no banner is set
  - The tweet compose window now features a "favorite image" view that
allows users to save often sent images and quickly add them to tweets
  - The media dialog now shows Previous/Next buttons to quickly switch
between multiple media attachments of a tweet>
  - The Vine support has been removed since the project is discontinued
  - Allow text selection in Direct Messages
  - New --account parameter allows opening the window for the given
account only
  - Support tweets with up to 50 replied-to users.
  - Add back verified icons next to user avatars
  - Redesigned account creation UI
  - Tons of bug fixes
---

Looking for OK's and feedback on the update.


PS.
While testing I hit hit one issue that I am having trouble reproducing

Core was generated by `corebird'.
Program terminated with signal 11, Segmentation fault.

Loaded symbols for /usr/local/lib/libhunspell-1.6.so.0.0
#0  g_utf8_offset_to_pointer (
str=0x13b776fc6a00 "e @gnorth @mrjafri @m0ccha616 @KenzoJones
@ItsMe_JessicaT @thejonnyharris @alluring_blue @JBon20 @WolverSteve
@GeekToMeRadio @Gisy_Angel @thenerdynurse89 @zoly_marie @th
ealexrossart  https://t.co/wuSY";..., offset=4294967286) at gutf8.c:351
351 gutf8.c: No such file or directory.
in gutf8.c
(gdb) exit

I hit it while subsequently loading my twitter tabs, it crashed when
I got to direct messages. If someone manages to reproduce please let me know.

Regards,
Adam
Index: Makefile
===
RCS file: /cvs/ports/net/corebird/Makefile,v
retrieving revision 1.11
diff -u -p -r1.11 Makefile
--- Makefile10 Apr 2017 11:46:23 -  1.11
+++ Makefile22 May 2017 21:16:57 -
@@ -2,7 +2,7 @@
 
 COMMENT =  native Gtk+ Twitter client
 
-V =1.4.2
+V =1.5
 DISTNAME = corebird-${V}
 MASTER_SITES = 
https://github.com/baedert/corebird/releases/download/${V}/
 
Index: distinfo
===
RCS file: /cvs/ports/net/corebird/distinfo,v
retrieving revision 1.8
diff -u -p -r1.8 distinfo
--- distinfo10 Feb 2017 18:44:56 -  1.8
+++ distinfo22 May 2017 21:16:57 -
@@ -1,2 +1,2 @@
-SHA256 (corebird-1.4.2.tar.xz) = HAemU4Lngwj33kBr6EZHicHsQtUxxRm2lRCmhSNLQHQ=
-SIZE (corebird-1.4.2.tar.xz) = 648448
+SHA256 (corebird-1.5.tar.xz) = ZBRmg4PAjzpLu4PaYexGQ+590EcPYLrCbx5ldx8D9+E=
+SIZE (corebird-1.5.tar.xz) = 668064
Index: pkg/PLIST
===
RCS file: /cvs/ports/net/corebird/pkg/PLIST,v
retrieving revision 1.4
diff -u -p -r1.4 PLIST
--- pkg/PLIST   19 Jan 2017 19:31:59 -  1.4
+++ pkg/PLIST   22 May 2017 21:16:57 -
@@ -33,6 +33,7 @@ share/locale/es_VE/LC_MESSAGES/corebird.
 share/locale/fa/LC_MESSAGES/corebird.mo
 share/locale/fi/LC_MESSAGES/corebird.mo
 share/locale/fr/LC_MESSAGES/corebird.mo
+share/locale/ga/LC_MESSAGES/corebird.mo
 share/locale/gl/LC_MESSAGES/corebird.mo
 share/locale/hi/LC_MESSAGES/corebird.mo
 share/locale/hu/LC_MESSAGES/corebird.mo


Re: [UPDATE] security/keybase 1.0.17 -> 1.0.18

2017-02-19 Thread Adam Wolk
On Tue, Jan 31, 2017 at 08:29:36AM -0700, Aaron Bieber wrote:
> Hi!
> 
> This update brings keybase to the latest. Most of the patches were
> pulled into the various upstreams, as such, they have been removed.
> 
> I have been using it for a month or two without issue (I don't use
> keybase often, so usage is fairly lite).
> 
> OK?
> 
> Cheers,
> Aaron
> 

works fine on -current amd64

OK awolk@



[UPDATE] net/corebird 1.4.1 => 1.4.2

2017-02-09 Thread Adam Wolk
Hi ports@

attaching a trivial bump for net/corebird

upstream changelog:
---
Version 1.4.2 (2017-01-27)

Changelog:
  - Fix a crash when going back from a tweet with media to another
tweet with media
  - Fix compilation with --disable-video
  - Decrease log level of UserStream output to debug when no internet
connection is present

---

Feedback? OK's?

Index: Makefile
===
RCS file: /cvs/ports/net/corebird/Makefile,v
retrieving revision 1.9
diff -u -p -r1.9 Makefile
--- Makefile19 Jan 2017 19:31:59 -  1.9
+++ Makefile9 Feb 2017 13:45:25 -
@@ -2,7 +2,7 @@
 
 COMMENT =  native Gtk+ Twitter client
 
-V =1.4.1
+V =1.4.2
 DISTNAME = corebird-${V}
 MASTER_SITES = 
https://github.com/baedert/corebird/releases/download/${V}/
 
Index: distinfo
===
RCS file: /cvs/ports/net/corebird/distinfo,v
retrieving revision 1.7
diff -u -p -r1.7 distinfo
--- distinfo19 Jan 2017 19:31:59 -  1.7
+++ distinfo9 Feb 2017 13:45:25 -
@@ -1,2 +1,2 @@
-SHA256 (corebird-1.4.1.tar.xz) = 9X5diTtmpchfsNs6EyGwAxHWwCo53t/JXJyGejFOgAk=
-SIZE (corebird-1.4.1.tar.xz) = 648564
+SHA256 (corebird-1.4.2.tar.xz) = HAemU4Lngwj33kBr6EZHicHsQtUxxRm2lRCmhSNLQHQ=
+SIZE (corebird-1.4.2.tar.xz) = 648448


Re: games/hedgewars EAccessViolation on game start

2017-02-04 Thread Adam Wolk
On Sat, Feb 04, 2017 at 10:17:22AM +0100, Pascal Stumpf wrote:
> On Fri, 3 Feb 2017 22:52:41 +0100, Adam Wolk wrote:
> > 
> > 
> > Note in this case instead of substituting library name on the command line 
> > it's
> > done in the source code itself with the {$linklib NAME} directive.
> > 
> > I'm roughly looking for some feedback on what might cause such issue.
> > I think the proper solution is updating the hedgewars CMakeLists.txt in this
> > case to not use the XLAlua pascal flag and for this port baking it in with
> > a patch to the source. I am going to work on a diff like that for a separate
> > email including a hotfix for the -server package but I wanted to share the
> > actual problem this will remove in case someone can spot a bigger problem.
> 
> Wow, thanks for investigating.  I've taken a closer look at the binaries
> produced, and - lo and behold - when passing -XAfoo, fpc will generate a
> binary that is linked against /usr/libexec/ld.so twice, once as runtime
> linker, once as a shared object.  Ugh.  The fix is straightforward:
> 
> 
> Index: Makefile
> ===
> RCS file: /cvs/ports/lang/fpc/Makefile,v
> retrieving revision 1.13
> diff -u -p -r1.13 Makefile
> --- Makefile  7 May 2016 12:40:57 -   1.13
> +++ Makefile  4 Feb 2017 09:16:30 -
> @@ -7,7 +7,7 @@ COMMENT = open source compiler for Pasc
>  V =  3.0.0
>  DISTNAME =   fpcbuild-${V}
>  PKGNAME =fpc-${V}
> -REVISION =   1
> +REVISION =   2
>  BOOT_GEN =   0
>  
>  CATEGORIES = lang
> Index: patches/patch-fpcsrc_compiler_systems_t_bsd_pas
> ===
> RCS file: 
> /cvs/ports/lang/fpc/patches/patch-fpcsrc_compiler_systems_t_bsd_pas,v
> retrieving revision 1.3
> diff -u -p -r1.3 patch-fpcsrc_compiler_systems_t_bsd_pas
> --- patches/patch-fpcsrc_compiler_systems_t_bsd_pas   22 Dec 2015 14:43:55 
> -  1.3
> +++ patches/patch-fpcsrc_compiler_systems_t_bsd_pas   4 Feb 2017 09:16:30 
> -
> @@ -2,8 +2,10 @@ $OpenBSD: patch-fpcsrc_compiler_systems_
>  
>  Correct library search path for OpenBSD; always add -nopie to linker flags.
>  
> +Do not link to /usr/libexec/ld.so as a shared library.
> +
>  --- fpcsrc/compiler/systems/t_bsd.pas.orig   Sun Dec  7 21:27:02 2014
> -+++ fpcsrc/compiler/systems/t_bsd.pasThu Nov 26 15:17:06 2015
>  fpcsrc/compiler/systems/t_bsd.pasSat Feb  4 09:51:46 2017
>  @@ -126,7 +126,10 @@ begin
> Inherited Create;
> if not Dontlinkstdlibpath Then
> @@ -12,11 +14,21 @@ Correct library search path for OpenBSD;
>  + if not(target_info.system in systems_openbsd) then
>  +   
> LibrarySearchPath.AddPath(sysrootpath,'/lib;/usr/lib;/usr/X11R6/lib',true)
>  + else
> -+   
> LibrarySearchPath.AddPath(sysrootpath,'/usr/lib;/usr/X11R6/lib;${LOCALBASE}/lib',true)
> ++   
> LibrarySearchPath.AddPath(sysrootpath,'/usr/lib;${X11BASE}/lib;${LOCALBASE}/lib',true)
>  else
>{ Mac OS X doesn't have a /lib }
>LibrarySearchPath.AddPath(sysrootpath,'/usr/lib',true)
> -@@ -697,8 +700,7 @@ begin
> +@@ -581,7 +584,8 @@ begin
> +  { when we have -static for the linker the we also need libgcc }
> +  if (cs_link_staticflag in current_settings.globalswitches) then
> +   LinkRes.Add('-lgcc');
> +- if linkdynamic and (Info.DynamicLinker<>'') then
> ++ if linkdynamic and (Info.DynamicLinker<>'') and
> ++not(target_info.system in systems_openbsd) then
> +   LinkRes.AddFileName(Info.DynamicLinker);
> +  if not LdSupportsNoResponseFile then
> +LinkRes.Add(')');
> +@@ -697,8 +701,7 @@ begin
>  end;
>  
>   { Use -nopie on OpenBSD }
> 

I just applied your diff, rebuilt lang/fpc and rebuild games/hedgewars using the
new compiler. The game now works. I was able to start a normal quickmatch
against the AI.

OK awolk@

> 
> 
> > PS.
> > While working on the port 2 things happened.
> > 
> > 1. Upstream took some of our patches
> >  http://hg.hedgewars.org/hedgewars/rev/ffc7bb9fde01
> > 2. The -server package didn't build for me and according to hedgewars 
> > upstream
> > it's a ghc bug. FreeBSD ships a workaround patch for it
> >  
> > https://svnweb.freebsd.org/ports/head/games/hedgewars/files/patch-CMakeLists.txt?revision=377852&view=markup
> 
> 
> Has this happened in any bulk build?  If yes, we shou

games/hedgewars EAccessViolation on game start

2017-02-03 Thread Adam Wolk
Hi ports,

Two weeks ago Damien Couderc reported[1] on /r/openbd_gaming that 
games/hedgewars
is broken on 6.0. I did a test on -current back then and found the same thing
planning to test a bit later when I find some time to bump my box to a newer
snapshot.

I finally found some time and behold, it still crashesh with a weird
backtrace[2].

I started digging around and got upstream involved. Together with unC0Rr from
the #hedgewars @ freenode IRC channel we reduced the breaking code to a single
pascal file that fails the same way without any hedgewars code:

$ cat hello.pas
uses GL;
begin
system.writeln('hello');
end.

Now, the trick is - the failure only occurs if one compiles with:

$ fpc -XLAlua=lua5.1 hello.pas
Free Pascal Compiler version 3.0.0 [2016/12/07] for x86_64
Copyright (c) 1993-2015 by Florian Klaempfl and others
Target OS: OpenBSD for x86-64
Compiling hello.pas
Linking hello
4 lines compiled, 0.3 sec
$ ./hello
An unhandled exception occurred at $000223C29C43:
EAccessViolation: Access violation
  $000223C29C43

The -XLAlua=lua5.1 line was taken from the build output of our ports system.
There is a way to build a pascal file without passing that flag and it doesn't
lead to broken runtimes, redesigned example:

$ cat code.pas
{$linklib lua5.1}
uses GL;
begin
system.writeln('hello');
end.
$ fpc code.pas
Free Pascal Compiler version 3.0.0 [2016/12/07] for x86_64
Copyright (c) 1993-2015 by Florian Klaempfl and others
Target OS: OpenBSD for x86-64
Compiling code.pas
Linking code
/usr/local/lib//liblua5.1.so.5.1: warning: warning: rand() may return
deterministic values, is that what you want?
/usr/local/lib//liblua5.1.so.5.1: warning: warning: sprintf() is often misused,
please use snprintf()
/usr/local/lib//liblua5.1.so.5.1: warning: warning: strcat() is almost always
misused, please use strlcat()
/usr/local/lib//liblua5.1.so.5.1: warning: warning: strcpy() is almost always
misused, please use strlcpy()
5 lines compiled, 0.3 sec
$ ./code
hello

Note in this case instead of substituting library name on the command line it's
done in the source code itself with the {$linklib NAME} directive.

I'm roughly looking for some feedback on what might cause such issue.
I think the proper solution is updating the hedgewars CMakeLists.txt in this
case to not use the XLAlua pascal flag and for this port baking it in with
a patch to the source. I am going to work on a diff like that for a separate
email including a hotfix for the -server package but I wanted to share the
actual problem this will remove in case someone can spot a bigger problem.

PS.
While working on the port 2 things happened.

1. Upstream took some of our patches
 http://hg.hedgewars.org/hedgewars/rev/ffc7bb9fde01
2. The -server package didn't build for me and according to hedgewars upstream
it's a ghc bug. FreeBSD ships a workaround patch for it
 
https://svnweb.freebsd.org/ports/head/games/hedgewars/files/patch-CMakeLists.txt?revision=377852&view=markup

This problem manifests itself with the following compile error

[25 of 25] Compiling Main ( 
/usr/local/pobj/hedgewars-src-0.9.22/hedgewars-src-0.9.22/gameServer/hedgewars-server.hs,
/usr/local/pobj/hedgewars-src-0.9.22/build-amd64/gameServer/Main.o )
*** Core Lint errors : in result of Simplifier ***
: Warning:


[1]
https://www.reddit.com/r/openbsd_gaming/comments/5phgho/hedgewars/?st=iyq4oro0&sh=46dea45e
[2]
https://gist.github.com/mulander/b5aa0e825b88910c6d0e446db30dd526t



Re: [UPDATE] devel/zeal 0.2.1 => 0.3.1

2017-01-20 Thread Adam Wolk
On Sat, Jan 07, 2017 at 06:27:53PM +0100, Adam Wolk wrote:
> On Sun, Jan 01, 2017 at 03:15:47PM +0100, Adam Wolk wrote:
> > On Sun, Jan 01, 2017 at 03:02:10PM +0100, Antoine Jacoutot wrote:
> > > On Sun, Jan 01, 2017 at 02:57:30PM +0100, Adam Wolk wrote:
> > > > Hi ports@
> > > > 
> > > > Attaching a bump to devel/zeal
> > > > 
> > > > port changes:
> > > >  - add x11/qt5/qt5base dependency to LIB_DEPENDS
> > > >  - add Qt5Concurrent to WANTLIB (from qtbase)
> > > >  - mark USE_WXNEEDED = Yes (uses qt webkit)
> > > 
> > > Hmm, can't the qtwebkit port be patched like www/webkitgtk4 so that 
> > > pkg-config
> > > files or whatever cmake magic pass the proper -Wl flags insead of adding 
> > > USE_WXNEEDED to all qtwebkit consumers?
> > 
> > I just checked why www/otter-browser doesn't need it as a qtwebkit consumer 
> > and
> > in that case devel/cmake was patched to pass the wx flags during the build.
> > 
> > This port uses devel/qmake to build hence it doens't get the same automatic
> > benefit. 
> > 
> > x11/qt5/qtwebkit/ has USE_WNEEDED = Yes in what way could it impact 
> > producing a
> > downstream (consumer) binary that's also marked as wxneeded?
> > 
> > > Also we usually put USE_WXNEEDED at the top of the Makefile.
> > > 
> > > 
> > 
> > Will move that to the top of the file before comitting (assuming I will get 
> > an
> > OK of course ;)).
> > 
> > Regards,
> > Adam
> > 
> 
> Ping for reviews & a discussion on how to handle WXNEEDED with this port.
> 

Ping.



Re: [UPDATE] net/corebird 1.3.3 => 1.4.1

2017-01-19 Thread Adam Wolk
On Mon, Jan 16, 2017 at 11:19:14PM +0100, Adam Wolk wrote:
> Hi ports@
> 
> Attaching an update for net/corebird
> 
> port changes:
>   - new dependency on textproc/gspell for spellchecking
>   - 2 new locale files (PLIST regenerated)
>   - WANTLIB regenerated, new entries are enchant, gspell-1
> and stdc++
> 

Additionally added databases/sqlite3 as a direct dependency
in LIB_DEPENDS (spotted by fcambus@ thanks)

> upstream changelog:
> ---
> Version 1.4.1 (2017-01-08)
> 
> Changelog:
>   - Lower the required gspell version to 1.0
>   - Work around a binding problem in the gtk+ 3.18 vapi of vala 0.30
> 
> Version 1.4 (2017-01-06)
> 
> Changelog:
>   - Images in quoted tweets now look more like they actually belong to
> the quoted tweet instead of the quoting tweet.
>   - Allow deleting tweets from the tweet info page and not just from
> timelines.
>   - Fix the user completion not showing all possible results
>   - Focus the already opened window for an account if the account gets
> selected in the accounts popover
>   - Avoid window resizing when typing in the Direct Message text box
>   - Add 'q' accelerator to tweet rows for quoting
>   - Add spellchecking. This add a dependency to gspell
>   - Increase gtk+ dependency to 3.18 to get rid of some workarounds.
>   - Increase avatar size in profiles and slightly overlap them over the
> banner. Also, always show the full banner.
>   - Fix completion popup positioning under Wayland
>   - Add experimental meson build files
>   - Fix some videos not playing correctly
>   - Add a minimal video progress indicator to the video dialog
>   - Fix the retweet/favorite count updating in the tweet info page
> 
> ---
> 
> Feedback? OK's?

The updated diff:

Index: Makefile
===
RCS file: /cvs/ports/net/corebird/Makefile,v
retrieving revision 1.8
diff -u -p -r1.8 Makefile
--- Makefile28 Dec 2016 17:10:54 -  1.8
+++ Makefile19 Jan 2017 11:18:25 -
@@ -2,7 +2,7 @@
 
 COMMENT =  native Gtk+ Twitter client
 
-V =1.3.3
+V =1.4.1
 DISTNAME = corebird-${V}
 MASTER_SITES = 
https://github.com/baedert/corebird/releases/download/${V}/
 
@@ -18,14 +18,14 @@ PERMIT_PACKAGE_CDROM =  Yes
 
 WANTLIB += ICE SM X11 Xcomposite Xcursor Xdamage Xext Xfixes Xi
 WANTLIB += Xinerama Xrandr Xrender atk-1.0 atk-bridge-2.0 atspi
-WANTLIB += c cairo cairo-gobject dbus-1 epoxy expat ffi fontconfig
-WANTLIB += freetype gdk-3 gdk_pixbuf-2.0 gio-2.0 glib-2.0
-WANTLIB += gmodule-2.0 gobject-2.0 graphite2 gstbase-1.0 gstreamer-1.0
-WANTLIB += gstvideo-1.0 gthread-2.0 gtk-3 harfbuzz json-glib-1.0
-WANTLIB += lzma m orc-0.4 pango-1.0 pangocairo-1.0 pangoft2-1.0
-WANTLIB += pcre pixman-1 png pthread pthread-stubs rest-0.7 soup-2.4
-WANTLIB += soup-gnome-2.4 sqlite3 xcb xcb-render xcb-shm xml2
-WANTLIB += z
+WANTLIB += c cairo cairo-gobject dbus-1 enchant epoxy expat ffi
+WANTLIB += fontconfig freetype gdk-3 gdk_pixbuf-2.0 gio-2.0 glib-2.0
+WANTLIB += gmodule-2.0 gobject-2.0 graphite2 gspell-1 gstbase-1.0
+WANTLIB += gstreamer-1.0 gstvideo-1.0 gthread-2.0 gtk-3 harfbuzz
+WANTLIB += json-glib-1.0 lzma m orc-0.4 pango-1.0 pangocairo-1.0
+WANTLIB += pangoft2-1.0 pcre pixman-1 png pthread pthread-stubs
+WANTLIB += rest-0.7 soup-2.4 soup-gnome-2.4 sqlite3 stdc++ xcb
+WANTLIB += xcb-render xcb-shm xml2 z
 
 MODULES =  devel/dconf \
devel/gettext \
@@ -35,12 +35,14 @@ MODGNOME_TOOLS =desktop-file-utils \
gtk-update-icon-cache \
vala
 
-LIB_DEPENDS =  devel/json-glib \
+LIB_DEPENDS =  databases/sqlite3 \
+   devel/json-glib \
devel/libsoup>=2.4 \
devel/orc \
net/librest>=0.7.91 \
multimedia/gstreamer1/core \
multimedia/gstreamer1/plugins-base \
+   textproc/gspell \
x11/gtk+3
 
 RUN_DEPENDS =  multimedia/gstreamer1/plugins-bad \
Index: distinfo
===
RCS file: /cvs/ports/net/corebird/distinfo,v
retrieving revision 1.6
diff -u -p -r1.6 distinfo
--- distinfo28 Dec 2016 17:10:54 -  1.6
+++ distinfo19 Jan 2017 11:18:25 -
@@ -1,2 +1,2 @@
-SHA256 (corebird-1.3.3.tar.xz) = vssIS1wv0gmgOux3vv2SLClvOOFqIJjiKB6PBY9ef6g=
-SIZE (corebird-1.3.3.tar.xz) = 635980
+SHA256 (corebird-1.4.1.tar.xz) = 9X5diTtmpchfsNs6EyGwAxHWwCo53t/JXJyGejFOgAk=
+SIZE (corebird-1.4.1.tar.xz) = 648564
Index: pkg/PLIST
===
RCS file: /cvs/ports/net/corebird/pkg/PLIST,v
retrieving revision 1.3
diff

Re: [NEW] editors/ghostwriter

2017-01-17 Thread Adam Wolk
On Tue, Jan 17, 2017 at 01:46:22PM +0100, Landry Breuil wrote:
> On Tue, Jan 17, 2017 at 01:12:07PM +0100, Adam Wolk wrote:
> > Hi ports@
> > 
> > Attaching a new port editors/ghostwriter
> > 
> >  - https://wereturtle.github.io/ghostwriter/
> >  - https://github.com/wereturtle/ghostwriter
> > 
> > Feedback? OK's?
> 
> As for the port itself, looks good, maybe just remove the leading 'a' in
> COMMENT and put the BDEPs on their own line (ie newline between guic and
> desktop-file-utils)
> 

Removed the leading 'a' and split dependencies to separate lines.

> ok to import it.
> 
> Landry
> 

Imported. Thanks!

Regards,
Adam



[NEW] editors/ghostwriter

2017-01-17 Thread Adam Wolk
Hi ports@

Attaching a new port editors/ghostwriter

 - https://wereturtle.github.io/ghostwriter/
 - https://github.com/wereturtle/ghostwriter

Feedback? OK's?

$ cat pkg/DESCR
ghostwriter is a Windows and Unix text editor for Markdown, providing a
relaxing, distraction-free writing environment, whether your masterpiece
be that next blog post, your school paper, or your NaNoWriMo novel.

* Syntax highlighting of Markdown
* Navigation of document headings
* Full-screen mode
* Focus mode that highlights the current sentence, line, three lines, or
  paragraph
* Two built-in themes, one light and one dark
* Theme creator for custom colors and background images
* Spell checking with Hunspell
* A Live word count
* A live HTML preview as you type
* Use of custom CSS style sheets for HTML preview
* Image URL insertion via dragging and dropping an image file into the editor
* Sundown processor built in for preview and export to HTML
* Interoperability with the following Markdown processors (if installed) for
  preview and export to popular document formats, such as PDF, ODT, and RTF:
  * Pandoc
  * MultiMarkdown
  * Discount
  * cmark


ghostwriter.tar.gz
Description: application/tar-gz


[UPDATE] net/corebird 1.3.3 => 1.4.1

2017-01-16 Thread Adam Wolk
Hi ports@

Attaching an update for net/corebird

port changes:
  - new dependency on textproc/gspell for spellchecking
  - 2 new locale files (PLIST regenerated)
  - WANTLIB regenerated, new entries are enchant, gspell-1
and stdc++

upstream changelog:
---
Version 1.4.1 (2017-01-08)

Changelog:
  - Lower the required gspell version to 1.0
  - Work around a binding problem in the gtk+ 3.18 vapi of vala 0.30

Version 1.4 (2017-01-06)

Changelog:
  - Images in quoted tweets now look more like they actually belong to
the quoted tweet instead of the quoting tweet.
  - Allow deleting tweets from the tweet info page and not just from
timelines.
  - Fix the user completion not showing all possible results
  - Focus the already opened window for an account if the account gets
selected in the accounts popover
  - Avoid window resizing when typing in the Direct Message text box
  - Add 'q' accelerator to tweet rows for quoting
  - Add spellchecking. This add a dependency to gspell
  - Increase gtk+ dependency to 3.18 to get rid of some workarounds.
  - Increase avatar size in profiles and slightly overlap them over the
banner. Also, always show the full banner.
  - Fix completion popup positioning under Wayland
  - Add experimental meson build files
  - Fix some videos not playing correctly
  - Add a minimal video progress indicator to the video dialog
  - Fix the retweet/favorite count updating in the tweet info page

---

Feedback? OK's?
Index: Makefile
===
RCS file: /cvs/ports/net/corebird/Makefile,v
retrieving revision 1.8
diff -u -p -r1.8 Makefile
--- Makefile28 Dec 2016 17:10:54 -  1.8
+++ Makefile16 Jan 2017 22:04:17 -
@@ -2,7 +2,7 @@
 
 COMMENT =  native Gtk+ Twitter client
 
-V =1.3.3
+V =1.4.1
 DISTNAME = corebird-${V}
 MASTER_SITES = 
https://github.com/baedert/corebird/releases/download/${V}/
 
@@ -18,14 +18,14 @@ PERMIT_PACKAGE_CDROM =  Yes
 
 WANTLIB += ICE SM X11 Xcomposite Xcursor Xdamage Xext Xfixes Xi
 WANTLIB += Xinerama Xrandr Xrender atk-1.0 atk-bridge-2.0 atspi
-WANTLIB += c cairo cairo-gobject dbus-1 epoxy expat ffi fontconfig
-WANTLIB += freetype gdk-3 gdk_pixbuf-2.0 gio-2.0 glib-2.0
-WANTLIB += gmodule-2.0 gobject-2.0 graphite2 gstbase-1.0 gstreamer-1.0
-WANTLIB += gstvideo-1.0 gthread-2.0 gtk-3 harfbuzz json-glib-1.0
-WANTLIB += lzma m orc-0.4 pango-1.0 pangocairo-1.0 pangoft2-1.0
-WANTLIB += pcre pixman-1 png pthread pthread-stubs rest-0.7 soup-2.4
-WANTLIB += soup-gnome-2.4 sqlite3 xcb xcb-render xcb-shm xml2
-WANTLIB += z
+WANTLIB += c cairo cairo-gobject dbus-1 enchant epoxy expat ffi
+WANTLIB += fontconfig freetype gdk-3 gdk_pixbuf-2.0 gio-2.0 glib-2.0
+WANTLIB += gmodule-2.0 gobject-2.0 graphite2 gspell-1 gstbase-1.0
+WANTLIB += gstreamer-1.0 gstvideo-1.0 gthread-2.0 gtk-3 harfbuzz
+WANTLIB += json-glib-1.0 lzma m orc-0.4 pango-1.0 pangocairo-1.0
+WANTLIB += pangoft2-1.0 pcre pixman-1 png pthread pthread-stubs
+WANTLIB += rest-0.7 soup-2.4 soup-gnome-2.4 sqlite3 stdc++ xcb
+WANTLIB += xcb-render xcb-shm xml2 z
 
 MODULES =  devel/dconf \
devel/gettext \
@@ -41,6 +41,7 @@ LIB_DEPENDS = devel/json-glib \
net/librest>=0.7.91 \
multimedia/gstreamer1/core \
multimedia/gstreamer1/plugins-base \
+   textproc/gspell \
x11/gtk+3
 
 RUN_DEPENDS =  multimedia/gstreamer1/plugins-bad \
Index: distinfo
===
RCS file: /cvs/ports/net/corebird/distinfo,v
retrieving revision 1.6
diff -u -p -r1.6 distinfo
--- distinfo28 Dec 2016 17:10:54 -  1.6
+++ distinfo16 Jan 2017 22:04:17 -
@@ -1,2 +1,2 @@
-SHA256 (corebird-1.3.3.tar.xz) = vssIS1wv0gmgOux3vv2SLClvOOFqIJjiKB6PBY9ef6g=
-SIZE (corebird-1.3.3.tar.xz) = 635980
+SHA256 (corebird-1.4.1.tar.xz) = 9X5diTtmpchfsNs6EyGwAxHWwCo53t/JXJyGejFOgAk=
+SIZE (corebird-1.4.1.tar.xz) = 648564
Index: pkg/PLIST
===
RCS file: /cvs/ports/net/corebird/pkg/PLIST,v
retrieving revision 1.3
diff -u -p -r1.3 PLIST
--- pkg/PLIST   8 Sep 2016 00:07:10 -   1.3
+++ pkg/PLIST   16 Jan 2017 22:04:17 -
@@ -33,6 +33,7 @@ share/locale/es_VE/LC_MESSAGES/corebird.
 share/locale/fa/LC_MESSAGES/corebird.mo
 share/locale/fi/LC_MESSAGES/corebird.mo
 share/locale/fr/LC_MESSAGES/corebird.mo
+share/locale/gl/LC_MESSAGES/corebird.mo
 share/locale/hi/LC_MESSAGES/corebird.mo
 share/locale/hu/LC_MESSAGES/corebird.mo
 share/locale/id/LC_MESSAGES/corebird.mo
@@ -40,6 +41,7 @@ share/locale/it/LC_MESSAGES/corebird.mo
 share/locale/ja/LC_MESSAGES/corebird.mo
 share/locale/ko/LC_MESSAGES/corebird.mo
 share/locale/lt/LC_MESSAGES/corebird.mo
+share/locale/nb/LC_MESSAGES/corebird.mo
 share/locale/nl/LC_MESSAGES/co

Re: NEW: games/lugaru

2017-01-08 Thread Adam Wolk
On Mon, Jan 02, 2017 at 11:49:05PM -0700, Anthony J. Bentley wrote:
> Hi,
> 
> Lugaru (pronounced Loo-GAH-roo) is a cross-platform third-person action
> game. The main character, Turner, is an anthropomorphic rebel bunny rabbit
> with impressive combat skills. In his quest to find those responsible for
> slaughtering his village, he uncovers a far-reaching conspiracy involving
> the corrupt leaders of the rabbit republic and the starving wolves from a
> nearby den. Turner takes it upon himself to fight against their plot and
> save his fellow rabbits from slavery.
> 
> ok?
> 
> -- 
> Anthony J. Bentley

OK awolk@ works great on amd64



Re: [UPDATE] www/otter-browser 0.9.12-dev152 => 0.9.12

2017-01-08 Thread Adam Wolk
On Sun, Jan 08, 2017 at 09:59:29AM +0100, Adam Wolk wrote:
> On Sun, Jan 08, 2017 at 12:40:18AM +, Stuart Henderson wrote:
> > On 2017/01/07 18:26, Adam Wolk wrote:
> > > On Sun, Jan 01, 2017 at 09:45:00PM +0100, Adam Wolk wrote:
> > > > On Sun, Jan 01, 2017 at 09:20:50PM +0100, Adam Wolk wrote:
> > > > > On Sun, Jan 01, 2017 at 03:58:10PM +0100, Adam Wolk wrote:
> > > > > > Hi ports@
> > > > > > 
> > > > > > Attaching a bump to the latest otter browser weekly
> > > > > > 
> > > > > > Feedback? OK's?
> > > > > 
> > > > > Just got noticed from upstream tag a new beta will be
> > > > > released shortly. Will follow up with a diff after
> > > > > upstream tarballs show up.
> > > > > 
> > > > > Regards,
> > > > > Adam
> > > > > 
> > > > 
> > > > Attaching the diff for the new 0.9.12 beta full upstream
> > > > changelog since 0.9.11 will show up on http://otter-browser.org
> > > > in a day or two.
> > > > 
> > > > Feedback? OK's?
> > > 
> > > Ping.
> > 
> > Runtime seems fine, but I see a segfault at shutdown (which I don't
> > see with the current amd64 snapshot packages) - any ideas?
> > 

Looks like it's an upstream Qt issue:
 https://github.com/OtterBrowser/otter-browser/issues/889
 https://bugreports.qt.io/browse/QTBUG-50829

> > $ otter-browser  
> > No language dictionaries for the language: "C"
> > Segmentation fault (core dumped) 
> 
> I was able to reproduce the crash when closing the browser using the windows
> manager (ie. super - shift - q in i3) the crash doesn't seem to happen on 
> option
> exit (Menu -> exit).
> 
> I'll bring the issue up with upstream.
> 
> Regards,
> Adam
> 



Re: [UPDATE] www/otter-browser 0.9.12-dev152 => 0.9.12

2017-01-08 Thread Adam Wolk
On Sun, Jan 08, 2017 at 12:40:18AM +, Stuart Henderson wrote:
> On 2017/01/07 18:26, Adam Wolk wrote:
> > On Sun, Jan 01, 2017 at 09:45:00PM +0100, Adam Wolk wrote:
> > > On Sun, Jan 01, 2017 at 09:20:50PM +0100, Adam Wolk wrote:
> > > > On Sun, Jan 01, 2017 at 03:58:10PM +0100, Adam Wolk wrote:
> > > > > Hi ports@
> > > > > 
> > > > > Attaching a bump to the latest otter browser weekly
> > > > > 
> > > > > Feedback? OK's?
> > > > 
> > > > Just got noticed from upstream tag a new beta will be
> > > > released shortly. Will follow up with a diff after
> > > > upstream tarballs show up.
> > > > 
> > > > Regards,
> > > > Adam
> > > > 
> > > 
> > > Attaching the diff for the new 0.9.12 beta full upstream
> > > changelog since 0.9.11 will show up on http://otter-browser.org
> > > in a day or two.
> > > 
> > > Feedback? OK's?
> > 
> > Ping.
> 
> Runtime seems fine, but I see a segfault at shutdown (which I don't
> see with the current amd64 snapshot packages) - any ideas?
> 
> $ otter-browser  
> No language dictionaries for the language: "C"
> Segmentation fault (core dumped) 

I was able to reproduce the crash when closing the browser using the windows
manager (ie. super - shift - q in i3) the crash doesn't seem to happen on option
exit (Menu -> exit).

I'll bring the issue up with upstream.

Regards,
Adam



Re: [UPDATE] devel/zeal 0.2.1 => 0.3.1

2017-01-07 Thread Adam Wolk
On Sun, Jan 01, 2017 at 03:15:47PM +0100, Adam Wolk wrote:
> On Sun, Jan 01, 2017 at 03:02:10PM +0100, Antoine Jacoutot wrote:
> > On Sun, Jan 01, 2017 at 02:57:30PM +0100, Adam Wolk wrote:
> > > Hi ports@
> > > 
> > > Attaching a bump to devel/zeal
> > > 
> > > port changes:
> > >  - add x11/qt5/qt5base dependency to LIB_DEPENDS
> > >  - add Qt5Concurrent to WANTLIB (from qtbase)
> > >  - mark USE_WXNEEDED = Yes (uses qt webkit)
> > 
> > Hmm, can't the qtwebkit port be patched like www/webkitgtk4 so that 
> > pkg-config
> > files or whatever cmake magic pass the proper -Wl flags insead of adding 
> > USE_WXNEEDED to all qtwebkit consumers?
> 
> I just checked why www/otter-browser doesn't need it as a qtwebkit consumer 
> and
> in that case devel/cmake was patched to pass the wx flags during the build.
> 
> This port uses devel/qmake to build hence it doens't get the same automatic
> benefit. 
> 
> x11/qt5/qtwebkit/ has USE_WNEEDED = Yes in what way could it impact producing 
> a
> downstream (consumer) binary that's also marked as wxneeded?
> 
> > Also we usually put USE_WXNEEDED at the top of the Makefile.
> > 
> > 
> 
> Will move that to the top of the file before comitting (assuming I will get an
> OK of course ;)).
> 
> Regards,
> Adam
> 

Ping for reviews & a discussion on how to handle WXNEEDED with this port.



Re: [UPDATE] www/otter-browser 0.9.12-dev152 => 0.9.12

2017-01-07 Thread Adam Wolk
On Sun, Jan 01, 2017 at 09:45:00PM +0100, Adam Wolk wrote:
> On Sun, Jan 01, 2017 at 09:20:50PM +0100, Adam Wolk wrote:
> > On Sun, Jan 01, 2017 at 03:58:10PM +0100, Adam Wolk wrote:
> > > Hi ports@
> > > 
> > > Attaching a bump to the latest otter browser weekly
> > > 
> > > Feedback? OK's?
> > 
> > Just got noticed from upstream tag a new beta will be
> > released shortly. Will follow up with a diff after
> > upstream tarballs show up.
> > 
> > Regards,
> > Adam
> > 
> 
> Attaching the diff for the new 0.9.12 beta full upstream
> changelog since 0.9.11 will show up on http://otter-browser.org
> in a day or two.
> 
> Feedback? OK's?

Ping.

> Index: Makefile
> ===
> RCS file: /cvs/ports/www/otter-browser/Makefile,v
> retrieving revision 1.21
> diff -u -p -r1.21 Makefile
> --- Makefile  25 Dec 2016 14:34:40 -  1.21
> +++ Makefile  1 Jan 2017 20:42:56 -
> @@ -3,9 +3,7 @@
>  COMMENT =browser aiming to recreate classic Opera (12.x) UI 
> using Qt5
>  
>  MASTER_SITES =   ${MASTER_SITE_SOURCEFORGE:=otter-browser/}
> -DISTNAME =   otter-browser-0.9.12-dev152
> -PKGNAME =${DISTNAME:S/-dev/pre/}
> -REVISION =   0
> +DISTNAME =   otter-browser-0.9.12
>  
>  EXTRACT_SUFX =   .tar.bz2
>  
> @@ -21,7 +19,7 @@ PERMIT_PACKAGE_CDROM =  Yes
>  WANTLIB +=   c m pthread Qt5Core Qt5DBus Qt5Gui Qt5Multimedia
>  WANTLIB +=   Qt5Network Qt5PrintSupport Qt5Qml Qt5WebKit
>  WANTLIB +=   Qt5WebKitWidgets Qt5Widgets Qt5XmlPatterns
> -WANTLIB +=   hunspell-1.3
> +WANTLIB +=   hunspell-1.3 X11
>  
>  
>  RUN_DEPENDS =devel/desktop-file-utils x11/gtk+3,-guic
> Index: distinfo
> ===
> RCS file: /cvs/ports/www/otter-browser/distinfo,v
> retrieving revision 1.15
> diff -u -p -r1.15 distinfo
> --- distinfo  6 Dec 2016 13:00:19 -   1.15
> +++ distinfo  1 Jan 2017 20:42:56 -
> @@ -1,2 +1,2 @@
> -SHA256 (otter-browser-0.9.12-dev152.tar.bz2) = 
> JMFAyIMKNGpE0mBoMfrrIiUnD4WJbhyrMS0kne/xpgQ=
> -SIZE (otter-browser-0.9.12-dev152.tar.bz2) = 4810750
> +SHA256 (otter-browser-0.9.12.tar.bz2) = 
> 9mW9zDrzlufFz1iiI50BNuQ3vyLwOW/mvZeFaFa4sVc=
> +SIZE (otter-browser-0.9.12.tar.bz2) = 2805058



Re: textproc/the_silver_searcher: fix heap buffer overflow

2017-01-04 Thread Adam Wolk
On Tue, Jan 03, 2017 at 11:12:19PM +0100, Theo Buehler wrote:
> There is an annoying non-deterministic crash present since at least 0.32
> that occurs when ag is given an absolute path as root of the tree to search:
> 
> $ for f in $(jot 10); do ag pledge /usr/src/bin/pax >/dev/null; done
> Segmentation fault (core dumped)
> Segmentation fault (core dumped)
> Segmentation fault (core dumped)
> Segmentation fault (core dumped)
> $
> 

I was able to reliably reproduce the bug with the sample one-liner you provided.

> 
> Here's a diff with the revision bump plus a tarball with the four patches.
> 
> ok to commit this until a new release is cut?
>

OK awolk@ after applying the patch I can't reproduce the bug.



Re: [UPDATE] www/otter-browser 0.9.12-dev152 => 0.9.12

2017-01-01 Thread Adam Wolk
On Sun, Jan 01, 2017 at 09:20:50PM +0100, Adam Wolk wrote:
> On Sun, Jan 01, 2017 at 03:58:10PM +0100, Adam Wolk wrote:
> > Hi ports@
> > 
> > Attaching a bump to the latest otter browser weekly
> > 
> > Feedback? OK's?
> 
> Just got noticed from upstream tag a new beta will be
> released shortly. Will follow up with a diff after
> upstream tarballs show up.
> 
> Regards,
> Adam
> 

Attaching the diff for the new 0.9.12 beta full upstream
changelog since 0.9.11 will show up on http://otter-browser.org
in a day or two.

Feedback? OK's?
Index: Makefile
===
RCS file: /cvs/ports/www/otter-browser/Makefile,v
retrieving revision 1.21
diff -u -p -r1.21 Makefile
--- Makefile25 Dec 2016 14:34:40 -  1.21
+++ Makefile1 Jan 2017 20:42:56 -
@@ -3,9 +3,7 @@
 COMMENT =  browser aiming to recreate classic Opera (12.x) UI 
using Qt5
 
 MASTER_SITES = ${MASTER_SITE_SOURCEFORGE:=otter-browser/}
-DISTNAME = otter-browser-0.9.12-dev152
-PKGNAME =  ${DISTNAME:S/-dev/pre/}
-REVISION = 0
+DISTNAME = otter-browser-0.9.12
 
 EXTRACT_SUFX = .tar.bz2
 
@@ -21,7 +19,7 @@ PERMIT_PACKAGE_CDROM =Yes
 WANTLIB += c m pthread Qt5Core Qt5DBus Qt5Gui Qt5Multimedia
 WANTLIB += Qt5Network Qt5PrintSupport Qt5Qml Qt5WebKit
 WANTLIB += Qt5WebKitWidgets Qt5Widgets Qt5XmlPatterns
-WANTLIB += hunspell-1.3
+WANTLIB += hunspell-1.3 X11
 
 
 RUN_DEPENDS =  devel/desktop-file-utils x11/gtk+3,-guic
Index: distinfo
===
RCS file: /cvs/ports/www/otter-browser/distinfo,v
retrieving revision 1.15
diff -u -p -r1.15 distinfo
--- distinfo6 Dec 2016 13:00:19 -   1.15
+++ distinfo1 Jan 2017 20:42:56 -
@@ -1,2 +1,2 @@
-SHA256 (otter-browser-0.9.12-dev152.tar.bz2) = 
JMFAyIMKNGpE0mBoMfrrIiUnD4WJbhyrMS0kne/xpgQ=
-SIZE (otter-browser-0.9.12-dev152.tar.bz2) = 4810750
+SHA256 (otter-browser-0.9.12.tar.bz2) = 
9mW9zDrzlufFz1iiI50BNuQ3vyLwOW/mvZeFaFa4sVc=
+SIZE (otter-browser-0.9.12.tar.bz2) = 2805058


  1   2   3   4   >