[lldb-dev] [cfe-dev] [llvm-dev] [RFC] Deprecate email code reviews in favor of Phabricator

[Krzysztof Parzyszek via lldb-dev]

I'll defer to Christian the discussion about this section.

+Christian

-- 
Krzysztof Parzyszek  kparzysz at quicinc.com   AI tools development

-----Original Message-----
From: cfe-dev <cfe-dev-bounces at lists.llvm.org> On Behalf Of Kevin P. Neal via cfe-dev
Sent: Monday, May 3, 2021 1:57 PM
To: Krzysztof Parzyszek via llvm-dev <llvm-dev at lists.llvm.org>
Cc: clangd-dev at lists.llvm.org; openmp-dev at lists.llvm.org; lldb-dev at lists.llvm.org; cfe-dev at lists.llvm.org; libcxx-dev at lists.llvm.org; flang-dev at lists.llvm.org; parallel_libs-dev at lists.llvm.org
Subject: [EXT] Re: [cfe-dev] [llvm-dev] [RFC] Deprecate email code reviews in favor of Phabricator

On Mon, May 03, 2021 at 05:24:24PM +0000, Krzysztof Parzyszek via llvm-dev wrote:
>    This section presents a potential future evolution of the review
>    process.  Christian has conducted experiments suggesting that we can
>    replace the XXX-commits mailing lists with notifications directly from
>    Phabricator:

Wouldn't this make it more difficult for sites that archive the lists?
Right now it all works. If the lists were eliminated then it would be harder to archive. Not impossible, but it would be more work.

Plus, how long would it take for archive sites to switch over? How much history would only exist in Phab's database?

Couldn't the commit lists be made read-only except from Phab? That would force reviews to happen on Phab but otherwise keep all existing email setups working.
--
"A method for inducing cats to exercise consists of directing a beam of invisible light produced by a hand-held laser apparatus onto the floor ...
in the vicinity of the cat, then moving the laser ... in an irregular way fascinating to cats,..." -- US patent 5443036, "Method of exercising a cat"
_______________________________________________
cfe-dev mailing list
cfe-dev at lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-dev