The State of PQM

[Daniel Watkins]

(This has also been posted on my blog which is prettier and has links,
 at http://blog.daniel-watkins.co.uk/state_of_pqm.html.  I'm just
 emailing it so that people are aware it exists, and so they don't
 have to go to my blog if they don't so wish.)

So, I thought I would take some time to break down what I've been
working on. Some of this has been posted to the list, so I'll skim over
that, but some of it is so dependent on the work that's awaiting review
that there wasn't much point in getting it reviewed separately. I'll
try to cover that in slightly more detail.

Patches up for review
---------------------

These patches are currently in BundleBuggy, waiting for review or
merging:

  * Already approved: Reporting details of successful merges, fixing
backtrace when checking signed emails
  * Trivial patches: Fixing test stipple, making tests pass, web UI
    fixes for legacy commands, fix email failures due to a lack of merge
    base
  * Merge directive support: This is largely a reworking of Aaron's
    previous work on this. It's functional, and reasonably well-tested,
    but I'm not entirely convinced that it fits the abstraction
    correctly. I'm not sure that this is the case, but it certainly
    needs thought.
  * Reporting details of successful merges: This is just a cleanup of
    Tom Haddon's patch on Bug #55717, and is reasonably trivial.
  * Reporting of errors caused by unknown branch formats: This is,
    again, a pretty trivial patch. It fixes Bug #246660, which a couple
    of people have hit recently.
  * Switching the web UI to SimpleTAL: This makes the Web UI code much
    nicer and easier to modify. It also lays the foundations for further
    work on the web UI, including RSS feeds. It has received several
    rounds of review, so should be pretty much there.
  * Removed Arch-specific commands: As Robert and I discussed in
    London, some existing commands shouldn't be needed by any Arch
    implementation (and certainly aren't required by bzr). This patch
    removes those commands.
  * 'nodebug' command: Adds a nodebug command, so that only specific
    parts of a given script have to have debug enabled.
  * TestCaseWithQueue: Converts the tests to use a TestCaseWithQueue,
    which abstracts away queue creation. This will allow for testing of
    several different scenarios, which is especially helpful in testing
    the UI.


Patches in the wings
--------------------

These patches are waiting on patches in the above list to be merged:

  * UI Unicode fixes: Depends on the SimpleTAL changes. Fixes encoding
    issues with displaying Unicode names in the web UI, fixing Bug
    #125888.
  * New UI tests: New UI tests using the TestCaseWithQueue changes.
    This improves the test coverage of the UI by a considerable amount.
  * RSS feeds: This adds RSS feeds for each PQM project, containing the
    same information as is given in the Web UI, fixing Bug #133229.
  * PatchCommand: This moves the logic for accepting patches from
    CommandRunner to a separate PatchCommand. This is the first step
    towards removing CommandRunner entirely.


Patches in progress
-------------------

These patches are currently in development:

  * XMLRPC: This is a loom containing two threads of work:
    * 'xmlrpc': The basic XMLRPC functionality. This includes some
      email-message-related abstractions, so that both the email and
      XMLRPC interfaces could use the same functions. This fixes Bug
      #246845.
    * 'xmlrpc-validation': Validation of incoming XMLRPC requests. This
      takes each command in the incoming script and checks that branches
      it references are valid and that the submitter has commit access
      to them. It goes some of the way towards fixing Bug #110433.
  * TagCommand: This branch adds a TagCommand, intended to replace the
    functionality in CommandRunner. The CommandRunner code is not
    removed, and there are some issues with 'tag' meaning different
    things in Arch and bzr.

-- 
Daniel Watkins (Odd_Bloke)
-------------- next part --------------
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 197 bytes
Desc: not available
Url : https://lists.ubuntu.com/archives/bazaar/attachments/20080801/2ed1794f/attachment.pgp