<?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>38225</bug_id>
          
          <creation_ts>2010-04-27 15:23:08 -0700</creation_ts>
          <short_desc>run-bindings-tests doesn&apos;t work in an SVN checkout</short_desc>
          <delta_ts>2010-04-27 21:49:54 -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>528+ (Nightly build)</version>
          <rep_platform>Other</rep_platform>
          <op_sys>OS X 10.5</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>
          
          
          <everconfirmed>1</everconfirmed>
          <reporter name="Adam Barth">abarth</reporter>
          <assigned_to name="Nobody">webkit-unassigned</assigned_to>
          <cc>commit-queue</cc>
    
    <cc>eric</cc>
    
    <cc>jamesr</cc>
    
    <cc>yaar</cc>
          

      

      

      

          <comment_sort_order>oldest_to_newest</comment_sort_order>  
          <long_desc isprivate="0" >
    <commentid>217990</commentid>
    <comment_count>0</comment_count>
    <who name="Adam Barth">abarth</who>
    <bug_when>2010-04-27 15:23:08 -0700</bug_when>
    <thetext>run-bindings-tests doesn&apos;t work in an SVN checkout</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>217991</commentid>
    <comment_count>1</comment_count>
      <attachid>54460</attachid>
    <who name="Adam Barth">abarth</who>
    <bug_when>2010-04-27 15:23:44 -0700</bug_when>
    <thetext>Created attachment 54460
Patch</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>217993</commentid>
    <comment_count>2</comment_count>
      <attachid>54460</attachid>
    <who name="Eric Seidel (no email)">eric</who>
    <bug_when>2010-04-27 15:25:50 -0700</bug_when>
    <thetext>Comment on attachment 54460
Patch

This seems like a bug in detect_scm_system and shoudl have a test.</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>218003</commentid>
    <comment_count>3</comment_count>
    <who name="Adam Barth">abarth</who>
    <bug_when>2010-04-27 15:42:04 -0700</bug_when>
    <thetext>Darn you for making me run test-webkitpy --all.  I&apos;m going to go read a book.  :)</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>218009</commentid>
    <comment_count>4</comment_count>
      <attachid>54464</attachid>
    <who name="Adam Barth">abarth</who>
    <bug_when>2010-04-27 15:51:43 -0700</bug_when>
    <thetext>Created attachment 54464
Patch</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>218012</commentid>
    <comment_count>5</comment_count>
      <attachid>54464</attachid>
    <who name="Eric Seidel (no email)">eric</who>
    <bug_when>2010-04-27 15:54:24 -0700</bug_when>
    <thetext>Comment on attachment 54464
Patch

You can probably get the tmp folder from getcwd() or from self._svn_repository or whatever it&apos;s set as.

I think I probably would have fixed SVN&apos;s usage instead of the caller of in_working_directory, since git&apos;s seems to take relative paths just fine.

typo:
+        # I wanted to asser that we got the right path, but there was some</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>218019</commentid>
    <comment_count>6</comment_count>
      <attachid>54467</attachid>
    <who name="Adam Barth">abarth</who>
    <bug_when>2010-04-27 16:06:11 -0700</bug_when>
    <thetext>Created attachment 54467
Patch for landing</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>218192</commentid>
    <comment_count>7</comment_count>
      <attachid>54467</attachid>
    <who name="WebKit Commit Bot">commit-queue</who>
    <bug_when>2010-04-27 21:49:49 -0700</bug_when>
    <thetext>Comment on attachment 54467
Patch for landing

Clearing flags on attachment: 54467

Committed r58376: &lt;http://trac.webkit.org/changeset/58376&gt;</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>218193</commentid>
    <comment_count>8</comment_count>
    <who name="WebKit Commit Bot">commit-queue</who>
    <bug_when>2010-04-27 21:49:54 -0700</bug_when>
    <thetext>All reviewed patches have been landed.  Closing bug.</thetext>
  </long_desc>
      
          <attachment
              isobsolete="1"
              ispatch="1"
              isprivate="0"
          >
            <attachid>54460</attachid>
            <date>2010-04-27 15:23:44 -0700</date>
            <delta_ts>2010-04-27 15:51:38 -0700</delta_ts>
            <desc>Patch</desc>
            <filename>bug-38225-20100427152343.patch</filename>
            <type>text/plain</type>
            <size>1105</size>
            <attacher name="Adam Barth">abarth</attacher>
            
              <data encoding="base64">SW5kZXg6IFdlYktpdFRvb2xzL0NoYW5nZUxvZwo9PT09PT09PT09PT09PT09PT09PT09PT09PT09
PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09Ci0tLSBXZWJLaXRUb29scy9D
aGFuZ2VMb2cJKHJldmlzaW9uIDU4MzQyKQorKysgV2ViS2l0VG9vbHMvQ2hhbmdlTG9nCSh3b3Jr
aW5nIGNvcHkpCkBAIC0xLDMgKzEsMTQgQEAKKzIwMTAtMDQtMjcgIEFkYW0gQmFydGggIDxhYmFy
dGhAd2Via2l0Lm9yZz4KKworICAgICAgICBSZXZpZXdlZCBieSBOT0JPRFkgKE9PUFMhKS4KKwor
ICAgICAgICBydW4tYmluZGluZ3MtdGVzdHMgZG9lc24ndCB3b3JrIGluIGFuIFNWTiBjaGVja291
dAorICAgICAgICBodHRwczovL2J1Z3Mud2Via2l0Lm9yZy9zaG93X2J1Zy5jZ2k/aWQ9MzgyMjUK
KworICAgICAgICBUdXJucyBvdXQgZGV0ZWN0X3NjbV9zeXN0ZW0gbmVlZHMgYW4gYWJzb2x1dGUg
cGF0aC4KKworICAgICAgICAqIFNjcmlwdHMvcnVuLWJpbmRpbmdzLXRlc3RzOgorCiAyMDEwLTA0
LTI3ICBBZGFtIEJhcnRoICA8YWJhcnRoQHdlYmtpdC5vcmc+CiAKICAgICAgICAgUmV2aWV3ZWQg
YnkgRGFyaW4gQWRsZXIuCkluZGV4OiBXZWJLaXRUb29scy9TY3JpcHRzL3J1bi1iaW5kaW5ncy10
ZXN0cwo9PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09
PT09PT09PT09PT09PT09Ci0tLSBXZWJLaXRUb29scy9TY3JpcHRzL3J1bi1iaW5kaW5ncy10ZXN0
cwkocmV2aXNpb24gNTgzNDIpCisrKyBXZWJLaXRUb29scy9TY3JpcHRzL3J1bi1iaW5kaW5ncy10
ZXN0cwkod29ya2luZyBjb3B5KQpAQCAtMTA2LDcgKzEwNiw3IEBAIGRlZiBtYWluKGFyZ3YpOgog
ICAgICIiIgogICAgIHJlc2V0X3Jlc3VsdHMgPSAiLS1yZXNldC1yZXN1bHRzIiBpbiBhcmd2CiAK
LSAgICBjdXJyZW50X3NjbSA9IHNjbS5kZXRlY3Rfc2NtX3N5c3RlbShvcy5jdXJkaXIpCisgICAg
Y3VycmVudF9zY20gPSBzY20uZGV0ZWN0X3NjbV9zeXN0ZW0ob3MucGF0aC5hYnNwYXRoKG9zLmN1
cmRpcikpCiAgICAgb3MuY2hkaXIoY3VycmVudF9zY20uY2hlY2tvdXRfcm9vdCkKIAogICAgIGFs
bF90ZXN0c19wYXNzZWQgPSBUcnVlCg==
</data>

          </attachment>
          <attachment
              isobsolete="1"
              ispatch="1"
              isprivate="0"
          >
            <attachid>54464</attachid>
            <date>2010-04-27 15:51:43 -0700</date>
            <delta_ts>2010-04-27 16:06:04 -0700</delta_ts>
            <desc>Patch</desc>
            <filename>bug-38225-20100427155141.patch</filename>
            <type>text/plain</type>
            <size>2324</size>
            <attacher name="Adam Barth">abarth</attacher>
            
              <data encoding="base64">SW5kZXg6IFdlYktpdFRvb2xzL0NoYW5nZUxvZwo9PT09PT09PT09PT09PT09PT09PT09PT09PT09
PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09Ci0tLSBXZWJLaXRUb29scy9D
aGFuZ2VMb2cJKHJldmlzaW9uIDU4MzQ0KQorKysgV2ViS2l0VG9vbHMvQ2hhbmdlTG9nCSh3b3Jr
aW5nIGNvcHkpCkBAIC0xLDMgKzEsMTYgQEAKKzIwMTAtMDQtMjcgIEFkYW0gQmFydGggIDxhYmFy
dGhAd2Via2l0Lm9yZz4KKworICAgICAgICBSZXZpZXdlZCBieSBOT0JPRFkgKE9PUFMhKS4KKwor
ICAgICAgICBydW4tYmluZGluZ3MtdGVzdHMgZG9lc24ndCB3b3JrIGluIGFuIFNWTiBjaGVja291
dAorICAgICAgICBodHRwczovL2J1Z3Mud2Via2l0Lm9yZy9zaG93X2J1Zy5jZ2k/aWQ9MzgyMjUK
KworICAgICAgICBQcmV2aW91c2x5IGRldGVjdF9zY21fc3lzdGVtIG5lZWRlZCBhbiBhYnNvbHV0
ZSBwYXRoIGZvciBTVk4uICBOb3cgd2UKKyAgICAgICAgYWNjZXB0IGEgcmVsYXRpdmUgcGF0aC4K
KworICAgICAgICAqIFNjcmlwdHMvd2Via2l0cHkvY29tbW9uL2NoZWNrb3V0L3NjbS5weToKKyAg
ICAgICAgKiBTY3JpcHRzL3dlYmtpdHB5L2NvbW1vbi9jaGVja291dC9zY21fdW5pdHRlc3QucHk6
CisKIDIwMTAtMDQtMjcgIEFkYW0gQmFydGggIDxhYmFydGhAd2Via2l0Lm9yZz4KIAogICAgICAg
ICBSZXZpZXdlZCBieSBEYXJpbiBBZGxlci4KSW5kZXg6IFdlYktpdFRvb2xzL1NjcmlwdHMvd2Vi
a2l0cHkvY29tbW9uL2NoZWNrb3V0L3NjbS5weQo9PT09PT09PT09PT09PT09PT09PT09PT09PT09
PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09Ci0tLSBXZWJLaXRUb29scy9T
Y3JpcHRzL3dlYmtpdHB5L2NvbW1vbi9jaGVja291dC9zY20ucHkJKHJldmlzaW9uIDU4MzQyKQor
KysgV2ViS2l0VG9vbHMvU2NyaXB0cy93ZWJraXRweS9jb21tb24vY2hlY2tvdXQvc2NtLnB5CSh3
b3JraW5nIGNvcHkpCkBAIC00MSwxMSArNDEsMTMgQEAgZnJvbSB3ZWJraXRweS5jb21tb24uc3lz
dGVtLmRlcHJlY2F0ZWRfbAogCiAKIGRlZiBkZXRlY3Rfc2NtX3N5c3RlbShwYXRoKToKLSAgICBp
ZiBTVk4uaW5fd29ya2luZ19kaXJlY3RvcnkocGF0aCk6Ci0gICAgICAgIHJldHVybiBTVk4oY3dk
PXBhdGgpCisgICAgYWJzb2x1dGVfcGF0aCA9IG9zLnBhdGguYWJzcGF0aChwYXRoKQorCisgICAg
aWYgU1ZOLmluX3dvcmtpbmdfZGlyZWN0b3J5KGFic29sdXRlX3BhdGgpOgorICAgICAgICByZXR1
cm4gU1ZOKGN3ZD1hYnNvbHV0ZV9wYXRoKQogICAgIAotICAgIGlmIEdpdC5pbl93b3JraW5nX2Rp
cmVjdG9yeShwYXRoKToKLSAgICAgICAgcmV0dXJuIEdpdChjd2Q9cGF0aCkKKyAgICBpZiBHaXQu
aW5fd29ya2luZ19kaXJlY3RvcnkoYWJzb2x1dGVfcGF0aCk6CisgICAgICAgIHJldHVybiBHaXQo
Y3dkPWFic29sdXRlX3BhdGgpCiAgICAgCiAgICAgcmV0dXJuIE5vbmUKIApJbmRleDogV2ViS2l0
VG9vbHMvU2NyaXB0cy93ZWJraXRweS9jb21tb24vY2hlY2tvdXQvc2NtX3VuaXR0ZXN0LnB5Cj09
PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09
PT09PT09PT0KLS0tIFdlYktpdFRvb2xzL1NjcmlwdHMvd2Via2l0cHkvY29tbW9uL2NoZWNrb3V0
L3NjbV91bml0dGVzdC5weQkocmV2aXNpb24gNTgzNDIpCisrKyBXZWJLaXRUb29scy9TY3JpcHRz
L3dlYmtpdHB5L2NvbW1vbi9jaGVja291dC9zY21fdW5pdHRlc3QucHkJKHdvcmtpbmcgY29weSkK
QEAgLTQ4Nyw2ICs0ODcsMTIgQEAgY2xhc3MgU1ZOVGVzdChTQ01UZXN0KToKICAgICBkZWYgdGVh
ckRvd24oc2VsZik6CiAgICAgICAgIFNWTlRlc3RSZXBvc2l0b3J5LnRlYXJfZG93bihzZWxmKQog
CisgICAgZGVmIHRlc3RfZGV0ZWN0X3NjbV9zeXN0ZW1fcmVsYXRpdmVfdXJsKHNlbGYpOgorICAg
ICAgICBzY20gPSBkZXRlY3Rfc2NtX3N5c3RlbSgiLiIpCisgICAgICAgICMgSSB3YW50ZWQgdG8g
YXNzZXIgdGhhdCB3ZSBnb3QgdGhlIHJpZ2h0IHBhdGgsIGJ1dCB0aGVyZSB3YXMgc29tZQorICAg
ICAgICAjIGNyYXp5IG1hZ2ljIHdpdGggdGVtcCBmb2xkZXIgbmFtZXMgdGhhdCBJIGNvdWxkbid0
IGZpZ3VyZSBvdXQuCisgICAgICAgIHNlbGYuYXNzZXJ0VHJ1ZShzY20uY2hlY2tvdXRfcm9vdCkK
KwogICAgIGRlZiB0ZXN0X2NyZWF0ZV9wYXRjaF9pc19mdWxsX3BhdGNoKHNlbGYpOgogICAgICAg
ICB0ZXN0X2Rpcl9wYXRoID0gb3MucGF0aC5qb2luKHNlbGYuc3ZuX2NoZWNrb3V0X3BhdGgsICJ0
ZXN0X2RpcjIiKQogICAgICAgICBvcy5ta2Rpcih0ZXN0X2Rpcl9wYXRoKQo=
</data>

          </attachment>
          <attachment
              isobsolete="0"
              ispatch="1"
              isprivate="0"
          >
            <attachid>54467</attachid>
            <date>2010-04-27 16:06:11 -0700</date>
            <delta_ts>2010-04-27 21:49:49 -0700</delta_ts>
            <desc>Patch for landing</desc>
            <filename>bug-38225-20100427160610.patch</filename>
            <type>text/plain</type>
            <size>2322</size>
            <attacher name="Adam Barth">abarth</attacher>
            
              <data encoding="base64">SW5kZXg6IFdlYktpdFRvb2xzL0NoYW5nZUxvZwo9PT09PT09PT09PT09PT09PT09PT09PT09PT09
PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09Ci0tLSBXZWJLaXRUb29scy9D
aGFuZ2VMb2cJKHJldmlzaW9uIDU4MzQ0KQorKysgV2ViS2l0VG9vbHMvQ2hhbmdlTG9nCSh3b3Jr
aW5nIGNvcHkpCkBAIC0xLDMgKzEsMTYgQEAKKzIwMTAtMDQtMjcgIEFkYW0gQmFydGggIDxhYmFy
dGhAd2Via2l0Lm9yZz4KKworICAgICAgICBSZXZpZXdlZCBieSBFcmljIFNlaWRlbC4KKworICAg
ICAgICBydW4tYmluZGluZ3MtdGVzdHMgZG9lc24ndCB3b3JrIGluIGFuIFNWTiBjaGVja291dAor
ICAgICAgICBodHRwczovL2J1Z3Mud2Via2l0Lm9yZy9zaG93X2J1Zy5jZ2k/aWQ9MzgyMjUKKwor
ICAgICAgICBQcmV2aW91c2x5IGRldGVjdF9zY21fc3lzdGVtIG5lZWRlZCBhbiBhYnNvbHV0ZSBw
YXRoIGZvciBTVk4uICBOb3cgd2UKKyAgICAgICAgYWNjZXB0IGEgcmVsYXRpdmUgcGF0aC4KKwor
ICAgICAgICAqIFNjcmlwdHMvd2Via2l0cHkvY29tbW9uL2NoZWNrb3V0L3NjbS5weToKKyAgICAg
ICAgKiBTY3JpcHRzL3dlYmtpdHB5L2NvbW1vbi9jaGVja291dC9zY21fdW5pdHRlc3QucHk6CisK
IDIwMTAtMDQtMjcgIEFkYW0gQmFydGggIDxhYmFydGhAd2Via2l0Lm9yZz4KIAogICAgICAgICBS
ZXZpZXdlZCBieSBEYXJpbiBBZGxlci4KSW5kZXg6IFdlYktpdFRvb2xzL1NjcmlwdHMvd2Via2l0
cHkvY29tbW9uL2NoZWNrb3V0L3NjbS5weQo9PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09
PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09Ci0tLSBXZWJLaXRUb29scy9TY3Jp
cHRzL3dlYmtpdHB5L2NvbW1vbi9jaGVja291dC9zY20ucHkJKHJldmlzaW9uIDU4MzQyKQorKysg
V2ViS2l0VG9vbHMvU2NyaXB0cy93ZWJraXRweS9jb21tb24vY2hlY2tvdXQvc2NtLnB5CSh3b3Jr
aW5nIGNvcHkpCkBAIC00MSwxMSArNDEsMTMgQEAgZnJvbSB3ZWJraXRweS5jb21tb24uc3lzdGVt
LmRlcHJlY2F0ZWRfbAogCiAKIGRlZiBkZXRlY3Rfc2NtX3N5c3RlbShwYXRoKToKLSAgICBpZiBT
Vk4uaW5fd29ya2luZ19kaXJlY3RvcnkocGF0aCk6Ci0gICAgICAgIHJldHVybiBTVk4oY3dkPXBh
dGgpCisgICAgYWJzb2x1dGVfcGF0aCA9IG9zLnBhdGguYWJzcGF0aChwYXRoKQorCisgICAgaWYg
U1ZOLmluX3dvcmtpbmdfZGlyZWN0b3J5KGFic29sdXRlX3BhdGgpOgorICAgICAgICByZXR1cm4g
U1ZOKGN3ZD1hYnNvbHV0ZV9wYXRoKQogICAgIAotICAgIGlmIEdpdC5pbl93b3JraW5nX2RpcmVj
dG9yeShwYXRoKToKLSAgICAgICAgcmV0dXJuIEdpdChjd2Q9cGF0aCkKKyAgICBpZiBHaXQuaW5f
d29ya2luZ19kaXJlY3RvcnkoYWJzb2x1dGVfcGF0aCk6CisgICAgICAgIHJldHVybiBHaXQoY3dk
PWFic29sdXRlX3BhdGgpCiAgICAgCiAgICAgcmV0dXJuIE5vbmUKIApJbmRleDogV2ViS2l0VG9v
bHMvU2NyaXB0cy93ZWJraXRweS9jb21tb24vY2hlY2tvdXQvc2NtX3VuaXR0ZXN0LnB5Cj09PT09
PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09
PT09PT0KLS0tIFdlYktpdFRvb2xzL1NjcmlwdHMvd2Via2l0cHkvY29tbW9uL2NoZWNrb3V0L3Nj
bV91bml0dGVzdC5weQkocmV2aXNpb24gNTgzNDIpCisrKyBXZWJLaXRUb29scy9TY3JpcHRzL3dl
YmtpdHB5L2NvbW1vbi9jaGVja291dC9zY21fdW5pdHRlc3QucHkJKHdvcmtpbmcgY29weSkKQEAg
LTQ4Nyw2ICs0ODcsMTIgQEAgY2xhc3MgU1ZOVGVzdChTQ01UZXN0KToKICAgICBkZWYgdGVhckRv
d24oc2VsZik6CiAgICAgICAgIFNWTlRlc3RSZXBvc2l0b3J5LnRlYXJfZG93bihzZWxmKQogCisg
ICAgZGVmIHRlc3RfZGV0ZWN0X3NjbV9zeXN0ZW1fcmVsYXRpdmVfdXJsKHNlbGYpOgorICAgICAg
ICBzY20gPSBkZXRlY3Rfc2NtX3N5c3RlbSgiLiIpCisgICAgICAgICMgSSB3YW50ZWQgdG8gYXNz
ZXJ0IHRoYXQgd2UgZ290IHRoZSByaWdodCBwYXRoLCBidXQgdGhlcmUgd2FzIHNvbWUKKyAgICAg
ICAgIyBjcmF6eSBtYWdpYyB3aXRoIHRlbXAgZm9sZGVyIG5hbWVzIHRoYXQgSSBjb3VsZG4ndCBm
aWd1cmUgb3V0LgorICAgICAgICBzZWxmLmFzc2VydFRydWUoc2NtLmNoZWNrb3V0X3Jvb3QpCisK
ICAgICBkZWYgdGVzdF9jcmVhdGVfcGF0Y2hfaXNfZnVsbF9wYXRjaChzZWxmKToKICAgICAgICAg
dGVzdF9kaXJfcGF0aCA9IG9zLnBhdGguam9pbihzZWxmLnN2bl9jaGVja291dF9wYXRoLCAidGVz
dF9kaXIyIikKICAgICAgICAgb3MubWtkaXIodGVzdF9kaXJfcGF0aCkK
</data>

          </attachment>
      

    </bug>

</bugzilla>