Re: [update] mpd/ncmpc/ncmpcpp

2021-02-20 Thread Bjorn Ketelaars
On Sat 20/02/2021 08:37, Landry Breuil wrote:
> Hi,
> 
> found those were a bit outdated, changelogs at
> https://rybczak.net/ncmpcpp/
> https://raw.githubusercontent.com/MusicPlayerDaemon/MPD/v0.22.6/NEWS
> https://github.com/MusicPlayerDaemon/ncmpc/blob/v0.45/NEWS
> 
> mpd updates from 0.21 to 0.22 because of some additions to the protocol,
> added BDEP on sphinx because the manpage isnt in the src tarball so
> needs to be generated from the rst file.
> 
> Please test !

Build and (lightly) run tested on amd64. OK mpd and ncmpc

It seems that ncmpcpp does not depend on libiconv, as such remove it
from LDEP and sync WANTLIB. While here sync patches. Based on your diff:


diff --git Makefile Makefile
index 5216990772e..168e25b4b0d 100644
--- Makefile
+++ Makefile
@@ -5,8 +5,7 @@ BROKEN-hppa =   bindings.h:85:4: error: type 'Binding' 
is not a direct base of 'B
 
 COMMENT =  ncurses mpd client inspired by ncmpc
 
-DISTNAME = ncmpcpp-0.8.2
-REVISION = 1
+DISTNAME = ncmpcpp-0.9.2
 
 EXTRACT_SUFX = .tar.bz2
 
@@ -17,17 +16,16 @@ HOMEPAGE =  https://rybczak.net/ncmpcpp/
 # GPLv2
 PERMIT_PACKAGE =   Yes
 
-WANTLIB += boost_date_time-mt boost_filesystem-mt boost_locale-mt
-WANTLIB += boost_program_options-mt boost_regex-mt boost_system-mt
-WANTLIB += boost_thread-mt c crypto curl ereadline iconv m mpdclient
-WANTLIB += curses nghttp2 pthread ssl ${LIBCXX} tag z
-WANTLIB += ${COMPILER_LIBCXX}
+WANTLIB += ${COMPILER_LIBCXX} boost_chrono-mt boost_date_time-mt
+WANTLIB += boost_filesystem-mt boost_locale-mt boost_program_options-mt
+WANTLIB += boost_regex-mt boost_system-mt boost_thread-mt c crypto
+WANTLIB += curl curses ereadline icudata icui18n icuuc m mpdclient
+WANTLIB += nghttp2 ssl tag z
 
 MASTER_SITES = ${HOMEPAGE}stable/
 
 LIB_DEPENDS =  audio/libmpdclient>=2.4 \
audio/taglib \
-   converters/libiconv \
devel/boost \
devel/readline \
net/curl
diff --git distinfo distinfo
index fd53b891b20..f1807f24c59 100644
--- distinfo
+++ distinfo
@@ -1,2 +1,2 @@
-SHA256 (ncmpcpp-0.8.2.tar.bz2) = ZQuj6AiWJLetnkzBm8GsYCjtt1I8wRH6FobqRMCSFVQ=
-SIZE (ncmpcpp-0.8.2.tar.bz2) = 465713
+SHA256 (ncmpcpp-0.9.2.tar.bz2) = +qv2FXyMsbJKBZryduFi+p+aO5zTgQxDuRKIYMk4Ohs=
+SIZE (ncmpcpp-0.9.2.tar.bz2) = 486266
diff --git patches/patch-configure patches/patch-configure
index 8b99053d03a..b3c1b09f417 100644
--- patches/patch-configure
+++ patches/patch-configure
@@ -3,7 +3,7 @@ $OpenBSD: patch-configure,v 1.11 2018/02/11 10:37:51 dcoppa Exp 
$
 Index: configure
 --- configure.orig
 +++ configure
-@@ -19500,7 +19500,7 @@ CPPFLAGS=$boost_thread_save_CPPFLAGS
+@@ -20542,7 +20542,7 @@ CPPFLAGS=$boost_thread_save_CPPFLAGS
  
  
  
diff --git patches/patch-doc_ncmpcpp_1 patches/patch-doc_ncmpcpp_1
index 68e41d9c40e..949c7934e1a 100644
--- patches/patch-doc_ncmpcpp_1
+++ patches/patch-doc_ncmpcpp_1
@@ -8,7 +8,7 @@ Index: doc/ncmpcpp.1
  .SH "NAME"
  ncmpcpp \- An ncurses Music Player Daemon (MPD) client.
  .SH "SYNOPSIS"
-@@ -298,7 +298,7 @@ Slave screen that has to be displayed at start (nothin
+@@ -325,7 +325,7 @@ Slave screen that has to be displayed at start (nothin
  .TP
  .B startup_slave_screen_focus = yes/no
  If set to yes, slave screen will be the active one after startup. Otherwise 
master screen will be.
@@ -17,11 +17,11 @@ Index: doc/ncmpcpp.1
  .B locked_screen_width_part = 20-80
  If you want to lock a screen, ncmpcpp asks for % of locked screen's width to 
be reserved before that and provides a default value, which is the one you can 
set here.
  .TP
-@@ -415,7 +415,6 @@ Border color of pop-up windows. If set to 'none', no b
+@@ -445,7 +445,6 @@ Border color of pop-up windows. If set to 'none', no b
  .TP
  .B active_window_border = COLOR
  Color of active window's border.
 -.TP
  .SH "BINDINGS"
- When ncmpcpp starts, it tries to read user's bindings from 
~/.ncmpcpp/bindings and $XDG_CONFIG_HOME/ncmpcpp/bindings files. If no bindings 
file is found, ncmpcpp uses the defaults. An example bindings file with the 
default values can be found usually in /usr/share/doc/ncmpcpp (exact location 
may depend on used distribution or configure prefix).
- 
+ When ncmpcpp starts, it tries to read bindings from
+ $XDG_CONFIG_HOME/ncmpcpp/bindings and ~/.ncmpcpp/bindings files. If no 
bindings
diff --git patches/patch-src_ncmpcpp_cpp patches/patch-src_ncmpcpp_cpp
deleted file mode 100644
index ee24f3c24c3..000
--- patches/patch-src_ncmpcpp_cpp
+++ /dev/null
@@ -1,35 +0,0 @@
-$OpenBSD: patch-src_ncmpcpp_cpp,v 1.13 2017/06/09 15:09:51 dcoppa Exp $
-
-OpenBSD lacks sigignore(), use the one from NetBSD
-
-Index: src/ncmpcpp.cpp
 src/ncmpcpp.cpp.orig
-+++ src/ncmpcpp.cpp
-@@ -68,6 +68,18 @@ void sighandler(int sig)
- #endif // __sun && __SVR4
- }
- 
-+int n_sigignore(int sig)
-+{
-+  struct sigaction sa;
-+
-+  sa.sa_handler

[update] mpd/ncmpc/ncmpcpp

2021-02-19 Thread Landry Breuil
Hi,

found those were a bit outdated, changelogs at
https://rybczak.net/ncmpcpp/
https://raw.githubusercontent.com/MusicPlayerDaemon/MPD/v0.22.6/NEWS
https://github.com/MusicPlayerDaemon/ncmpc/blob/v0.45/NEWS

mpd updates from 0.21 to 0.22 because of some additions to the protocol,
added BDEP on sphinx because the manpage isnt in the src tarball so
needs to be generated from the rst file.

Please test !

Landry
? 0.21.8.log
? mpd-0.20.20
? mpd-0.21.1
? patch-src_Main_cxx
Index: Makefile
===
RCS file: /cvs/ports/audio/mpd/Makefile,v
retrieving revision 1.132
diff -u -r1.132 Makefile
--- Makefile8 Jul 2020 13:02:24 -   1.132
+++ Makefile20 Feb 2021 07:24:49 -
@@ -2,7 +2,7 @@
 
 BROKEN-hppa =  no atomic ops
 COMMENT =  Music Player Daemon
-VER =  0.21.25
+VER =  0.22.6
 DISTNAME = mpd-${VER}
 EXTRACT_SUFX = .tar.xz
 CATEGORIES =   audio
@@ -11,7 +11,7 @@
 # GPLv2+
 PERMIT_PACKAGE =   Yes
 
-WANTLIB =  FLAC audiofile avcodec avformat avutil \
+WANTLIB =  FLAC audiofile avcodec avfilter avformat avutil \
bz2 c cdio cdio_cdda cdio_paranoia curl \
expat faad icui18n icuuc \
id3tag iso9660 m mikmod mp3lame mpcdec \
@@ -24,6 +24,7 @@
 COMPILER = base-clang ports-gcc
 
 BUILD_DEPENDS =devel/boost
+BUILD_DEPENDS +=   textproc/py-sphinx
 
 LIB_DEPENDS =  audio/faad>=2.6.1p0 \
audio/flac \
@@ -145,9 +146,6 @@
CPPFLAGS="-I${LOCALBASE}/include -D_BSD_SOURCE"
 
 post-install:
-   ${INSTALL_MAN} ${WRKSRC}/doc/mpd.1 ${PREFIX}/man/man1/mpd.1
-   ${INSTALL_MAN} ${WRKSRC}/doc/mpd.conf.5 ${PREFIX}/man/man5/mpd.conf.5
-
# avoid dependency on gtk3,-guic
@rm -Rf ${PREFIX}/share/icons/
${INSTALL_DATA_DIR} ${PREFIX}/share/examples/mpd
Index: distinfo
===
RCS file: /cvs/ports/audio/mpd/distinfo,v
retrieving revision 1.57
diff -u -r1.57 distinfo
--- distinfo8 Jul 2020 13:02:24 -   1.57
+++ distinfo20 Feb 2021 07:24:49 -
@@ -1,2 +1,2 @@
-SHA256 (mpd-0.21.25.tar.xz) = IKCtAb8yey2+tq6OGvlSyw3oPS1j+rD6S3GDp0dlwgE=
-SIZE (mpd-0.21.25.tar.xz) = 683768
+SHA256 (mpd-0.22.6.tar.xz) = K+FJpIlcPLYTR3+M8Rk1k+PYodOKdf+n0y2oyDFqTV4=
+SIZE (mpd-0.22.6.tar.xz) = 730524
Index: patches/patch-doc_meson_build
===
RCS file: /cvs/ports/audio/mpd/patches/patch-doc_meson_build,v
retrieving revision 1.1
diff -u -r1.1 patch-doc_meson_build
--- patches/patch-doc_meson_build   6 Aug 2019 17:28:12 -   1.1
+++ patches/patch-doc_meson_build   20 Feb 2021 07:24:49 -
@@ -3,12 +3,12 @@
 Index: doc/meson.build
 --- doc/meson.build.orig
 +++ doc/meson.build
-@@ -11,7 +11,7 @@ sphinx_output = custom_target(
- 'conf.py',
-   ],
-   command: [sphinx, '-q', '-b', 'html', '-d', '@OUTDIR@/doctrees', 
meson.current_source_dir(), '@OUTPUT@'],
--  build_by_default: true,
-+  build_by_default: false,
-   install: true,
-   install_dir: join_paths(get_option('datadir'), 'doc', meson.project_name()),
- )
+@@ -18,7 +18,7 @@ if get_option('html_manual')
+   'conf.py',
+ ],
+ command: [sphinx, '-q', '-b', 'html', '-d', '@OUTDIR@/doctrees', 
meson.current_source_dir(), '@OUTPUT@'],
+-build_by_default: true,
++build_by_default: false,
+ install: true,
+ install_dir: join_paths(get_option('datadir'), 'doc', 
meson.project_name()),
+   )
Index: patches/patch-doc_mpdconf_example
===
RCS file: /cvs/ports/audio/mpd/patches/patch-doc_mpdconf_example,v
retrieving revision 1.19
diff -u -r1.19 patch-doc_mpdconf_example
--- patches/patch-doc_mpdconf_example   6 Aug 2019 17:28:12 -   1.19
+++ patches/patch-doc_mpdconf_example   20 Feb 2021 07:24:49 -
@@ -7,25 +7,25 @@
  # file:// protocol) or streaming files over an accepted protocol.
  #
 -#music_directory  "~/music"
-+music_directory "/please-configure-your-music_directory"
++music_directory   "/please-configure-your-music_directory"
  #
  # This setting sets the MPD internal playlist directory. The purpose of this
- # directory is storage for playlists created by MPD. The server will use 
+ # directory is storage for playlists created by MPD. The server will use
  # playlist files not created by the server but only if they are in the MPD
  # format. This setting defaults to playlist saving being disabled.
  #
 -#playlist_directory   "~/.mpd/playlists"
-+playlist_directory  "/var/spool/mpd/playlists"
++playlist_directory"/var/spool/mpd/playlists"
  #
  # This setting sets the location of the MPD database. This file is u