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

Authored by epriestley on Feb 4 2023, 1:03 PM.

Tags

None

Referenced Files

	F12195566: D21856.diff
	Tue, Sep 26, 12:54 AM

	F12193932: D21856.diff
	Sun, Sep 24, 7:06 AM

	F12192729: D21856.diff
	Sat, Sep 23, 2:59 AM

	F12177240: D21856.diff
	Sun, Sep 10, 8:15 PM

	F12162415: D21856.diff
	Wed, Sep 6, 8:14 PM

	F12153886: D21856.diff
	Mon, Sep 4, 3:41 AM

	F12134658: D21856.id52093.diff
	Aug 25 2023, 4:54 AM

	F12134657: D21856.id52092.diff
	Aug 25 2023, 4:54 AM

View All 77 Files

Subscribers

None

Details

Reviewers: None
Maniphest Tasks: T13588: PHP 8 Compatibility
Commits: rARCc39ab20eb371: Correct a PHP8 compatibility issue when running "arc diff" with no active branch

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 requested review of this revision.Feb 4 2023, 1:03 PM

epriestley created this revision.

Harbormaster completed remote builds in B25765: Diff 52092.Feb 4 2023, 1:03 PM

This revision was not accepted when it landed; it landed in state Needs Review.Feb 4 2023, 1:16 PM

Closed by commit rARCc39ab20eb371: Correct a PHP8 compatibility issue when running "arc diff" with no active branch (authored by epriestley). · Explain Why

This revision was automatically updated to reflect the committed changes.

epriestley added a commit: rARCc39ab20eb371: Correct a PHP8 compatibility issue when running "arc diff" with no active branch.

Revision Contents
Changeset List

Path

Size

src/

workflow/

ArcanistDiffWorkflow.php

2 lines

Diff 52093

View Options

src/workflow/ArcanistDiffWorkflow.php

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