[haiku-commits] Re: haiku: hrev53104 - src/apps/debugger/user_interface/gui/team_window src/system/kernel/locks headers/private/kernel src/servers/debug

  • From: "Alexander von Gluck IV" <kallisti5@xxxxxxxxxxx>
  • To: haiku-commits@xxxxxxxxxxxxx
  • Date: Thu, 02 May 2019 21:34:31 +0000

May 2, 2019 4:19 PM, "Rene Gollent" <anevilyak@xxxxxxxxx> wrote:

On Thu, May 2, 2019 at 5:05 PM waddlesplash <waddlesplash@xxxxxxxxx> wrote:

5ef0649658e5: Debugger: Add a "Write core file" menu item alongside "Save 
debug report".

For the millionth time, please stop bypassing Gerrit and assuming
everybody is keeping 100% tabs on every ticket, chat channel and forum
discussion. This option was put in the crash dialog quite
deliberately, so it's easy for an end user to submit a core file in
the case where a crash report alone doesn't supply enough information.
Now, this is no longer possible and one must walk the user through the
extra steps needed to get to the appropriate menu item in the Debugger
instead every time, all so an (ideally) rarely seen dialog box can be
slightly smaller? This is completely backwards logic. Please revert.

And again. Use. Gerrit.

Up until now, as a project, we haven't decided to force the usage of Gerrit
for all commits. As the discussion last stood, we are working under the
best practice of doing gerrit code review for large commits / substantial
commits. (even this is squishy at the moment, however since the rules
were never laid in stone)

In the spirit of how things have been coordinated lately, I'd recommend
starting a public discussion/vote on the topic if there is real consensus
that code review should be mandatory going forward.

-- Alex

Other related posts: