<?xml version="1.0" encoding="UTF-8" standalone="yes" ?>
<!DOCTYPE bugzilla SYSTEM "https://bugs.webkit.org/page.cgi?id=bugzilla.dtd">

<bugzilla version="5.0.4.1"
          urlbase="https://bugs.webkit.org/"
          
          maintainer="admin@webkit.org"
>

    <bug>
          <bug_id>243979</bug_id>
          
          <creation_ts>2022-08-16 00:30:41 -0700</creation_ts>
          <short_desc>[ews-build.webkit.org] Ignore base commit when rebasing</short_desc>
          <delta_ts>2023-01-11 09:46:37 -0800</delta_ts>
          <reporter_accessible>1</reporter_accessible>
          <cclist_accessible>1</cclist_accessible>
          <classification_id>1</classification_id>
          <classification>Unclassified</classification>
          <product>WebKit</product>
          <component>Tools / Tests</component>
          <version>Other</version>
          <rep_platform>Unspecified</rep_platform>
          <op_sys>Unspecified</op_sys>
          <bug_status>RESOLVED</bug_status>
          <resolution>FIXED</resolution>
          
          <see_also>https://bugs.webkit.org/show_bug.cgi?id=243277</see_also>
          <bug_file_loc></bug_file_loc>
          <status_whiteboard></status_whiteboard>
          <keywords>InRadar</keywords>
          <priority>P2</priority>
          <bug_severity>Normal</bug_severity>
          <target_milestone>---</target_milestone>
          
          <blocked>239082</blocked>
          <everconfirmed>1</everconfirmed>
          <reporter name="Kimmo Kinnunen">kkinnunen</reporter>
          <assigned_to name="Jonathan Bedard">jbedard</assigned_to>
          <cc>jbedard</cc>
    
    <cc>mattwoodrow</cc>
    
    <cc>webkit-bug-importer</cc>
          

      

      

      

          <comment_sort_order>oldest_to_newest</comment_sort_order>  
          <long_desc isprivate="0" >
    <commentid>1891598</commentid>
    <comment_count>0</comment_count>
    <who name="Kimmo Kinnunen">kkinnunen</who>
    <bug_when>2022-08-16 00:30:41 -0700</bug_when>
    <thetext>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 &lt;newbase&gt; &lt;upstream&gt; &lt;branch&gt;)

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 -&gt; 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 &lt;upstream&gt; 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</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>1891600</commentid>
    <comment_count>1</comment_count>
    <who name="Kimmo Kinnunen">kkinnunen</who>
    <bug_when>2022-08-16 00:32:49 -0700</bug_when>
    <thetext>Pull request: https://github.com/WebKit/WebKit/pull/3350</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>1891602</commentid>
    <comment_count>2</comment_count>
    <who name="Kimmo Kinnunen">kkinnunen</who>
    <bug_when>2022-08-16 00:44:07 -0700</bug_when>
    <thetext>Pull request: https://github.com/WebKit/WebKit/pull/3351</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>1891604</commentid>
    <comment_count>3</comment_count>
    <who name="Kimmo Kinnunen">kkinnunen</who>
    <bug_when>2022-08-16 00:52:08 -0700</bug_when>
    <thetext>I probably am not able to test this, as the EWS will rebase only when trying to merge, and I don&apos;t want to try to merge a test commit.


To reproduce, I would imagine this would do it:

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

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

After this, you would need to make the EWS to try to run the rebase.</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>1891610</commentid>
    <comment_count>4</comment_count>
    <who name="Kimmo Kinnunen">kkinnunen</who>
    <bug_when>2022-08-16 01:26:08 -0700</bug_when>
    <thetext>Pull request: https://github.com/WebKit/WebKit/pull/3352</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>1893178</commentid>
    <comment_count>5</comment_count>
    <who name="Radar WebKit Bug Importer">webkit-bug-importer</who>
    <bug_when>2022-08-23 00:31:14 -0700</bug_when>
    <thetext>&lt;rdar://problem/99019798&gt;</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>1893352</commentid>
    <comment_count>6</comment_count>
    <who name="Jonathan Bedard">jbedard</who>
    <bug_when>2022-08-23 14:43:50 -0700</bug_when>
    <thetext>Pull request: https://github.com/WebKit/WebKit/pull/3591</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>1893392</commentid>
    <comment_count>7</comment_count>
    <who name="EWS">ews-feeder</who>
    <bug_when>2022-08-23 17:33:42 -0700</bug_when>
    <thetext>Committed 253713@main (f4104beafed4): &lt;https://commits.webkit.org/253713@main&gt;

Reviewed commits have been landed. Closing PR #3591 and removing active labels.</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>1924816</commentid>
    <comment_count>8</comment_count>
    <who name="Ryan Haddad">ryanhaddad</who>
    <bug_when>2023-01-11 09:46:37 -0800</bug_when>
    <thetext>*** Bug 243277 has been marked as a duplicate of this bug. ***</thetext>
  </long_desc>
      
      

    </bug>

</bugzilla>