Re: [FFmpeg-devel] [PATCH] configure: fix configure check for lilv-0
On Sun, May 13, 2018 at 7:12 PM, Paul B Maholwrote: > On 5/13/18, Jan Ekstroem wrote: >> On Sun, May 13, 2018 at 6:52 PM, Paul B Mahol wrote: >>> On 5/13/18, Niklas Haas wrote: From: Niklas Haas This should be included as ``, same as is done in af_lv2.c. Forcing the extra lilv-0 breaks platforms where the include dir is `/usr/include/lilv/lilv.h` rather than `/usr/include/lilv-0/lilv/lilv.h`. The new include path works for both, because the `pkg-config --cflags` includes `-I/usr/include/lilv-0`. --- configure | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) >>> >>> lgtm >> >> Thank you. Pushed in. >> >> Should this be back-ported to release/4.0 as well? > > If you care. And thus, back-ported to release/4.0. Jan ___ ffmpeg-devel mailing list ffmpeg-devel@ffmpeg.org http://ffmpeg.org/mailman/listinfo/ffmpeg-devel
Re: [FFmpeg-devel] [PATCH] configure: fix configure check for lilv-0
On Sun, May 13, 2018 at 7:12 PM, Paul B Maholwrote: > On 5/13/18, Jan Ekstroem wrote: >> On Sun, May 13, 2018 at 6:52 PM, Paul B Mahol wrote: >>> On 5/13/18, Niklas Haas wrote: From: Niklas Haas This should be included as ``, same as is done in af_lv2.c. Forcing the extra lilv-0 breaks platforms where the include dir is `/usr/include/lilv/lilv.h` rather than `/usr/include/lilv-0/lilv/lilv.h`. The new include path works for both, because the `pkg-config --cflags` includes `-I/usr/include/lilv-0`. --- configure | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) >>> >>> lgtm >> >> Thank you. Pushed in. >> >> Should this be back-ported to release/4.0 as well? > > If you care. You may say that I do not care, but I have this still vivid in my memory and I would guess you would like your filter to build better in releases as well. Best regards, Jan ___ ffmpeg-devel mailing list ffmpeg-devel@ffmpeg.org http://ffmpeg.org/mailman/listinfo/ffmpeg-devel
Re: [FFmpeg-devel] [PATCH] configure: fix configure check for lilv-0
On 5/13/18, Jan Ekstroemwrote: > On Sun, May 13, 2018 at 6:52 PM, Paul B Mahol wrote: >> On 5/13/18, Niklas Haas wrote: >>> From: Niklas Haas >>> >>> This should be included as ``, same as is done in af_lv2.c. >>> Forcing the extra lilv-0 breaks platforms where the include dir is >>> `/usr/include/lilv/lilv.h` rather than >>> `/usr/include/lilv-0/lilv/lilv.h`. >>> >>> The new include path works for both, because the `pkg-config --cflags` >>> includes `-I/usr/include/lilv-0`. >>> --- >>> configure | 2 +- >>> 1 file changed, 1 insertion(+), 1 deletion(-) >>> >> >> lgtm > > Thank you. Pushed in. > > Should this be back-ported to release/4.0 as well? If you care. ___ ffmpeg-devel mailing list ffmpeg-devel@ffmpeg.org http://ffmpeg.org/mailman/listinfo/ffmpeg-devel
Re: [FFmpeg-devel] [PATCH] configure: fix configure check for lilv-0
On Sun, May 13, 2018 at 6:52 PM, Paul B Maholwrote: > On 5/13/18, Niklas Haas wrote: >> From: Niklas Haas >> >> This should be included as ``, same as is done in af_lv2.c. >> Forcing the extra lilv-0 breaks platforms where the include dir is >> `/usr/include/lilv/lilv.h` rather than >> `/usr/include/lilv-0/lilv/lilv.h`. >> >> The new include path works for both, because the `pkg-config --cflags` >> includes `-I/usr/include/lilv-0`. >> --- >> configure | 2 +- >> 1 file changed, 1 insertion(+), 1 deletion(-) >> > > lgtm Thank you. Pushed in. Should this be back-ported to release/4.0 as well? Jan ___ ffmpeg-devel mailing list ffmpeg-devel@ffmpeg.org http://ffmpeg.org/mailman/listinfo/ffmpeg-devel
Re: [FFmpeg-devel] [PATCH] configure: fix configure check for lilv-0
On 5/13/18, Jan Ekstroemwrote: > On Sun, May 13, 2018 at 6:07 PM, Niklas Haas wrote: >> From: Niklas Haas >> >> This should be included as ``, same as is done in af_lv2.c. >> Forcing the extra lilv-0 breaks platforms where the include dir is >> `/usr/include/lilv/lilv.h` rather than >> `/usr/include/lilv-0/lilv/lilv.h`. >> >> The new include path works for both, because the `pkg-config --cflags` >> includes `-I/usr/include/lilv-0`. >> --- >> configure | 2 +- >> 1 file changed, 1 insertion(+), 1 deletion(-) >> >> diff --git a/configure b/configure >> index a1f13a7109..451d1937b7 100755 >> --- a/configure >> +++ b/configure >> @@ -5973,7 +5973,7 @@ enabled gnutls&& require_pkg_config >> gnutls gnutls gnutls/gnutls.h gn >> enabled jni && { [ $target_os = "android" ] && check_header >> jni.h && enabled pthreads || die "ERROR: jni not found"; } >> enabled ladspa&& require_header ladspa.h >> enabled libaom&& require_pkg_config libaom "aom >= 0.1.0" >> aom/aom_codec.h aom_codec_version >> -enabled lv2 && require_pkg_config lv2 lilv-0 >> "lilv-0/lilv/lilv.h" lilv_world_new >> +enabled lv2 && require_pkg_config lv2 lilv-0 "lilv/lilv.h" >> lilv_world_new >> enabled libiec61883 && require libiec61883 libiec61883/iec61883.h >> iec61883_cmp_connect -lraw1394 -lavc1394 -lrom1394 -liec61883 >> enabled libass&& require_pkg_config libass libass ass/ass.h >> ass_library_init >> enabled libbluray && require_pkg_config libbluray libbluray >> libbluray/bluray.h bd_open >> -- >> 2.17.0 >> > > Generally looks good, CC'ing Paul so he can verify that it still > builds for him as well. Why are you CCing me? ___ ffmpeg-devel mailing list ffmpeg-devel@ffmpeg.org http://ffmpeg.org/mailman/listinfo/ffmpeg-devel
Re: [FFmpeg-devel] [PATCH] configure: fix configure check for lilv-0
On 5/13/18, Niklas Haaswrote: > From: Niklas Haas > > This should be included as ``, same as is done in af_lv2.c. > Forcing the extra lilv-0 breaks platforms where the include dir is > `/usr/include/lilv/lilv.h` rather than > `/usr/include/lilv-0/lilv/lilv.h`. > > The new include path works for both, because the `pkg-config --cflags` > includes `-I/usr/include/lilv-0`. > --- > configure | 2 +- > 1 file changed, 1 insertion(+), 1 deletion(-) > lgtm ___ ffmpeg-devel mailing list ffmpeg-devel@ffmpeg.org http://ffmpeg.org/mailman/listinfo/ffmpeg-devel
Re: [FFmpeg-devel] [PATCH] configure: fix configure check for lilv-0
On Sun, May 13, 2018 at 6:07 PM, Niklas Haaswrote: > From: Niklas Haas > > This should be included as ``, same as is done in af_lv2.c. > Forcing the extra lilv-0 breaks platforms where the include dir is > `/usr/include/lilv/lilv.h` rather than > `/usr/include/lilv-0/lilv/lilv.h`. > > The new include path works for both, because the `pkg-config --cflags` > includes `-I/usr/include/lilv-0`. > --- > configure | 2 +- > 1 file changed, 1 insertion(+), 1 deletion(-) > > diff --git a/configure b/configure > index a1f13a7109..451d1937b7 100755 > --- a/configure > +++ b/configure > @@ -5973,7 +5973,7 @@ enabled gnutls&& require_pkg_config gnutls > gnutls gnutls/gnutls.h gn > enabled jni && { [ $target_os = "android" ] && check_header > jni.h && enabled pthreads || die "ERROR: jni not found"; } > enabled ladspa&& require_header ladspa.h > enabled libaom&& require_pkg_config libaom "aom >= 0.1.0" > aom/aom_codec.h aom_codec_version > -enabled lv2 && require_pkg_config lv2 lilv-0 > "lilv-0/lilv/lilv.h" lilv_world_new > +enabled lv2 && require_pkg_config lv2 lilv-0 "lilv/lilv.h" > lilv_world_new > enabled libiec61883 && require libiec61883 libiec61883/iec61883.h > iec61883_cmp_connect -lraw1394 -lavc1394 -lrom1394 -liec61883 > enabled libass&& require_pkg_config libass libass ass/ass.h > ass_library_init > enabled libbluray && require_pkg_config libbluray libbluray > libbluray/bluray.h bd_open > -- > 2.17.0 > Generally looks good, CC'ing Paul so he can verify that it still builds for him as well. Jan ___ ffmpeg-devel mailing list ffmpeg-devel@ffmpeg.org http://ffmpeg.org/mailman/listinfo/ffmpeg-devel
[FFmpeg-devel] [PATCH] configure: fix configure check for lilv-0
From: Niklas HaasThis should be included as ``, same as is done in af_lv2.c. Forcing the extra lilv-0 breaks platforms where the include dir is `/usr/include/lilv/lilv.h` rather than `/usr/include/lilv-0/lilv/lilv.h`. The new include path works for both, because the `pkg-config --cflags` includes `-I/usr/include/lilv-0`. --- configure | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/configure b/configure index a1f13a7109..451d1937b7 100755 --- a/configure +++ b/configure @@ -5973,7 +5973,7 @@ enabled gnutls&& require_pkg_config gnutls gnutls gnutls/gnutls.h gn enabled jni && { [ $target_os = "android" ] && check_header jni.h && enabled pthreads || die "ERROR: jni not found"; } enabled ladspa&& require_header ladspa.h enabled libaom&& require_pkg_config libaom "aom >= 0.1.0" aom/aom_codec.h aom_codec_version -enabled lv2 && require_pkg_config lv2 lilv-0 "lilv-0/lilv/lilv.h" lilv_world_new +enabled lv2 && require_pkg_config lv2 lilv-0 "lilv/lilv.h" lilv_world_new enabled libiec61883 && require libiec61883 libiec61883/iec61883.h iec61883_cmp_connect -lraw1394 -lavc1394 -lrom1394 -liec61883 enabled libass&& require_pkg_config libass libass ass/ass.h ass_library_init enabled libbluray && require_pkg_config libbluray libbluray libbluray/bluray.h bd_open -- 2.17.0 ___ ffmpeg-devel mailing list ffmpeg-devel@ffmpeg.org http://ffmpeg.org/mailman/listinfo/ffmpeg-devel