[MPlayer-dev-eng] [PATCH] Depend on bio2jack version 0.2

ismail dönmez ismail.donmez at gmail.com
Tue Aug 10 19:49:51 CEST 2004


Your humble awk-lovin' servant proposes attached patches

Cheers,
ismail

On Tue, 10 Aug 2004 19:16:42 +0200, Diego Biurrun <diego at biurrun.de> wrote:
> ismail dönmez writes:
> >
> > Lastly two patches attached one makes configure to check bio2jack
> > version , the latter makes ao_jack to compile with bio2jack v0.2 .
> 
> Patch rejected.
> 
> Some remarks:
> 
> -int  JACK_SetVolume(int deviceID, int left, int right); /* returns 0 on success */
> +int  JACK_SetVolumeForChannel(int deviceID, unsigned int channel, unsigned int volume);
> 
> Was that comment removed for a reason?
> 
> +        if ( ( test `bio2jack-config --version | awk -F"." '{print $2}'` != 2 ) ) ; then
> 
> There is no need to use full awk here, cut will do the job nicely, or
> - even better - you could simply check for the complete version number
> (0.2) and save yourself calling awk/cut.
> 
> Diego
> 
> _______________________________________________
> MPlayer-dev-eng mailing list
> MPlayer-dev-eng at mplayerhq.hu
> http://mplayerhq.hu/mailman/listinfo/mplayer-dev-eng
> 


-- 
Time is what you make of it
-------------- next part --------------
A non-text attachment was scrubbed...
Name: bio2jack_aojack.patch
Type: application/octet-stream
Size: 2071 bytes
Desc: not available
URL: <http://lists.mplayerhq.hu/pipermail/mplayer-dev-eng/attachments/20040810/7f526024/attachment.obj>
-------------- next part --------------
A non-text attachment was scrubbed...
Name: bio2jack_configure.patch
Type: application/octet-stream
Size: 618 bytes
Desc: not available
URL: <http://lists.mplayerhq.hu/pipermail/mplayer-dev-eng/attachments/20040810/7f526024/attachment-0001.obj>


More information about the MPlayer-dev-eng mailing list