Re: [UPDATE] audio/p5-Audio-MPD 1.110560 -> 2.004 (Perl+MPD 5/5)

2019-02-22 Thread Charlene Wendling
Hi,

(lots of things sniped) 

On Fri, 22 Feb 2019 09:34:46 +0100
Landry Breuil wrote:

> audio/p5-Audio-MPD/patches/patch-lib_Audio_MPD_Types_pm should be
> removed, applied upstream.

Thanks a lot, i've forgotten to 'cvs rm' it when doing the original
update 5 months ago and didn't remember. So here is a new diff.

> I dunno if the dependency on Audio-MPD-Common-2.003 should be
> enforced, since upstream says there's no breaking changes.. but the
> major version bump might say the contrary.

You're right, i think JQUELIN has just not thought about changing
META stuff, see [1] where he recommends moving everything to 2.x. So 
i changed accordingly.

> I noticed that it had a dependency on List::Utils 1.45 - 1.50 is in
> base only since the 5.28 update it seems, but that's fine for
> -current - it just complains with the 1.42 that was in 6.4 before.

In fact it requires devel/p5-List-AllUtils, that has been imported a
week ago, after Perl 5.28. The situation is similar to the GeoIP2
stack.

> On Fri, Feb 22, 2019 at 09:10:57AM +0100, Landry Breuil wrote:

> t/00-report-prereqs.t  ok
> Could not create socket: No route to host
> # Looks like you planned 20 tests but ran 3.
> t/20-connection.t  
> Dubious, test returned 255 (wstat 65280, 0xff00)
> Failed 17/20 subtests 
> (less 1 skipped subtest: 2 okay)
> 
> (and the other tests pass)
> 

If you can reproduce it consistently, tell me how, because no matter how
hard i try, i can't. If that's the case, can you please send the output
of: 

cd `make show=WRKSRC` && ./Build test verbose=1


Charlène.


[1] https://rt.cpan.org/Public/Bug/Display.html?id=91082#txn-1374402


Index: Makefile
===
RCS file: /cvs/ports/audio/p5-Audio-MPD/Makefile,v
retrieving revision 1.28
diff -u -p -u -p -r1.28 Makefile
--- Makefile20 Mar 2016 19:56:06 -  1.28
+++ Makefile22 Feb 2019 18:38:30 -
@@ -1,8 +1,7 @@
 # $OpenBSD: Makefile,v 1.28 2016/03/20 19:56:06 naddy Exp $
 
 COMMENT=   module for communicating with MPD servers
-DISTNAME=  Audio-MPD-1.110560
-REVISION=  3
+DISTNAME=  Audio-MPD-2.004
 CATEGORIES=audio
 
 # perl
@@ -10,18 +9,24 @@ PERMIT_PACKAGE_CDROM=Yes
 
 MODULES=   cpan
 PKG_ARCH=  *
-CPAN_AUTHOR=   JQUELIN
 MODCPAN_EXAMPLES=  Yes
+MODCPAN_EXAMPLES_DIST= examples
 
 CONFIGURE_STYLE=   modbuild
 
-RUN_DEPENDS=   audio/p5-Audio-MPD-Common>=1.110550 \
-   devel/p5-MooseX-Has-Sugar \
-   devel/p5-MooseX-SemiAffordanceAccessor>=0.09 \
-   devel/p5-Class-Accessor \
+RUN_DEPENDS=   audio/p5-Audio-MPD-Common>=2.003 \
devel/p5-Getopt-Euclid \
+   devel/p5-List-AllUtils \
+   devel/p5-List-MoreUtils \
+   devel/p5-Moose \
+   devel/p5-MooseX-Has-Sugar \
+   devel/p5-MooseX-SemiAffordanceAccessor \
devel/p5-Proc-Daemon
-TEST_DEPENDS=  devel/p5-Getopt-Euclid \
-   devel/p5-Test-Corpus-Audio-MPD
+
+TEST_DEPENDS=  devel/p5-Test-Corpus-Audio-MPD>=1.113282
+
+# Tests need to find mpd.
+pre-test:
+   @ln -sf ${LOCALBASE}/sbin/mpd ${WRKDIR}/bin/mpd
 
 .include 
Index: distinfo
===
RCS file: /cvs/ports/audio/p5-Audio-MPD/distinfo,v
retrieving revision 1.13
diff -u -p -u -p -r1.13 distinfo
--- distinfo17 Apr 2014 20:51:25 -  1.13
+++ distinfo22 Feb 2019 18:38:30 -
@@ -1,2 +1,2 @@
-SHA256 (Audio-MPD-1.110560.tar.gz) = 
EnQIRIdGxv58uweer9ZRC7IZLneZ2BAeBDts/BQK39M=
-SIZE (Audio-MPD-1.110560.tar.gz) = 36121
+SHA256 (Audio-MPD-2.004.tar.gz) = xPit/gQNU3PdyaM9umbqEI0C5FF4uOYPXPdKpAjGqpI=
+SIZE (Audio-MPD-2.004.tar.gz) = 41586
Index: pkg/PLIST
===
RCS file: /cvs/ports/audio/p5-Audio-MPD/pkg/PLIST,v
retrieving revision 1.5
diff -u -p -u -p -r1.5 PLIST
--- pkg/PLIST   29 Sep 2010 17:19:30 -  1.5
+++ pkg/PLIST   22 Feb 2019 18:38:30 -
@@ -1,10 +1,12 @@
 @comment $OpenBSD: PLIST,v 1.5 2010/09/29 17:19:30 jasper Exp $
+bin/mpd-dump-ratings
 bin/mpd-dynamic
 bin/mpd-rate
 ${P5SITE}/Audio/MPD.pm
 ${P5SITE}/Audio/MPD/Collection.pm
 ${P5SITE}/Audio/MPD/Playlist.pm
 ${P5SITE}/Audio/MPD/Types.pm
+@man man/man1/mpd-dump-ratings.1
 @man man/man1/mpd-dynamic.1
 @man man/man1/mpd-rate.1
 @man man/man3p/Audio::MPD.3p
Index: patches/patch-lib_Audio_MPD_Types_pm
===
RCS file: patches/patch-lib_Audio_MPD_Types_pm
diff -N patches/patch-lib_Audio_MPD_Types_pm
--- patches/patch-lib_Audio_MPD_Types_pm17 Apr 2014 20:51:25 -  
1.1
+++ /dev/null   1 Jan 1970 00:00:00 -
@@ -1,16 +0,0 @@
-$OpenBSD: 

Re: [UPDATE] audio/p5-Audio-MPD 1.110560 -> 2.004 (Perl+MPD 5/5)

2019-02-22 Thread Landry Breuil
On Fri, Feb 22, 2019 at 09:10:57AM +0100, Landry Breuil wrote:
> On Thu, Feb 21, 2019 at 11:23:15PM +0100, Charlene Wendling wrote:
> > 
> > Here is an update for Audio::MPD. 
> > 
> > What's new upstream (partial, see [1]): 
> > 
> > - support for mpd on a Unix socket
> > - syncing with mpd changes, especially allow dumping song ratings
> > - ipv6 support
> > 
> > What's new in the port: 
> 
> audio/p5-Audio-MPD/patches/patch-lib_Audio_MPD_Types_pm should be
> removed, applied upstream.
> 
> > - Dropped CPAN_AUTHOR, it's unneeded here
> > - Really install examples thanks to MODCPAN_EXAMPLES_DIST
> > - When tested in the ports tree, mpd isn't in PATH, so i've symlinked
> >   it in ${WRKDIR}/bin to enable them. If mpd isn't installed it skips 
> >   tests gracefully anyway.
> 
> If there's an already running mpd it fails to start one, but that's a
> corner case :)

running tests on current yields this:

t/00-report-prereqs.t  ok
Could not create socket: No route to host
# Looks like you planned 20 tests but ran 3.
t/20-connection.t  
Dubious, test returned 255 (wstat 65280, 0xff00)
Failed 17/20 subtests 
(less 1 skipped subtest: 2 okay)

(and the other tests pass)



Re: [UPDATE] audio/p5-Audio-MPD 1.110560 -> 2.004 (Perl+MPD 5/5)

2019-02-22 Thread Landry Breuil
On Thu, Feb 21, 2019 at 11:23:15PM +0100, Charlene Wendling wrote:
> 
> Here is an update for Audio::MPD. 
> 
> What's new upstream (partial, see [1]): 
> 
> - support for mpd on a Unix socket
> - syncing with mpd changes, especially allow dumping song ratings
> - ipv6 support
> 
> What's new in the port: 

audio/p5-Audio-MPD/patches/patch-lib_Audio_MPD_Types_pm should be
removed, applied upstream.

> - Dropped CPAN_AUTHOR, it's unneeded here
> - Really install examples thanks to MODCPAN_EXAMPLES_DIST
> - When tested in the ports tree, mpd isn't in PATH, so i've symlinked
>   it in ${WRKDIR}/bin to enable them. If mpd isn't installed it skips 
>   tests gracefully anyway.

If there's an already running mpd it fails to start one, but that's a
corner case :)

I dunno if the dependency on Audio-MPD-Common-2.003 should be enforced,
since upstream says there's no breaking changes.. but the major version
bump might say the contrary.

I noticed that it had a dependency on List::Utils 1.45 - 1.50 is in base
only since the 5.28 update it seems, but that's fine for -current - it
just complains with the 1.42 that was in 6.4 before.

> - It has no direct reverse dependencies but is pulled by
>   audio/p5-POE-Component-Client-MPD, and tests pass there - anyway
>   i've an update for it as well :) 

my own 100-lines perl script pushing mpd songs to last.fm via
Audio::Scrobbler still works with it, so ok for me (and -Common & the
POE component too)

Landry



[UPDATE] audio/p5-Audio-MPD 1.110560 -> 2.004 (Perl+MPD 5/5)

2019-02-21 Thread Charlene Wendling


Here is an update for Audio::MPD. 

What's new upstream (partial, see [1]): 

- support for mpd on a Unix socket
- syncing with mpd changes, especially allow dumping song ratings
- ipv6 support

What's new in the port: 

- Dropped CPAN_AUTHOR, it's unneeded here
- Really install examples thanks to MODCPAN_EXAMPLES_DIST
- When tested in the ports tree, mpd isn't in PATH, so i've symlinked
  it in ${WRKDIR}/bin to enable them. If mpd isn't installed it skips 
  tests gracefully anyway.

Testing:

- 'make test' pass, we don't have Pod::Coverage::TrustPod, so 
  qa tests would fail and thus aren't enabled.
- It has no direct reverse dependencies but is pulled by
  audio/p5-POE-Component-Client-MPD, and tests pass there - anyway
  i've an update for it as well :) 


Comments/feedback are welcome, 

Charlène. 


[1] https://metacpan.org/changes/distribution/Audio-MPD


Index: Makefile
===
RCS file: /cvs/ports/audio/p5-Audio-MPD/Makefile,v
retrieving revision 1.28
diff -u -p -u -p -r1.28 Makefile
--- Makefile20 Mar 2016 19:56:06 -  1.28
+++ Makefile21 Feb 2019 21:26:16 -
@@ -1,8 +1,7 @@
 # $OpenBSD: Makefile,v 1.28 2016/03/20 19:56:06 naddy Exp $
 
 COMMENT=   module for communicating with MPD servers
-DISTNAME=  Audio-MPD-1.110560
-REVISION=  3
+DISTNAME=  Audio-MPD-2.004
 CATEGORIES=audio
 
 # perl
@@ -10,18 +9,24 @@ PERMIT_PACKAGE_CDROM=Yes
 
 MODULES=   cpan
 PKG_ARCH=  *
-CPAN_AUTHOR=   JQUELIN
 MODCPAN_EXAMPLES=  Yes
+MODCPAN_EXAMPLES_DIST= examples
 
 CONFIGURE_STYLE=   modbuild
 
-RUN_DEPENDS=   audio/p5-Audio-MPD-Common>=1.110550 \
-   devel/p5-MooseX-Has-Sugar \
-   devel/p5-MooseX-SemiAffordanceAccessor>=0.09 \
-   devel/p5-Class-Accessor \
+RUN_DEPENDS=   audio/p5-Audio-MPD-Common \
devel/p5-Getopt-Euclid \
+   devel/p5-List-AllUtils \
+   devel/p5-List-MoreUtils \
+   devel/p5-Moose \
+   devel/p5-MooseX-Has-Sugar \
+   devel/p5-MooseX-SemiAffordanceAccessor \
devel/p5-Proc-Daemon
-TEST_DEPENDS=  devel/p5-Getopt-Euclid \
-   devel/p5-Test-Corpus-Audio-MPD
+
+TEST_DEPENDS=  devel/p5-Test-Corpus-Audio-MPD>=1.113282
+
+# Tests need to find mpd.
+pre-test:
+   @ln -sf ${LOCALBASE}/sbin/mpd ${WRKDIR}/bin/mpd
 
 .include 
Index: distinfo
===
RCS file: /cvs/ports/audio/p5-Audio-MPD/distinfo,v
retrieving revision 1.13
diff -u -p -u -p -r1.13 distinfo
--- distinfo17 Apr 2014 20:51:25 -  1.13
+++ distinfo21 Feb 2019 21:26:16 -
@@ -1,2 +1,2 @@
-SHA256 (Audio-MPD-1.110560.tar.gz) = 
EnQIRIdGxv58uweer9ZRC7IZLneZ2BAeBDts/BQK39M=
-SIZE (Audio-MPD-1.110560.tar.gz) = 36121
+SHA256 (Audio-MPD-2.004.tar.gz) = xPit/gQNU3PdyaM9umbqEI0C5FF4uOYPXPdKpAjGqpI=
+SIZE (Audio-MPD-2.004.tar.gz) = 41586
Index: pkg/PLIST
===
RCS file: /cvs/ports/audio/p5-Audio-MPD/pkg/PLIST,v
retrieving revision 1.5
diff -u -p -u -p -r1.5 PLIST
--- pkg/PLIST   29 Sep 2010 17:19:30 -  1.5
+++ pkg/PLIST   21 Feb 2019 21:26:16 -
@@ -1,10 +1,12 @@
 @comment $OpenBSD: PLIST,v 1.5 2010/09/29 17:19:30 jasper Exp $
+bin/mpd-dump-ratings
 bin/mpd-dynamic
 bin/mpd-rate
 ${P5SITE}/Audio/MPD.pm
 ${P5SITE}/Audio/MPD/Collection.pm
 ${P5SITE}/Audio/MPD/Playlist.pm
 ${P5SITE}/Audio/MPD/Types.pm
+@man man/man1/mpd-dump-ratings.1
 @man man/man1/mpd-dynamic.1
 @man man/man1/mpd-rate.1
 @man man/man3p/Audio::MPD.3p