Bug 30084 - webkit-patch commit-queue does not notice modifications to committers.py
Summary: webkit-patch commit-queue does not notice modifications to committers.py
Status: RESOLVED FIXED
Alias: None
Product: WebKit
Classification: Unclassified
Component: New Bugs (show other bugs)
Version: 528+ (Nightly build)
Hardware: PC OS X 10.5
: P2 Normal
Assignee: Eric Seidel (no email)
URL:
Keywords:
: 30869 (view as bug list)
Depends on: 30634 32911
Blocks:
  Show dependency treegraph
 
Reported: 2009-10-05 11:17 PDT by Eric Seidel (no email)
Modified: 2010-05-10 15:40 PDT (History)
2 users (show)

See Also:


Attachments
Add re-exec support (9.27 KB, patch)
2009-10-21 11:22 PDT, Eric Seidel (no email)
no flags Details | Formatted Diff | Diff
Add re-exec support (9.26 KB, patch)
2009-10-21 11:40 PDT, Eric Seidel (no email)
no flags Details | Formatted Diff | Diff

Note You need to log in before you can comment on or make changes to this bug.
Description Eric Seidel (no email) 2009-10-05 11:17:00 PDT
bugzilla-tool commit-queue does not notice modifications to committers.py

I'm not sure if it notices them after an update-webkit call or not.  I think it might be requiring a restart.  Either way, we should make it so that no restart, or patch landing is required.  Once someone adds themselves to committers.py, setting commit-queue+ should "just work" for them.
Comment 1 Eric Seidel (no email) 2009-10-21 11:22:55 PDT
Created attachment 41587 [details]
Add re-exec support
Comment 2 Eric Seidel (no email) 2009-10-21 11:40:55 PDT
Created attachment 41595 [details]
Add re-exec support
Comment 3 Eric Seidel (no email) 2009-10-21 11:49:00 PDT
Comment on attachment 41595 [details]
Add re-exec support

Removing cq? until bug 30634 is landed.
Comment 4 Adam Barth 2009-10-22 00:01:55 PDT
Comment on attachment 41595 [details]
Add re-exec support

You're blowing my mind with your crazy continuation style here.
Comment 5 Eric Seidel (no email) 2009-10-26 15:34:26 PDT
Committed r50105: <http://trac.webkit.org/changeset/50105>
Comment 6 Eric Seidel (no email) 2009-10-30 15:41:35 PDT
Rolled out in:
Committed r50362: <http://trac.webkit.org/changeset/50362>

This was causing https://bugs.webkit.org/show_bug.cgi?id=30869
Comment 7 Eric Seidel (no email) 2009-10-30 15:43:24 PDT
*** Bug 30869 has been marked as a duplicate of this bug. ***
Comment 8 Eric Seidel (no email) 2009-11-02 18:15:00 PST
I think the best plan for now is to add some explicit code to reload just the committers.py module on every run.
Comment 9 Eric Seidel (no email) 2009-12-23 15:51:46 PST
Using bug 32911 for a workaround for now.
Comment 10 Eric Seidel (no email) 2010-05-10 15:40:27 PDT
This has been solved by making the commit-queue automatically restart itself every 10 cycles using a wrapper script.