[Reviewers] Recommending Phabricator

Augie Fackler raf at durin42.com
Sat Feb 10 23:03:23 UTC 2018


> On Feb 9, 2018, at 12:53 PM, Ryan McElroy <rm at fb.com> wrote:
> 
> Long ago (2012?), LLVM got a patch into the upstream for this but we wised up over the years and I removed it in D15850. I believe no other install ever used it -- at the least, no one objected when it stopped working, and installs stopped complaining about the feature being awful and ugly and started complimenting the change. I think LLVM still maintains some kind of patch locally, and see here for guesses about how to build such a patch. I can write this patch for you, but usual mana/unmainted caveats apply.

That’s interesting. I’ll poke LLVM folks that I work with on Monday and see if they can point to any useful customizations they’ve done on their phabricator.

I still find Evan’s opinion baffling, but I can at least respect that he’s got a vision and he’s sticking with it.


More information about the Reviewers mailing list