[Sugar-devel] Bug tracking Vs Patch review

David Farning dfarning at gmail.com
Mon Jul 26 11:27:02 EDT 2010


Over the past couple of months there have been a number of thread
regarding patch management.  In reviewing the threads in preparation
for herding a number of patches through the process, I notice that
there seem to be two distinct workflows that are getting interchanged.

Tracking Bugs vs Reviewing Patches

Processing a patch via the bug tracker seems inappropriate.  There are
two distinct differences between bugs and patches.
1.  Purpose.
  A. Bug is a problem waiting for a solution.
  B. Patch implements a change in code.  (It is a solution to a problem)
2. Owner.
  A. Bug is owned by the maintainer -- It is generally accepted to be
the module maintainers responibility to track bugs.
  B. Patch is owned by the submitter -- It is generally accepted to be
the patch submitters Responsibility to herd the patch up the to point
it as been ACKed by the maintainer.

The purpose of a patch review is to ensure that each patch is reviewed
for quality and that each patch implements a change which is desirable
to have in the mainline.  It is not a discussion about the best
possible solution to a problem.  Most of the discussion to date has
been centered around two points (a) syncing the bug tracker to the
patch review process and (b) loosing patches.  If a patch is a
solution to a problem it is not necessary to discuss the problem and
if it is the patch submitters responsibility to track the patch the
need for a maintainer to track the patch goes away.

david


More information about the Sugar-devel mailing list