Page MenuHomePhorge

T15064: Several arcanist PHP 8.1 compat issues on Windows
ClosedPublic

Authored by speck on Feb 6 2024, 00:37.

Details

Summary

Ran into strlen/strpos issues while using Arcanist on Windows:

  • Running arc version when not in a project/repository directory blew up (may not be specific to Windows).
  • Determining mime type of binary file in repository fails.

Refs T15064

Test Plan

Ran diff including a binary file. Ran arc version when not in a project/repository directory.

Diff Detail

Repository
rARC Arcanist
Lint
Lint Not Applicable
Unit
Tests Not Applicable

Event Timeline

speck requested review of this revision.Feb 6 2024, 00:37

I wonder how many Windows users we have. The xhpast linter does not work, and the unit test failures listed here are pre-existing and occur without these changes.

avivey subscribed.

To be fair, it's possible that most of the arc features work for Windows users that don't use PHP. Or at least, arc diff might work for some setup of lint/unit configuration.

This revision is now accepted and ready to land.Feb 6 2024, 09:23

Both of these failures are edge cases so it does seem likely we wouldn't have found them as quickly. I do still wonder how many Windows users we have.

This revision was landed with ongoing or failed builds.Feb 6 2024, 11:56
This revision was automatically updated to reflect the committed changes.