[SCM] rtaudio/master: Add libpulse-dev to deps of -dev package. (Closes: #833271)

[Jaromír Mikeš]

2016-08-02 15:39 GMT+02:00 Felipe Sateler <fsateler at debian.org>:
> On 2 August 2016 at 07:43,  <mira-guest at users.alioth.debian.org> wrote:
>> The following commit has been merged in the master branch:
>> commit e34c103bad0f3ab33e7c6dd578cd061d6f4a57b1
>> Author: Jaromír Mikeš <mira.mikes at seznam.cz>
>> Date:   Tue Aug 2 13:37:36 2016 +0200
>>
>>     Add libpulse-dev to deps of -dev package. (Closes: #833271)
>
>
> Is this the  correct fix? AFAICT there is no reason for an
> rtaudio-using package to require libpulse. The correct fix seems to me
> to be dropping libpulse-dev (and alsa) from Requires, as rtaudio is
> supposed to be an abstraction over those libraries, and the dev
> packages are not required:
>
> % grep '#include' RtAudio.h
> #include <string>
> #include <vector>
> #include <exception>
> #include <iostream>
>   #include <windows.h>
>   #include <process.h>
>   #include <pthread.h>
>   #include <sys/time.h>
> #include <sstream>
> #include <CoreAudio/AudioHardware.h>

Hi Felipe,

I hope this is correct fix.
It should be according bug description.
https://bugs.debian.org/cgi-bin/bugreport.cgi?bug=833271
IOhannes can you comment on this please?

best regards

mira