[FFmpeg-devel] [PATCH] configure: use just the pkg-config for sndio

[Michael Niedermayer]

On Mon, Aug 07, 2023 at 03:04:35PM -0400, Brad Smith wrote:
> On 2023-07-20 3:17 p.m., Brad Smith wrote:
> > On 7/7/2023 3:16 PM, Brad Smith wrote:
> > > On 2023-07-01 2:58 p.m., Brad Smith wrote:
> > > > On 2023-06-23 7:36 p.m., Brad Smith wrote:
> > > > > On 2023-06-23 7:35 p.m., Michael Niedermayer wrote:
> > > > > > On Fri, Jun 23, 2023 at 06:56:30PM -0400, Brad Smith wrote:
> > > > > > > On 2023-06-23 6:55 p.m., Michael Niedermayer wrote:
> > > > > > > > On Fri, Jun 23, 2023 at 06:41:08PM -0400, Brad Smith wrote:
> > > > > > > > > ping.
> > > > > > > > > 
> > > > > > > > > On 2023-06-17 6:48 p.m., Brad Smith wrote:
> > > > > > > > > > On Sun, Jun 18, 2023 at 12:01:14AM
> > > > > > > > > > +0200, Michael Niedermayer wrote:
> > > > > > > > > > > this breaks a plain configure
> > > > > > > > > > > here on ubuntu
> > > > > > > > > > > 
> > > > > > > > > > > ./configure
> > > > > > > > > > > ERROR: sndio not found using pkg-config
> > > > > > > > > > > 
> > > > > > > > > > > If you think configure made a
> > > > > > > > > > > mistake, make sure you are using the
> > > > > > > > > > > latest
> > > > > > > > > > > version from Git.  If the latest
> > > > > > > > > > > version fails, report the problem to
> > > > > > > > > > > the
> > > > > > > > > > > ffmpeg-user at ffmpeg.org mailing list
> > > > > > > > > > > or IRC #ffmpeg on irc.libera.chat.
> > > > > > > > > > > Include the log file
> > > > > > > > > > > "ffbuild/config.log" produced by
> > > > > > > > > > > configure as this will help
> > > > > > > > > > > solve the problem.
> > > > > > > > > > > 
> > > > > > > > > > > 
> > > > > > > > > > > 
> > > > > > > > > > > [...]
> > > > > > > > > > > -- 
> > > > > > > > > > > Michael     GnuPG fingerprint:
> > > > > > > > > > > 9FF2128B147EF6730BADF133611EC787040B0FAB
> > > > > > > > > > > 
> > > > > > > > > > > The misfortune of the wise is better
> > > > > > > > > > > than the prosperity of the fool.
> > > > > > > > > > > -- Epicurus
> > > > > > > > > > This is what I had intended.
> > > > > > > > You intended to break a plain ./configure on ubuntu ?
> > > > > > > > If so i think we better dont apply that patch :)
> > > > > > > > 
> > > > > > > > thx
> > > > > > > No, there was a second patch there.
> > > > > > oops i missed that, the 2nd patch works fine on ubuntu
> > > > > > no objections from me
> > > > > > 
> > > > > > thx
> > > > 
> > > > 
> > > > ping.
> > > > 
> > > 
> > > ping.
> > 
> > 
> > ping.
> 
> 
> Is there something wrong with the patch?

no, i was just a bit hesitant because i didnt know if anyone or anything
would depend on that removed case

ill apply it

thx

[...]
-- 
Michael     GnuPG fingerprint: 9FF2128B147EF6730BADF133611EC787040B0FAB

I have often repented speaking, but never of holding my tongue.
-- Xenocrates
-------------- next part --------------
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 195 bytes
Desc: not available
URL: <https://ffmpeg.org/pipermail/ffmpeg-devel/attachments/20230807/acc48ba3/attachment.sig>