[Stk] rtaudio / pulse & build

[Stephen Sinclair]

In the pkg-config branch, I got an error when I ran "autoconf && make"
on my system:

checking for audio API... ./configure: line 4156: syntax error near
unexpected token `PULSE,'
./configure: line 4156: `    PKG_CHECK_MODULES(PULSE, libpulse-simple,
, as_fn_error $? "PulseAudio support requires the pulse-simple
library!" "$LINENO" 5 )

It seemed that it couldn't find the pkg-config-related m4 modules,
which I can imagine will be a problem on some systems.  I fixed it by
adding the following lines near the top of the file, making it ignore
the pkg-config-related macros if they don't exist.  This is taken
from,
http://osdir.com/ml/automake-gnu/2011-03/msg00007.html

dnl Check for pkg-config program, used for configuring some libraries.
m4_define_default([PKG_PROG_PKG_CONFIG],
[AC_MSG_CHECKING([pkg-config])
AC_MSG_RESULT([no])])

PKG_PROG_PKG_CONFIG

dnl If the pkg-config autoconf support isn't installed, define its
dnl autoconf macro to disable any packages depending on it.
m4_define_default([PKG_CHECK_MODULES],
[AC_MSG_CHECKING([$1])
AC_MSG_RESULT([no])
$4])

After this, and being slightly confused since pkg.m4 was clearly in
the right place on my system, I realized I had to run "aclocal" in the
folder to get it to import the correct macros from pkg.m4.  Once I did
that and re-ran autoconf, I can confirm that this branch compiles and
tests fine for me.  It also complained nicely when I didn't have
pulse-simple installed.  (Although, I had to guess that this
corresponded with the libpulse-dev package.)

I'd recommend adding the lines that make autoconf work even without
pkg-config installed, since I think that will be a problem on OS X and
Windows.

By the way it's weird that "configure" is in the repository.  I
thought it was working all fine until I thought of running autoconf to
re-generate configure, so I almost missed this.  I only did it because
I happened to notice that librtaudio.pc wasn't being generated.
(Which it now is.)

Looking at it.. it would be interesting maybe to add -L and -l and
-framework flags to librtaudio.pc.in as necessary for whatever audio
backend is compiled in.

Steve

On Sun, Jul 22, 2012 at 9:36 PM, Tristan Matthews
<le.businessman at gmail.com> wrote:
> Hi,
>
> 2012/7/19 Peter Meerwald <pmeerw at pmeerw.net>:
>> Hello,
>>
>>> > I plan to propose/submit patches for two things:
>>> > - rework RTAUDIO_ALSA_USE_DEFAULT, goal is to use the default ALSA
>>> > device but it seems some API calls such as GetDeviceCount(),
>>> > GetDeviceInfo() do not work with default
>>> > - provide a librtaudio.pc file for pkgconfig
>>
>>> Both of those would be helpful changes for GNU/Linux developers. I can
>>> update the github repo today to be in sync with the latest RtAudio
>>> release, otherwise sending patches is fine too.
>>
>> attached my proposed patch, some comments can be found on top of some
>> patches; feedback is appreciated
>>
>> I think it would be good if Tristan could stage these patches on github
>
> I had to scrap the old github rtaudio repo, because it was imported
> with an older version of svn2git, and as a result I wasn't able to
> update it with the latest changes in rtaudio's svn.
> So I've uploaded a new git mirror of the rtaudo subversion repository
> to gitorious, the master branch is now in sync with the trunk of the
> official RtAudio svn repo:
>
> https://gitorious.org/rtaudio/rtaudio/commits/master
>
> and the pkgconfig related patches have been applied in a branch
> appropriately named "pkgconfig":
>
> I haven't applied the other two patches yet, as they are for different
> features. It would be good if this initial set got some testing and
> feedback first, and then hopefully they can be merged back into the
> RtAudio trunk. To test, simply clone the repository and checkout the
> pkgconfig branch:
>
> git clone https://gitorious.org/rtaudio/rtaudio/commits/master
> git checkout pkgconfig
>
> Best,
> Tristan
>
> --
> Tristan Matthews
> web: http://tristanswork.blogspot.com
>
> _______________________________________________
> Stk mailing list
> Stk at ccrma.stanford.edu
> http://ccrma-mail.stanford.edu/mailman/listinfo/stk
>
>
-------------- next part --------------
A non-text attachment was scrubbed...
Name: 0001-Ignore-pkg-config-related-m4-macros-if-they-aren-t-a.patch
Type: application/octet-stream
Size: 1218 bytes
Desc: not available
Url : http://ccrma-mail.stanford.edu/pipermail/stk/attachments/20120722/0e3a0118/attachment.obj