<?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>193234</bug_id>
          
          <creation_ts>2019-01-08 07:10:58 -0800</creation_ts>
          <short_desc>[ews-build] unapply-patch step should not update source</short_desc>
          <delta_ts>2019-01-17 05:55:27 -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>
          
          
          <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>
          
          
          <everconfirmed>1</everconfirmed>
          <reporter name="Aakash Jain">aakash_jain</reporter>
          <assigned_to name="Aakash Jain">aakash_jain</assigned_to>
          <cc>aakash_jain</cc>
    
    <cc>ap</cc>
    
    <cc>ews-watchlist</cc>
    
    <cc>lforschler</cc>
    
    <cc>webkit-bug-importer</cc>
          

      

      

      

          <comment_sort_order>oldest_to_newest</comment_sort_order>  
          <long_desc isprivate="0" >
    <commentid>1492586</commentid>
    <comment_count>0</comment_count>
    <who name="Aakash Jain">aakash_jain</who>
    <bug_when>2019-01-08 07:10:58 -0800</bug_when>
    <thetext>Currently unapply-patch step uses buildbot git methods with alwaysUseLatest=True. This cleans the working directory, but also updates the repository. We should not pull down new source changes.</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>1492587</commentid>
    <comment_count>1</comment_count>
      <attachid>358586</attachid>
    <who name="Aakash Jain">aakash_jain</who>
    <bug_when>2019-01-08 07:12:54 -0800</bug_when>
    <thetext>Created attachment 358586
Proposed patch</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>1492589</commentid>
    <comment_count>2</comment_count>
    <who name="EWS Watchlist">ews-watchlist</who>
    <bug_when>2019-01-08 07:16:10 -0800</bug_when>
    <thetext>Attachment 358586 did not pass style-queue:


ERROR: Tools/BuildSlaveSupport/ews-build/steps.py:108:  expected 2 blank lines, found 1  [pep8/E302] [5]
Total errors found: 1 in 2 files


If any of these errors are false positives, please file a bug against check-webkit-style.</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>1492593</commentid>
    <comment_count>3</comment_count>
      <attachid>358588</attachid>
    <who name="Aakash Jain">aakash_jain</who>
    <bug_when>2019-01-08 07:25:14 -0800</bug_when>
    <thetext>Created attachment 358588
Proposed patch</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>1492931</commentid>
    <comment_count>4</comment_count>
    <who name="Aakash Jain">aakash_jain</who>
    <bug_when>2019-01-09 06:30:02 -0800</bug_when>
    <thetext>Sample run: https://ews-build.webkit-uat.org/#/builders/25/builds/984</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>1493053</commentid>
    <comment_count>5</comment_count>
      <attachid>358588</attachid>
    <who name="Lucas Forschler">lforschler</who>
    <bug_when>2019-01-09 13:19:45 -0800</bug_when>
    <thetext>Comment on attachment 358588
Proposed patch

View in context: https://bugs.webkit.org/attachment.cgi?id=358588&amp;action=review

r=me with typo fix.

&gt; Tools/BuildSlaveSupport/ews-build/steps.py:111
&gt; +    description = [&apos;clean-workring-directory running&apos;]

typo: working</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>1495444</commentid>
    <comment_count>6</comment_count>
    <who name="Aakash Jain">aakash_jain</who>
    <bug_when>2019-01-17 05:54:05 -0800</bug_when>
    <thetext>Committed r240107: &lt;https://trac.webkit.org/changeset/240107&gt;</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>1495445</commentid>
    <comment_count>7</comment_count>
    <who name="Radar WebKit Bug Importer">webkit-bug-importer</who>
    <bug_when>2019-01-17 05:55:27 -0800</bug_when>
    <thetext>&lt;rdar://problem/47349291&gt;</thetext>
  </long_desc>
      
          <attachment
              isobsolete="1"
              ispatch="1"
              isprivate="0"
          >
            <attachid>358586</attachid>
            <date>2019-01-08 07:12:54 -0800</date>
            <delta_ts>2019-01-08 07:25:14 -0800</delta_ts>
            <desc>Proposed patch</desc>
            <filename>ews-build-clean-webkit.patch</filename>
            <type>text/plain</type>
            <size>1831</size>
            <attacher name="Aakash Jain">aakash_jain</attacher>
            
              <data encoding="base64">SW5kZXg6IFRvb2xzL0NoYW5nZUxvZwo9PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09
PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09Ci0tLSBUb29scy9DaGFuZ2VMb2cJKHJl
dmlzaW9uIDIzOTcyMykKKysrIFRvb2xzL0NoYW5nZUxvZwkod29ya2luZyBjb3B5KQpAQCAtMSwz
ICsxLDE0IEBACisyMDE5LTAxLTA4ICBBYWthc2ggSmFpbiAgPGFha2FzaF9qYWluQGFwcGxlLmNv
bT4KKworICAgICAgICBbZXdzLWJ1aWxkXSB1bmFwcGx5LXBhdGNoIHN0ZXAgc2hvdWxkIG5vdCB1
cGRhdGUgc291cmNlCisgICAgICAgIGh0dHBzOi8vYnVncy53ZWJraXQub3JnL3Nob3dfYnVnLmNn
aT9pZD0xOTMyMzQKKworICAgICAgICBSZXZpZXdlZCBieSBOT0JPRFkgKE9PUFMhKS4KKworICAg
ICAgICAqIEJ1aWxkU2xhdmVTdXBwb3J0L2V3cy1idWlsZC9zdGVwcy5weToKKyAgICAgICAgKENs
ZWFuV29ya2luZ0RpcmVjdG9yeSk6IFVzZSBjbGVhbi13ZWJraXQgc2NyaXB0IHRvIGNsZWFuIHRo
ZSB3b3JraW5nIGRpcmVjdG9yeS4KKyAgICAgICAgKFVuQXBwbHlQYXRjaElmUmVxdWlyZWQpOiBV
c2UgQ2xlYW5Xb3JraW5nRGlyZWN0b3J5IGFzIGJhc2UgY2xhc3MuCisKIDIwMTktMDEtMDcgIERh
dmlkIEtpbHplciAgPGRka2lsemVyQGFwcGxlLmNvbT4KIAogICAgICAgICBMZWFrIG9mIFNjcm9s
bENvbXBsZXRpb25DYWxsYmFja0RhdGEgKDE2IGJ5dGVzKSBpbiBjb20uYXBwbGUuV2ViS2l0Lldl
YkNvbnRlbnQgcnVubmluZyBXZWJLaXQgbGF5b3V0IHRlc3RzCkluZGV4OiBUb29scy9CdWlsZFNs
YXZlU3VwcG9ydC9ld3MtYnVpbGQvc3RlcHMucHkKPT09PT09PT09PT09PT09PT09PT09PT09PT09
PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PQotLS0gVG9vbHMvQnVpbGRT
bGF2ZVN1cHBvcnQvZXdzLWJ1aWxkL3N0ZXBzLnB5CShyZXZpc2lvbiAyMzk3MjMpCisrKyBUb29s
cy9CdWlsZFNsYXZlU3VwcG9ydC9ld3MtYnVpbGQvc3RlcHMucHkJKHdvcmtpbmcgY29weSkKQEAg
LTEwNSw2ICsxMDUsMTQgQEAgY2xhc3MgQ2hlY2tPdXRTb3VyY2UoZ2l0LkdpdCk6CiAgICAgICAg
ICAgICAgICAgICAgICAgICAgICAgICAgICAgICAgICAgICAgICAgICBwcm9ncmVzcz1UcnVlLAog
ICAgICAgICAgICAgICAgICAgICAgICAgICAgICAgICAgICAgICAgICAgICAgICAgKiprd2FyZ3Mp
CiAKK2NsYXNzIENsZWFuV29ya2luZ0RpcmVjdG9yeShzaGVsbC5TaGVsbENvbW1hbmQpOgorICAg
IG5hbWUgPSAnY2xlYW4td29ya2luZy1kaXJlY3RvcnknCisgICAgZGVzY3JpcHRpb24gPSBbJ2Ns
ZWFuLXdvcmtyaW5nLWRpcmVjdG9yeSBydW5uaW5nJ10KKyAgICBkZXNjcmlwdGlvbkRvbmUgPSBb
J2NsZWFuLXdvcmtpbmctZGlyZWN0b3J5J10KKyAgICBmbHVua09uRmFpbHVyZSA9IFRydWUKKyAg
ICBoYWx0T25GYWlsdXJlID0gVHJ1ZQorICAgIGNvbW1hbmQgPSBbJ1Rvb2xzL1NjcmlwdHMvY2xl
YW4td2Via2l0J10KKwogCiBjbGFzcyBBcHBseVBhdGNoKHNoZWxsLlNoZWxsQ29tbWFuZCwgQ29t
cG9zaXRlU3RlcE1peGluKToKICAgICBuYW1lID0gJ2FwcGx5LXBhdGNoJwpAQCAtMzUzLDcgKzM2
MSw3IEBAIGNsYXNzIFZhbGlkYXRlUGF0Y2goYnVpbGRzdGVwLkJ1aWxkU3RlcCkKICAgICAgICAg
cmV0dXJuIE5vbmUKIAogCi1jbGFzcyBVbkFwcGx5UGF0Y2hJZlJlcXVpcmVkKENoZWNrT3V0U291
cmNlKToKK2NsYXNzIFVuQXBwbHlQYXRjaElmUmVxdWlyZWQoQ2xlYW5Xb3JraW5nRGlyZWN0b3J5
KToKICAgICBuYW1lID0gJ3VuYXBwbHktcGF0Y2gnCiAKICAgICBkZWYgZG9TdGVwSWYoc2VsZiwg
c3RlcCk6Cg==
</data>

          </attachment>
          <attachment
              isobsolete="0"
              ispatch="1"
              isprivate="0"
          >
            <attachid>358588</attachid>
            <date>2019-01-08 07:25:14 -0800</date>
            <delta_ts>2019-01-17 05:54:21 -0800</delta_ts>
            <desc>Proposed patch</desc>
            <filename>ews-build-clean-webkit-v2.patch</filename>
            <type>text/plain</type>
            <size>1807</size>
            <attacher name="Aakash Jain">aakash_jain</attacher>
            
              <data encoding="base64">SW5kZXg6IFRvb2xzL0NoYW5nZUxvZwo9PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09
PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09Ci0tLSBUb29scy9DaGFuZ2VMb2cJKHJl
dmlzaW9uIDIzOTcyNCkKKysrIFRvb2xzL0NoYW5nZUxvZwkod29ya2luZyBjb3B5KQpAQCAtMSwz
ICsxLDE0IEBACisyMDE5LTAxLTA4ICBBYWthc2ggSmFpbiAgPGFha2FzaF9qYWluQGFwcGxlLmNv
bT4KKworICAgICAgICBbZXdzLWJ1aWxkXSB1bmFwcGx5LXBhdGNoIHN0ZXAgc2hvdWxkIG5vdCB1
cGRhdGUgc291cmNlCisgICAgICAgIGh0dHBzOi8vYnVncy53ZWJraXQub3JnL3Nob3dfYnVnLmNn
aT9pZD0xOTMyMzQKKworICAgICAgICBSZXZpZXdlZCBieSBOT0JPRFkgKE9PUFMhKS4KKworICAg
ICAgICAqIEJ1aWxkU2xhdmVTdXBwb3J0L2V3cy1idWlsZC9zdGVwcy5weToKKyAgICAgICAgKENs
ZWFuV29ya2luZ0RpcmVjdG9yeSk6IFVzZSBjbGVhbi13ZWJraXQgc2NyaXB0IHRvIGNsZWFuIHRo
ZSB3b3JraW5nIGRpcmVjdG9yeS4KKyAgICAgICAgKFVuQXBwbHlQYXRjaElmUmVxdWlyZWQpOiBV
c2UgQ2xlYW5Xb3JraW5nRGlyZWN0b3J5IGFzIGJhc2UgY2xhc3MuCisKIDIwMTktMDEtMDcgIERh
dmlkIEtpbHplciAgPGRka2lsemVyQGFwcGxlLmNvbT4KIAogICAgICAgICBMZWFrIG9mIFNjcm9s
bENvbXBsZXRpb25DYWxsYmFja0RhdGEgKDE2IGJ5dGVzKSBpbiBjb20uYXBwbGUuV2ViS2l0Lldl
YkNvbnRlbnQgcnVubmluZyBXZWJLaXQgbGF5b3V0IHRlc3RzCkluZGV4OiBUb29scy9CdWlsZFNs
YXZlU3VwcG9ydC9ld3MtYnVpbGQvc3RlcHMucHkKPT09PT09PT09PT09PT09PT09PT09PT09PT09
PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PQotLS0gVG9vbHMvQnVpbGRT
bGF2ZVN1cHBvcnQvZXdzLWJ1aWxkL3N0ZXBzLnB5CShyZXZpc2lvbiAyMzk3MjMpCisrKyBUb29s
cy9CdWlsZFNsYXZlU3VwcG9ydC9ld3MtYnVpbGQvc3RlcHMucHkJKHdvcmtpbmcgY29weSkKQEAg
LTEwNiw2ICsxMDYsMTUgQEAgY2xhc3MgQ2hlY2tPdXRTb3VyY2UoZ2l0LkdpdCk6CiAgICAgICAg
ICAgICAgICAgICAgICAgICAgICAgICAgICAgICAgICAgICAgICAgICAqKmt3YXJncykKIAogCitj
bGFzcyBDbGVhbldvcmtpbmdEaXJlY3Rvcnkoc2hlbGwuU2hlbGxDb21tYW5kKToKKyAgICBuYW1l
ID0gJ2NsZWFuLXdvcmtpbmctZGlyZWN0b3J5JworICAgIGRlc2NyaXB0aW9uID0gWydjbGVhbi13
b3JrcmluZy1kaXJlY3RvcnkgcnVubmluZyddCisgICAgZGVzY3JpcHRpb25Eb25lID0gWydjbGVh
bi13b3JraW5nLWRpcmVjdG9yeSddCisgICAgZmx1bmtPbkZhaWx1cmUgPSBUcnVlCisgICAgaGFs
dE9uRmFpbHVyZSA9IFRydWUKKyAgICBjb21tYW5kID0gWydUb29scy9TY3JpcHRzL2NsZWFuLXdl
YmtpdCddCisKKwogY2xhc3MgQXBwbHlQYXRjaChzaGVsbC5TaGVsbENvbW1hbmQsIENvbXBvc2l0
ZVN0ZXBNaXhpbik6CiAgICAgbmFtZSA9ICdhcHBseS1wYXRjaCcKICAgICBkZXNjcmlwdGlvbiA9
IFsnYXBwbHlpbmctcGF0Y2gnXQpAQCAtMzUzLDcgKzM2Miw3IEBAIGNsYXNzIFZhbGlkYXRlUGF0
Y2goYnVpbGRzdGVwLkJ1aWxkU3RlcCkKICAgICAgICAgcmV0dXJuIE5vbmUKIAogCi1jbGFzcyBV
bkFwcGx5UGF0Y2hJZlJlcXVpcmVkKENoZWNrT3V0U291cmNlKToKK2NsYXNzIFVuQXBwbHlQYXRj
aElmUmVxdWlyZWQoQ2xlYW5Xb3JraW5nRGlyZWN0b3J5KToKICAgICBuYW1lID0gJ3VuYXBwbHkt
cGF0Y2gnCiAKICAgICBkZWYgZG9TdGVwSWYoc2VsZiwgc3RlcCk6Cg==
</data>
<flag name="review"
          id="375409"
          type_id="1"
          status="+"
          setter="lforschler"
    />
          </attachment>
      

    </bug>

</bugzilla>