[PATCH] Add a source_tree field to Project.

[Guilherme Salgado]

On Wed, 2011-03-30 at 18:25 -0300, Guilherme Salgado wrote:
> On Wed, 2011-03-30 at 12:10 +0800, Jeremy Kerr wrote:
> > Hi Guilherme,
> > 
> > >  apps/patchwork/models.py                      |    1 +
> > >  lib/sql/migration/008-project-source-tree.sql |    3 +++
> > >  2 files changed, 4 insertions(+), 0 deletions(-)
> > >  create mode 100644 lib/sql/migration/008-project-source-tree.sql
> > 
> > Looks good. I'd like to wait until there is a user of this field before 
> > merging the change though - or are you using this for linaro-internal things?
> 
> I'm using it on a script I'm writing to fetch the git history of every
> project and scan that looking for patches that have been committed. Just
> like the existing patchwork-update-commits script does but this one is
> fully automated, and to make it more easily testable I'm experimenting
> with python-dulwich to scan the git history. 

So, although I could do most things using python-dulwich it was failing
in some cases and I just don't have the time to chase down these
failures and fix them, so I resorted to running git on a subprocess. I'm
still writing the script in python as that allows me to have at least
some test coverage, which is something very valuable to me.

This new version of the script works similarly to the previous version.
There are a few things that need to be changed but it has some tests and
is able to scan the qemu tree and update the status of a couple patches.
Also note that it updates the commit_ref of a patch, which will allow us
to generate a link to the project's gitweb once we have the commit_url
field.

I'd appreciate some feedback on this; specially whether or not this is
something that's going to be useful upstream and, if so, if the current
approach is reasonable.

Cheers,

-- 
Guilherme Salgado <https://launchpad.net/~salgado>
-------------- next part --------------
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 198 bytes
Desc: This is a digitally signed message part
URL: <http://lists.ozlabs.org/pipermail/patchwork/attachments/20110331/e523df43/attachment.pgp>