git/list[1] front-page[2] threads[3] people[4] search[5] about
 

Re: [PATCH v1 1/2] config.mak.in: set NO_OPENSSL and APPLE_COMMON_CRYPTO for macOS >10.11

From
Torsten Bögershausen <tboegi@web.de>
Date
Nov 9, 2016, 08:18 UTC
Message-ID
<11cc8bbd-2e67-f53a-c8f4-2244409fd6af@web.de>
In-Reply-To
<20161107172617.tlcrpwbjy2w7aoyc@sigill.intra.peff.net>
On 07.11.16 18:26, Jeff King wrote:
Show 21 quoted lines
> On Sun, Nov 06, 2016 at 08:35:04PM +0100, Lars Schneider wrote:
> 
>> Good point. I think I found an even easier way to achieve the same.
>> What do you think about the patch below?
>>
>> [...]
>>
>> diff --git a/Makefile b/Makefile
>> index 9d6c245..f53fcc9 100644
>> --- a/Makefile
>> +++ b/Makefile
>> @@ -1047,6 +1047,7 @@ ifeq ($(uname_S),Darwin)
>>  		endif
>>  	endif
>>  	ifndef NO_APPLE_COMMON_CRYPTO
>> +		NO_OPENSSL = YesPlease
>>  		APPLE_COMMON_CRYPTO = YesPlease
>>  		COMPAT_CFLAGS += -DAPPLE_COMMON_CRYPTO
>>  	endif
> 
> That is much simpler.

[] I don't know if that is a correct solution.

If I have Mac OS 10.12 and Mac Ports installed, I may want to use OPENSSL from Mac Ports. How about this:

diff --git a/Makefile b/Makefile
index ee89c06..e93511f 100644
--- a/Makefile
+++ b/Makefile
@@ -1038,17 +1038,22 @@ ifeq ($(uname_S),Darwin)
                ifeq ($(shell test -d /sw/lib && echo y),y)
                        BASIC_CFLAGS += -I/sw/include
                        BASIC_LDFLAGS += -L/sw/lib
+                       HAS_OPENSSL = Yes
                endif
        endif
        ifndef NO_DARWIN_PORTS
                ifeq ($(shell test -d /opt/local/lib && echo y),y)
                        BASIC_CFLAGS += -I/opt/local/include
                        BASIC_LDFLAGS += -L/opt/local/lib
+                       HAS_OPENSSL = Yes
                endif
        endif
        ifndef NO_APPLE_COMMON_CRYPTO
                APPLE_COMMON_CRYPTO = YesPlease
                COMPAT_CFLAGS += -DAPPLE_COMMON_CRYPTO
+               ifndef HAS_OPENSSL
+                       NO_OPENSSL = YesPlease
+               endif
        endif
        NO_REGEX = YesPlease
        PTHREAD_LIBS =
Previous: Paul SmithNext: Lars Schneider
Message 9 of 35 in “Fix default macOS build locally and on Travis CI”
  1. 0/2 Fix default macOS build locally and on Travis CIlarsxschneider@gmail.com, Oct 17, 2016
  2. 1/2 config.mak.in: set NO_OPENSSL and APPLE_COMMON_CRYPTO for macOS >10.11larsxschneider@gmail.com, Oct 17, 2016
  3. Jeff KingOct 17, 2016
  4. Lars SchneiderNov 6, 2016
  5. Jeff KingNov 7, 2016
  6. Paul SmithNov 7, 2016
  7. Jeff KingNov 7, 2016
  8. Paul SmithNov 7, 2016
  9. Torsten BögershausenNov 9, 2016
  10. Lars SchneiderNov 9, 2016
  11. Torsten BögershausenNov 9, 2016
  12. Makefile: set NO_OPENSSL on macOS by defaultlarsxschneider@gmail.com, Nov 10, 2016
  13. 2/2 travis-ci: disable GIT_TEST_HTTPD for macOSlarsxschneider@gmail.com, Oct 17, 2016
  14. Lars SchneiderNov 6, 2016
  15. Jeff KingNov 7, 2016
  16. Lars SchneiderNov 10, 2016
  17. Jeff KingNov 10, 2016
  18. Junio C HamanoNov 10, 2016
  19. Jeff KingNov 10, 2016
  20. Junio C HamanoNov 10, 2016
  21. Jeff KingNov 10, 2016
  22. Lars SchneiderNov 11, 2016
  23. Jeff KingNov 11, 2016
  24. Lars SchneiderNov 11, 2016
  25. Jeff KingNov 11, 2016
  26. Heiko VoigtNov 15, 2016
  27. Lars SchneiderNov 15, 2016
  28. Jeff KingNov 15, 2016
  29. Heiko VoigtNov 16, 2016
  30. Torsten BögershausenNov 16, 2016
  31. Junio C HamanoNov 9, 2016
  32. Torsten BögershausenNov 10, 2016
  33. Lars SchneiderNov 10, 2016
  34. Junio C HamanoNov 10, 2016
  35. Lars SchneiderNov 11, 2016

Read the whole thread, see it on lore, or plain text.

$ cat FOOTERMessages come from the public archive at lore.kernel.org/git, fetched every hour. The front page is chosen and written each morning by an AI editor and can be wrong; the threads themselves are the record. About and API. For agents: an MCP server at https://gitlist.dev/mcp, and any thread, story or person page as Markdown by adding .md to its URL (or sending Accept: text/markdown). Details in /llms.txt.