Premising that very probably a fix there would not 100% resolve that upstream issue but surely this deserves a fix :D

I invested so much energies in this patch in trying some facets, and I think I really cannot verify more stuff without becoming completely insane.

By the way the only usages are:

@Matthew please remove the red flag from this revision :) Thank you so much

Hi @Dylsss, feel free to land :) A reviewer approved this patch!

Hi @speck, feel free to land :)

I tested both values locally with phlog() and using some forms as the test plan suggests.

Thanks again

FWIW I think the test plan is configurable (that requirement can be disabled in phabricator's config server side which should remove it from commit messages in arc)

One other thing we’ll need to address is the current code sharing between arcanist and phorge. I’m not sure the full scope of what Phorge relies on from arcanist but I think there’sa fair amount. We can probably get an idea by looking at the libphutil repo which was the shared library between the two which was eventually merged into arcanist

Yeah no problem!

In T15368#8316, @valerio.bozzolan wrote:

Hi @aklapper. Are you affected by this as well?

@valerio.bozzolan right, I forgot the important bit, sorry :D

Note that probably @ton you have resolved, but anybody else nope :D The patch D25205 still needs review and land

Since this question has an answer and a related Task and a related Patch, I mark this as closed :)

@amit Can you please try again with PHP 8.1 enabled, check whenever this patch fixes your problem? (Or, at least, it unlocks another problem)

Works fine after downgrading to PHP 7.4

Oh no.......... I think some auto-update has updated the PHP version to 8.2

The bad news is that you are using PHP 8.1 or above that is not supported officially since never

I see this strlen() error keeps repeating, and it is flooding the daemon.log

Ah! I was looking in php error log

• update celerity map again

Premising that it is very strange that the daemon log is not telling anything in your /var/tmp/phd/log/daemons.log about this thing

Kind of

Can I ask your current commit hash of Phorge?

Sorry for not giving more info

Thanks! uhm...... still unclear how to help. Feel free to clarify these:

Uhm, remove now duplicated package name. Heh.

I close this answer since probably now we have an answer that explains the current situation. But, feel free to also file a specific Task under Conpherence (also feel free to put myself in Subscribers since I'm interested in that)

Can you please share more details in the question? Is that a new installation? have you seen the logs? are your daemons running? Thanks :)

Thanks for this patch

Thanks for this patch (again)