<?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>50715</bug_id>
          
          <creation_ts>2010-12-08 14:03:39 -0800</creation_ts>
          <short_desc>webkit-patch should warn users when they&apos;re using a 32-bit git on a 64-bit system</short_desc>
          <delta_ts>2010-12-14 01:44:08 -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>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="Eric Seidel (no email)">eric</reporter>
          <assigned_to name="Eric Seidel (no email)">eric</assigned_to>
          <cc>abarth</cc>
    
    <cc>commit-queue</cc>
    
    <cc>eric</cc>
    
    <cc>kling</cc>
    
    <cc>mihaip</cc>
    
    <cc>ojan</cc>
          

      

      

      

          <comment_sort_order>oldest_to_newest</comment_sort_order>  
          <long_desc isprivate="0" >
    <commentid>319014</commentid>
    <comment_count>0</comment_count>
    <who name="Eric Seidel (no email)">eric</who>
    <bug_when>2010-12-08 14:03:39 -0800</bug_when>
    <thetext>webkit-patch should warn users when they&apos;re using a 32-bit git on a 64-bit system</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>319026</commentid>
    <comment_count>1</comment_count>
      <attachid>75960</attachid>
    <who name="Eric Seidel (no email)">eric</who>
    <bug_when>2010-12-08 14:17:33 -0800</bug_when>
    <thetext>Created attachment 75960
Patch</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>319033</commentid>
    <comment_count>2</comment_count>
      <attachid>75960</attachid>
    <who name="Eric Seidel (no email)">eric</who>
    <bug_when>2010-12-08 14:22:52 -0800</bug_when>
    <thetext>Comment on attachment 75960
Patch

Hmm... Nevermind, platform.architecture() is not working as expected.  Investigating.</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>319060</commentid>
    <comment_count>3</comment_count>
      <attachid>75974</attachid>
    <who name="Eric Seidel (no email)">eric</who>
    <bug_when>2010-12-08 14:55:23 -0800</bug_when>
    <thetext>Created attachment 75974
Patch</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>319071</commentid>
    <comment_count>4</comment_count>
      <attachid>75977</attachid>
    <who name="Eric Seidel (no email)">eric</who>
    <bug_when>2010-12-08 15:13:06 -0800</bug_when>
    <thetext>Created attachment 75977
move _check_git_architecture to after SCM.__init__</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>319076</commentid>
    <comment_count>5</comment_count>
      <attachid>75977</attachid>
    <who name="Mihai Parparita">mihaip</who>
    <bug_when>2010-12-08 15:22:46 -0800</bug_when>
    <thetext>Comment on attachment 75977
move _check_git_architecture to after SCM.__init__

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

&gt; WebKitTools/ChangeLog:19
&gt; +        This message will print twice during &apos;webkit-patch upload&apos;,

You could keep track of whether or not you printed out the message in a static, so that it only shows up once.</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>319090</commentid>
    <comment_count>6</comment_count>
    <who name="Eric Seidel (no email)">eric</who>
    <bug_when>2010-12-08 15:46:32 -0800</bug_when>
    <thetext>(In reply to comment #5)
&gt; (From update of attachment 75977 [details])
&gt; View in context: https://bugs.webkit.org/attachment.cgi?id=75977&amp;action=review
&gt; 
&gt; &gt; WebKitTools/ChangeLog:19
&gt; &gt; +        This message will print twice during &apos;webkit-patch upload&apos;,
&gt; 
&gt; You could keep track of whether or not you printed out the message in a static, so that it only shows up once.

webkit-patch calls out to check-webkit-style (a separate process).</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>319523</commentid>
    <comment_count>7</comment_count>
      <attachid>75977</attachid>
    <who name="Ojan Vafai">ojan</who>
    <bug_when>2010-12-09 09:53:06 -0800</bug_when>
    <thetext>Comment on attachment 75977
move _check_git_architecture to after SCM.__init__

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

&gt; WebKitTools/ChangeLog:23
&gt; +        the code is.  I considered writing a test for the log message, but it seemed not worth it.

I think testing _check_git_architecture is worthwhile just to ensure that the code actually runs. Don&apos;t even need to verify the output. It helps avoid simple programming errors (e.g. typos).</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>319580</commentid>
    <comment_count>8</comment_count>
    <who name="Eric Seidel (no email)">eric</who>
    <bug_when>2010-12-09 10:49:46 -0800</bug_when>
    <thetext>(In reply to comment #7)
&gt; (From update of attachment 75977 [details])
&gt; View in context: https://bugs.webkit.org/attachment.cgi?id=75977&amp;action=review
&gt; 
&gt; &gt; WebKitTools/ChangeLog:23
&gt; &gt; +        the code is.  I considered writing a test for the log message, but it seemed not worth it.
&gt; 
&gt; I think testing _check_git_architecture is worthwhile just to ensure that the code actually runs. Don&apos;t even need to verify the output. It helps avoid simple programming errors (e.g. typos).

Possible, however having git installed is not required for WebKit.  So I could check that it throws an OSError or returns a value?</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>321762</commentid>
    <comment_count>9</comment_count>
    <who name="WebKit Commit Bot">commit-queue</who>
    <bug_when>2010-12-14 01:26:10 -0800</bug_when>
    <thetext>The commit-queue encountered the following flaky tests while processing attachment 75977:

fast/dom/onerror-img.html bug 51019
fast/preloader/script.html bug 50879 (author: abarth@webkit.org)
The commit-queue is continuing to process your patch.</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>321802</commentid>
    <comment_count>10</comment_count>
    <who name="WebKit Commit Bot">commit-queue</who>
    <bug_when>2010-12-14 01:42:48 -0800</bug_when>
    <thetext>The commit-queue encountered the following flaky tests while processing attachment 75977:

http/tests/xmlhttprequest/cross-site-denied-response-sync.html bug 51020 (author: ap@webkit.org)
The commit-queue is continuing to process your patch.</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>321806</commentid>
    <comment_count>11</comment_count>
      <attachid>75977</attachid>
    <who name="WebKit Commit Bot">commit-queue</who>
    <bug_when>2010-12-14 01:44:01 -0800</bug_when>
    <thetext>Comment on attachment 75977
move _check_git_architecture to after SCM.__init__

Clearing flags on attachment: 75977

Committed r74008: &lt;http://trac.webkit.org/changeset/74008&gt;</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>321808</commentid>
    <comment_count>12</comment_count>
    <who name="WebKit Commit Bot">commit-queue</who>
    <bug_when>2010-12-14 01:44:08 -0800</bug_when>
    <thetext>All reviewed patches have been landed.  Closing bug.</thetext>
  </long_desc>
      
          <attachment
              isobsolete="1"
              ispatch="1"
              isprivate="0"
          >
            <attachid>75960</attachid>
            <date>2010-12-08 14:17:33 -0800</date>
            <delta_ts>2010-12-08 14:55:17 -0800</delta_ts>
            <desc>Patch</desc>
            <filename>bug-50715-20101208141731.patch</filename>
            <type>text/plain</type>
            <size>3286</size>
            <attacher name="Eric Seidel (no email)">eric</attacher>
            
              <data encoding="base64">ZGlmZiAtLWdpdCBhL1dlYktpdFRvb2xzL0NoYW5nZUxvZyBiL1dlYktpdFRvb2xzL0NoYW5nZUxv
ZwppbmRleCA2ZWQ1YWM3MmUxNDg2OTE3MDE0YzdlZDg5Nzg1MmI2ZGIyYTYzYTJjLi4zMDRmYWVj
ZWFiOTU2ZDEyZmY4ODE1MjRlNTNjYjA5MDU3YjMwOThkIDEwMDY0NAotLS0gYS9XZWJLaXRUb29s
cy9DaGFuZ2VMb2cKKysrIGIvV2ViS2l0VG9vbHMvQ2hhbmdlTG9nCkBAIC0xLDMgKzEsMjYgQEAK
KzIwMTAtMTItMDggIEVyaWMgU2VpZGVsICA8ZXJpY0B3ZWJraXQub3JnPgorCisgICAgICAgIFJl
dmlld2VkIGJ5IE5PQk9EWSAoT09QUyEpLgorCisgICAgICAgIHdlYmtpdC1wYXRjaCBzaG91bGQg
d2FybiB1c2VycyB3aGVuIHRoZXkncmUgdXNpbmcgYSAzMi1iaXQgZ2l0IG9uIGEgNjQtYml0IHN5
c3RlbQorICAgICAgICBodHRwczovL2J1Z3Mud2Via2l0Lm9yZy9zaG93X2J1Zy5jZ2k/aWQ9NTA3
MTUKKworICAgICAgICBUaGlzIHBhdGNoIG1ha2VzIHdlYmtpdC1wYXRjaCBwcmludCB0aGUgZm9s
bG93aW5nOgorCisgICAgICAgIFdhcm5pbmc6IFRoaXMgbWFjaGluZSBpcyA2NC1iaXQsIGJ1dCB0
aGUgZ2l0IGJpbmFyeSAoL3Vzci9sb2NhbC9naXQvYmluL2dpdCkgZG9lcyBub3Qgc3VwcG9ydCA2
NC1iaXQuCisgICAgICAgIEluc3RhbGwgYSA2NC1iaXQgZ2l0IGZvciBiZXR0ZXIgcGVyZm9ybWFu
Y2UsIHNlZToKKyAgICAgICAgaHR0cHM6Ly9saXN0cy53ZWJraXQub3JnL3BpcGVybWFpbC93ZWJr
aXQtZGV2LzIwMTAtRGVjZW1iZXIvMDE1MjQ5Lmh0bWwKKworICAgICAgICBJIHdyb3RlIHRoaXMg
bW9zdGx5IGJlY2F1c2UgSSBoYXZlIGxpa2UgOCBtYWNoaW5lcyB0aGF0IEkgdXNlCisgICAgICAg
IGFuZCBtYWtpbmcgc3VyZSBlYWNoIG9uZSBpcyB1c2luZyBhIGdvb2QgR2l0IGluc3RhbGwgc2Vl
bWVkIGZvbGx5LgorICAgICAgICB3ZWJraXQtcGF0Y2ggbWFrZXMgYSBsb3Qgb2YgZ2l0IGNhbGxz
LCBzbyB1c2luZyBhIGZhc3QgZ2l0IGNhbiBzaGF2ZQorICAgICAgICBzZXZlcmFsIHNlY29uZHMg
aW4gZXZlcnkgaW52b2NhdGlvbi4gIFNlZSB0aGUgd2Via2l0LWRldiB0aHJlYWQgZm9yIG1vcmUg
aW5mby4KKworICAgICAgICBVbmZvcnR1bmF0ZWx5IHRoZXJlIGlzIG5vIGdvb2Qgd2F5IHRvIHRl
c3QgdGhpcyBkdWUgdG8gaG93IG1hY2hpbmUtZGVwZW5kYW50CisgICAgICAgIHRoZSBjb2RlIGlz
LiAgSSBjb25zaWRlcmVkIHdyaXRpbmcgYSB0ZXN0IGZvciB0aGUgbG9nIG1lc3NhZ2UsIGJ1dCBp
dCBzZWVtZWQgbm90IHdvcnRoIGl0LgorCisgICAgICAgICogU2NyaXB0cy93ZWJraXRweS9jb21t
b24vY2hlY2tvdXQvc2NtLnB5OgorCiAyMDEwLTEyLTA4ICBKZXNzaWUgQmVybGluICA8amJlcmxp
bkBhcHBsZS5jb20+CiAKICAgICAgICAgUmV2aWV3ZWQgYnkgQWRhbSBSb2Jlbi4KZGlmZiAtLWdp
dCBhL1dlYktpdFRvb2xzL1NjcmlwdHMvd2Via2l0cHkvY29tbW9uL2NoZWNrb3V0L3NjbS5weSBi
L1dlYktpdFRvb2xzL1NjcmlwdHMvd2Via2l0cHkvY29tbW9uL2NoZWNrb3V0L3NjbS5weQppbmRl
eCBlZTJiYjQxMmJhMDU1Y2FiZDIyMjFjNmYxNWJiNjNmODliMjdhMDFmLi4wMTBmNGU2MzczMzNi
ODZkN2IyN2MwNDlhYmI3MjMxMWM2MmNjODM5IDEwMDY0NAotLS0gYS9XZWJLaXRUb29scy9TY3Jp
cHRzL3dlYmtpdHB5L2NvbW1vbi9jaGVja291dC9zY20ucHkKKysrIGIvV2ViS2l0VG9vbHMvU2Ny
aXB0cy93ZWJraXRweS9jb21tb24vY2hlY2tvdXQvc2NtLnB5CkBAIC01NzQsOCArNTc0LDMwIEBA
IGNsYXNzIFNWTihTQ00pOgogIyBBbGwgZ2l0LXNwZWNpZmljIGxvZ2ljIHNob3VsZCBnbyBoZXJl
LgogY2xhc3MgR2l0KFNDTSk6CiAgICAgZGVmIF9faW5pdF9fKHNlbGYsIGN3ZCk6CisgICAgICAg
IHNlbGYuX2NoZWNrX2dpdF9hcmNoaXRlY3R1cmUoKQogICAgICAgICBTQ00uX19pbml0X18oc2Vs
ZiwgY3dkKQogCisgICAgZGVmIF9jaGVja19naXRfYXJjaGl0ZWN0dXJlKHNlbGYpOgorICAgICAg
ICBpbXBvcnQgcGxhdGZvcm0KKyAgICAgICAgIyBUaGlzIG9ubHkgaXMgdGVzdGVkIG9uIE1hYy4K
KyAgICAgICAgaWYgbm90IHBsYXRmb3JtLm1hY192ZXIoKVswXToKKyAgICAgICAgICAgIHJldHVy
bgorCisgICAgICAgICMgVGhpcyBpcyBjaGVja2luZyB0aGUgYXJjaGl0ZWN0dXJlIG9mIHRoZSBw
eXRob24gZXhlY3V0YWJsZSwKKyAgICAgICAgIyBvbiBTbm93IExlb2FwcmQsIC91c3IvYmluL3B5
dGhvbiBpcyA2NC1iaXQsIHNvIGlmIHRoZQorICAgICAgICAjIG1hY2hpbmUgaXMgNjQtYml0IHRo
aXMgd2lsIHJldHVybiAoJzY0Yml0JywgJycpCisgICAgICAgICMgQWx0ZXJuYXRpdmVseSB3ZSBj
b3VsZCBjaGVjayAnc3lzY3RsIC1uIGh3LmNwdTY0Yml0X2NhcGFibGUnIG9uIE1hYworICAgICAg
ICAjIGlmIHdlIG5lZWRlZCB0byBzdXBwb3J0IDY0LWJpdCBNYWMgbWFjaGluZXMgd2l0aCBhIDMy
LWJpdCBweXRob24uCisgICAgICAgIGlmIHBsYXRmb3JtLmFyY2hpdGVjdHVyZSgpWzBdICE9ICc2
NGJpdCc6CisgICAgICAgICAgICByZXR1cm4KKworICAgICAgICAjIFdlIGNvdWxkIHBhdGgtc2Vh
cmNoIGVudGlyZWx5IGluIHB5dGhvbiBvciB3aXRoCisgICAgICAgICMgd2hpY2gucHkgKGh0dHA6
Ly9jb2RlLmdvb2dsZS5jb20vcC93aGljaCksIGJ1dCB0aGlzIGlzIGVhc2llcjoKKyAgICAgICAg
Z2l0X3BhdGggPSBzZWxmLnJ1bihbJ3doaWNoJywgJ2dpdCddKS5yc3RyaXAoKQorICAgICAgICB3
ZWJraXRfZGV2X3RoZWFkX3VybCA9ICJodHRwczovL2xpc3RzLndlYmtpdC5vcmcvcGlwZXJtYWls
L3dlYmtpdC1kZXYvMjAxMC1EZWNlbWJlci8wMTUyNDkuaHRtbCIKKyAgICAgICAgaWYgcGxhdGZv
cm0uYXJjaGl0ZWN0dXJlKGdpdF9wYXRoKSAhPSAnNjRiaXQnOgorICAgICAgICAgICAgbG9nKCJX
YXJuaW5nOiBUaGlzIG1hY2hpbmUgaXMgNjQtYml0LCBidXQgdGhlIGdpdCBiaW5hcnkgKCVzKSBk
b2VzIG5vdCBzdXBwb3J0IDY0LWJpdC5cbkluc3RhbGwgYSA2NC1iaXQgZ2l0IGZvciBiZXR0ZXIg
cGVyZm9ybWFuY2UsIHNlZTpcbiVzXG4iICUgKGdpdF9wYXRoLCB3ZWJraXRfZGV2X3RoZWFkX3Vy
bCkpCisKICAgICBAY2xhc3NtZXRob2QKICAgICBkZWYgaW5fd29ya2luZ19kaXJlY3RvcnkoY2xz
LCBwYXRoKToKICAgICAgICAgcmV0dXJuIHJ1bl9jb21tYW5kKFsnZ2l0JywgJ3Jldi1wYXJzZScs
ICctLWlzLWluc2lkZS13b3JrLXRyZWUnXSwgY3dkPXBhdGgsIGVycm9yX2hhbmRsZXI9RXhlY3V0
aXZlLmlnbm9yZV9lcnJvcikucnN0cmlwKCkgPT0gInRydWUiCg==
</data>

          </attachment>
          <attachment
              isobsolete="1"
              ispatch="1"
              isprivate="0"
          >
            <attachid>75974</attachid>
            <date>2010-12-08 14:55:23 -0800</date>
            <delta_ts>2010-12-08 15:13:02 -0800</delta_ts>
            <desc>Patch</desc>
            <filename>bug-50715-20101208145521.patch</filename>
            <type>text/plain</type>
            <size>3816</size>
            <attacher name="Eric Seidel (no email)">eric</attacher>
            
              <data encoding="base64">ZGlmZiAtLWdpdCBhL1dlYktpdFRvb2xzL0NoYW5nZUxvZyBiL1dlYktpdFRvb2xzL0NoYW5nZUxv
ZwppbmRleCA2ZWQ1YWM3MmUxNDg2OTE3MDE0YzdlZDg5Nzg1MmI2ZGIyYTYzYTJjLi5jNTc2Y2Zh
ZmU1OTdkM2RlMjFjMjkzZDc3MjBjNTkyNjkzNmZjN2ZhIDEwMDY0NAotLS0gYS9XZWJLaXRUb29s
cy9DaGFuZ2VMb2cKKysrIGIvV2ViS2l0VG9vbHMvQ2hhbmdlTG9nCkBAIC0xLDMgKzEsMjkgQEAK
KzIwMTAtMTItMDggIEVyaWMgU2VpZGVsICA8ZXJpY0B3ZWJraXQub3JnPgorCisgICAgICAgIFJl
dmlld2VkIGJ5IE5PQk9EWSAoT09QUyEpLgorCisgICAgICAgIHdlYmtpdC1wYXRjaCBzaG91bGQg
d2FybiB1c2VycyB3aGVuIHRoZXkncmUgdXNpbmcgYSAzMi1iaXQgZ2l0IG9uIGEgNjQtYml0IHN5
c3RlbQorICAgICAgICBodHRwczovL2J1Z3Mud2Via2l0Lm9yZy9zaG93X2J1Zy5jZ2k/aWQ9NTA3
MTUKKworICAgICAgICBUaGlzIHBhdGNoIG1ha2VzIHdlYmtpdC1wYXRjaCBwcmludCB0aGUgZm9s
bG93aW5nOgorCisgICAgICAgIFdhcm5pbmc6IFRoaXMgbWFjaGluZSBpcyA2NC1iaXQsIGJ1dCB0
aGUgZ2l0IGJpbmFyeSAoL3Vzci9sb2NhbC9naXQvYmluL2dpdCkgZG9lcyBub3Qgc3VwcG9ydCA2
NC1iaXQuCisgICAgICAgIEluc3RhbGwgYSA2NC1iaXQgZ2l0IGZvciBiZXR0ZXIgcGVyZm9ybWFu
Y2UsIHNlZToKKyAgICAgICAgaHR0cHM6Ly9saXN0cy53ZWJraXQub3JnL3BpcGVybWFpbC93ZWJr
aXQtZGV2LzIwMTAtRGVjZW1iZXIvMDE1MjQ5Lmh0bWwKKworICAgICAgICBJIHdyb3RlIHRoaXMg
bW9zdGx5IGJlY2F1c2UgSSBoYXZlIGFwcHJveGltYXRlbHkgOCBtYWNoaW5lcyB0aGF0IEkgdXNl
CisgICAgICAgIGFuZCBtYWtpbmcgc3VyZSBlYWNoIG9uZSBpcyB1c2luZyBhIGdvb2QgR2l0IGlu
c3RhbGwgc2VlbWVkIGZvbGx5LgorICAgICAgICB3ZWJraXQtcGF0Y2ggbWFrZXMgYSBsb3Qgb2Yg
Z2l0IGNhbGxzLCBzbyB1c2luZyBhIGZhc3QgZ2l0IGNhbiBzaGF2ZQorICAgICAgICBzZXZlcmFs
IHNlY29uZHMgaW4gZXZlcnkgaW52b2NhdGlvbi4gIFNlZSB0aGUgd2Via2l0LWRldiB0aHJlYWQg
Zm9yIG1vcmUgaW5mby4KKworICAgICAgICBUaGlzIG1lc3NhZ2Ugd2lsbCBwcmludCB0d2ljZSBk
dXJpbmcgJ3dlYmtpdC1wYXRjaCB1cGxvYWQnLAorICAgICAgICBvbmNlIGZyb20gd2Via2l0LXBh
dGNoIGFuZCBvbmNlIGZyb20gY2hlY2std2Via2l0LXN0eWxlLgorCisgICAgICAgIFVuZm9ydHVu
YXRlbHkgdGhlcmUgaXMgbm8gZ29vZCB3YXkgdG8gdGVzdCB0aGlzIGR1ZSB0byBob3cgbWFjaGlu
ZS1kZXBlbmRlbnQKKyAgICAgICAgdGhlIGNvZGUgaXMuICBJIGNvbnNpZGVyZWQgd3JpdGluZyBh
IHRlc3QgZm9yIHRoZSBsb2cgbWVzc2FnZSwgYnV0IGl0IHNlZW1lZCBub3Qgd29ydGggaXQuCisK
KyAgICAgICAgKiBTY3JpcHRzL3dlYmtpdHB5L2NvbW1vbi9jaGVja291dC9zY20ucHk6CisKIDIw
MTAtMTItMDggIEplc3NpZSBCZXJsaW4gIDxqYmVybGluQGFwcGxlLmNvbT4KIAogICAgICAgICBS
ZXZpZXdlZCBieSBBZGFtIFJvYmVuLgpkaWZmIC0tZ2l0IGEvV2ViS2l0VG9vbHMvU2NyaXB0cy93
ZWJraXRweS9jb21tb24vY2hlY2tvdXQvc2NtLnB5IGIvV2ViS2l0VG9vbHMvU2NyaXB0cy93ZWJr
aXRweS9jb21tb24vY2hlY2tvdXQvc2NtLnB5CmluZGV4IGVlMmJiNDEyYmEwNTVjYWJkMjIyMWM2
ZjE1YmI2M2Y4OWIyN2EwMWYuLjYyZjFlMWM1Yzc3YjhlMjBmZmI1ZDEzNWY0NmU3MzJiM2ZjMTZl
YjggMTAwNjQ0Ci0tLSBhL1dlYktpdFRvb2xzL1NjcmlwdHMvd2Via2l0cHkvY29tbW9uL2NoZWNr
b3V0L3NjbS5weQorKysgYi9XZWJLaXRUb29scy9TY3JpcHRzL3dlYmtpdHB5L2NvbW1vbi9jaGVj
a291dC9zY20ucHkKQEAgLTU3NCw4ICs1NzQsNDAgQEAgY2xhc3MgU1ZOKFNDTSk6CiAjIEFsbCBn
aXQtc3BlY2lmaWMgbG9naWMgc2hvdWxkIGdvIGhlcmUuCiBjbGFzcyBHaXQoU0NNKToKICAgICBk
ZWYgX19pbml0X18oc2VsZiwgY3dkKToKKyAgICAgICAgc2VsZi5fY2hlY2tfZ2l0X2FyY2hpdGVj
dHVyZSgpCiAgICAgICAgIFNDTS5fX2luaXRfXyhzZWxmLCBjd2QpCiAKKyAgICBkZWYgX21hY2hp
bmVfaXNfNjRiaXQoc2VsZik6CisgICAgICAgIGltcG9ydCBwbGF0Zm9ybQorICAgICAgICAjIFRo
aXMgb25seSBpcyB0ZXN0ZWQgb24gTWFjLgorICAgICAgICBpZiBub3QgcGxhdGZvcm0ubWFjX3Zl
cigpWzBdOgorICAgICAgICAgICAgcmV0dXJuIEZhbHNlCisKKyAgICAgICAgIyBwbGF0Zm9ybS5h
cmNoaXRlY3R1cmUoKVswXSBjYW4gYmUgJzY0Yml0JyBldmVuIGlmIHRoZSBtYWNoaW5lIGlzIDMy
Yml0OgorICAgICAgICAjIGh0dHA6Ly9tYWlsLnB5dGhvbi5vcmcvcGlwZXJtYWlsL3B5dGhvbm1h
Yy1zaWcvMjAwOS1TZXB0ZW1iZXIvMDIxNjQ4Lmh0bWwKKyAgICAgICAgIyBVc2UgdGhlIHN5c2N0
bCBjb21tYW5kIHRvIGZpbmQgb3V0IHdoYXQgdGhlIHByb2Nlc3NvciBhY3R1YWxseSBzdXBwb3J0
cy4KKyAgICAgICAgcmV0dXJuIHNlbGYucnVuKFsnc3lzY3RsJywgJy1uJywgJ2h3LmNwdTY0Yml0
X2NhcGFibGUnXSkucnN0cmlwKCkgPT0gJzEnCisKKyAgICBkZWYgX2V4ZWN1dGFibGVfaXNfNjRi
aXQoc2VsZiwgcGF0aCk6CisgICAgICAgICMgQWdhaW4sIHBsYXRmb3JtLmFyY2hpdGVjdHVyZSgp
IGZhaWxzIHVzLiAgT24gbXkgbWFjaGluZQorICAgICAgICAjIGdpdF9iaXRzID0gcGxhdGZvcm0u
YXJjaGl0ZWN0dXJlKGV4ZWN1dGFibGU9Z2l0X3BhdGgsIGJpdHM9J2RlZmF1bHQnKVswXQorICAg
ICAgICAjIGdpdF9iaXRzIGlzIGp1c3QgJ2RlZmF1bHQnLCBtZWFuaW5nIHRoZSBjYWxsIGZhaWxl
ZC4KKyAgICAgICAgZmlsZV9vdXRwdXQgPSBzZWxmLnJ1bihbJ2ZpbGUnLCBwYXRoXSkKKyAgICAg
ICAgcmV0dXJuIHJlLnNlYXJjaCgneDg2XzY0JywgZmlsZV9vdXRwdXQpCisKKyAgICBkZWYgX2No
ZWNrX2dpdF9hcmNoaXRlY3R1cmUoc2VsZik6CisgICAgICAgIGlmIG5vdCBzZWxmLl9tYWNoaW5l
X2lzXzY0Yml0KCk6CisgICAgICAgICAgICByZXR1cm4KKworICAgICAgICAjIFdlIGNvdWxkIHBh
dGgtc2VhcmNoIGVudGlyZWx5IGluIHB5dGhvbiBvciB3aXRoCisgICAgICAgICMgd2hpY2gucHkg
KGh0dHA6Ly9jb2RlLmdvb2dsZS5jb20vcC93aGljaCksIGJ1dCB0aGlzIGlzIGVhc2llcjoKKyAg
ICAgICAgZ2l0X3BhdGggPSBzZWxmLnJ1bihbJ3doaWNoJywgJ2dpdCddKS5yc3RyaXAoKQorICAg
ICAgICBpZiBzZWxmLl9leGVjdXRhYmxlX2lzXzY0Yml0KGdpdF9wYXRoKToKKyAgICAgICAgICAg
IHJldHVybgorCisgICAgICAgIHdlYmtpdF9kZXZfdGhlYWRfdXJsID0gImh0dHBzOi8vbGlzdHMu
d2Via2l0Lm9yZy9waXBlcm1haWwvd2Via2l0LWRldi8yMDEwLURlY2VtYmVyLzAxNTI0OS5odG1s
IgorICAgICAgICBsb2coIldhcm5pbmc6IFRoaXMgbWFjaGluZSBpcyA2NC1iaXQsIGJ1dCB0aGUg
Z2l0IGJpbmFyeSAoJXMpIGRvZXMgbm90IHN1cHBvcnQgNjQtYml0LlxuSW5zdGFsbCBhIDY0LWJp
dCBnaXQgZm9yIGJldHRlciBwZXJmb3JtYW5jZSwgc2VlOlxuJXNcbiIgJSAoZ2l0X3BhdGgsIHdl
YmtpdF9kZXZfdGhlYWRfdXJsKSkKKwogICAgIEBjbGFzc21ldGhvZAogICAgIGRlZiBpbl93b3Jr
aW5nX2RpcmVjdG9yeShjbHMsIHBhdGgpOgogICAgICAgICByZXR1cm4gcnVuX2NvbW1hbmQoWydn
aXQnLCAncmV2LXBhcnNlJywgJy0taXMtaW5zaWRlLXdvcmstdHJlZSddLCBjd2Q9cGF0aCwgZXJy
b3JfaGFuZGxlcj1FeGVjdXRpdmUuaWdub3JlX2Vycm9yKS5yc3RyaXAoKSA9PSAidHJ1ZSIK
</data>

          </attachment>
          <attachment
              isobsolete="0"
              ispatch="1"
              isprivate="0"
          >
            <attachid>75977</attachid>
            <date>2010-12-08 15:13:06 -0800</date>
            <delta_ts>2010-12-14 01:44:01 -0800</delta_ts>
            <desc>move _check_git_architecture to after SCM.__init__</desc>
            <filename>bug-50715-20101208151306.patch</filename>
            <type>text/plain</type>
            <size>3632</size>
            <attacher name="Eric Seidel (no email)">eric</attacher>
            
              <data encoding="base64">ZGlmZiAtLWdpdCBhL1dlYktpdFRvb2xzL0NoYW5nZUxvZyBiL1dlYktpdFRvb2xzL0NoYW5nZUxv
ZwppbmRleCA2ZWQ1YWM3MmUxNDg2OTE3MDE0YzdlZDg5Nzg1MmI2ZGIyYTYzYTJjLi5jNTc2Y2Zh
ZmU1OTdkM2RlMjFjMjkzZDc3MjBjNTkyNjkzNmZjN2ZhIDEwMDY0NAotLS0gYS9XZWJLaXRUb29s
cy9DaGFuZ2VMb2cKKysrIGIvV2ViS2l0VG9vbHMvQ2hhbmdlTG9nCkBAIC0xLDMgKzEsMjkgQEAK
KzIwMTAtMTItMDggIEVyaWMgU2VpZGVsICA8ZXJpY0B3ZWJraXQub3JnPgorCisgICAgICAgIFJl
dmlld2VkIGJ5IE5PQk9EWSAoT09QUyEpLgorCisgICAgICAgIHdlYmtpdC1wYXRjaCBzaG91bGQg
d2FybiB1c2VycyB3aGVuIHRoZXkncmUgdXNpbmcgYSAzMi1iaXQgZ2l0IG9uIGEgNjQtYml0IHN5
c3RlbQorICAgICAgICBodHRwczovL2J1Z3Mud2Via2l0Lm9yZy9zaG93X2J1Zy5jZ2k/aWQ9NTA3
MTUKKworICAgICAgICBUaGlzIHBhdGNoIG1ha2VzIHdlYmtpdC1wYXRjaCBwcmludCB0aGUgZm9s
bG93aW5nOgorCisgICAgICAgIFdhcm5pbmc6IFRoaXMgbWFjaGluZSBpcyA2NC1iaXQsIGJ1dCB0
aGUgZ2l0IGJpbmFyeSAoL3Vzci9sb2NhbC9naXQvYmluL2dpdCkgZG9lcyBub3Qgc3VwcG9ydCA2
NC1iaXQuCisgICAgICAgIEluc3RhbGwgYSA2NC1iaXQgZ2l0IGZvciBiZXR0ZXIgcGVyZm9ybWFu
Y2UsIHNlZToKKyAgICAgICAgaHR0cHM6Ly9saXN0cy53ZWJraXQub3JnL3BpcGVybWFpbC93ZWJr
aXQtZGV2LzIwMTAtRGVjZW1iZXIvMDE1MjQ5Lmh0bWwKKworICAgICAgICBJIHdyb3RlIHRoaXMg
bW9zdGx5IGJlY2F1c2UgSSBoYXZlIGFwcHJveGltYXRlbHkgOCBtYWNoaW5lcyB0aGF0IEkgdXNl
CisgICAgICAgIGFuZCBtYWtpbmcgc3VyZSBlYWNoIG9uZSBpcyB1c2luZyBhIGdvb2QgR2l0IGlu
c3RhbGwgc2VlbWVkIGZvbGx5LgorICAgICAgICB3ZWJraXQtcGF0Y2ggbWFrZXMgYSBsb3Qgb2Yg
Z2l0IGNhbGxzLCBzbyB1c2luZyBhIGZhc3QgZ2l0IGNhbiBzaGF2ZQorICAgICAgICBzZXZlcmFs
IHNlY29uZHMgaW4gZXZlcnkgaW52b2NhdGlvbi4gIFNlZSB0aGUgd2Via2l0LWRldiB0aHJlYWQg
Zm9yIG1vcmUgaW5mby4KKworICAgICAgICBUaGlzIG1lc3NhZ2Ugd2lsbCBwcmludCB0d2ljZSBk
dXJpbmcgJ3dlYmtpdC1wYXRjaCB1cGxvYWQnLAorICAgICAgICBvbmNlIGZyb20gd2Via2l0LXBh
dGNoIGFuZCBvbmNlIGZyb20gY2hlY2std2Via2l0LXN0eWxlLgorCisgICAgICAgIFVuZm9ydHVu
YXRlbHkgdGhlcmUgaXMgbm8gZ29vZCB3YXkgdG8gdGVzdCB0aGlzIGR1ZSB0byBob3cgbWFjaGlu
ZS1kZXBlbmRlbnQKKyAgICAgICAgdGhlIGNvZGUgaXMuICBJIGNvbnNpZGVyZWQgd3JpdGluZyBh
IHRlc3QgZm9yIHRoZSBsb2cgbWVzc2FnZSwgYnV0IGl0IHNlZW1lZCBub3Qgd29ydGggaXQuCisK
KyAgICAgICAgKiBTY3JpcHRzL3dlYmtpdHB5L2NvbW1vbi9jaGVja291dC9zY20ucHk6CisKIDIw
MTAtMTItMDggIEplc3NpZSBCZXJsaW4gIDxqYmVybGluQGFwcGxlLmNvbT4KIAogICAgICAgICBS
ZXZpZXdlZCBieSBBZGFtIFJvYmVuLgpkaWZmIC0tZ2l0IGEvV2ViS2l0VG9vbHMvU2NyaXB0cy93
ZWJraXRweS9jb21tb24vY2hlY2tvdXQvc2NtLnB5IGIvV2ViS2l0VG9vbHMvU2NyaXB0cy93ZWJr
aXRweS9jb21tb24vY2hlY2tvdXQvc2NtLnB5CmluZGV4IGVlMmJiNDEyYmEwNTVjYWJkMjIyMWM2
ZjE1YmI2M2Y4OWIyN2EwMWYuLmNhNzYyMTQzMzNkMTI2OGY3ODA5NTM4Y2YxNTAyYzVmNDQxY2Vi
YzAgMTAwNjQ0Ci0tLSBhL1dlYktpdFRvb2xzL1NjcmlwdHMvd2Via2l0cHkvY29tbW9uL2NoZWNr
b3V0L3NjbS5weQorKysgYi9XZWJLaXRUb29scy9TY3JpcHRzL3dlYmtpdHB5L2NvbW1vbi9jaGVj
a291dC9zY20ucHkKQEAgLTU3NSw2ICs1NzUsMzggQEAgY2xhc3MgU1ZOKFNDTSk6CiBjbGFzcyBH
aXQoU0NNKToKICAgICBkZWYgX19pbml0X18oc2VsZiwgY3dkKToKICAgICAgICAgU0NNLl9faW5p
dF9fKHNlbGYsIGN3ZCkKKyAgICAgICAgc2VsZi5fY2hlY2tfZ2l0X2FyY2hpdGVjdHVyZSgpCisK
KyAgICBkZWYgX21hY2hpbmVfaXNfNjRiaXQoc2VsZik6CisgICAgICAgIGltcG9ydCBwbGF0Zm9y
bQorICAgICAgICAjIFRoaXMgb25seSBpcyB0ZXN0ZWQgb24gTWFjLgorICAgICAgICBpZiBub3Qg
cGxhdGZvcm0ubWFjX3ZlcigpWzBdOgorICAgICAgICAgICAgcmV0dXJuIEZhbHNlCisKKyAgICAg
ICAgIyBwbGF0Zm9ybS5hcmNoaXRlY3R1cmUoKVswXSBjYW4gYmUgJzY0Yml0JyBldmVuIGlmIHRo
ZSBtYWNoaW5lIGlzIDMyYml0OgorICAgICAgICAjIGh0dHA6Ly9tYWlsLnB5dGhvbi5vcmcvcGlw
ZXJtYWlsL3B5dGhvbm1hYy1zaWcvMjAwOS1TZXB0ZW1iZXIvMDIxNjQ4Lmh0bWwKKyAgICAgICAg
IyBVc2UgdGhlIHN5c2N0bCBjb21tYW5kIHRvIGZpbmQgb3V0IHdoYXQgdGhlIHByb2Nlc3NvciBh
Y3R1YWxseSBzdXBwb3J0cy4KKyAgICAgICAgcmV0dXJuIHNlbGYucnVuKFsnc3lzY3RsJywgJy1u
JywgJ2h3LmNwdTY0Yml0X2NhcGFibGUnXSkucnN0cmlwKCkgPT0gJzEnCisKKyAgICBkZWYgX2V4
ZWN1dGFibGVfaXNfNjRiaXQoc2VsZiwgcGF0aCk6CisgICAgICAgICMgQWdhaW4sIHBsYXRmb3Jt
LmFyY2hpdGVjdHVyZSgpIGZhaWxzIHVzLiAgT24gbXkgbWFjaGluZQorICAgICAgICAjIGdpdF9i
aXRzID0gcGxhdGZvcm0uYXJjaGl0ZWN0dXJlKGV4ZWN1dGFibGU9Z2l0X3BhdGgsIGJpdHM9J2Rl
ZmF1bHQnKVswXQorICAgICAgICAjIGdpdF9iaXRzIGlzIGp1c3QgJ2RlZmF1bHQnLCBtZWFuaW5n
IHRoZSBjYWxsIGZhaWxlZC4KKyAgICAgICAgZmlsZV9vdXRwdXQgPSBzZWxmLnJ1bihbJ2ZpbGUn
LCBwYXRoXSkKKyAgICAgICAgcmV0dXJuIHJlLnNlYXJjaCgneDg2XzY0JywgZmlsZV9vdXRwdXQp
CisKKyAgICBkZWYgX2NoZWNrX2dpdF9hcmNoaXRlY3R1cmUoc2VsZik6CisgICAgICAgIGlmIG5v
dCBzZWxmLl9tYWNoaW5lX2lzXzY0Yml0KCk6CisgICAgICAgICAgICByZXR1cm4KKworICAgICAg
ICAjIFdlIGNvdWxkIHBhdGgtc2VhcmNoIGVudGlyZWx5IGluIHB5dGhvbiBvciB3aXRoCisgICAg
ICAgICMgd2hpY2gucHkgKGh0dHA6Ly9jb2RlLmdvb2dsZS5jb20vcC93aGljaCksIGJ1dCB0aGlz
IGlzIGVhc2llcjoKKyAgICAgICAgZ2l0X3BhdGggPSBzZWxmLnJ1bihbJ3doaWNoJywgJ2dpdCdd
KS5yc3RyaXAoKQorICAgICAgICBpZiBzZWxmLl9leGVjdXRhYmxlX2lzXzY0Yml0KGdpdF9wYXRo
KToKKyAgICAgICAgICAgIHJldHVybgorCisgICAgICAgIHdlYmtpdF9kZXZfdGhlYWRfdXJsID0g
Imh0dHBzOi8vbGlzdHMud2Via2l0Lm9yZy9waXBlcm1haWwvd2Via2l0LWRldi8yMDEwLURlY2Vt
YmVyLzAxNTI0OS5odG1sIgorICAgICAgICBsb2coIldhcm5pbmc6IFRoaXMgbWFjaGluZSBpcyA2
NC1iaXQsIGJ1dCB0aGUgZ2l0IGJpbmFyeSAoJXMpIGRvZXMgbm90IHN1cHBvcnQgNjQtYml0Llxu
SW5zdGFsbCBhIDY0LWJpdCBnaXQgZm9yIGJldHRlciBwZXJmb3JtYW5jZSwgc2VlOlxuJXNcbiIg
JSAoZ2l0X3BhdGgsIHdlYmtpdF9kZXZfdGhlYWRfdXJsKSkKIAogICAgIEBjbGFzc21ldGhvZAog
ICAgIGRlZiBpbl93b3JraW5nX2RpcmVjdG9yeShjbHMsIHBhdGgpOgo=
</data>

          </attachment>
      

    </bug>

</bugzilla>