On Sat, Aug 15, 2009 at 18:46, Brian Gernhardt<brian@xxxxxxxxxxxxxxxxxxxxx> wrote: > > After adding BLK_SHA1 to my config.mak, git-imap-send started giving me link > errors: > > Undefined symbols: > "_ERR_get_error", referenced from: > _ssl_socket_perror in imap-send.o > "_ERR_error_string", referenced from: > _ssl_socket_perror in imap-send.o > > Some investigation led me to the fact that BLK_SHA1 removes LIB_4_CRYPTO from > EXTLIBS. That let me find the missing functions in libcrypto. At first I > considered making NEEDS_SSL_WITH_CRYPTO add -lcrypto to the SSL build flags > but decided to go this route in case there are platforms that need it one way > around and not the other. As imap-send is the only one which uses the symbols, why not just add LIB_4_CRYPTO to its linking command? Like in the broken GMail-patch below: diff --git a/Makefile b/Makefile index 02ff867..33971f3 100644 --- a/Makefile +++ b/Makefile @@ -1503,7 +1503,7 @@ git-%$X: %.o $(GITLIBS) git-imap-send$X: imap-send.o $(GITLIBS) $(QUIET_LINK)$(CC) $(ALL_CFLAGS) -o $@ $(ALL_LDFLAGS) $(filter %.o,$^) \ - $(LIBS) $(OPENSSL_LINK) $(OPENSSL_LIBSSL) + $(LIBS) $(OPENSSL_LINK) $(OPENSSL_LIBSSL) $(LIB_4_CRYPTO) http.o http-walker.o http-push.o: http.h -- To unsubscribe from this list: send the line "unsubscribe git" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html