svn commit: r415903 - head/x11/lxpanel
Olivier Duchateau
olivierd at freebsd.org
Thu May 26 17:39:21 UTC 2016
On Thu, 26 May 2016 16:29:44 +0000 (UTC)
Dmitry Marakasov <amdmi3 at FreeBSD.org> wrote:
> Author: amdmi3
> Date: Thu May 26 16:29:44 2016
> New Revision: 415903
> URL: https://svnweb.freebsd.org/changeset/ports/415903
>
> Log:
> - Fix build for disabled NLS case
> - Switch to options helpers
> - Simplify stripping
> - Simplify plist handling
>
> PR: 202076
> Reported by: sascha at root-login.org
> Approved by: maintainer timeout (horia at racoviceanu.com, 9 months)
I'm working to update LXpanel to 0.8.2 [1], and for me ALSA option is bogus
1. If ALSA is not set it's OSS (Open Sound System), so I think it's better to distinguish two sound systems. Moreover sound support is a panel widget plugin, we must also check --with-plugins= option.
Panel plugins can be disabled with prefix '-'.
ALSA:
CONFIGURE_ARGS should be --enable-alsa --with-plugins=-volume,volumealsa,...
OSS (we can use OPTIONS_SINGLE macro):
CONFIGURE_ARGS must be --disable-alsa --with-plugins=volume,-volumealsa,...
Why still keep USE_AUTOTOOLS? It's deprecated
[1] https://bitbucket.org/olivierduchateau/ports/src/434988b9ed6c5d5199be5c960cd7aee358aa1fba/x11/lxpanel/?at=default
>
> Modified:
> head/x11/lxpanel/Makefile
> head/x11/lxpanel/pkg-plist
>
> Modified: head/x11/lxpanel/Makefile
> ==============================================================================
> --- head/x11/lxpanel/Makefile Thu May 26 16:25:39 2016 (r415902)
> +++ head/x11/lxpanel/Makefile Thu May 26 16:29:44 2016 (r415903)
> @@ -26,7 +26,7 @@ USES= gmake pathfix pkgconfig
> GNU_CONFIGURE= yes
> CONFIGURE_ARGS= --prefix="${PREFIX}" \
> --with-plugins=netstatus,volume,volumealsa,deskno,batt,kbled,xkb,thermal,cpu,cpufreq,monitors,wnckpager \
> - --with-x \
> + --with-x --disable-silent-rules \
> CPPFLAGS="-I${LOCALBASE}/include" \
> LDFLAGS="-L${LOCALBASE}/lib"
> CFLAGS+= -I${WRKSRC}
> @@ -41,14 +41,7 @@ ALSA_LIB_DEPENDS= libasound.so:audio/als
> ALSA_CONFIGURE_ENABLE= alsa
> NLS_CONFIGURE_ENABLE= nls
> NLS_USES= gettext
> -
> -.include <bsd.port.options.mk>
> -
> -.if ${PORT_OPTIONS:MALSA}
> -PLIST_SUB+= VOLUME="@comment "
> -.else
> -PLIST_SUB+= VOLUME=""
> -.endif
> +NLS_USES_OFF= gettext-tools
>
> post-patch:
> @${REINPLACE_CMD} -e '/g_thread_create/ \
> @@ -105,22 +98,14 @@ post-patch:
> @${REINPLACE_CMD} -e 's|linux|sys|' \
> ${WRKSRC}/src/plugins/volume/volume-impl.c
>
> -.if ! ${PORT_OPTIONS:MNLS}
> +post-patch-NLS-off:
> @${REINPLACE_CMD} -e 's|po man|man|' \
> ${WRKSRC}/Makefile.in
> -.endif
>
> post-install:
> -.for l in netstatus deskno batt kbled xkb thermal cpu cpufreq monitors wnckpager
> - ${STRIP_CMD} ${STAGEDIR}${PREFIX}/lib/lxpanel/plugins/${l}.so
> -.endfor
> -
> -.if ${PORT_OPTIONS:MALSA}
> - ${STRIP_CMD} ${STAGEDIR}${PREFIX}/lib/lxpanel/plugins/volumealsa.so
> -.else
> - ${STRIP_CMD} ${STAGEDIR}${PREFIX}/lib/lxpanel/plugins/volume.so
> -.endif
> + @${STRIP_CMD} ${STAGEDIR}${PREFIX}/lib/lxpanel/plugins/*.so
>
> +post-install-DOCS-on:
> @${MKDIR} ${STAGEDIR}${DOCSDIR}
> ${INSTALL_DATA} ${PORTDOCS:S|^|${WRKSRC}/|} ${STAGEDIR}${DOCSDIR}
>
>
> Modified: head/x11/lxpanel/pkg-plist
> ==============================================================================
> --- head/x11/lxpanel/pkg-plist Thu May 26 16:25:39 2016 (r415902)
> +++ head/x11/lxpanel/pkg-plist Thu May 26 16:29:44 2016 (r415903)
> @@ -9,7 +9,7 @@ lib/lxpanel/plugins/kbled.so
> lib/lxpanel/plugins/monitors.so
> lib/lxpanel/plugins/netstatus.so
> lib/lxpanel/plugins/thermal.so
> -%%VOLUME%%lib/lxpanel/plugins/volume.so
> +%%NO_ALSA%%lib/lxpanel/plugins/volume.so
> %%ALSA%%lib/lxpanel/plugins/volumealsa.so
> lib/lxpanel/plugins/xkb.so
> lib/lxpanel/plugins/wnckpager.so
>
--
Olivier Duchateau <olivierd at freebsd.org>
More information about the svn-ports-head
mailing list