[pulseaudio-discuss] [RFC] [PATCH] build-sys: Make esound bits optional
Colin Guthrie
gmane at colin.guthr.ie
Thu Dec 22 05:35:28 PST 2011
'Twas brillig, and Maarten Bosmans at 21/12/11 21:50 did gyre and gimble:
> 2011/12/21 Arun Raghavan <arun.raghavan at collabora.co.uk>:
>> IMO EsounD is really quite irrelevant on most modern systems today, and more
>> so for embedded systems.
>>
>> Any objections to making it optional?
>
> No, seems fine.
>
> I'd go for HAVE_ESOUND though just to keep it consistent with the rest.
>
> As seen in Makefile.am as a conditional or in source files as a macro,
> the semantics of the symbol for esound are not different from those
> for e.g. solaris. It just determines whether a certain piece of code
> gets compiled or not.
>
> The only place where esound is different from the others is in
> configure.ac. But even here we have already lost a precise link
> between HAVE_SOLARIS and the availability of the solaris headers. As
> when --disable-solaris is passed to configure, HAVE_SOLARIS=0 even
> when the headers are available.
>
> So it's probably better to be consistently imprecise with our variable
> naming than to introduce a new name prefix.
Yeah what Maarten said :D
Looks good.
Col
--
Colin Guthrie
gmane(at)colin.guthr.ie
http://colin.guthr.ie/
Day Job:
Tribalogic Limited http://www.tribalogic.net/
Open Source:
Mageia Contributor http://www.mageia.org/
PulseAudio Hacker http://www.pulseaudio.org/
Trac Hacker http://trac.edgewall.org/
More information about the pulseaudio-discuss
mailing list