posts/debbugs_forcemerge.mdwn

   1 [[!meta title="Debbugs forcemerge support"]]
   2
   3
   4 One of the earliest features I wrote for the Debian bug tracking
   5 system (Debbugs) after joining the team was support for forcibly
   6 merging bugs. Originally, merging two bugs required that the bugs be
   7 in exactly the same state before merging them; `forcemerge` removed
   8 this requirement.
   9
  10 Unfortunately, the way I originally implemented this was shortsighted,
  11 and merely forced the merge partners to have the same values as the
  12 merge master. This meant that owners, blocking bugs, and many other
  13 things were silently changed, which meant that people weren't notified
  14 of changes, and bugs could end up in an inconsistent state.
  15
  16 A while ago, I decided to fix this by calculating the changes required
  17 to actually merge the bugs, making those changes, and then merging the
  18 bugs normally; thus, doing everything that a maintainer would normally
  19 have done for them. This necessitated abstracting out the entire
  20 control apparatus into the `Debbugs::Control` module.
  21
  22 Now that it's complete, you can do the following:
  23
  24     > forcemerge 1 2
  25     Bug #1 [foo] new title
  26     Bug #2 {Done: foo@bugs.something} [foo] foo
  27     Unset bug forwarded-to-address
  28     Severity set to 'wishlist' from 'grave'
  29     3 was blocked by: 2
  30     3 was not blocking any bugs.
  31     Removed blocking bug(s) of 3: 2
  32     2 was blocked by: 4
  33     2 was not blocking any bugs.
  34     Removed blocking bug(s) of 2: 4
  35     Bug reopened
  36     Removed annotation that bug was owned by bar@baz.com.
  37
  38     Removed indication that 2 affects bleargh
  39     Removed tag(s) unreproducible and moreinfo.
  40     Merged 1 2
  41     > thanks
  42     Stopping processing here.
  43
  44 and bug 2 now is merged with 1 and matches the state of 1.
  45 [The above is the control output from the appropriate bit of the `06_mail_handling.t` test.]
  46
  47 This change also means that I'll be able to finally write support for
  48 control@ operations at submit@ time. Also, all of the bug
  49 modifications that happen at submit@ or nnn@ time (setting title,
  50 found, etc.) will be implemented as calls to Debbugs::Control so we
  51 can eventually keep a postgresql database updated in addition to the
  52 flatfile database.
  53
  54 [[!tag debian tech debbugs]]