<?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>51804</bug_id>
          
          <creation_ts>2011-01-02 13:23:49 -0800</creation_ts>
          <short_desc>Make EFL a core builder</short_desc>
          <delta_ts>2011-01-12 23:27:47 -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>Tools / Tests</component>
          <version>528+ (Nightly build)</version>
          <rep_platform>All</rep_platform>
          <op_sys>All</op_sys>
          <bug_status>RESOLVED</bug_status>
          <resolution>LATER</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>
          <dependson>52357</dependson>
          
          <everconfirmed>1</everconfirmed>
          <reporter name="Patrick R. Gansterer">paroga</reporter>
          <assigned_to name="Patrick R. Gansterer">paroga</assigned_to>
          <cc>abarth</cc>
    
    <cc>ap</cc>
    
    <cc>commit-queue</cc>
    
    <cc>eric</cc>
    
    <cc>wsiegrist</cc>
          

      

      

      

          <comment_sort_order>oldest_to_newest</comment_sort_order>  
          <long_desc isprivate="0" >
    <commentid>328423</commentid>
    <comment_count>0</comment_count>
    <who name="Patrick R. Gansterer">paroga</who>
    <bug_when>2011-01-02 13:23:49 -0800</bug_when>
    <thetext>see patch</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>328424</commentid>
    <comment_count>1</comment_count>
      <attachid>77780</attachid>
    <who name="Patrick R. Gansterer">paroga</who>
    <bug_when>2011-01-02 13:26:18 -0800</bug_when>
    <thetext>Created attachment 77780
Patch</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>328426</commentid>
    <comment_count>2</comment_count>
      <attachid>77780</attachid>
    <who name="Adam Barth">abarth</who>
    <bug_when>2011-01-02 13:45:45 -0800</bug_when>
    <thetext>Comment on attachment 77780
Patch

Thanks.  I&apos;m surprised you don&apos;t need to update the webkitpy unit tests.  Did you try running ./Tools/Scripts/test-webkitpy ?</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>328427</commentid>
    <comment_count>3</comment_count>
    <who name="Patrick R. Gansterer">paroga</who>
    <bug_when>2011-01-02 13:51:27 -0800</bug_when>
    <thetext>(In reply to comment #2)
&gt; (From update of attachment 77780 [details])
&gt; Thanks.  I&apos;m surprised you don&apos;t need to update the webkitpy unit tests.  Did you try running ./Tools/Scripts/test-webkitpy ?
Not all build slaves are in the unit tests (e.g. WinCairo too). So i don&apos;t find sth to change.</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>328428</commentid>
    <comment_count>4</comment_count>
    <who name="Adam Barth">abarth</who>
    <bug_when>2011-01-02 14:10:55 -0800</bug_when>
    <thetext>Ok.  We might need to update the test to cover all the builders we have now.</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>328433</commentid>
    <comment_count>5</comment_count>
      <attachid>77780</attachid>
    <who name="WebKit Commit Bot">commit-queue</who>
    <bug_when>2011-01-02 15:09:13 -0800</bug_when>
    <thetext>Comment on attachment 77780
Patch

Rejecting attachment 77780 from commit-queue.

Failed to run &quot;[&apos;./Tools/Scripts/webkit-patch&apos;, &apos;--status-host=queues.webkit.org&apos;, &apos;--bot-id=cr-jail-4&apos;, &apos;build-and-test&apos;, &apos;--no-clean&apos;, &apos;--no-update&apos;, &apos;--test&apos;, &apos;--non-interactive&apos;]&quot; exit_code: 2
Last 500 characters of output:
   self.assertEquals(buildbot.core_builder_names_regexps, name_regexps)
AssertionError: [&apos;SnowLeopard.*Build&apos;, &apos;SnowLeopard.*\\(Test&apos;, &apos;Leopard&apos;, &apos;Tiger&apos;, &apos;Windows.*Build&apos;, &apos;EFL&apos;, &apos;GTK.*32&apos;, &apos;GTK.*64.*Debug&apos;, &apos;Qt&apos;, &apos;Chromium.*Release$&apos;] != [&apos;SnowLeopard.*Build&apos;, &apos;SnowLeopard.*\\(Test&apos;, &apos;Leopard&apos;, &apos;Tiger&apos;, &apos;Windows.*Build&apos;, &apos;GTK.*32&apos;, &apos;GTK.*64.*Debug&apos;, &apos;Qt&apos;, &apos;Chromium.*Release$&apos;]

----------------------------------------------------------------------
Ran 766 tests in 19.645s

FAILED (failures=1)

Full output: http://queues.webkit.org/results/7251392</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>328435</commentid>
    <comment_count>6</comment_count>
      <attachid>77781</attachid>
    <who name="Patrick R. Gansterer">paroga</who>
    <bug_when>2011-01-02 15:18:34 -0800</bug_when>
    <thetext>Created attachment 77781
Patch

(In reply to comment #2)
&gt; (From update of attachment 77780 [details])
&gt; Did you try running ./Tools/Scripts/test-webkitpy ?
Now it works on my machine.</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>328449</commentid>
    <comment_count>7</comment_count>
      <attachid>77781</attachid>
    <who name="WebKit Commit Bot">commit-queue</who>
    <bug_when>2011-01-02 17:07:13 -0800</bug_when>
    <thetext>Comment on attachment 77781
Patch

Clearing flags on attachment: 77781

Committed r74881: &lt;http://trac.webkit.org/changeset/74881&gt;</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>328450</commentid>
    <comment_count>8</comment_count>
    <who name="WebKit Commit Bot">commit-queue</who>
    <bug_when>2011-01-02 17:07:19 -0800</bug_when>
    <thetext>All reviewed patches have been landed.  Closing bug.</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>328457</commentid>
    <comment_count>9</comment_count>
    <who name="WebKit Commit Bot">commit-queue</who>
    <bug_when>2011-01-02 17:27:03 -0800</bug_when>
    <thetext>The commit-queue encountered the following flaky tests while processing attachment 77781:

fast/images/load-img-with-empty-src.html bug 50855 (author: mitz@webkit.org)
The commit-queue is continuing to process your patch.</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>328467</commentid>
    <comment_count>10</comment_count>
    <who name="Alexey Proskuryakov">ap</who>
    <bug_when>2011-01-02 19:21:42 -0800</bug_when>
    <thetext>Shouldn&apos;t making a builder &quot;core&quot; require webkit-dev discussion? This is a burden on all WebKit developers.</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>328473</commentid>
    <comment_count>11</comment_count>
    <who name="Adam Barth">abarth</who>
    <bug_when>2011-01-02 20:33:02 -0800</bug_when>
    <thetext>(In reply to comment #10)
&gt; Shouldn&apos;t making a builder &quot;core&quot; require webkit-dev discussion? This is a burden on all WebKit developers.

Historically we haven&apos;t.  Being a core builder is just an empirical property.  If the builder is consistently green, it&apos;s a core builder.  According to the buildbot logs, the EFL builder has been pretty consistently green.  If it stops being green consistently, we can remove it from the list of core builders.</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>331295</commentid>
    <comment_count>12</comment_count>
    <who name="Eric Seidel (no email)">eric</who>
    <bug_when>2011-01-08 15:02:57 -0800</bug_when>
    <thetext>The EFL builder is now core, but has been read for at least the last day due to some sort of compile failure.</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>331297</commentid>
    <comment_count>13</comment_count>
    <who name="Patrick R. Gansterer">paroga</who>
    <bug_when>2011-01-08 15:11:19 -0800</bug_when>
    <thetext>(In reply to comment #12)
&gt; The EFL builder is now core, but has been read for at least the last day due to some sort of compile failure.

There&apos;s some problem with the machine. The port is ok at the moment and compiles on my machine. I didn&apos;t see one of the slave admin in IRC since it&apos;s down. :-(</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>331298</commentid>
    <comment_count>14</comment_count>
    <who name="Eric Seidel (no email)">eric</who>
    <bug_when>2011-01-08 15:14:29 -0800</bug_when>
    <thetext>Thanks.</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>332006</commentid>
    <comment_count>15</comment_count>
    <who name="Adam Barth">abarth</who>
    <bug_when>2011-01-10 15:46:51 -0800</bug_when>
    <thetext>This bot has been red for a while.  Looks like we might have acted too soon.  I didn&apos;t realize that paroga owns the WinCE bots and not the EFL bot.</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>333566</commentid>
    <comment_count>16</comment_count>
    <who name="Adam Barth">abarth</who>
    <bug_when>2011-01-12 23:27:47 -0800</bug_when>
    <thetext>Revert the change in https://bugs.webkit.org/show_bug.cgi?id=52357</thetext>
  </long_desc>
      
          <attachment
              isobsolete="1"
              ispatch="1"
              isprivate="0"
          >
            <attachid>77780</attachid>
            <date>2011-01-02 13:26:18 -0800</date>
            <delta_ts>2011-01-02 15:18:34 -0800</delta_ts>
            <desc>Patch</desc>
            <filename>bug-51804.patch</filename>
            <type>text/plain</type>
            <size>1009</size>
            <attacher name="Patrick R. Gansterer">paroga</attacher>
            
              <data encoding="base64">ZGlmZiAtLWdpdCBhL1Rvb2xzL0NoYW5nZUxvZyBiL1Rvb2xzL0NoYW5nZUxvZwppbmRleCBhM2Jh
ZTEzLi43NTM3MDY0IDEwMDY0NAotLS0gYS9Ub29scy9DaGFuZ2VMb2cKKysrIGIvVG9vbHMvQ2hh
bmdlTG9nCkBAIC0xLDMgKzEsMTIgQEAKKzIwMTEtMDEtMDIgIFBhdHJpY2sgR2Fuc3RlcmVyICA8
cGFyb2dhQHdlYmtpdC5vcmc+CisKKyAgICAgICAgUmV2aWV3ZWQgYnkgTk9CT0RZIChPT1BTISku
CisKKyAgICAgICAgTWFrZSBFRkwgYSBjb3JlIGJ1aWxkZXIKKyAgICAgICAgaHR0cHM6Ly9idWdz
LndlYmtpdC5vcmcvc2hvd19idWcuY2dpP2lkPTUxODA0CisKKyAgICAgICAgKiBTY3JpcHRzL3dl
YmtpdHB5L2NvbW1vbi9uZXQvYnVpbGRib3QvYnVpbGRib3QucHk6CisKIDIwMTEtMDEtMDIgIFJv
YmVydCBIb2dhbiAgPHJvYmVydEB3ZWJraXQub3JnPgogCiAgICAgICAgIFVucmV2aWV3ZWQsIGJ1
aWxkIGZpeC4KZGlmZiAtLWdpdCBhL1Rvb2xzL1NjcmlwdHMvd2Via2l0cHkvY29tbW9uL25ldC9i
dWlsZGJvdC9idWlsZGJvdC5weSBiL1Rvb2xzL1NjcmlwdHMvd2Via2l0cHkvY29tbW9uL25ldC9i
dWlsZGJvdC9idWlsZGJvdC5weQppbmRleCAzY2I2ZGE1Li45OGUyZmFlIDEwMDY0NAotLS0gYS9U
b29scy9TY3JpcHRzL3dlYmtpdHB5L2NvbW1vbi9uZXQvYnVpbGRib3QvYnVpbGRib3QucHkKKysr
IGIvVG9vbHMvU2NyaXB0cy93ZWJraXRweS9jb21tb24vbmV0L2J1aWxkYm90L2J1aWxkYm90LnB5
CkBAIC0yNzAsNiArMjcwLDcgQEAgY2xhc3MgQnVpbGRCb3Qob2JqZWN0KToKICAgICAgICAgICAg
ICJMZW9wYXJkIiwKICAgICAgICAgICAgICJUaWdlciIsCiAgICAgICAgICAgICAiV2luZG93cy4q
QnVpbGQiLAorICAgICAgICAgICAgIkVGTCIsCiAgICAgICAgICAgICAiR1RLLiozMiIsCiAgICAg
ICAgICAgICAiR1RLLio2NC4qRGVidWciLCAgIyBEaXNhbGxvdyB0aGUgNjQtYml0IFJlbGVhc2Ug
Ym90IHdoaWNoIGlzIGJyb2tlbi4KICAgICAgICAgICAgICJRdCIsCg==
</data>
<flag name="review"
          id="68963"
          type_id="1"
          status="+"
          setter="abarth"
    />
    <flag name="commit-queue"
          id="68964"
          type_id="3"
          status="-"
          setter="commit-queue"
    />
          </attachment>
          <attachment
              isobsolete="0"
              ispatch="1"
              isprivate="0"
          >
            <attachid>77781</attachid>
            <date>2011-01-02 15:18:34 -0800</date>
            <delta_ts>2011-01-02 17:07:13 -0800</delta_ts>
            <desc>Patch</desc>
            <filename>bug-51804b.patch</filename>
            <type>text/plain</type>
            <size>1670</size>
            <attacher name="Patrick R. Gansterer">paroga</attacher>
            
              <data encoding="base64">ZGlmZiAtLWdpdCBhL1Rvb2xzL0NoYW5nZUxvZyBiL1Rvb2xzL0NoYW5nZUxvZwppbmRleCBhM2Jh
ZTEzLi43NDExYzIxIDEwMDY0NAotLS0gYS9Ub29scy9DaGFuZ2VMb2cKKysrIGIvVG9vbHMvQ2hh
bmdlTG9nCkBAIC0xLDMgKzEsMTMgQEAKKzIwMTEtMDEtMDIgIFBhdHJpY2sgR2Fuc3RlcmVyICA8
cGFyb2dhQHdlYmtpdC5vcmc+CisKKyAgICAgICAgUmV2aWV3ZWQgYnkgQWRhbSBCYXJ0aC4KKwor
ICAgICAgICBNYWtlIEVGTCBhIGNvcmUgYnVpbGRlcgorICAgICAgICBodHRwczovL2J1Z3Mud2Vi
a2l0Lm9yZy9zaG93X2J1Zy5jZ2k/aWQ9NTE4MDQKKworICAgICAgICAqIFNjcmlwdHMvd2Via2l0
cHkvY29tbW9uL25ldC9idWlsZGJvdC9idWlsZGJvdC5weToKKyAgICAgICAgKiBTY3JpcHRzL3dl
YmtpdHB5L2NvbW1vbi9uZXQvYnVpbGRib3QvYnVpbGRib3RfdW5pdHRlc3QucHk6CisKIDIwMTEt
MDEtMDIgIFJvYmVydCBIb2dhbiAgPHJvYmVydEB3ZWJraXQub3JnPgogCiAgICAgICAgIFVucmV2
aWV3ZWQsIGJ1aWxkIGZpeC4KZGlmZiAtLWdpdCBhL1Rvb2xzL1NjcmlwdHMvd2Via2l0cHkvY29t
bW9uL25ldC9idWlsZGJvdC9idWlsZGJvdC5weSBiL1Rvb2xzL1NjcmlwdHMvd2Via2l0cHkvY29t
bW9uL25ldC9idWlsZGJvdC9idWlsZGJvdC5weQppbmRleCAzY2I2ZGE1Li45OGUyZmFlIDEwMDY0
NAotLS0gYS9Ub29scy9TY3JpcHRzL3dlYmtpdHB5L2NvbW1vbi9uZXQvYnVpbGRib3QvYnVpbGRi
b3QucHkKKysrIGIvVG9vbHMvU2NyaXB0cy93ZWJraXRweS9jb21tb24vbmV0L2J1aWxkYm90L2J1
aWxkYm90LnB5CkBAIC0yNzAsNiArMjcwLDcgQEAgY2xhc3MgQnVpbGRCb3Qob2JqZWN0KToKICAg
ICAgICAgICAgICJMZW9wYXJkIiwKICAgICAgICAgICAgICJUaWdlciIsCiAgICAgICAgICAgICAi
V2luZG93cy4qQnVpbGQiLAorICAgICAgICAgICAgIkVGTCIsCiAgICAgICAgICAgICAiR1RLLioz
MiIsCiAgICAgICAgICAgICAiR1RLLio2NC4qRGVidWciLCAgIyBEaXNhbGxvdyB0aGUgNjQtYml0
IFJlbGVhc2UgYm90IHdoaWNoIGlzIGJyb2tlbi4KICAgICAgICAgICAgICJRdCIsCmRpZmYgLS1n
aXQgYS9Ub29scy9TY3JpcHRzL3dlYmtpdHB5L2NvbW1vbi9uZXQvYnVpbGRib3QvYnVpbGRib3Rf
dW5pdHRlc3QucHkgYi9Ub29scy9TY3JpcHRzL3dlYmtpdHB5L2NvbW1vbi9uZXQvYnVpbGRib3Qv
YnVpbGRib3RfdW5pdHRlc3QucHkKaW5kZXggYTEwZTQzMi4uODQ5OWM5ZCAxMDA2NDQKLS0tIGEv
VG9vbHMvU2NyaXB0cy93ZWJraXRweS9jb21tb24vbmV0L2J1aWxkYm90L2J1aWxkYm90X3VuaXR0
ZXN0LnB5CisrKyBiL1Rvb2xzL1NjcmlwdHMvd2Via2l0cHkvY29tbW9uL25ldC9idWlsZGJvdC9i
dWlsZGJvdF91bml0dGVzdC5weQpAQCAtMjU0LDYgKzI1NCw3IEBAIGNsYXNzIEJ1aWxkQm90VGVz
dCh1bml0dGVzdC5UZXN0Q2FzZSk6CiAgICAgICAgICAgICAiTGVvcGFyZCIsCiAgICAgICAgICAg
ICAiVGlnZXIiLAogICAgICAgICAgICAgIldpbmRvd3MuKkJ1aWxkIiwKKyAgICAgICAgICAgICJF
RkwiLAogICAgICAgICAgICAgIkdUSy4qMzIiLAogICAgICAgICAgICAgIkdUSy4qNjQuKkRlYnVn
IiwgICMgRGlzYWxsb3cgdGhlIDY0LWJpdCBSZWxlYXNlIGJvdCB3aGljaCBpcyBicm9rZW4uCiAg
ICAgICAgICAgICAiUXQiLAo=
</data>

          </attachment>
      

    </bug>

</bugzilla>