Page MenuHomePhabricator

[Nix] Re-package arcanist as just the php scripts
ClosedPublic

Authored by jon on Jul 27 2022, 6:47 PM.
Tags
None
Referenced Files
Unknown Object (File)
Mon, Dec 16, 12:18 AM
Unknown Object (File)
Mon, Dec 16, 12:18 AM
Unknown Object (File)
Mon, Dec 16, 12:18 AM
Unknown Object (File)
Mon, Dec 16, 12:17 AM
Unknown Object (File)
Sun, Dec 15, 4:00 PM
Unknown Object (File)
Sun, Dec 15, 4:00 PM
Unknown Object (File)
Sun, Dec 15, 3:58 PM
Unknown Object (File)
Sun, Dec 15, 3:31 PM

Details

Summary

Nixpkgs arcanist pacakge includes "phage" shell which
is a fairly large c++ and php code base. However,
we only really care about using the arc command, so we should reduce
the scope of the arcanist package to be what we actually use.

This avoids situations such as https://phab.comm.dev/D4483#131136
where the build for phage (and it's requirements), aren't thread safe. They
use bison and flex, and the generated files aren't delcared in a way in which
make will be aware that generation of those files need to be ordered before
usage of them.

Test Plan
# this can be used from any branch
nix develop github:commE2E/comm?ref=update-nix-flakes
# use any arc command, 8.1 would fail on `arc diff`

Diff Detail

Repository
rCOMM Comm
Lint
Lint Not Applicable
Unit
Tests Not Applicable