NetBSD Problem Report #53613
From www@NetBSD.org Mon Sep 17 13:54:09 2018
Return-Path: <www@NetBSD.org>
Received: from mail.netbsd.org (mail.netbsd.org [199.233.217.200])
(using TLSv1.2 with cipher ECDHE-RSA-AES256-GCM-SHA384 (256/256 bits))
(Client CN "mail.NetBSD.org", Issuer "mail.NetBSD.org CA" (not verified))
by mollari.NetBSD.org (Postfix) with ESMTPS id 924517A18A
for <gnats-bugs@gnats.NetBSD.org>; Mon, 17 Sep 2018 13:54:09 +0000 (UTC)
Message-Id: <20180917135406.DFA7D7A237@mollari.NetBSD.org>
Date: Mon, 17 Sep 2018 13:54:06 +0000 (UTC)
From: bastiangermann@fishpost.de
Reply-To: bastiangermann@fishpost.de
To: gnats-bugs@NetBSD.org
Subject: New package: audio/alure-1.2
X-Send-Pr-Version: www-1.0
>Number: 53613
>Category: pkg
>Synopsis: New package: audio/alure-1.2
>Confidential: no
>Severity: non-critical
>Priority: medium
>Responsible: pkg-manager
>State: closed
>Class: change-request
>Submitter-Id: net
>Arrival-Date: Mon Sep 17 13:55:01 +0000 2018
>Closed-Date: Fri Sep 21 13:26:19 +0000 2018
>Last-Modified: Wed Sep 26 16:30:01 +0000 2018
>Originator: Bastian Germann
>Release:
>Organization:
>Environment:
>Description:
ALURE is a utility library to help manage common tasks with OpenAL applications. This includes device enumeration and initialization, file loading, and streaming.
The purpose of this library is to provide pre-made functionality that would otherwise be repetitive or difficult to (re)code for various projects and platforms, such as loading a sound file into an OpenAL buffer and streaming an audio file through a buffer queue. Support for different formats is consistant across platforms, so no special checks are needed when loading files, and all formats are handled through the same API.
>How-To-Repeat:
>Fix:
begin 644 alure.tar.gz
M'XL(`-RNGUL``^U9:W/:2!;-5^M7]";Y8!=!Z/U(+;/&1G;8\"J$LS.?J%:K
M!8J%I)6$,\Y4_OO>EL3+QD!F8Z=V5J<2"V[?OGWZH=/=%QPL$MIX]:P0`+JJ
MLJ>HJ\+F<XE7HBPKNB[*DJ*_$D1!5?572'U>6@46:883A%XY>$KW^1TJ_Q\%
MSN=_V.W8XV=K@TVPIBA/S;^HZ>IJ_@51AOD793`AX=D8;>#_?/[/232?TS!#
M;_LTN[#;;SD_),'"I8U6MY$O#G[&!;[3@/_YUSJ,5^83'F]9^31Z^)T7'UG>
M_C'\>/W)&MF=0?\;+^3E\>V41*'G3QM;X6.RJY29TQF&%>M&I#`U9MD\:'A^
M0-,R`HECGAGW>3H+SZ/)4:Y^FB44SX_R_6[7.,#W!]S]T*6_'RIO7"U"DOE1
MF![TO*8A37"PQ^\SOL,I2?PX:\RQ'_*?TX-^*<4)F;DXPT]Z%RYKHJT]#$K?
MDNKQGE='>_:.]K0/>_:C$4T70;9O\-/LGDV[R)/TJ0$J/*2#'OFD,*<?\/X7
MT=N6?3GZ`=%VXY#^BY+P2/\EN=+_ET"K>S.RD)\BC!:9'_C9/0+=37!RC[((
MS6@0HSD.H>^(;111B#*<WJ;HBY_-T""F8:O+X3@.?((+]4'C&00K]Y`4N?3.
M)Q31<#&'=XFY(!RZ4.YG/@[\K[GI'6*2R`41=OUP^B[W*.01OO(<-YY1%"^2
M.$HIBCR4L1:6).$C\(R3Z,YWP0OD?X[A@U>J#,Y[E,UPAKY$B\#EHFQ&DR\^
M1'(H2FA,,V!R!W$3Y/J>YQ-XC5G$TX2>D8A%@I([G/C1(F7-?*8D2QE##J0[
M@\)Y^@ZE"S)#&$@5/8"Q3*,%=()U"_H*X7!8CA8J=I[M/L(W#B]</RIJ9+,D
M6DPAXM+YWPNZH#RR%W$<)5E.B9&E"=NX&0<,G&`D8*M,8;O"8<9ADD0I,-X@
M&:$P0FE,"8P\(C-*8!Y!9%!(J4M=]&5&PU4/\CVJF`D<!-RR#>8^`V,`_DN6
M,*`HQ7.*6L,._R,DJ<(+HM!_9^$';N"'MS(_O_WA;1S0?T&1E:7^:Y+,[G^R
MJ%;Z_R)XLS[X<Q<WG6Z[V^E_G(Q'EE5KGN2+@^-XWT-_<ZGGA]0]S3>,R<I5
MGO0^GG$[C.^;FP%!&R9M:VCUVS:?1UU&_Z4I\M*&(UP0[-%ENS,JW/[1/.'Y
M!OS+Y;&Q(E3L+^CU9F$$`HN#>AIYV?:"?LWQ-`3!/'F#=A#=T>]ZT<[/GIN7
M0/'^QSB##2%]ICS0]^1_BO.?)++[?Y7_>7YLSW_^K*<)F;"S#YE,O&#AN^E]
MF,UXN*7^R38.ZK\N;<^_),ARE?]Y$:S5?Y9E\?M&`\[$$1\E//E:IERF?E:L
MBX9L4%53!,U3=$KABB8IFD(UW3$\B@U#D@Q5PYXF.AQ7K]<1K*)&L8JV%Q$$
M]Z<G$ESRZH)>ETPD&.]E_;UB\"LU0#7VEZO5:D]&X<[/$51^IZ$:_#70^3F'
MWBRWA;_C-*5)QL]^848/MBXT^5>G+TN;/G[$RFMO:)!2>*SLBQ".T&Y1-]\V
M.+15K3BU__(7V1UVO?^7/7Q+N]#1E,]^S_[[-@[F?P7M@?YK0O7^OPS6[W\7
M3DOHJMNZA&MDDL`U-[A'IUX2S=$^96!9`7AHIJP04?=,3*CN$=&DV%3@,.]B
M1\"*(^NN*&)#-\\*;=A>8=^G"-MU"QTPM'<Z"$'^`$/GZI1UY(Q#2]C6N#RY
M#B[^::.W?ZR_?-O2&$R8NI0UX;@*H3ZT[$D9SNK:5AFZO@S;[5PPRP2>H]:H
M8]GH]>LSKK8LWE%6QBUC@KB@WO!:E&24%A?\O$^RH;,^%0\P+.O<=-KV;_WQ
MA[PFQ[K:'XQ1^[=^=]!J;]"R?AV/6JQAUEN[W[[J=*TUCV]@_#08773LQ_9'
M'6+&@N&VK7W3NWC@-6@/NS?7V\8UYVW[FN&WC?'Z\[Q?GG2Q'K;&/U\WK78;
M;CM7G7YGW!GT[=-ZNRQOBJO9WZKU\][_0O_9*\52/L_3QB']%S3QX?E?$I1*
M_U\"F_?_=L<>]UL]J[R:U]G-_+(UMJX';/6#E5VSN5[+'ENCB=T9,UNY-]P2
MG/%P,L+AE+(+.!_2K/PY+J$!Q2D<+CBHVNF/X;\U:IXX.,U\YI[,<1B>>WXZ
MBZ,TXUW*?1CTK&'KVCHB.G<YZ/6L_KAY\H$&<7HH5XTV<]5<MW-I]6UH!G8P
MCKNQK<EEK_41OM_3-/\Z'@RZ=JUY$M].Z\7/D+FYV^I?WP`_Z#Y!I%;C\FJ3
MUNB:.=?;>3YA8OW:Z@V[X#2XND*@`*`[RX^E&,`WCAM^O+X<]*\ZUY/!)VLT
MZK17B9?U3Z&\'^[Q*QTVLB)17"3C\]3'=^=*'E28WS:<U.5A#/+BG[U@*_Q0
M%/J_7C#/T<8A_==U_:'^LRM!I?\O@$W]!XF9#(;YD67RJ04:O6$HTK&YBWTS
M'`Y&8ZN]+&N>L',S6E_0T3QRXV`Q1?-XFI]K0S?_8>LN2AP_?9R_+25F4[:*
MI#.=Q]G]Z0:-]SW6U-E#Q65'O^:@_X38L2J/,\+LVK\S#*ARD2_>1V+9U1U4
ME@?'?826U8^DM0YY!+ER\!\Q*\^Y3],*?*>L>Q2M5;PC.)4KX!&G\H"_EU-9
M]RA.JWA'<"I6XR-*Q?5B+Z.BYE&$EM&.F;?\97D\;?DEYFD^1;7C9JP,M293
MZK\+UVD_]*+GT9A#^5]=?9C_D61)J/3_);!6?_M#2T2GJY,_#\/"3[^>H28R
M)*(('A5$(L$G0S-AFAS)="5-E$T=KFJ"B%59E+E1KRUJPNX@)DRZ)+A85:B'
M#1T3;#J8*IK@F9[BZJ[N2#+&LL=XJ**T.XBN$QF6DDNI(V!J8D$AHFR8JJ88
MKD)4W74U[!F";&@&45Q'5PQ3UC2PFH;K$"RYHB8YU%`H40W1]71-$75JFM11
M'%,C,M24P%>@CBP#+2HZNB.KDJ-@+!(#VN)L_RM]8I!461:1<Y_!W:$8REW)
M5.8I:)ZA2:JIF`;6)9E0@XBJJ>O0BH0]!UB+!GRB6V&>_$TF'Q9!D5QLFHXI
MNSK,D2PIHF#"J#J.J8B2*A!'E%5/JP[O%2I4J%"A0H4*%2I4J%"A0H4*%2I4
;J%"A0H4*%2I4J%"A0H4*?SG\!V:9P>``4```
`
end
>Release-Note:
>Audit-Trail:
From: "Maya Rashish" <maya@netbsd.org>
To: gnats-bugs@gnats.NetBSD.org
Cc:
Subject: PR/53613 CVS commit: pkgsrc/audio/alure
Date: Fri, 21 Sep 2018 13:24:34 +0000
Module Name: pkgsrc
Committed By: maya
Date: Fri Sep 21 13:24:34 UTC 2018
Added Files:
pkgsrc/audio/alure: DESCR Makefile PLIST buildlink3.mk distinfo
options.mk
pkgsrc/audio/alure/patches: patch-CMakeLists.txt
patch-src_codec__fluidsynth.cpp
Log Message:
Add alure 1.2
ALURE is a utility library to help manage common tasks with OpenAL
applications. This includes device enumeration and initialization, file
loading, and streaming.
The purpose of this library is to provide pre-made functionality that would
otherwise be repetitive or difficult to (re)code for various projects and
platforms, such as loading a sound file into an OpenAL buffer and streaming an
audio file through a buffer queue. Support for different formats is consistant
across platforms, so no special checks are needed when loading files, and all
formats are handled through the same API.
Packaged by Bastian Germann in PR pkg/53613.
To generate a diff of this commit:
cvs rdiff -u -r0 -r1.1 pkgsrc/audio/alure/DESCR pkgsrc/audio/alure/Makefile \
pkgsrc/audio/alure/PLIST pkgsrc/audio/alure/buildlink3.mk \
pkgsrc/audio/alure/distinfo pkgsrc/audio/alure/options.mk
cvs rdiff -u -r0 -r1.1 pkgsrc/audio/alure/patches/patch-CMakeLists.txt \
pkgsrc/audio/alure/patches/patch-src_codec__fluidsynth.cpp
Please note that diffs are not public domain; they are subject to the
copyright notices on the relevant files.
State-Changed-From-To: open->closed
State-Changed-By: maya@NetBSD.org
State-Changed-When: Fri, 21 Sep 2018 13:26:19 +0000
State-Changed-Why:
Added, thanks!
From: Leonardo Taccari <leot@NetBSD.org>
To: gnats-bugs@NetBSD.org
Cc:
Subject: Re: pkg/53613: New package: audio/alure-1.2
Date: Fri, 21 Sep 2018 15:47:47 +0200
Hello Bastian,
bastiangermann@fishpost.de writes:
> >Number: 53613
> >Category: pkg
> >Synopsis: New package: audio/alure-1.2
> >Confidential: no
> >Severity: non-critical
> >Priority: medium
> >Responsible: pkg-manager
> >State: open
> >Class: change-request
> >Submitter-Id: net
> >Arrival-Date: Mon Sep 17 13:55:01 +0000 2018
> >Originator: Bastian Germann
> >Release:
> >Organization:
> >Environment:
> >Description:
> [...]
At the moment alure buildlink3.mk is not used so practically it is
not an issue but...
Does the buildlink3.mk needs to be aware of the possible options and
include corresponding bl3 as well on buildlink3.mk based on the options
used?
Thanks!
From: Bastian Germann <bastiangermann@fishpost.de>
To: gnats-bugs@NetBSD.org, pkg-manager@netbsd.org, gnats-admin@netbsd.org,
pkgsrc-bugs@netbsd.org
Cc:
Subject: Re: pkg/53613: New package: audio/alure-1.2
Date: Tue, 25 Sep 2018 12:24:28 +0200
In the submitted version the buildlink3 is aware of the options. You can
check that with wip/gambas3-gb-openal.
Am 21.09.18 um 15:50 schrieb Leonardo Taccari:
> The following reply was made to PR pkg/53613; it has been noted by GNATS.
>
> From: Leonardo Taccari <leot@NetBSD.org>
> To: gnats-bugs@NetBSD.org
> Cc:
> Subject: Re: pkg/53613: New package: audio/alure-1.2
> Date: Fri, 21 Sep 2018 15:47:47 +0200
>
> Hello Bastian,
>
> bastiangermann@fishpost.de writes:
> > >Number: 53613
> > >Category: pkg
> > >Synopsis: New package: audio/alure-1.2
> > >Confidential: no
> > >Severity: non-critical
> > >Priority: medium
> > >Responsible: pkg-manager
> > >State: open
> > >Class: change-request
> > >Submitter-Id: net
> > >Arrival-Date: Mon Sep 17 13:55:01 +0000 2018
> > >Originator: Bastian Germann
> > >Release:
> > >Organization:
> > >Environment:
> > >Description:
> > [...]
>
> At the moment alure buildlink3.mk is not used so practically it is
> not an issue but...
>
> Does the buildlink3.mk needs to be aware of the possible options and
> include corresponding bl3 as well on buildlink3.mk based on the options
> used?
>
>
> Thanks!
>
>
From: Leonardo Taccari <leot@NetBSD.org>
To: gnats-bugs@NetBSD.org
Cc:
Subject: Re: pkg/53613: New package: audio/alure-1.2
Date: Tue, 25 Sep 2018 12:45:23 +0200
Hello Bastian,
Bastian Germann writes:
> In the submitted version the buildlink3 is aware of the options. You can
> check that with wip/gambas3-gb-openal.
Sorry, I don't follow. wip/gambas3-gb-openal seems to include alure
bl3 and that's okay if alure was built with the default package
options, but (maybe!) can be problematic if alure was built with
non-default (extra) options.
What I meant is, should audio/alure/buildlink3.mk be adjusted like that:
Index: buildlink3.mk
===================================================================
RCS file: /cvsroot/pkgsrc/audio/alure/buildlink3.mk,v
retrieving revision 1.1
diff -u -p -r1.1 buildlink3.mk
--- buildlink3.mk 21 Sep 2018 13:24:34 -0000 1.1
+++ buildlink3.mk 25 Sep 2018 10:43:56 -0000
@@ -8,6 +8,33 @@ ALURE_BUILDLINK3_MK:=
BUILDLINK_API_DEPENDS.alure+= alure>=1.2
BUILDLINK_PKGSRCDIR.alure?= ../../audio/alure
+pkgbase := alure
+.include "../../mk/pkg-build-options.mk"
+
+.if !empty(PKG_BUILD_OPTIONS.alure:Mflac)
+.include "../../audio/flac/buildlink3.mk"
+.endif
+
+.if !empty(PKG_BUILD_OPTIONS.alure:Mfluidsynth)
+.include "../../audio/fluidsynth/buildlink3.mk"
+.endif
+
+.if !empty(PKG_BUILD_OPTIONS.alure:Mmodplug)
+.include "../../audio/libmodplug/buildlink3.mk"
+.endif
+
+.if !empty(PKG_BUILD_OPTIONS.alure:Msndfile)
+.include "../../audio/libsndfile/buildlink3.mk"
+.endif
+
+.if !empty(PKG_BUILD_OPTIONS.alure:Mvorbis)
+.include "../../audio/libvorbis/buildlink3.mk"
+.endif
+
+.if !empty(PKG_BUILD_OPTIONS.alure:Mmpg123)
+.include "../../audio/mpg123/buildlink3.mk"
+.endif
+
.include "../../audio/openal-soft/buildlink3.mk"
.endif # ALURE_BUILDLINK3_MK
Thank you!
From: Bastian Germann <bastiangermann@fishpost.de>
To: gnats-bugs@NetBSD.org
Cc:
Subject: Re: pkg/53613: New package: audio/alure-1.2
Date: Tue, 25 Sep 2018 12:53:14 +0200
If that is what packages with buildlink3 and options do, you can add the
patch. I am new to this.
Am 25.09.18 um 12:50 schrieb Leonardo Taccari:
> The following reply was made to PR pkg/53613; it has been noted by GNATS.
>
> From: Leonardo Taccari <leot@NetBSD.org>
> To: gnats-bugs@NetBSD.org
> Cc:
> Subject: Re: pkg/53613: New package: audio/alure-1.2
> Date: Tue, 25 Sep 2018 12:45:23 +0200
>
> Hello Bastian,
>
> Bastian Germann writes:
> > In the submitted version the buildlink3 is aware of the options. You can
> > check that with wip/gambas3-gb-openal.
>
> Sorry, I don't follow. wip/gambas3-gb-openal seems to include alure
> bl3 and that's okay if alure was built with the default package
> options, but (maybe!) can be problematic if alure was built with
> non-default (extra) options.
>
> What I meant is, should audio/alure/buildlink3.mk be adjusted like that:
>
> Index: buildlink3.mk
> ===================================================================
> RCS file: /cvsroot/pkgsrc/audio/alure/buildlink3.mk,v
> retrieving revision 1.1
> diff -u -p -r1.1 buildlink3.mk
> --- buildlink3.mk 21 Sep 2018 13:24:34 -0000 1.1
> +++ buildlink3.mk 25 Sep 2018 10:43:56 -0000
> @@ -8,6 +8,33 @@ ALURE_BUILDLINK3_MK:=
> BUILDLINK_API_DEPENDS.alure+= alure>=1.2
> BUILDLINK_PKGSRCDIR.alure?= ../../audio/alure
>
> +pkgbase := alure
> +.include "../../mk/pkg-build-options.mk"
> +
> +.if !empty(PKG_BUILD_OPTIONS.alure:Mflac)
> +.include "../../audio/flac/buildlink3.mk"
> +.endif
> +
> +.if !empty(PKG_BUILD_OPTIONS.alure:Mfluidsynth)
> +.include "../../audio/fluidsynth/buildlink3.mk"
> +.endif
> +
> +.if !empty(PKG_BUILD_OPTIONS.alure:Mmodplug)
> +.include "../../audio/libmodplug/buildlink3.mk"
> +.endif
> +
> +.if !empty(PKG_BUILD_OPTIONS.alure:Msndfile)
> +.include "../../audio/libsndfile/buildlink3.mk"
> +.endif
> +
> +.if !empty(PKG_BUILD_OPTIONS.alure:Mvorbis)
> +.include "../../audio/libvorbis/buildlink3.mk"
> +.endif
> +
> +.if !empty(PKG_BUILD_OPTIONS.alure:Mmpg123)
> +.include "../../audio/mpg123/buildlink3.mk"
> +.endif
> +
> .include "../../audio/openal-soft/buildlink3.mk"
> .endif # ALURE_BUILDLINK3_MK
>
>
> Thank you!
>
>
From: David Holland <dholland-pbugs@netbsd.org>
To: gnats-bugs@NetBSD.org
Cc:
Subject: Re: pkg/53613: New package: audio/alure-1.2
Date: Wed, 26 Sep 2018 16:28:14 +0000
On Tue, Sep 25, 2018 at 10:50:01AM +0000, Leonardo Taccari wrote:
> What I meant is, should audio/alure/buildlink3.mk be adjusted like that:
> [...]
> +.include "../../mk/pkg-build-options.mk"
> +
> +.if !empty(PKG_BUILD_OPTIONS.alure:Mflac)
> +.include "../../audio/flac/buildlink3.mk"
> +.endif
> [...]
It depends on how the library works -- if its public include files
include (for this example) flac's, then you need this or the package
using alure won't be able to get to them. Otherwise (if the library
presents its own self-contained API and ABI) you don't.
I think. Now that I think about it I'm not sure what happens with
static linking :-/
--
David A. Holland
dholland@netbsd.org
>Unformatted:
(Contact us)
$NetBSD: query-full-pr,v 1.43 2018/01/16 07:36:43 maya Exp $
$NetBSD: gnats_config.sh,v 1.9 2014/08/02 14:16:04 spz Exp $
Copyright © 1994-2017
The NetBSD Foundation, Inc. ALL RIGHTS RESERVED.