Page MenuHomePhorge

arnold (Arnold Noronha)
Screenshotbot.io

Projects

Today

  • Clear sailing ahead.

Tomorrow

  • Clear sailing ahead.

Thursday

  • Clear sailing ahead.

User Details

User Since
Fri, May 12, 14:41 (3 w, 3 d)
Availability
Available

Recent Activity

Wed, May 17

arnold closed T15400: Unable to arc-diff against this install as Resolved.

Issue on my end, will try to get the Mysql credentials set up for running tests

Wed, May 17, 15:19 · phorge.it install
arnold added a comment to T15400: Unable to arc-diff against this install.

Ah! That makes sense!

Wed, May 17, 15:18 · phorge.it install
arnold added a comment to T15400: Unable to arc-diff against this install.

(Unless there's some PHP 8.2 bug in my arc invocation... )

Wed, May 17, 15:07 · phorge.it install
arnold added a comment to T15400: Unable to arc-diff against this install.

Pretty sure, here's the full --trace output, you can see it's trying to talk to we.phorge.it (also, I see we.phorge.it in my ~/.arcrc)

Wed, May 17, 15:04 · phorge.it install

Mon, May 15

arnold created T15400: Unable to arc-diff against this install.
Mon, May 15, 19:47 · phorge.it install
arnold created T15399: strlen() null issues when arc-diffing with PHP 8.2.
Mon, May 15, 19:36 · PHP 8 support

Fri, May 12

arnold accepted D25220: PhabricatorModularTransactionType: fix regression.

lgtm

Fri, May 12, 17:29
arnold added a comment to T15385: Fix Almanac page /almanac/interface/edit/1/.

@valerio.bozzolan I can verify that the proposed patch works for me

Fri, May 12, 16:47 · Bug Reports, PHP 8 support
arnold added inline comments to D25202: Fix PHP 8.1 "strlen(null)" exception which blocks rendering error pages for Packages' Create Package and Fund's Create Initiative.
Fri, May 12, 16:43
arnold added a comment to D25202: Fix PHP 8.1 "strlen(null)" exception which blocks rendering error pages for Packages' Create Package and Fund's Create Initiative.

When editing almanac hosts, I get this: (url /almanac/interface/edit/1/)

Fri, May 12, 15:39
arnold added a comment to T15064: Make Phorge compatible with PHP 8.1/8.2.

This is a good example of why fixing each case individually might be a bad idea, this fix was from D25202. It appears the old strlen allowed for integers too...

Fri, May 12, 15:38 · PHP 8 support
arnold added a comment to T15064: Make Phorge compatible with PHP 8.1/8.2.

@valerio.bozzolan Here's a small sampling of all the errors that I encountered (I fixed each of them and got to the next one): https://we.phorge.it/P10

Fri, May 12, 15:10 · PHP 8 support
arnold created P10 (An Untitled Masterwork).
Fri, May 12, 15:09
arnold added a comment to T15064: Make Phorge compatible with PHP 8.1/8.2.

@valerio.bozzolan You know, I went on a spree fixing this in my local checkout. But it's super spread out all throughout the repo. The current master is broken even when I just view a revision. Then there are issues that show up in the SSH auth code when trying to push where the stack trace gets hidden (and it'll say something about ENORMOUS CHANGES, but that's a lie)

Fri, May 12, 15:04 · PHP 8 support
arnold updated arnold.
Fri, May 12, 14:56
arnold added a comment to T15064: Make Phorge compatible with PHP 8.1/8.2.

Re: all the null errors.

Fri, May 12, 14:50 · PHP 8 support