On Mon, Oct 02, 2017 at 11:49:27AM +0200, Diego Biurrun wrote: > On Fri, Sep 29, 2017 at 10:56:08PM -0300, James Almer wrote: > > --- a/configure > > +++ b/configure > > @@ -1025,8 +1025,8 @@ check_lib(){ > > > > -check_pkg_config(){ > > - log check_pkg_config "$@" > > +test_pkg_config(){ > > + log test_pkg_config "$@" > > name="$1" > > pkg_version="$2" > > pkg="${2%% *}" > > @@ -1039,8 +1039,15 @@ check_pkg_config(){ > > pkg_libs=$($pkg_config --libs $pkg_config_flags $pkg) > > check_func_headers "$headers" "$funcs" $pkg_cflags $pkg_libs "$@" && > > enable $name && > > - add_cflags "$pkg_cflags" && > > - eval $(sanitize_var_name ${name}_extralibs)="\$pkg_libs" > > + set_sanitized "${name}_cflags" $pkg_cflags && > > + set_sanitized "${name}_extralibs" $pkg_libs > > +} > > + > > +check_pkg_config(){ > > + log check_pkg_config "$@" > > + pkg="${2%% *}" > > + test_pkg_config "$@" || return > > + add_cflags $(get_sanitized "${name}_cflags") > > } > > At a quick glance, this looks like name and pkg are confused.
No. I should not review patches on the airport after what felt like 6 hours of sleep in the last three days... Diego _______________________________________________ libav-devel mailing list libav-devel@libav.org https://lists.libav.org/mailman/listinfo/libav-devel