Re: [PATCH] Have sndio check use the pkg-config file

Reimar Döffinger <[email protected]> Sun, 27 Mar 2022 12:38:53 +0200
Newsgroups gmane.comp.video.mplayer.devel
Message-ID <[email protected]>
> On 27 Feb 2022, at 23:46, Brad Smith <[email protected]> wrote:
>> I think it might also cause issues with static linking.
>> So yes it would be better to change that.
>> Also why did you change from statement_check to header_check?
> 
> I was copying what Pulse was doing, not being sure what was appropriate. Pulse has a library so it should
> be checking that as well I would think.

To be honest I think all the code here is bad, so the comments are more from a "no regressions" point.
It seems you can assume it is there on OpenBSD, but pkg-config often ends up not installed on Linux quite often, and sndio exists on Linux.
So I would normally suggest to add pkg-config only as a fallback option like for the JACK case you mentioned.
But I am aware that also has its issues, and you didn't say why you want this change (which I kind of also need to know for the commit message).

Best regards,
Reimar