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

	F15458681: D21856.diff
	Mon, Mar 31, 6:22 AM

	F15450727: D21856.id52092.diff
	Fri, Mar 28, 5:21 PM

	F15450723: D21856.id.diff
	Fri, Mar 28, 5:20 PM

	F15439070: D21856.diff
	Wed, Mar 26, 5:32 AM

	F15421183: D21856.diff
	Fri, Mar 21, 9:35 PM

	F15391160: D21856.id52093.diff
	Sat, Mar 15, 8:29 AM

	F15391158: D21856.id52092.diff
	Sat, Mar 15, 8:28 AM

	F15391157: D21856.id.diff
	Sat, Mar 15, 8:28 AM

View All 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