Welcome! Log In Create A New Profile

Advanced

Re: [PATCH] Configure: add support for Homebrew on Apple Silicon

Sergey Kandaurov
March 27, 2024 02:00PM
> On 11 Mar 2024, at 19:14, Sergey Kandaurov <pluknet@nginx.com> wrote:
>
>
>> On 8 Mar 2024, at 19:31, Piotr Sikora via nginx-devel <nginx-devel@nginx.org> wrote:
>>
>> Hi Sergey,
>>
>>> An obvious question is why do you need this change. Homebrew seems
>>> to be quite niche to pay attention.
>>
>> Homebrew [1] is orders of magnitude more popular than MacPorts [2],
>> which is already supported by the configure script.
>
> Thanks for clarifying this.
>
>>
>>> Using appropriate paths in
>>> --with-cc-opt / --with-ld-opt should work (not tested).
>>
>> Everything under auto/lib can be replaced with --with-{cc,ld}-opt,
>> so I don't really understand this reasoning.
>
> Given the high popularity, it may be indeed not an option.
>
>>
>>> A quick grep for MacPorts search paths suggests that some libraries
>>> are missing in the change. If this is on purpose, please reflect
>>> this in the description.
>>
>> libxml2, libxslt, and libexslt are all installed as part of Xcode,
>> which is required to use Homebrew and compile anything on macOS.
>
> This raises a question whether we need to test it for MacPorts as well,
> which also requires SDK. Obviously, it is out of scope of this patch.
>
>>
>> I'll ship update patch in a moment.
>>
>>> Apple Silicon is something from the marketing language,
>>> using Apple ARM instead should be fine.
>>>
>>> Notably, Homebrew uses Hardware::CPU.arm Ruby language boolean
>>> to make the distinction.
>>
>> There is no such thing as "Apple ARM".
>>
>> The official documentation uses the term "Apple silicon" [3],
>> Homebrew refers to the supported platform as "Apple Silicon" [4],
>> and Wikipedia has an article about "Apple silicon" [5].
>
> Thank you for clarification.
>
>>
>>> Further, given the smooth decay on Intel-based hardware,
>>> I'd reduce this just to "Homebrew".
>>
>> But that would be misleading, seeing that the new code path doesn't do
>> anything for Homebrew on Intel.
>>
>> And then, there is Homebrew on Linux [6].
>
> Given all the above argumentation, it makes sense
> to push the patch just as is.

Pushed, thanks.

--
Sergey Kandaurov
_______________________________________________
nginx-devel mailing list
nginx-devel@nginx.org
https://mailman.nginx.org/mailman/listinfo/nginx-devel
Subject Author Views Posted

[PATCH] Configure: add support for Homebrew on Apple Silicon

Piotr Sikora via nginx-devel 287 February 27, 2024 08:26PM

Re: [PATCH] Configure: add support for Homebrew on Apple Silicon

Roman Arutyunyan 15 March 06, 2024 07:58AM

Re: [PATCH] Configure: add support for Homebrew on Apple Silicon

Sergey Kandaurov 15 March 06, 2024 10:38AM

Re: [PATCH] Configure: add support for Homebrew on Apple Silicon

J Carter 25 March 07, 2024 09:24AM

Re: [PATCH] Configure: add support for Homebrew on Apple Silicon

Piotr Sikora via nginx-devel 20 March 08, 2024 10:32AM

Re: [PATCH] Configure: add support for Homebrew on Apple Silicon

Sergey Kandaurov 25 March 11, 2024 11:16AM

Re: [PATCH] Configure: add support for Homebrew on Apple Silicon

Sergey Kandaurov 24 March 27, 2024 02:00PM



Sorry, you do not have permission to post/reply in this forum.

Online Users

Guests: 302
Record Number of Users: 8 on April 13, 2023
Record Number of Guests: 421 on December 02, 2018
Powered by nginx      Powered by FreeBSD      PHP Powered      Powered by MariaDB      ipv6 ready