Thread (1 message) 1 message, 1 author, 2016-06-15

Re: [PATCH] Makefile: add Minix configuration options.

From: Junio C Hamano <hidden>
Date: 2016-06-15 22:51:37

Thomas Cort [off-list ref] writes:
 * Adds a $(uname_S) case for Minix with the correct options.
Ok, but lose the asterisk and speak as if you are giving an order to the
codebase or the person who maintains it, i.e. "Add $(uname_S) case...".
 * Adds NEEDS_SSL_WITH_CURL and NEEDS_CRYPTO_WITH_SSL to the
   libcurl section. On Minix both -lssl and -lcrypto are
   needed when using -lcurl.
These...
+	NEEDS_IDN_WITH_CURL = YesPlease
+	NEEDS_SSL_WITH_CURL = YesPlease
... are not described in the beginning of Makefile; add them.
 * Changes OPENSSL_LINK to OPENSSL_LIBSSL in the
   NEEDS_CRYPTO_WITH_SSL conditional in the libopenssl section.
Why is this change necessary?
   The -lcrypto needs to be in OPENSSL_LIBSSL.
Ditto.

Thanks.
Keyboard shortcuts
hback out one level
jnext message in thread
kprevious message in thread
ldrill in
Escclose help / fold thread tree
?toggle this help