On Mon, 17 Oct 2016, Peter Stuge wrote:

I've attached my current git diff master state for review as well as testing. It has a couple of things which still need to be done, but is already functional.

The diff also includes a few unrelated cleanups; a few type issues and not using RSA if the crypto backend does not implement it.

Here's some quick first questions/notes:

Which axTLS version or versions does this work with? (I couldn't even figure out how to build axTLS 2.0.1 so I didn't try it out yet)

Your configure check doesn't work at all like the other crypto backends and it doesn't seem to support a custom install path. I would expect that you be fairly common with this crypto lib. Does it even ship a pkg-config file itself? It also adds a requirement for the pkg-config autoconf stuff, which I guess I'm fine with but I know that in other projects that make people uncomfortable because it adds more prerequisits to the build process.

src/axtls.[ch] don't use our source code style: tabs, wrong indent level, long lines, starting brace in function declaration on the wrong line, case + return on the same line.

--

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

Reply via email to