[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