WebKit Bugzilla
New
Browse
Log In
×
Sign in with GitHub
or
Remember my login
Create Account
·
Forgot Password
Forgotten password account recovery
NEW
99507
Commit queue tried to land the wrong patch
https://bugs.webkit.org/show_bug.cgi?id=99507
Summary
Commit queue tried to land the wrong patch
Emil A Eklund
Reported
2012-10-16 14:08:56 PDT
In setting cq+ on
bug 99376
the commit queue instead tried to land the patch for
bug 99435
. Luckily this resulted in a merge conflict preventing it from being landed. See
https://bugs.webkit.org/show_bug.cgi?id=99376
for details and output.
Attachments
Add attachment
proposed patch, testcase, etc.
Adam Barth
Comment 1
2012-10-16 14:37:32 PDT
The bug is just that the two patches landed close to each other and the commit-queue didn't know how to resolve the conflict in the ChangeLog. We used to have a merge-driver on the bots that knew how to fix these things, but it had other problems. We just need to teach the bot that it shouldn't bother trying to resolve conflicts when landing.
Emil A Eklund
Comment 2
2012-10-16 14:40:57 PDT
(In reply to
comment #1
)
> The bug is just that the two patches landed close to each other and the commit-queue didn't know how to resolve the conflict in the ChangeLog. We used to have a merge-driver on the bots that knew how to fix these things, but it had other problems. We just need to teach the bot that it shouldn't bother trying to resolve conflicts when landing.
Are you sure? It looks a lot like it tried to grab the wrong patch as the file it is complaining about (Tools/ChangeLog) isn't even in the patch it should have applied.
Adam Barth
Comment 3
2012-10-16 14:48:38 PDT
> Are you sure?
I'm not sure. I know we see that error message from time to time and it makes me sad. I'd like to fix it, but I haven't found the time. :(
Emil A Eklund
Comment 4
2012-10-16 14:50:48 PDT
Sadness :( Thanks for looking.
Note
You need to
log in
before you can comment on or make changes to this bug.
Top of Page
Format For Printing
XML
Clone This Bug