<?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>185291</bug_id>
          
          <creation_ts>2018-05-03 21:37:11 -0700</creation_ts>
          <short_desc>Use subprocess.call instead of os.system to handle path with spaces</short_desc>
          <delta_ts>2018-05-03 22:29:33 -0700</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="Yusuke Suzuki">ysuzuki</reporter>
          <assigned_to name="Yusuke Suzuki">ysuzuki</assigned_to>
          <cc>achristensen</cc>
    
    <cc>cdumez</cc>
    
    <cc>darin</cc>
    
    <cc>fpizlo</cc>
    
    <cc>rniwa</cc>
    
    <cc>saam</cc>
    
    <cc>sam</cc>
    
    <cc>webkit-bug-importer</cc>
          

      

      

      

          <comment_sort_order>oldest_to_newest</comment_sort_order>  
          <long_desc isprivate="0" >
    <commentid>1420555</commentid>
    <comment_count>0</comment_count>
    <who name="Yusuke Suzuki">ysuzuki</who>
    <bug_when>2018-05-03 21:37:11 -0700</bug_when>
    <thetext>Use subprocess.call instead of os.system to handle path with spaces</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>1420556</commentid>
    <comment_count>1</comment_count>
      <attachid>339518</attachid>
    <who name="Yusuke Suzuki">ysuzuki</who>
    <bug_when>2018-05-03 21:37:53 -0700</bug_when>
    <thetext>Created attachment 339518
Patch</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>1420566</commentid>
    <comment_count>2</comment_count>
    <who name="Yusuke Suzuki">ysuzuki</who>
    <bug_when>2018-05-03 22:28:17 -0700</bug_when>
    <thetext>Committed r231346: &lt;https://trac.webkit.org/changeset/231346&gt;</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>1420567</commentid>
    <comment_count>3</comment_count>
    <who name="Yusuke Suzuki">ysuzuki</who>
    <bug_when>2018-05-03 22:28:42 -0700</bug_when>
    <thetext>Thanks for your review!</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>1420568</commentid>
    <comment_count>4</comment_count>
    <who name="Radar WebKit Bug Importer">webkit-bug-importer</who>
    <bug_when>2018-05-03 22:29:33 -0700</bug_when>
    <thetext>&lt;rdar://problem/39967056&gt;</thetext>
  </long_desc>
      
          <attachment
              isobsolete="0"
              ispatch="1"
              isprivate="0"
          >
            <attachid>339518</attachid>
            <date>2018-05-03 21:37:53 -0700</date>
            <delta_ts>2018-05-03 22:23:54 -0700</delta_ts>
            <desc>Patch</desc>
            <filename>bug-185291-20180504133752.patch</filename>
            <type>text/plain</type>
            <size>4176</size>
            <attacher name="Yusuke Suzuki">ysuzuki</attacher>
            
              <data encoding="base64">U3VidmVyc2lvbiBSZXZpc2lvbjogMjMxMzQ0CmRpZmYgLS1naXQgYS9Tb3VyY2UvV2ViQ29yZS9D
aGFuZ2VMb2cgYi9Tb3VyY2UvV2ViQ29yZS9DaGFuZ2VMb2cKaW5kZXggZDIzMWNmZTU2YmJiYWEx
MTExODA3Y2NkMmY2NDk3OGNhMWUxYjA0MC4uODhlMGIxMzBmY2YwOWJlOWE2OTZlNTVmOTM1MjQ0
MjFmM2MzZmQzOSAxMDA2NDQKLS0tIGEvU291cmNlL1dlYkNvcmUvQ2hhbmdlTG9nCisrKyBiL1Nv
dXJjZS9XZWJDb3JlL0NoYW5nZUxvZwpAQCAtMSwzICsxLDE3IEBACisyMDE4LTA1LTAzICBZdXN1
a2UgU3V6dWtpICA8dXRhdGFuZS50ZWFAZ21haWwuY29tPgorCisgICAgICAgIFVzZSBzdWJwcm9j
ZXNzLmNhbGwgaW5zdGVhZCBvZiBvcy5zeXN0ZW0gdG8gaGFuZGxlIHBhdGggd2l0aCBzcGFjZXMK
KyAgICAgICAgaHR0cHM6Ly9idWdzLndlYmtpdC5vcmcvc2hvd19idWcuY2dpP2lkPTE4NTI5MQor
CisgICAgICAgIFJldmlld2VkIGJ5IE5PQk9EWSAoT09QUyEpLgorCisgICAgICAgIElmIGdwZXJm
IHBhdGggaW5jbHVkZXMgc3BhY2VzLCB0aGVzZSBweXRob24gc2NyaXB0cyBmYWlsIHRvIGV4ZWN1
dGUgZ3BlcmYuCisgICAgICAgIFdlIHVzZSBzdWJwcm9jZXNzIG1vZHVsZSBpbnN0ZWFkIG9mIG9z
LnN5c3RlbSB0byBpbnZva2UgZ3BlcmYuCisKKyAgICAgICAgKiBjc3MvbWFrZVNlbGVjdG9yUHNl
dWRvQ2xhc3NBbmRDb21wYXRpYmlsaXR5RWxlbWVudE1hcC5weToKKyAgICAgICAgKiBjc3MvbWFr
ZVNlbGVjdG9yUHNldWRvRWxlbWVudHNNYXAucHk6CisgICAgICAgICogcGxhdGZvcm0vbmV0d29y
ay9jcmVhdGUtaHR0cC1oZWFkZXItbmFtZS10YWJsZToKKwogMjAxOC0wNS0wMyAgWXVzdWtlIFN1
enVraSAgPHV0YXRhbmUudGVhQGdtYWlsLmNvbT4KIAogICAgICAgICBVbnJldmlld2VkLCBhdHRl
bXB0IHRvIGZpeCBXaW5DYWlybyBidWlsZCBmYWlsdXJlCmRpZmYgLS1naXQgYS9Tb3VyY2UvV2Vi
Q29yZS9jc3MvbWFrZVNlbGVjdG9yUHNldWRvQ2xhc3NBbmRDb21wYXRpYmlsaXR5RWxlbWVudE1h
cC5weSBiL1NvdXJjZS9XZWJDb3JlL2Nzcy9tYWtlU2VsZWN0b3JQc2V1ZG9DbGFzc0FuZENvbXBh
dGliaWxpdHlFbGVtZW50TWFwLnB5CmluZGV4IDc4MzMyMjcxYWM2MTU2OTU3NDNkM2NkYWE3Zjk3
YzlhMTc4NzY3YjEuLjBjMzAwODc2ZGE3NWVjOTNjYzUxOTI0NWQzMTU2Yzg1MTgxMjJmNTMgMTAw
NjQ0Ci0tLSBhL1NvdXJjZS9XZWJDb3JlL2Nzcy9tYWtlU2VsZWN0b3JQc2V1ZG9DbGFzc0FuZENv
bXBhdGliaWxpdHlFbGVtZW50TWFwLnB5CisrKyBiL1NvdXJjZS9XZWJDb3JlL2Nzcy9tYWtlU2Vs
ZWN0b3JQc2V1ZG9DbGFzc0FuZENvbXBhdGliaWxpdHlFbGVtZW50TWFwLnB5CkBAIC0yNSw2ICsy
NSw3IEBACiAKIGltcG9ydCBvcwogaW1wb3J0IHN5cworaW1wb3J0IHN1YnByb2Nlc3MKIAogCiBk
ZWYgZW51bWVyYWJsZVBzZXVkb1R5cGUoc3RyaW5nUHNldWRvVHlwZSk6CkBAIC0yMTEsNyArMjEy
LDYgQEAgZGVmIGV4cGFuZF9pZmRlZl9jb25kaXRpb24oY29uZGl0aW9uKToKIGlmICdHUEVSRicg
aW4gb3MuZW52aXJvbjoKICAgICBncGVyZl9jb21tYW5kID0gb3MuZW52aXJvblsnR1BFUkYnXQog
Ci1ncGVyZl9yZXR1cm4gPSBvcy5zeXN0ZW0oIiVzIC0ta2V5LXBvc2l0aW9ucz1cIipcIiAtbSAx
MCAtcyAyIFNlbGVjdG9yUHNldWRvQ2xhc3NBbmRDb21wYXRpYmlsaXR5RWxlbWVudE1hcC5ncGVy
ZiAtLW91dHB1dC1maWxlPVNlbGVjdG9yUHNldWRvQ2xhc3NBbmRDb21wYXRpYmlsaXR5RWxlbWVu
dE1hcC5jcHAiICUgZ3BlcmZfY29tbWFuZCkKLWlmIGdwZXJmX3JldHVybiAhPSAwOgoraWYgc3Vi
cHJvY2Vzcy5jYWxsKFtncGVyZl9jb21tYW5kLCAnLS1rZXktcG9zaXRpb25zPSonLCAnLW0nLCAn
MTAnLCAnLXMnLCAnMicsICdTZWxlY3RvclBzZXVkb0NsYXNzQW5kQ29tcGF0aWJpbGl0eUVsZW1l
bnRNYXAuZ3BlcmYnLCAnLS1vdXRwdXQtZmlsZT1TZWxlY3RvclBzZXVkb0NsYXNzQW5kQ29tcGF0
aWJpbGl0eUVsZW1lbnRNYXAuY3BwJ10pICE9IDA6CiAgICAgcHJpbnQoIkVycm9yIHdoZW4gZ2Vu
ZXJhdGluZyBTZWxlY3RvclBzZXVkb0NsYXNzQW5kQ29tcGF0aWJpbGl0eUVsZW1lbnRNYXAuY3Bw
IGZyb20gU2VsZWN0b3JQc2V1ZG9DbGFzc0FuZENvbXBhdGliaWxpdHlFbGVtZW50TWFwLmdwZXJm
IDooIikKICAgICBzeXMuZXhpdChncGVyZl9yZXR1cm4pCmRpZmYgLS1naXQgYS9Tb3VyY2UvV2Vi
Q29yZS9jc3MvbWFrZVNlbGVjdG9yUHNldWRvRWxlbWVudHNNYXAucHkgYi9Tb3VyY2UvV2ViQ29y
ZS9jc3MvbWFrZVNlbGVjdG9yUHNldWRvRWxlbWVudHNNYXAucHkKaW5kZXggODNiMjM4ZTJmNDZk
NDNiNDg0MDQwMTJjMDVkZDc2MDkxNjBhYWY3Mi4uMmM5Mjc2Yzk4MWUxMDlkODk1NjM2N2Y3ZjRi
Yjg5MWQyZThmZTkxYiAxMDA2NDQKLS0tIGEvU291cmNlL1dlYkNvcmUvY3NzL21ha2VTZWxlY3Rv
clBzZXVkb0VsZW1lbnRzTWFwLnB5CisrKyBiL1NvdXJjZS9XZWJDb3JlL2Nzcy9tYWtlU2VsZWN0
b3JQc2V1ZG9FbGVtZW50c01hcC5weQpAQCAtMjUsNiArMjUsNyBAQAogCiBpbXBvcnQgb3MKIGlt
cG9ydCBzeXMKK2ltcG9ydCBzdWJwcm9jZXNzCiAKIAogZGVmIGVudW1lcmFibGVQc2V1ZG9UeXBl
KHN0cmluZ1BzZXVkb1R5cGUpOgpAQCAtMjA2LDcgKzIwNyw2IEBAIGRlZiBleHBhbmRfaWZkZWZf
Y29uZGl0aW9uKGNvbmRpdGlvbik6CiBpZiAnR1BFUkYnIGluIG9zLmVudmlyb246CiAgICAgZ3Bl
cmZfY29tbWFuZCA9IG9zLmVudmlyb25bJ0dQRVJGJ10KIAotZ3BlcmZfcmV0dXJuID0gb3Muc3lz
dGVtKCIlcyAtLWtleS1wb3NpdGlvbnM9XCIqXCIgLW0gMTAgLXMgMiBTZWxlY3RvclBzZXVkb0Vs
ZW1lbnRUeXBlTWFwLmdwZXJmIC0tb3V0cHV0LWZpbGU9U2VsZWN0b3JQc2V1ZG9FbGVtZW50VHlw
ZU1hcC5jcHAiICUgZ3BlcmZfY29tbWFuZCkKLWlmIGdwZXJmX3JldHVybiAhPSAwOgoraWYgc3Vi
cHJvY2Vzcy5jYWxsKFtncGVyZl9jb21tYW5kLCAnLS1rZXktcG9zaXRpb25zPSonLCAnLW0nLCAn
MTAnLCAnLXMnLCAnMicsICdTZWxlY3RvclBzZXVkb0VsZW1lbnRUeXBlTWFwLmdwZXJmJywgJy0t
b3V0cHV0LWZpbGU9U2VsZWN0b3JQc2V1ZG9FbGVtZW50VHlwZU1hcC5jcHAnXSkgIT0gMDoKICAg
ICBwcmludCgiRXJyb3Igd2hlbiBnZW5lcmF0aW5nIFNlbGVjdG9yUHNldWRvRWxlbWVudFR5cGVN
YXAuY3BwIGZyb20gU2VsZWN0b3JQc2V1ZG9FbGVtZW50VHlwZU1hcC5ncGVyZiA6KCIpCiAgICAg
c3lzLmV4aXQoZ3BlcmZfcmV0dXJuKQpkaWZmIC0tZ2l0IGEvU291cmNlL1dlYkNvcmUvcGxhdGZv
cm0vbmV0d29yay9jcmVhdGUtaHR0cC1oZWFkZXItbmFtZS10YWJsZSBiL1NvdXJjZS9XZWJDb3Jl
L3BsYXRmb3JtL25ldHdvcmsvY3JlYXRlLWh0dHAtaGVhZGVyLW5hbWUtdGFibGUKaW5kZXggNzU1
ZDIyZTk0Njg3YTkyZmYzMjdkOTgxM2ViZDJlNzkwYThkMTgyNy4uMWQ0ZWJjYWViZDczM2NkODY0
NDQzMjVmMWU4NzEwYWU5ZGJhZWE0MyAxMDA3NTUKLS0tIGEvU291cmNlL1dlYkNvcmUvcGxhdGZv
cm0vbmV0d29yay9jcmVhdGUtaHR0cC1oZWFkZXItbmFtZS10YWJsZQorKysgYi9Tb3VyY2UvV2Vi
Q29yZS9wbGF0Zm9ybS9uZXR3b3JrL2NyZWF0ZS1odHRwLWhlYWRlci1uYW1lLXRhYmxlCkBAIC0y
OSw2ICsyOSw3IEBACiAKIGltcG9ydCBvcwogaW1wb3J0IHN5cworaW1wb3J0IHN1YnByb2Nlc3MK
IAogcHJvZ3JhbV9uYW1lID0gb3MucGF0aC5iYXNlbmFtZShfX2ZpbGVfXykKIGlmIGxlbihzeXMu
YXJndikgPCAyOgpAQCAtMjQwLDYgKzI0MSw2IEBAIGhlYWRlcl9maWxlLmNsb3NlKCkKIAogZ3Bl
cmYgPSBvcy5nZXRlbnYoJ0dQRVJGJykgb3Igc3lzLmFyZ3ZbMl0KIAotaWYgb3Muc3lzdGVtKCcl
cyAtLWtleS1wb3NpdGlvbnM9IioiIC1EIC1uIC1zIDIgSFRUUEhlYWRlck5hbWVzLmdwZXJmIC0t
b3V0cHV0LWZpbGU9SFRUUEhlYWRlck5hbWVzLmNwcCcgJSBncGVyZik6CitpZiBzdWJwcm9jZXNz
LmNhbGwoW2dwZXJmLCAnLS1rZXktcG9zaXRpb25zPSonLCAnLUQnLCAnLW4nLCAnLXMnLCAnMics
ICdIVFRQSGVhZGVyTmFtZXMuZ3BlcmYnLCAnLS1vdXRwdXQtZmlsZT1IVFRQSGVhZGVyTmFtZXMu
Y3BwJ10pICE9IDA6CiAgICAgc3lzLnN0ZGVyci53cml0ZSgnRmFpbGVkIHRvIHJ1biBncGVyZi4n
KQogICAgIGV4aXQoMSkK
</data>
<flag name="review"
          id="357756"
          type_id="1"
          status="+"
          setter="darin"
    />
          </attachment>
      

    </bug>

</bugzilla>