[FFmpeg-devel] [PATCH] configure: jni no longer requires -ldl

[Aaron Levinson]

On 5/12/2017 11:34 AM, Aman Gupta wrote:
> From: Aman Gupta <aman at tmm1.net>
>
> this dependency was removed in 33d69a90085d30af8a292d9364b835a26565d6b9
> ---
>  configure | 3 +--
>  1 file changed, 1 insertion(+), 2 deletions(-)
>
> diff --git a/configure b/configure
> index 5ae5227868..ba33241a45 100755
> --- a/configure
> +++ b/configure
> @@ -5766,8 +5766,7 @@ enabled decklink          && { { check_header DeckLinkAPI.h || die "ERROR: DeckL
>  enabled frei0r            && { check_header frei0r.h || die "ERROR: frei0r.h header not found"; }
>  enabled gmp               && require gmp gmp.h mpz_export -lgmp
>  enabled gnutls            && require_pkg_config gnutls gnutls/gnutls.h gnutls_global_init
> -enabled jni               && { [ $target_os = "android" ] && check_header jni.h && enabled pthreads &&
> -                               check_lib jni "dlfcn.h" dlopen -ldl || die "ERROR: jni not found"; }
> +enabled jni               && { [ $target_os = "android" ] && check_header jni.h && enabled pthreads || die "ERROR: jni not found"; }
>  enabled ladspa            && { check_header ladspa.h || die "ERROR: ladspa.h header not found"; }
>  enabled libiec61883       && require libiec61883 libiec61883/iec61883.h iec61883_cmp_connect -lraw1394 -lavc1394 -lrom1394 -liec61883
>  enabled libass            && require_pkg_config libass ass/ass.h ass_library_init
>

LGTM--I see that the use of dl was eliminated in 33d69a9 (at 
https://github.com/FFmpeg/FFmpeg/commit/33d69a90085d30af8a292d9364b835a26565d6b9 
).

Aaron Levinson