Newsgroups: php.internals
Path: news.php.net
Xref: news.php.net php.internals:121072
Message-ID: <ac239e29-edb5-3976-4df0-4a92ec6ea4ad@bastelstu.be>
Date: Sat, 16 Sep 2023 00:22:01 +0200
MIME-Version: 1.0
Content-Language: en-US
To: internals@lists.php.net
References: <CAPVMwwtcLwq8AGEBYCXB1-HngS1R0D-xM5kTGjYqEdjmB7M=ag@mail.gmail.com>
 <CAPVMwwu9+RK13ByqFpexrQ4Sc+xARvyQb=WVxruhVtGKzxjo+A@mail.gmail.com>
 <8b9a1aae-5aec-91a8-5377-27f6f53581cd@php.net>
In-Reply-To: <8b9a1aae-5aec-91a8-5377-27f6f53581cd@php.net>
Content-Type: text/plain; charset=UTF-8; format=flowed
Content-Transfer-Encoding: 8bit
Subject: Re: [PHP-DEV] A new JIT engine for PHP-8.4/9
From: tim@bastelstu.be (=?UTF-8?Q?Tim_D=c3=bcsterhus?=)

Hi

On 9/15/23 17:50, Ben Ramsey wrote:
> Additionally, despite the use of a Git submodule complicating things for
> "everyone else," it provides a clear dependency and development
> boundary, avoiding situations where the php-src version of IR drifts
> from the upstream version. I think we can adjust tooling and messaging
> to help folks know how to use the submodule. :-)

Do not want: If the submodule repository goes away for whatever reason, 
the dependency will no longer be available. IR is currently sitting in 
Dmitry's personal GitHub account and it would not be the first time that 
a GitHub account is suspended for good or less good reasons [1].

Bundle IR in some dedicated directory with php-src. Then updating it is 
as easy as "throw away the directory and copy over the new files". We 
could even automate that using GitHub actions to sync in the changes 
every night or so.

Dmitry also said that he gave 'git subtree' a try. I don't know that 
command myself, but it looks exactly like what is required here. I would 
be interested in hearing why it didn't work well.

Best regards
Tim Düsterhus

[1] Such as sanctions, see: 
https://techcrunch.com/2019/07/29/github-ban-sanctioned-countries/