[Sugar-devel] Patchwork (was: Re: Deprecating class and service_name in activity.info file)

[Sascha Silbe]

Excerpts from Simon Schampijer's message of Wed Sep 07 08:29:59 +0200 2011:

> PS: Who is maintaining patchwork? What keywords does it look for to 
> close tickets? Is a cron-job handling that? Works rather badly for me.

Nobody is actively maintaining Patchwork. Marco offered to do it [1,2],
but didn't follow up. I've been working on it from time to time to fix
the most important issues, but already have too many other things to
work on.

git.sl.o has a custom module [3] (developed by me) to update patches in
Patchwork whenever it can find a match. It's based on existing Patchwork
functionality, so the diff portion must match exactly: changing the
commit message is fine, but if you change anything in the code it won't
update the patch in Patchwork (since there's no exact match).

Each package that has its own repository needs to be explicitly added to
the configuration file (besides other reasons because we don't want
commits to personal clones to remove the patch from the Patchwork
queue). Similarly, the subject prefix must mention the name of the
repository (again to prevent us from updating the wrong patch for
projects containing similar sources).

If patches that a) match exactly, b) are committed to a configured
repository (e.g. sugar, sugar-toolkit) and c) mentioned the module name in
the subject prefix don't get updated in Patchwork, please file a bug on
bugs.sl.o, mentioning a) the commit id in the git repository and b) the
patch id on Patchwork so that someone (maybe me) can investigate.

Sascha

[1] mid:3367C086-5C5C-4E87-9709-3023025EF4AC at marcopg.org
[2] http://lists.sugarlabs.org/archive/sugar-devel/2010-September/026466.html
[3] http://gitorious.org/~alsroot/gitorious/sugarlabs-org/blobs/master/lib/patchwork.rb
-- 
http://sascha.silbe.org/
http://www.infra-silbe.de/
-------------- next part --------------
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 490 bytes
Desc: not available
URL: <http://lists.sugarlabs.org/archive/sugar-devel/attachments/20110907/ff9d341e/attachment.pgp>