<?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>220795</bug_id>
          
          <creation_ts>2021-01-21 07:06:10 -0800</creation_ts>
          <short_desc>[webkitcorepy] Don&apos;t import six at the top level</short_desc>
          <delta_ts>2021-01-21 14:37:23 -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>New Bugs</component>
          <version>WebKit Nightly Build</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="Angelos Oikonomopoulos">angelos</reporter>
          <assigned_to name="Nobody">webkit-unassigned</assigned_to>
          <cc>aakash_jain</cc>
    
    <cc>dewei_zhu</cc>
    
    <cc>jbedard</cc>
    
    <cc>webkit-bug-importer</cc>
          

      

      

      

          <comment_sort_order>oldest_to_newest</comment_sort_order>  
          <long_desc isprivate="0" >
    <commentid>1721153</commentid>
    <comment_count>0</comment_count>
    <who name="Angelos Oikonomopoulos">angelos</who>
    <bug_when>2021-01-21 07:06:10 -0800</bug_when>
    <thetext>[webkitcorepy] Don&apos;t import six at the top level</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>1721154</commentid>
    <comment_count>1</comment_count>
      <attachid>418037</attachid>
    <who name="Angelos Oikonomopoulos">angelos</who>
    <bug_when>2021-01-21 07:10:47 -0800</bug_when>
    <thetext>Created attachment 418037
Patch</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>1721167</commentid>
    <comment_count>2</comment_count>
      <attachid>418037</attachid>
    <who name="Aakash Jain">aakash_jain</who>
    <bug_when>2021-01-21 08:05:32 -0800</bug_when>
    <thetext>Comment on attachment 418037
Patch

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

&gt; Tools/Scripts/libraries/webkitcorepy/webkitcorepy/task_pool.py:338
&gt; +        from six import reraise

Do we need to import it again whiling exiting?</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>1721170</commentid>
    <comment_count>3</comment_count>
      <attachid>418037</attachid>
    <who name="Jonathan Bedard">jbedard</who>
    <bug_when>2021-01-21 08:20:37 -0800</bug_when>
    <thetext>Comment on attachment 418037
Patch

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

&gt;&gt; Tools/Scripts/libraries/webkitcorepy/webkitcorepy/task_pool.py:338
&gt;&gt; +        from six import reraise
&gt; 
&gt; Do we need to import it again whiling exiting?

We do, because we use it on line 352</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>1721176</commentid>
    <comment_count>4</comment_count>
    <who name="EWS">ews-feeder</who>
    <bug_when>2021-01-21 08:52:11 -0800</bug_when>
    <thetext>Committed r271692: &lt;https://trac.webkit.org/changeset/271692&gt;

All reviewed patches have been landed. Closing bug and clearing flags on attachment 418037.</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>1721178</commentid>
    <comment_count>5</comment_count>
    <who name="Radar WebKit Bug Importer">webkit-bug-importer</who>
    <bug_when>2021-01-21 08:54:51 -0800</bug_when>
    <thetext>&lt;rdar://problem/73454252&gt;</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>1721345</commentid>
    <comment_count>6</comment_count>
    <who name="Aakash Jain">aakash_jain</who>
    <bug_when>2021-01-21 14:37:23 -0800</bug_when>
    <thetext>*** Bug 220791 has been marked as a duplicate of this bug. ***</thetext>
  </long_desc>
      
          <attachment
              isobsolete="0"
              ispatch="1"
              isprivate="0"
          >
            <attachid>418037</attachid>
            <date>2021-01-21 07:10:47 -0800</date>
            <delta_ts>2021-01-21 08:52:12 -0800</delta_ts>
            <desc>Patch</desc>
            <filename>bug-220795-20210121161044.patch</filename>
            <type>text/plain</type>
            <size>2045</size>
            <attacher name="Angelos Oikonomopoulos">angelos</attacher>
            
              <data encoding="base64">U3VidmVyc2lvbiBSZXZpc2lvbjogMjcxNjkwCmRpZmYgLS1naXQgYS9Ub29scy9DaGFuZ2VMb2cg
Yi9Ub29scy9DaGFuZ2VMb2cKaW5kZXggMGQyYjllNDQ1M2M3NjE4NTdmNGU0YTJjZTFhOWQ2NGY2
NGYzMDAwNy4uYWRmNzkxMzRmMjU4YzRlM2FkMmFhNTI3NmZlYjg4MzY3YjY1NzJjZSAxMDA2NDQK
LS0tIGEvVG9vbHMvQ2hhbmdlTG9nCisrKyBiL1Rvb2xzL0NoYW5nZUxvZwpAQCAtMSwzICsxLDE5
IEBACisyMDIxLTAxLTIxICBBbmdlbG9zIE9pa29ub21vcG91bG9zICA8YW5nZWxvc0BpZ2FsaWEu
Y29tPgorCisgICAgICAgIFt3ZWJraXRjb3JlcHldIERvbid0IGltcG9ydCBzaXggYXQgdGhlIHRv
cCBsZXZlbAorICAgICAgICBodHRwczovL2J1Z3Mud2Via2l0Lm9yZy9zaG93X2J1Zy5jZ2k/aWQ9
MjIwNzk1CisKKyAgICAgICAgUmV2aWV3ZWQgYnkgTk9CT0RZIChPT1BTISkuCisKKyAgICAgICAg
SW5zdGVhZCwgZG8gdGhlIGltcG9ydCBhZnRlciB0aGUgbW9kdWxlIGhhcyBiZWVuIGxvYWRlZCwg
d2hlbgorICAgICAgICBBdXRvSW5zdGFsbCBoYXMgYmVlbiBzZXQgdXAgcHJvcGVybHkuIEZpeGVz
IGEgcmVncmVzc2lvbiBhZnRlcgorICAgICAgICAxZDM0MmE1MDY3ZTIyYWM4MDY5ZmYzZmYzMzdi
MmQ1OGUxZDA2NTE3LCB3aGVyZSB0YXNrX3Bvb2wucHkgdXNlcworICAgICAgICB0aGUgc2l4IG1v
ZHVsZSB3aXRob3V0IGl0IGJlaW5nIGluc3RhbGxlZC4KKworICAgICAgICAqIFNjcmlwdHMvbGli
cmFyaWVzL3dlYmtpdGNvcmVweS93ZWJraXRjb3JlcHkvdGFza19wb29sLnB5OgorICAgICAgICAo
Q2hpbGRFeGNlcHRpb24uX19jYWxsX18pOgorICAgICAgICAoVGFza1Bvb2wuX19leGl0X18pOgor
CiAyMDIxLTAxLTIwICBKb25hdGhhbiBCZWRhcmQgIDxqYmVkYXJkQGFwcGxlLmNvbT4KIAogICAg
ICAgICBbd2Via2l0c2NtcHldIEdyYWNlZnVsbHkgaGFuZGxlIG1pc3Npbmcga2V5cmluZyBiYWNr
ZW5kCmRpZmYgLS1naXQgYS9Ub29scy9TY3JpcHRzL2xpYnJhcmllcy93ZWJraXRjb3JlcHkvd2Vi
a2l0Y29yZXB5L3Rhc2tfcG9vbC5weSBiL1Rvb2xzL1NjcmlwdHMvbGlicmFyaWVzL3dlYmtpdGNv
cmVweS93ZWJraXRjb3JlcHkvdGFza19wb29sLnB5CmluZGV4IGNhOTM1MmFjMGRlZDJjODNhMzhk
MWJlMTRiYzg2ZDJhNDg4ZTRmMWMuLmQ3N2E5ZTk5NmJjNjYyNzczNTUyOTY4NzExZTAzY2ZhYjY2
MmRiNjcgMTAwNjQ0Ci0tLSBhL1Rvb2xzL1NjcmlwdHMvbGlicmFyaWVzL3dlYmtpdGNvcmVweS93
ZWJraXRjb3JlcHkvdGFza19wb29sLnB5CisrKyBiL1Rvb2xzL1NjcmlwdHMvbGlicmFyaWVzL3dl
YmtpdGNvcmVweS93ZWJraXRjb3JlcHkvdGFza19wb29sLnB5CkBAIC0yNyw3ICsyNyw2IEBAIGlt
cG9ydCBtdWx0aXByb2Nlc3NpbmcKIGltcG9ydCBzaWduYWwKIGltcG9ydCBzeXMKIAotZnJvbSBz
aXggaW1wb3J0IHJlcmFpc2UKIGZyb20gd2Via2l0Y29yZXB5IGltcG9ydCBPdXRwdXRDYXB0dXJl
LCBUaW1lb3V0LCBsb2cKIAogCkBAIC0xMjAsNiArMTE5LDcgQEAgY2xhc3MgQ2hpbGRFeGNlcHRp
b24oTWVzc2FnZSk6CiAgICAgICAgIHNlbGYuZXhjX2luZm8gPSBleGNfaW5mbyBvciBzeXMuZXhj
X2luZm8oKQogCiAgICAgZGVmIF9fY2FsbF9fKHNlbGYsIGNhbGxlcik6CisgICAgICAgIGZyb20g
c2l4IGltcG9ydCByZXJhaXNlCiAgICAgICAgIF8sIGV4Y2VwdGlvbiwgdHJhY2UgPSBzeXMuZXhj
X2luZm8oKQogICAgICAgICBpZiBleGNlcHRpb246CiAgICAgICAgICAgICBsb2cuY3JpdGljYWwo
IkV4Y2VwdGlvbiBpbiBmbGlnaHQsICd7fScgaWdub3JlZCIuZm9ybWF0KHNlbGYuZXhjX2luZm9b
MV0pKQpAQCAtMzM1LDYgKzMzNSw3IEBAIGNsYXNzIFRhc2tQb29sKG9iamVjdCk6CiAgICAgICAg
ICAgICBzZWxmLnF1ZXVlLnJlY2VpdmUoKShzZWxmKQogCiAgICAgZGVmIF9fZXhpdF9fKHNlbGYs
ICphcmdzLCAqKmt3YXJncyk6CisgICAgICAgIGZyb20gc2l4IGltcG9ydCByZXJhaXNlCiAgICAg
ICAgIHRyeToKICAgICAgICAgICAgIGluZmxpZ2h0ID0gc3lzLmV4Y19pbmZvKCkKIAo=
</data>

          </attachment>
      

    </bug>

</bugzilla>