Subject: Re: [PATCH] configure.ac: Add single --with-crypto= instead of --with-$backend:s

Re: [PATCH] configure.ac: Add single --with-crypto= instead of --with-$backend:s

From: Peter Stuge <peter_at_stuge.se>
Date: Mon, 31 Oct 2016 13:23:42 +0000

Hi Kamil,

Thanks a lot for testing this!

Kamil Dudka wrote:
> The patch does not apply on the current upstream master branch, so I tested
> the withcrypto branch at http://git.stuge.se/libssh2.git (68b330d2).

Yes - it applies on top of the handful simple fixes in my simple180
branch, which I posted before. Sorry, should have made that more clear.

> Basic crypto backend selection seems to work.
>
> --with-crypto=auto works fine for OpenSSL but does not work for libgcrypt:
>
> $ ./configure --with-crypto=auto
> [...]
> configure: ERROR: No openssl crypto library found!
> configure: error: Required dependencies are missing!

Good find - I believe libgcrypt was actually successfully detected by
the tests, but the end logic needs a little fixing. Patch attached.

> As a side note, the --with-libssl-prefix option did not take any
> effect but it seems unrelated to your patch.

Oh! How did you notice that it does not take effect, and how did you
deal with that instead, if you need it?

The --with-x-prefix options are a bit unrelated; they are created and
supposed to be handled by AC_LIB_HAVE_LINKFLAGS, but they are related
because they deal with library finding, so I'd like to know more
about the problem you found. If you have config.log from a problem
run maybe you can send it to me. (Maybe off-list if it's large.)

Thanks again

//Peter

_______________________________________________
libssh2-devel https://cool.haxx.se/cgi-bin/mailman/listinfo/libssh2-devel

Received on 2016-10-31