[Buildroot] [PATCH v2, 1/1] package/ffmpeg: add -latomic to pkg-config files

Fabrice Fontaine fontaine.fabrice at gmail.com
Sun Aug 8 21:30:28 UTC 2021


Le dim. 8 août 2021 à 23:22, Thomas Petazzoni
<thomas.petazzoni at bootlin.com> a écrit :
>
> On Sat,  7 Aug 2021 21:40:31 +0200
> Fabrice Fontaine <fontaine.fabrice at gmail.com> wrote:
>
> > -latomic was added to extralibs to fix static build of ffmpeg in commit
> > fc8798197b57488a852e2c14b4677b5eb0114a0b. However, extralibs is not
> > added to libavformat.pc resulting in the following static build failure
> > of motion:
> >
> > /home/buildroot/autobuild/instance-1/output-1/host/opt/ext-toolchain/bin/../lib/gcc/arm-buildroot-linux-uclibcgnueabi/9.3.0/../../../../arm-buildroot-linux-uclibcgnueabi/bin/ld: /home/buildroot/autobuild/instance-1/output-1/host/bin/../arm-buildroot-linux-uclibcgnueabi/sysroot/usr/lib/libavformat.a(fifo.o): in function `fifo_init':
> > /home/buildroot/autobuild/instance-1/output-1/build/ffmpeg-4.4/libavformat/fifo.c:519: undefined reference to `__atomic_store_8'
> >
> > So add a patch to add extralibs (and so -latomic) to all pkg-config
> > files
> >
> > Fixes:
> >  - http://autobuild.buildroot.org/results/62ec618e40081a250b8129ec6f5a178eb06fba1d
> >
> > Signed-off-by: Fabrice Fontaine <fontaine.fabrice at gmail.com>
> > ---
> > Changes v1 -> v2 (after review of Yann E. Morin):
> >  - Add extralibs to all pkg-config files and not only libavformat.pc
>
> Applied to master, thanks. Why do you consider the patch not acceptable
> upstream?
Because I think it only makes sense because we're passing -latomic to
extralibs but I can try to get their feedback.
>
> Thomas
> --
> Thomas Petazzoni, CTO, Bootlin
> Embedded Linux and Kernel engineering
> https://bootlin.com
Best Regards,

Fabrice


More information about the buildroot mailing list