Re: UPDATE: telephony/baresip/{baresip,re,rem}

2018-05-06 Thread Ingo Feinerer
On Sun, May 06, 2018 at 03:34:20PM +0100, Stuart Henderson wrote:
> > > Index: baresip/Makefile
> > > ===
> > > RCS file: /cvs/ports/telephony/baresip/baresip/Makefile,v
> > > retrieving revision 1.20
> > > diff -u -p -r1.20 Makefile
> > > --- baresip/Makefile  11 Mar 2018 18:48:59 -  1.20
> > > +++ baresip/Makefile  25 Apr 2018 15:50:36 -
> > > @@ -3,7 +3,7 @@
> > >  COMMENT-main =   modular SIP User-Agent with audio and video 
> > > support
> > >  COMMENT-gtk2 =   GTK+2-based modules for baresip
> > >  
> > > -V =  0.5.8
> > > +V =  0.5.9
> > >  DISTNAME =   baresip-$V
> > >  PKGNAME-main =   baresip-$V
> > >  PKGNAME-gtk2 =   baresip-gtk2-$V
> > > @@ -14,7 +14,7 @@ WANTLIB-main += re rem sndfile sndio spa
> > >  WANTLIB-main += twolame v4l2 vpx x264 x265
> > >  
> > >  WANTLIB-gtk2 += X11 Xcomposite Xcursor Xdamage Xext Xfixes Xi Xinerama
> > > -WANTLIB-gtk2 += Xrandr Xrender atk-1.0 cairo fontconfig freetype 
> > > gdk-x11-2.0
> > > +WANTLIB-gtk2 += Xrandr Xrender atk-1.0 cairo fontconfig freetype fribidi 
> > > gdk-x11-2.0
> > >  WANTLIB-gtk2 += gdk_pixbuf-2.0 gio-2.0 glib-2.0 gobject-2.0 gstapp-1.0
> > >  WANTLIB-gtk2 += gstbase-1.0 gstreamer-1.0 gtk-x11-2.0 intl mpg123 
> > > pango-1.0
> > >  WANTLIB-gtk2 += pangocairo-1.0 pangoft2-1.0 re z
> > > @@ -37,7 +37,8 @@ LIB_DEPENDS-main =  audio/gsm \
> > >   telephony/baresip/re>=0.5.7 \
> > >   telephony/baresip/rem>=0.5.2 \
> > >   telephony/spandsp
> > > -LIB_DEPENDS-gtk2 =   multimedia/gstreamer1/plugins-base \
> > > +LIB_DEPENDS-gtk2 =   devel/fribidi \
> 
> I think fribidi should only be a WANTLIB (unless baresip started
> using fribidi directly itself).

Indeed. Thanks for spotting!

> Rest reads good, did you check the shared libraries for ABI changes to
> identify whether bumps are needed?

Yes, I checked them (both reading upstream change logs and using
/usr/src/lib/check_sym).

I assume after the fribidi LIB_DEPENDS removal this implies your OK?
I'll commit the update in the next days under this assumption (or
earlier with an explicit OK).

Best regards,
Ingo



Re: UPDATE: telephony/baresip/{baresip,re,rem}

2018-05-06 Thread Stuart Henderson
On 2018/05/06 08:19, Ingo Feinerer wrote:
> Ping (maintainer timeout)
> (original posting at https://marc.info/?l=openbsd-ports=152467183305081=2)
> 
> On Wed, Apr 25, 2018 at 05:55:44PM +0200, Ingo Feinerer wrote:
> > Hi,
> > 
> > update telephony/baresip/baresip 0.5.8 -> 0.5.9
> > update telephony/baresip/re  0.5.7 -> 0.5.8
> > update telephony/baresip/rem 0.5.2 -> 0.5.3
> > 
> > ChangeLogs:
> > 
> > http://lists.creytiv.com/pipermail/re-devel/2018-April/001186.html
> > http://lists.creytiv.com/pipermail/re-devel/2018-April/001185.html
> > http://lists.creytiv.com/pipermail/re-devel/2018-April/001184.html
> > 
> > OK?
> > 
> > Best regards,
> > Ingo
> > 
> > Index: baresip/Makefile
> > ===
> > RCS file: /cvs/ports/telephony/baresip/baresip/Makefile,v
> > retrieving revision 1.20
> > diff -u -p -r1.20 Makefile
> > --- baresip/Makefile11 Mar 2018 18:48:59 -  1.20
> > +++ baresip/Makefile25 Apr 2018 15:50:36 -
> > @@ -3,7 +3,7 @@
> >  COMMENT-main = modular SIP User-Agent with audio and video 
> > support
> >  COMMENT-gtk2 = GTK+2-based modules for baresip
> >  
> > -V =0.5.8
> > +V =0.5.9
> >  DISTNAME = baresip-$V
> >  PKGNAME-main = baresip-$V
> >  PKGNAME-gtk2 = baresip-gtk2-$V
> > @@ -14,7 +14,7 @@ WANTLIB-main += re rem sndfile sndio spa
> >  WANTLIB-main += twolame v4l2 vpx x264 x265
> >  
> >  WANTLIB-gtk2 += X11 Xcomposite Xcursor Xdamage Xext Xfixes Xi Xinerama
> > -WANTLIB-gtk2 += Xrandr Xrender atk-1.0 cairo fontconfig freetype 
> > gdk-x11-2.0
> > +WANTLIB-gtk2 += Xrandr Xrender atk-1.0 cairo fontconfig freetype fribidi 
> > gdk-x11-2.0
> >  WANTLIB-gtk2 += gdk_pixbuf-2.0 gio-2.0 glib-2.0 gobject-2.0 gstapp-1.0
> >  WANTLIB-gtk2 += gstbase-1.0 gstreamer-1.0 gtk-x11-2.0 intl mpg123 pango-1.0
> >  WANTLIB-gtk2 += pangocairo-1.0 pangoft2-1.0 re z
> > @@ -37,7 +37,8 @@ LIB_DEPENDS-main =audio/gsm \
> > telephony/baresip/re>=0.5.7 \
> > telephony/baresip/rem>=0.5.2 \
> > telephony/spandsp
> > -LIB_DEPENDS-gtk2 = multimedia/gstreamer1/plugins-base \
> > +LIB_DEPENDS-gtk2 = devel/fribidi \

I think fribidi should only be a WANTLIB (unless baresip started
using fribidi directly itself).

Rest reads good, did you check the shared libraries for ABI changes to
identify whether bumps are needed?

> > +   multimedia/gstreamer1/plugins-base \
> > telephony/baresip/re>=0.5.7 \
> > x11/gtk+2,-main
> >  RUN_DEPENDS-gtk2 = telephony/baresip/baresip,-main>=$V \
> > Index: baresip/distinfo
> > ===
> > RCS file: /cvs/ports/telephony/baresip/baresip/distinfo,v
> > retrieving revision 1.14
> > diff -u -p -r1.14 distinfo
> > --- baresip/distinfo11 Mar 2018 18:48:59 -  1.14
> > +++ baresip/distinfo25 Apr 2018 15:50:36 -
> > @@ -1,2 +1,2 @@
> > -SHA256 (baresip-0.5.8.tar.gz) = 
> > 2fEdpQ/WyTWatHhhi108EyR0qDj+n2aMJJ+dWgfyZmI=
> > -SIZE (baresip-0.5.8.tar.gz) = 584406
> > +SHA256 (baresip-0.5.9.tar.gz) = 
> > 7t3Pue+ACi2JIAi9maIj6n+pO6hYhswGPGu4HLkjbgk=
> > +SIZE (baresip-0.5.9.tar.gz) = 589680
> > Index: baresip/patches/patch-src_config_c
> > ===
> > RCS file: /cvs/ports/telephony/baresip/baresip/patches/patch-src_config_c,v
> > retrieving revision 1.13
> > diff -u -p -r1.13 patch-src_config_c
> > --- baresip/patches/patch-src_config_c  11 Mar 2018 18:48:59 -  
> > 1.13
> > +++ baresip/patches/patch-src_config_c  25 Apr 2018 15:50:36 -
> > @@ -2,16 +2,16 @@ $OpenBSD: patch-src_config_c,v 1.13 2018
> >  Index: src/config.c
> >  --- src/config.c.orig
> >  +++ src/config.c
> > -@@ -505,7 +505,7 @@ static const char *default_video_device(void)
> > - #endif
> > - 
> > +@@ -511,7 +511,7 @@ static const char *default_video_device(void)
> > + #elif defined (WIN32)
> > +   return "dshow,nil";
> >   #else
> >  -  return "v4l2,/dev/video0";
> >  +  return "v4l2,/dev/video";
> >   #endif
> >   }
> >   
> > -@@ -753,31 +753,31 @@ int config_write_template(const char *file, const 
> > stru
> > +@@ -761,31 +761,31 @@ int config_write_template(const char *file, const 
> > stru
> > (void)re_fprintf(f, "module\t\t\t" MOD_PRE "stdio" MOD_EXT "\n");
> >   #endif
> > (void)re_fprintf(f, "#module\t\t\t" MOD_PRE "cons" MOD_EXT "\n");
> > @@ -60,7 +60,7 @@ Index: src/config.c
> > (void)re_fprintf(f, "#module\t\t\t" MOD_PRE "plc" MOD_EXT "\n");
> >   
> > (void)re_fprintf(f, "\n# Audio driver Modules\n");
> > -@@ -796,8 +796,8 @@ int config_write_template(const char *file, const stru
> > +@@ -804,8 +804,8 @@ int config_write_template(const char *file, const stru
> > (void)re_fprintf(f, "module\t\t\t" MOD_PRE "alsa" MOD_EXT "\n");

Re: UPDATE: telephony/baresip/{baresip,re,rem}

2018-05-06 Thread Ingo Feinerer
Ping (maintainer timeout)
(original posting at https://marc.info/?l=openbsd-ports=152467183305081=2)

On Wed, Apr 25, 2018 at 05:55:44PM +0200, Ingo Feinerer wrote:
> Hi,
> 
> update telephony/baresip/baresip 0.5.8 -> 0.5.9
> update telephony/baresip/re  0.5.7 -> 0.5.8
> update telephony/baresip/rem 0.5.2 -> 0.5.3
> 
> ChangeLogs:
> 
> http://lists.creytiv.com/pipermail/re-devel/2018-April/001186.html
> http://lists.creytiv.com/pipermail/re-devel/2018-April/001185.html
> http://lists.creytiv.com/pipermail/re-devel/2018-April/001184.html
> 
> OK?
> 
> Best regards,
> Ingo
> 
> Index: baresip/Makefile
> ===
> RCS file: /cvs/ports/telephony/baresip/baresip/Makefile,v
> retrieving revision 1.20
> diff -u -p -r1.20 Makefile
> --- baresip/Makefile  11 Mar 2018 18:48:59 -  1.20
> +++ baresip/Makefile  25 Apr 2018 15:50:36 -
> @@ -3,7 +3,7 @@
>  COMMENT-main =   modular SIP User-Agent with audio and video 
> support
>  COMMENT-gtk2 =   GTK+2-based modules for baresip
>  
> -V =  0.5.8
> +V =  0.5.9
>  DISTNAME =   baresip-$V
>  PKGNAME-main =   baresip-$V
>  PKGNAME-gtk2 =   baresip-gtk2-$V
> @@ -14,7 +14,7 @@ WANTLIB-main += re rem sndfile sndio spa
>  WANTLIB-main += twolame v4l2 vpx x264 x265
>  
>  WANTLIB-gtk2 += X11 Xcomposite Xcursor Xdamage Xext Xfixes Xi Xinerama
> -WANTLIB-gtk2 += Xrandr Xrender atk-1.0 cairo fontconfig freetype gdk-x11-2.0
> +WANTLIB-gtk2 += Xrandr Xrender atk-1.0 cairo fontconfig freetype fribidi 
> gdk-x11-2.0
>  WANTLIB-gtk2 += gdk_pixbuf-2.0 gio-2.0 glib-2.0 gobject-2.0 gstapp-1.0
>  WANTLIB-gtk2 += gstbase-1.0 gstreamer-1.0 gtk-x11-2.0 intl mpg123 pango-1.0
>  WANTLIB-gtk2 += pangocairo-1.0 pangoft2-1.0 re z
> @@ -37,7 +37,8 @@ LIB_DEPENDS-main =  audio/gsm \
>   telephony/baresip/re>=0.5.7 \
>   telephony/baresip/rem>=0.5.2 \
>   telephony/spandsp
> -LIB_DEPENDS-gtk2 =   multimedia/gstreamer1/plugins-base \
> +LIB_DEPENDS-gtk2 =   devel/fribidi \
> + multimedia/gstreamer1/plugins-base \
>   telephony/baresip/re>=0.5.7 \
>   x11/gtk+2,-main
>  RUN_DEPENDS-gtk2 =   telephony/baresip/baresip,-main>=$V \
> Index: baresip/distinfo
> ===
> RCS file: /cvs/ports/telephony/baresip/baresip/distinfo,v
> retrieving revision 1.14
> diff -u -p -r1.14 distinfo
> --- baresip/distinfo  11 Mar 2018 18:48:59 -  1.14
> +++ baresip/distinfo  25 Apr 2018 15:50:36 -
> @@ -1,2 +1,2 @@
> -SHA256 (baresip-0.5.8.tar.gz) = 2fEdpQ/WyTWatHhhi108EyR0qDj+n2aMJJ+dWgfyZmI=
> -SIZE (baresip-0.5.8.tar.gz) = 584406
> +SHA256 (baresip-0.5.9.tar.gz) = 7t3Pue+ACi2JIAi9maIj6n+pO6hYhswGPGu4HLkjbgk=
> +SIZE (baresip-0.5.9.tar.gz) = 589680
> Index: baresip/patches/patch-src_config_c
> ===
> RCS file: /cvs/ports/telephony/baresip/baresip/patches/patch-src_config_c,v
> retrieving revision 1.13
> diff -u -p -r1.13 patch-src_config_c
> --- baresip/patches/patch-src_config_c11 Mar 2018 18:48:59 -  
> 1.13
> +++ baresip/patches/patch-src_config_c25 Apr 2018 15:50:36 -
> @@ -2,16 +2,16 @@ $OpenBSD: patch-src_config_c,v 1.13 2018
>  Index: src/config.c
>  --- src/config.c.orig
>  +++ src/config.c
> -@@ -505,7 +505,7 @@ static const char *default_video_device(void)
> - #endif
> - 
> +@@ -511,7 +511,7 @@ static const char *default_video_device(void)
> + #elif defined (WIN32)
> + return "dshow,nil";
>   #else
>  -return "v4l2,/dev/video0";
>  +return "v4l2,/dev/video";
>   #endif
>   }
>   
> -@@ -753,31 +753,31 @@ int config_write_template(const char *file, const stru
> +@@ -761,31 +761,31 @@ int config_write_template(const char *file, const stru
>   (void)re_fprintf(f, "module\t\t\t" MOD_PRE "stdio" MOD_EXT "\n");
>   #endif
>   (void)re_fprintf(f, "#module\t\t\t" MOD_PRE "cons" MOD_EXT "\n");
> @@ -60,7 +60,7 @@ Index: src/config.c
>   (void)re_fprintf(f, "#module\t\t\t" MOD_PRE "plc" MOD_EXT "\n");
>   
>   (void)re_fprintf(f, "\n# Audio driver Modules\n");
> -@@ -796,8 +796,8 @@ int config_write_template(const char *file, const stru
> +@@ -804,8 +804,8 @@ int config_write_template(const char *file, const stru
>   (void)re_fprintf(f, "module\t\t\t" MOD_PRE "alsa" MOD_EXT "\n");
>   (void)re_fprintf(f, "#module\t\t\t" MOD_PRE "pulse" MOD_EXT "\n");
>   #endif
> @@ -71,7 +71,7 @@ Index: src/config.c
>   (void)re_fprintf(f, "#module\t\t\t" MOD_PRE "aubridge" MOD_EXT "\n");
>   (void)re_fprintf(f, "#module\t\t\t" MOD_PRE "aufile" MOD_EXT "\n");
>   
> -@@ -809,9 +809,9 @@ int config_write_template(const char *file, const stru
> +@@ -817,9 +817,9 @@ int config_write_template(const char *file, const stru
>   #else
>   (void)re_fprintf(f, 

UPDATE: telephony/baresip/{baresip,re,rem}

2018-04-25 Thread Ingo Feinerer
Hi,

update telephony/baresip/baresip 0.5.8 -> 0.5.9
update telephony/baresip/re  0.5.7 -> 0.5.8
update telephony/baresip/rem 0.5.2 -> 0.5.3

ChangeLogs:

http://lists.creytiv.com/pipermail/re-devel/2018-April/001186.html
http://lists.creytiv.com/pipermail/re-devel/2018-April/001185.html
http://lists.creytiv.com/pipermail/re-devel/2018-April/001184.html

OK?

Best regards,
Ingo

Index: baresip/Makefile
===
RCS file: /cvs/ports/telephony/baresip/baresip/Makefile,v
retrieving revision 1.20
diff -u -p -r1.20 Makefile
--- baresip/Makefile11 Mar 2018 18:48:59 -  1.20
+++ baresip/Makefile25 Apr 2018 15:50:36 -
@@ -3,7 +3,7 @@
 COMMENT-main = modular SIP User-Agent with audio and video support
 COMMENT-gtk2 = GTK+2-based modules for baresip
 
-V =0.5.8
+V =0.5.9
 DISTNAME = baresip-$V
 PKGNAME-main = baresip-$V
 PKGNAME-gtk2 = baresip-gtk2-$V
@@ -14,7 +14,7 @@ WANTLIB-main += re rem sndfile sndio spa
 WANTLIB-main += twolame v4l2 vpx x264 x265
 
 WANTLIB-gtk2 += X11 Xcomposite Xcursor Xdamage Xext Xfixes Xi Xinerama
-WANTLIB-gtk2 += Xrandr Xrender atk-1.0 cairo fontconfig freetype gdk-x11-2.0
+WANTLIB-gtk2 += Xrandr Xrender atk-1.0 cairo fontconfig freetype fribidi 
gdk-x11-2.0
 WANTLIB-gtk2 += gdk_pixbuf-2.0 gio-2.0 glib-2.0 gobject-2.0 gstapp-1.0
 WANTLIB-gtk2 += gstbase-1.0 gstreamer-1.0 gtk-x11-2.0 intl mpg123 pango-1.0
 WANTLIB-gtk2 += pangocairo-1.0 pangoft2-1.0 re z
@@ -37,7 +37,8 @@ LIB_DEPENDS-main =audio/gsm \
telephony/baresip/re>=0.5.7 \
telephony/baresip/rem>=0.5.2 \
telephony/spandsp
-LIB_DEPENDS-gtk2 = multimedia/gstreamer1/plugins-base \
+LIB_DEPENDS-gtk2 = devel/fribidi \
+   multimedia/gstreamer1/plugins-base \
telephony/baresip/re>=0.5.7 \
x11/gtk+2,-main
 RUN_DEPENDS-gtk2 = telephony/baresip/baresip,-main>=$V \
Index: baresip/distinfo
===
RCS file: /cvs/ports/telephony/baresip/baresip/distinfo,v
retrieving revision 1.14
diff -u -p -r1.14 distinfo
--- baresip/distinfo11 Mar 2018 18:48:59 -  1.14
+++ baresip/distinfo25 Apr 2018 15:50:36 -
@@ -1,2 +1,2 @@
-SHA256 (baresip-0.5.8.tar.gz) = 2fEdpQ/WyTWatHhhi108EyR0qDj+n2aMJJ+dWgfyZmI=
-SIZE (baresip-0.5.8.tar.gz) = 584406
+SHA256 (baresip-0.5.9.tar.gz) = 7t3Pue+ACi2JIAi9maIj6n+pO6hYhswGPGu4HLkjbgk=
+SIZE (baresip-0.5.9.tar.gz) = 589680
Index: baresip/patches/patch-src_config_c
===
RCS file: /cvs/ports/telephony/baresip/baresip/patches/patch-src_config_c,v
retrieving revision 1.13
diff -u -p -r1.13 patch-src_config_c
--- baresip/patches/patch-src_config_c  11 Mar 2018 18:48:59 -  1.13
+++ baresip/patches/patch-src_config_c  25 Apr 2018 15:50:36 -
@@ -2,16 +2,16 @@ $OpenBSD: patch-src_config_c,v 1.13 2018
 Index: src/config.c
 --- src/config.c.orig
 +++ src/config.c
-@@ -505,7 +505,7 @@ static const char *default_video_device(void)
- #endif
- 
+@@ -511,7 +511,7 @@ static const char *default_video_device(void)
+ #elif defined (WIN32)
+   return "dshow,nil";
  #else
 -  return "v4l2,/dev/video0";
 +  return "v4l2,/dev/video";
  #endif
  }
  
-@@ -753,31 +753,31 @@ int config_write_template(const char *file, const stru
+@@ -761,31 +761,31 @@ int config_write_template(const char *file, const stru
(void)re_fprintf(f, "module\t\t\t" MOD_PRE "stdio" MOD_EXT "\n");
  #endif
(void)re_fprintf(f, "#module\t\t\t" MOD_PRE "cons" MOD_EXT "\n");
@@ -60,7 +60,7 @@ Index: src/config.c
(void)re_fprintf(f, "#module\t\t\t" MOD_PRE "plc" MOD_EXT "\n");
  
(void)re_fprintf(f, "\n# Audio driver Modules\n");
-@@ -796,8 +796,8 @@ int config_write_template(const char *file, const stru
+@@ -804,8 +804,8 @@ int config_write_template(const char *file, const stru
(void)re_fprintf(f, "module\t\t\t" MOD_PRE "alsa" MOD_EXT "\n");
(void)re_fprintf(f, "#module\t\t\t" MOD_PRE "pulse" MOD_EXT "\n");
  #endif
@@ -71,7 +71,7 @@ Index: src/config.c
(void)re_fprintf(f, "#module\t\t\t" MOD_PRE "aubridge" MOD_EXT "\n");
(void)re_fprintf(f, "#module\t\t\t" MOD_PRE "aufile" MOD_EXT "\n");
  
-@@ -809,9 +809,9 @@ int config_write_template(const char *file, const stru
+@@ -817,9 +817,9 @@ int config_write_template(const char *file, const stru
  #else
(void)re_fprintf(f, "#module\t\t\t" MOD_PRE "avcodec" MOD_EXT "\n");
  #endif
@@ -84,7 +84,7 @@ Index: src/config.c
  
(void)re_fprintf(f, "\n# Video filter Modules (in encoding order)\n");
(void)re_fprintf(f, "#module\t\t\t" MOD_PRE "selfview" MOD_EXT "\n");
-@@ -832,14 +832,14 @@ int config_write_template(const char *file, const stru
+@@ -840,14 +840,14 @@ int