[Tarantool-patches] [PATCH luajit 3/3] macOS: Remove obsolete -single_module flag.

Sergey Kaplun skaplun at tarantool.org
Thu Nov 27 12:44:34 MSK 2025


Hi, Sergey!
Thanks for the review!
See my clarification below.

On 26.11.25, Sergey Bronnikov wrote:
> Hi, Sergey,
> 
> thanks for the patch! LGTM with two minor comments.
> 
> Sergey
> 
> On 11/24/25 23:26, Sergey Kaplun wrote:
> > From: Mike Pall <mike>
> >
> > Thanks to dundargoc.
> >
> > (cherry picked from commit 97813fb924edf822455f91a5fbbdfdb349e5984f)
> >
> > The aforementioned flag is the default since Xcode 2.2 [1].
> 
> You refer to the description in the previous commit, right?
> 
> I would name a flag here instead

It is mentioned in the commit message header, so I refer to the commit
message header, not the previous commit. Hence, I don't see the reason
to mention it again.

> 
> > This patch removes this flag to avoid warnings for modern Xcode
> > versions at the linking stage.
> it is worth to mention our own modifications for CMake build system

I suppose this is excess since this is the details of the
implemenatation.

> >

<snipped>

-- 
Best regards,
Sergey Kaplun


More information about the Tarantool-patches mailing list