Re: [flac-dev] [PATCH] Optionally, allow distros to use openssl for MD5 verification

2012-05-08 Thread Erik de Castro Lopo
Cristian Rodríguez wrote: flac and metaflac do not use openSSL, only libFLAC does. But flac and metaflac are GPL and link (possibly statically) to libFLAC. Personally I don't see any technical reason to use a different library other than politics and obscure potential license

Re: [flac-dev] [PATCH] Optionally, allow distros to use openssl for MD5 verification

2012-05-07 Thread Eric Wong
Cristian Rodríguez crrodrig...@opensuse.org wrote: El 07/05/12 06:23, Miroslav Lichvar escribió: On Sat, May 05, 2012 at 05:34:31PM -0400, Cristian Rodríguez wrote: This has the advantage of being more efficient than the included Btw, how much more efficient and on what hardware? Based on

Re: [flac-dev] [PATCH] Optionally, allow distros to use openssl for MD5 verification

2012-05-07 Thread Cristian Rodríguez
El 07/05/12 16:19, Eric Wong escribió: flac and metaflac do not use openSSL, only libFLAC does. But since flac/metaflac use libFLAC, I suspect (IANAL) the relationship can be transitive. (But being an optional dependency helps as distros can avoid potential issues). That's why it is

[flac-dev] [PATCH] Optionally, allow distros to use openssl for MD5 verification

2012-05-05 Thread Cristian Rodríguez
This has the advantage of being more efficient than the included routines and allows distros to centralize crypto mainteniance on a few libraries. --- configure.ac |4 +- m4/ax_check_openssl.m4| 124 +

Re: [flac-dev] [PATCH] Optionally, allow distros to use openssl for MD5 verification

2012-05-05 Thread Eric Wong
Cristian Rodríguez crrodrig...@opensuse.org wrote: +#if defined(HAVE_OPENSSL) +/* decoder-private_-computed_md5sum is NULL when decoder-private_-do_md5_checking == false +* that causes assertion failure crash in openSSL. +*/ +if(decoder-private_-do_md5_checking) { +

Re: [flac-dev] [PATCH] Optionally, allow distros to use openssl for MD5 verification

2012-05-05 Thread Erik de Castro Lopo
Eric Wong wrote: Can you do this without sprinkling #ifdefs all over the place? +1 Mixing #ifdefs and normal C control structures make code hard to read/maintain. This is *especially* true for folks who aren't regular contributors to flac, myself included. Very much agree. Erik --