On Fri, 2021-10-08 at 10:32 -0300, James Almer wrote:
> Some packages may not define custom cflags, in which case a simple
> "pkg-config --cflags" call will return an empty string.
> This change will be useful to get a valid include path that can be
> used in library checks.
>
> Signed-off-by: James Almer <jamr...@gmail.com>
> ---
> configure | 5 ++++-
> 1 file changed, 4 insertions(+), 1 deletion(-)
>
> diff --git a/configure b/configure
> index 5e630e6e5d..92610c7edc 100755
> --- a/configure
> +++ b/configure
> @@ -1446,9 +1446,11 @@ test_pkg_config(){
> test_cmd $pkg_config --exists --print-errors $pkg_version || return
> pkg_cflags=$($pkg_config --cflags $pkg_config_flags $pkg)
> pkg_libs=$($pkg_config --libs $pkg_config_flags $pkg)
> + pkg_incdir=$($pkg_config --variable=includedir $pkg_config_flags $pkg)
> check_func_headers "$headers" "$funcs" $pkg_cflags $pkg_libs "$@" &&
> enable $name &&
> set_sanitized "${name}_cflags" $pkg_cflags &&
> + set_sanitized "${name}_incdir" $pkg_incdir &&
> set_sanitized "${name}_extralibs" $pkg_libs
> }
>
> @@ -1466,7 +1468,8 @@ test_pkg_config_cpp(){
> pkg_incdir=$($pkg_config --variable=includedir $pkg_config_flags $pkg)
> test_cpp_condition "$pkg_incdir/$headers" "$cond" "$@" &&
> enable $name &&
> - set_sanitized "${name}_cflags" $pkg_cflags
> + set_sanitized "${name}_cflags" $pkg_cflags &&
> + set_sanitized "${name}_incdir" $pkg_incdir
> }
>
> check_pkg_config(){
LGTM, and I can update my patchset to use ${libmfx_incdir} once this patch is
merged.
Regards
Haihao
_______________________________________________
ffmpeg-devel mailing list
ffmpeg-devel@ffmpeg.org
https://ffmpeg.org/mailman/listinfo/ffmpeg-devel
To unsubscribe, visit link above, or email
ffmpeg-devel-requ...@ffmpeg.org with subject "unsubscribe".
