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

Reply via email to