This is the mail archive of the gcc@gcc.gnu.org mailing list for the GCC project.


Index Nav: [Date Index] [Subject Index] [Author Index] [Thread Index]
Message Nav: [Date Prev] [Date Next] [Thread Prev] [Thread Next]
Other format: [Raw text]

Re: [GCC Steering Committee attention] [PING] [PING] [PING] libgomp: In OpenACC testing, cycle though $offload_targets, and by default only build for the offload target that we're actually going to test


On 08/05/2016 10:27 AM, Ian Lance Taylor wrote:
I believe that Diego tried setting up an alternative patch review
system using Reitveld, but it did not catch on.
And there were some before that :-)


For Go development I have been using Gerrit, an instance hosted at
Google (https://go-review.googlesource.com/).  It's just a much better
way of doing patch review.  You can do line by line comments, you can
easily see the set of all patches and their state, you can assign the
patch review to specific people, you can easily commit the patch.  For
Go we have trybots, which can be used to build the toolchain with the
patch and run all the tests on a range of different architectures and
operating systems.

Gerrit is free software under the Apache license
(https://www.gerritcodereview.com/).
I've heard a lot of good things about Gerrit and we use it internally within Red Hat for other things (well outside the compiler tools space).


It's not perfect: in particular, there is no offline code review.  But
overall it's just much much better than what we do today in GCC.

But the active maintainers are accustomed to the current system, and
it would be very hard to change.
Yup. That inertia is one of the hardest things to change. I'd be willing to try though.

Jeff


Index Nav: [Date Index] [Subject Index] [Author Index] [Thread Index]
Message Nav: [Date Prev] [Date Next] [Thread Prev] [Thread Next]