Page MenuHomePhabricator

Correct a PHP8 compatibility issue when running "arc diff" with no active branch
ClosedPublic

Authored by epriestley on Feb 4 2023, 1:03 PM.
Tags
None
Referenced Files
F13994902: D21856.id52093.diff
Wed, Oct 23, 9:21 AM
F13994740: D21856.id52093.diff
Wed, Oct 23, 8:11 AM
F13994637: D21856.id52093.diff
Wed, Oct 23, 7:29 AM
F13993270: D21856.id52092.diff
Tue, Oct 22, 9:46 PM
F13986578: D21856.id52092.diff
Mon, Oct 21, 3:58 AM
F13986525: D21856.id52092.diff
Mon, Oct 21, 3:40 AM
F13984541: D21856.id52092.diff
Sun, Oct 20, 2:05 PM
Unknown Object (File)
Wed, Oct 9, 2:43 PM
Subscribers
None

Details

Summary

Ref T13588. See PHI2228. When there is no active branch name (for example, because you are in Git on a detached HEAD), this section currently fails under PHP8 when we try to strlen(null).

Test Plan

Under PHP 8.1: ran git checkout <hash of head>, then arc diff to generate this revision.

Diff Detail

Repository
rARC Arcanist
Lint
Lint Not Applicable
Unit
Tests Not Applicable

Event Timeline

epriestley created this revision.
This revision was not accepted when it landed; it landed in state Needs Review.Feb 4 2023, 1:16 PM
This revision was automatically updated to reflect the committed changes.