Bug 243979 – [ews-build.webkit.org] Ignore base commit when rebasing

• Status: RESOLVED
• Resolution: FIXED
• Component: Tools / Tests

[Kimmo Kinnunen]

EWS failed to apply a patch correctly when it had committer date in the patch

1. I created a patch and a PR around Aug 12, uploaded with `git webkit pr --no-update`
2. I rebased, amended the commit on Aug 15. It was now on top of Aug 15 ToT.
3. I uploaded the new patch with `git webkit pr --no-update`, on Aug 15.
4. I set the merge-queue

EWS tried to rebase the patch on top of Aug 15 ToT with a upstream commit of Aug 11.
(git rebase --onto <newbase> <upstream> <branch>)

Naturally this rebase fails, at it tries to re-apply commits between Aug 11-15 on top of Aug 15 ToT. 

https://ews-build.webkit.org/#/builders/69/builds/29926/steps/10/logs/stdio

git rebase --onto 7654da12692d947efe23243092725dc6abc96436 6bb3f1342f342358061b525c7b8f077b7b5ed15b eng/machsendright-to-ipc

eng/machsendright-to-ipc is correctly on Aug 15 ToT: 7654da12692d

18021d1c87d2 (HEAD -> eng/machsendright-to-ipc, fork/eng/machsendright-to-ipc) Darwin specific IPC encoding primitives are implemented in WebKit/Shared but used in WebKit/Platform/IPC https://bugs.webkit.org/show_bug.cgi?id=243870 Include a Radar link (OOPS!).
7654da12692d (origin/main, origin/HEAD, main) Navigator.share() rejects with wrong exception when called multiple times https://bugs.webkit.org/show_bug.cgi?id=243652



The <upstream> was Aug 11:
6bb3f1342f34 [GTK] Fix build on macOS https://bugs.webkit.org/show_bug.cgi?id=243759


Most likely same cause as bug 243277

[Kimmo Kinnunen]

Pull request: https://github.com/WebKit/WebKit/pull/3350

[Kimmo Kinnunen]

Pull request: https://github.com/WebKit/WebKit/pull/3351

[Kimmo Kinnunen]

I probably am not able to test this, as the EWS will rebase only when trying to merge, and I don't want to try to merge a test commit.


To reproduce, I would imagine this would do it:

git reset $(git log --before "3 weeks ago" -1 --pretty=%H)
echo "   " >> Source/WebCore/platform/graphics/GraphicsContextGL.cpp
git commit -a --date="3 weeks ago"
git webkit pr --no-update

git rebase $(git log --before "2 weeks ago") -1 pretty=%H)
git webkit pr --no-update

After this, you would need to make the EWS to try to run the rebase.

[Kimmo Kinnunen]

Pull request: https://github.com/WebKit/WebKit/pull/3352

[Radar WebKit Bug Importer]

<rdar://problem/99019798>

[Jonathan Bedard]

Pull request: https://github.com/WebKit/WebKit/pull/3591

[EWS]

Committed 253713@main (f4104beafed4): <https://commits.webkit.org/253713@main>

Reviewed commits have been landed. Closing PR #3591 and removing active labels.

[Ryan Haddad]

*** Bug 243277 has been marked as a duplicate of this bug. ***