<?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>61250</bug_id>
          
          <creation_ts>2011-05-22 09:27:52 -0700</creation_ts>
          <short_desc>Use double quotes for filename arguments</short_desc>
          <delta_ts>2011-05-22 11:08:19 -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>Tools / Tests</component>
          <version>528+ (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></keywords>
          <priority>P2</priority>
          <bug_severity>Normal</bug_severity>
          <target_milestone>---</target_milestone>
          
          <blocked>55811</blocked>
          <everconfirmed>1</everconfirmed>
          <reporter name="Patrick R. Gansterer">paroga</reporter>
          <assigned_to name="Patrick R. Gansterer">paroga</assigned_to>
          <cc>abarth</cc>
    
    <cc>commit-queue</cc>
          

      

      

      

          <comment_sort_order>oldest_to_newest</comment_sort_order>  
          <long_desc isprivate="0" >
    <commentid>407651</commentid>
    <comment_count>0</comment_count>
    <who name="Patrick R. Gansterer">paroga</who>
    <bug_when>2011-05-22 09:27:52 -0700</bug_when>
    <thetext>Use double quotes for filename arguments</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>407657</commentid>
    <comment_count>1</comment_count>
      <attachid>94346</attachid>
    <who name="Patrick R. Gansterer">paroga</who>
    <bug_when>2011-05-22 09:31:21 -0700</bug_when>
    <thetext>Created attachment 94346
Patch</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>407659</commentid>
    <comment_count>2</comment_count>
      <attachid>94346</attachid>
    <who name="Adam Barth">abarth</who>
    <bug_when>2011-05-22 09:34:49 -0700</bug_when>
    <thetext>Comment on attachment 94346
Patch

Test?</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>407660</commentid>
    <comment_count>3</comment_count>
    <who name="Patrick R. Gansterer">paroga</who>
    <bug_when>2011-05-22 09:38:57 -0700</bug_when>
    <thetext>(In reply to comment #2)
&gt; (From update of attachment 94346 [details])
&gt; Test?

How? Are there any tests for the non VCSUtils perl scripts?</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>407673</commentid>
    <comment_count>4</comment_count>
    <who name="Adam Barth">abarth</who>
    <bug_when>2011-05-22 10:26:18 -0700</bug_when>
    <thetext>&gt; How? Are there any tests for the non VCSUtils perl scripts?

Dunno.  Maybe we need to add more infrastructure to test-webkitperl ?  Someone else might be willing to r+ your patch without tests, but, personally, I think we should have tests for all this stuff.</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>407674</commentid>
    <comment_count>5</comment_count>
    <who name="Patrick R. Gansterer">paroga</who>
    <bug_when>2011-05-22 10:28:32 -0700</bug_when>
    <thetext>(In reply to comment #4)
&gt; &gt; How? Are there any tests for the non VCSUtils perl scripts?
&gt; 
&gt; Dunno.  Maybe we need to add more infrastructure to test-webkitperl ?  Someone else might be willing to r+ your patch without tests, but, personally, I think we should have tests for all this stuff.

I&apos;m not against the tests, but I don&apos;t want to write the whole test infrastructure. :-/</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>407688</commentid>
    <comment_count>6</comment_count>
      <attachid>94346</attachid>
    <who name="WebKit Commit Bot">commit-queue</who>
    <bug_when>2011-05-22 11:08:14 -0700</bug_when>
    <thetext>Comment on attachment 94346
Patch

Clearing flags on attachment: 94346

Committed r87040: &lt;http://trac.webkit.org/changeset/87040&gt;</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>407689</commentid>
    <comment_count>7</comment_count>
    <who name="WebKit Commit Bot">commit-queue</who>
    <bug_when>2011-05-22 11:08:19 -0700</bug_when>
    <thetext>All reviewed patches have been landed.  Closing bug.</thetext>
  </long_desc>
      
          <attachment
              isobsolete="0"
              ispatch="1"
              isprivate="0"
          >
            <attachid>94346</attachid>
            <date>2011-05-22 09:31:21 -0700</date>
            <delta_ts>2011-05-22 11:08:13 -0700</delta_ts>
            <desc>Patch</desc>
            <filename>bug-61250-20110522183119.patch</filename>
            <type>text/plain</type>
            <size>1216</size>
            <attacher name="Patrick R. Gansterer">paroga</attacher>
            
              <data encoding="base64">U3VidmVyc2lvbiBSZXZpc2lvbjogODY1NTIKZGlmZiAtLWdpdCBhL1Rvb2xzL0NoYW5nZUxvZyBi
L1Rvb2xzL0NoYW5nZUxvZwppbmRleCAxN2UxODU1MzZhZWU0MTkyNWMwYzZiN2NlZmUxMzJhZThk
YTRlZWNkLi4wMmI3MDlhYWNkMWFjNzAwMzEyZmZlZDBmYWZlMWViMDFmMWIwMjkyIDEwMDY0NAot
LS0gYS9Ub29scy9DaGFuZ2VMb2cKKysrIGIvVG9vbHMvQ2hhbmdlTG9nCkBAIC0xLDMgKzEsMTUg
QEAKKzIwMTEtMDUtMjIgIFBhdHJpY2sgR2Fuc3RlcmVyICA8cGFyb2dhQHdlYmtpdC5vcmc+CisK
KyAgICAgICAgUmV2aWV3ZWQgYnkgTk9CT0RZIChPT1BTISkuCisKKyAgICAgICAgVXNlIGRvdWJs
ZSBxdW90ZXMgZm9yIGZpbGVuYW1lIGFyZ3VtZW50cworICAgICAgICBodHRwczovL2J1Z3Mud2Vi
a2l0Lm9yZy9zaG93X2J1Zy5jZ2k/aWQ9NjEyNTAKKworICAgICAgICBXaW5kb3dzIGRvZXMgbm90
IHN1cHBvcnQgc2luZ2xlIHF1b3RlcyBvbiB0aGUgY29tbWFuZCBsaW5lLAorICAgICAgICBzbyBy
ZXBsYWNlIHRoZW0gd2l0aCBkb3VibGUgcXVvdGVzLgorCisgICAgICAgICogU2NyaXB0cy9wcmVw
YXJlLUNoYW5nZUxvZzoKKwogMjAxMS0wNS0xNSAgRG9taW5pYyBDb29uZXkgIDxkb21pbmljY0Bj
aHJvbWl1bS5vcmc+CiAKICAgICAgICAgVW5yZXZpZXdlZDogYWRkaW5nIG15c2VsZiB0byBjb21t
aXR0ZXJzLnB5LgpkaWZmIC0tZ2l0IGEvVG9vbHMvU2NyaXB0cy9wcmVwYXJlLUNoYW5nZUxvZyBi
L1Rvb2xzL1NjcmlwdHMvcHJlcGFyZS1DaGFuZ2VMb2cKaW5kZXggZTZmZWY0MDhhN2JhMGQ0ZmJh
OWYyMDUyNmMyZWVhNTlmZWI1ZjliNi4uM2I2NWZjMmFjN2VmN2JiODk3Yjk2M2M5NTRlOWQ2MjYy
NjkxZTk1MiAxMDA3NTUKLS0tIGEvVG9vbHMvU2NyaXB0cy9wcmVwYXJlLUNoYW5nZUxvZworKysg
Yi9Ub29scy9TY3JpcHRzL3ByZXBhcmUtQ2hhbmdlTG9nCkBAIC0xMzAzLDcgKzEzMDMsNyBAQCBz
dWIgc3RhdHVzQ29tbWFuZChAKQogewogICAgIG15IEBmaWxlcyA9IEBfOwogCi0gICAgbXkgJGZp
bGVzU3RyaW5nID0gIiciIC4gam9pbiAoIicgJyIsIEBmaWxlcykgLiAiJyI7CisgICAgbXkgJGZp
bGVzU3RyaW5nID0gIlwiIiAuIGpvaW4gKCJcIiBcIiIsIEBmaWxlcykgLiAiXCIiOwogICAgIG15
ICRjb21tYW5kOwogICAgIGlmICgkaXNTVk4pIHsKICAgICAgICAgJGNvbW1hbmQgPSAiJFNWTiBz
dGF0ICRmaWxlc1N0cmluZyI7Cg==
</data>

          </attachment>
      

    </bug>

</bugzilla>