[Webkit-unassigned] [Bug 255675] [REGRESSION] Reverting a patch no longer posts anything in the original bug

[bugzilla-daemon at webkit.org]

https://bugs.webkit.org/show_bug.cgi?id=255675

--- Comment #3 from Alexey Proskuryakov <ap at webkit.org> ---
This is generally happening in practice though.

https://bugs.webkit.org/show_bug.cgi?id=255610 was a revert bug created by webkitbot. It was filed at 10:02:18 PDT, and https://bugs.webkit.org/show_bug.cgi?id=255506 got reopened three seconds later. Commit queue only landed the revert at 10:15:04 PDT.

Looking at the original case that motivated this report, https://bugs.webkit.org/show_bug.cgi?id=255473, I see that Fujii reopened it manually BEFORE creating the revert bug. Webkitbot did add a dependency relationship.

So, seems like correct behavior - it would be wrong for webkitbot to say that it reopened the bug when it didn't. Remember that the comment says "Re-opened since this is blocked by <...>". Myles, is your expectation different?

> When WebKitBot reverts a patch, it should _always_ comment in the bug of the patch it reverted.

Note however that webkitbot does not comment after reverting, it only ever comments when creating a revert bug. I think that this is wrong, both commenting and reopening should happen when the revert lands. But maybe I'm missing why it's being done this way. In any case, there is no regression here, and we'd want a new bug report to investigate improvements.

-- 
You are receiving this mail because:
You are the assignee for the bug.
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.webkit.org/pipermail/webkit-unassigned/attachments/20230426/3c6d79ee/attachment.htm>