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
F3367501: D4662.id15110.diff
Mon, Nov 25, 2:57 PM
Unknown Object (File)
Fri, Nov 22, 7:12 AM
Unknown Object (File)
Fri, Nov 22, 1:34 AM
Unknown Object (File)
Sun, Nov 10, 10:24 AM
Unknown Object (File)
Mon, Oct 28, 10:03 PM
Unknown Object (File)
Sun, Oct 27, 4:23 AM
Unknown Object (File)
Oct 25 2024, 2:50 PM
Unknown Object (File)
Oct 15 2024, 8:42 AM

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
Branch
update-nix-flakes (branched from master)
Lint
No Lint Coverage
Unit
No Test Coverage