<?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>81007</bug_id>
          
          <creation_ts>2012-03-13 09:57:56 -0700</creation_ts>
          <short_desc>check-webkit-style fails on LayoutTests/platform/efl/test_expectations.txt</short_desc>
          <delta_ts>2012-03-16 05:14:57 -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>
          
          
          <everconfirmed>0</everconfirmed>
          <reporter name="Kishore Bolisetty">kbolisetty</reporter>
          <assigned_to name="Nobody">webkit-unassigned</assigned_to>
          <cc>abarth</cc>
    
    <cc>gyuyoung.kim</cc>
    
    <cc>jchaffraix</cc>
    
    <cc>mrahaman</cc>
    
    <cc>ojan</cc>
    
    <cc>webkit.review.bot</cc>
          

      

      

      

          <comment_sort_order>oldest_to_newest</comment_sort_order>  
          <long_desc isprivate="0" >
    <commentid>577466</commentid>
    <comment_count>0</comment_count>
    <who name="Kishore Bolisetty">kbolisetty</who>
    <bug_when>2012-03-13 09:57:56 -0700</bug_when>
    <thetext>./Tools/Scripts/check-webkit-style LayoutTests/platform/efl/test_expectations.txt 
fails even after removing duplicate entries in Skipped &amp; test_expectations.txt


./Tools/Scripts/check-webkit-style LayoutTests/platform/efl/test_expectations.txt
Traceback (most recent call last):
  File &quot;./Tools/Scripts/check-webkit-style&quot;, line 48, in &lt;module&gt;
    sys.exit(CheckWebKitStyle().main())
  File &quot;/home/administrator/kishore/osc/code/8march/WebKit/Tools/Scripts/webkitpy/style/main.py&quot;, line 149, in main
    file_reader.process_paths(paths)
  File &quot;/home/administrator/kishore/osc/code/8march/WebKit/Tools/Scripts/webkitpy/style/filereader.py&quot;, line 145, in process_paths
    self.process_file(path)
  File &quot;/home/administrator/kishore/osc/code/8march/WebKit/Tools/Scripts/webkitpy/style/filereader.py&quot;, line 130, in process_file
    self._processor.process(lines, file_path, **kwargs)
  File &quot;/home/administrator/kishore/osc/code/8march/WebKit/Tools/Scripts/webkitpy/style/checker.py&quot;, line 838, in process
    checker.check(lines)
  File &quot;/home/administrator/kishore/osc/code/8march/WebKit/Tools/Scripts/webkitpy/style/checkers/test_expectations.py&quot;, line 103, in check
    overrides = self._port_obj.test_expectations_overrides()
AttributeError: &apos;NoneType&apos; object has no attribute &apos;test_expectations_overrides&apos;</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>577476</commentid>
    <comment_count>1</comment_count>
    <who name="Kishore Bolisetty">kbolisetty</who>
    <bug_when>2012-03-13 10:05:02 -0700</bug_when>
    <thetext>I also have done little debugging in python scripts and could find the workaround for this issue, but not sure if it is a right fix.

Adding &quot;efl&quot; in Tools/Scripts/webkitpy/layout_tests/port/builders.py

as shown below would solve the issue.

_ports_without_builders = [
    &quot;google-chrome-linux32&quot;,
    &quot;google-chrome-linux64&quot;,
    &quot;qt-mac&quot;,
    &quot;qt-win&quot;,
    &quot;qt-wk2&quot;,
    &quot;efl&quot;,
]</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>580151</commentid>
    <comment_count>2</comment_count>
      <attachid>132200</attachid>
    <who name="Kishore Bolisetty">kbolisetty</who>
    <bug_when>2012-03-15 22:46:06 -0700</bug_when>
    <thetext>Created attachment 132200
patch</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>580399</commentid>
    <comment_count>3</comment_count>
      <attachid>132200</attachid>
    <who name="WebKit Review Bot">webkit.review.bot</who>
    <bug_when>2012-03-16 05:14:50 -0700</bug_when>
    <thetext>Comment on attachment 132200
patch

Clearing flags on attachment: 132200

Committed r110990: &lt;http://trac.webkit.org/changeset/110990&gt;</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>580400</commentid>
    <comment_count>4</comment_count>
    <who name="WebKit Review Bot">webkit.review.bot</who>
    <bug_when>2012-03-16 05:14:57 -0700</bug_when>
    <thetext>All reviewed patches have been landed.  Closing bug.</thetext>
  </long_desc>
      
          <attachment
              isobsolete="0"
              ispatch="1"
              isprivate="0"
          >
            <attachid>132200</attachid>
            <date>2012-03-15 22:46:06 -0700</date>
            <delta_ts>2012-03-16 05:14:49 -0700</delta_ts>
            <desc>patch</desc>
            <filename>patch-81007</filename>
            <type>text/plain</type>
            <size>1602</size>
            <attacher name="Kishore Bolisetty">kbolisetty</attacher>
            
              <data encoding="base64">SW5kZXg6IFRvb2xzL0NoYW5nZUxvZwo9PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09
PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09Ci0tLSBUb29scy9DaGFuZ2VMb2cJKHJl
dmlzaW9uIDExMDk0NCkKKysrIFRvb2xzL0NoYW5nZUxvZwkod29ya2luZyBjb3B5KQpAQCAtMSwz
ICsxLDE2IEBACisyMDEyLTAzLTE1ICBLaXNob3JlIEJvbGlzZXR0eSAgPGtib2xpc2V0dHlAaW5u
b21pbmRzLmNvbT4KKworICAgICAgICBjaGVjay13ZWJraXQtc3R5bGUgZmFpbHMgb24gTGF5b3V0
VGVzdHMvcGxhdGZvcm0vZWZsL3Rlc3RfZXhwZWN0YXRpb25zLnR4dAorICAgICAgICBodHRwczov
L2J1Z3Mud2Via2l0Lm9yZy9zaG93X2J1Zy5jZ2k/aWQ9ODEwMDcKKworICAgICAgICBSZXZpZXdl
ZCBieSBOT0JPRFkgKE9PUFMhKS4KKworICAgICAgICBBIGRlZGlkY2F0ZWQgYnVpbGQgc2xhdmUg
aXMgYXZhaWxhYmxlIGZvciBlZmwgb24gYnVpbGQud2Via2l0Lm9yZy4gCisgICAgICAgIFRoaXMg
Y2hhbmdlIGFkZCdzIGFuIGVudHJ5IGZvciB0aGUgZWxmIHBvcnQgaW4gbGlzdCBvZiBhdmFpbGFi
bGUgcG9ydCBidWlsZGVycy4KKyAgICAgICAgVGhpcyBpcyByZXF1aXJlZCBiZWNhdXNlIHRlc3Rf
ZXhwZWN0YXRpb25zLnB5IHdoZW4gdHJpZ2dlcmVkLCBsb29rcyBmb3IgYSB2YWxpZCBwb3J0IG9i
amVjdCBmb3IgdGhlIHNwZWNpZmllZCBwb3J0LiAKKworICAgICAgICAqIFNjcmlwdHMvd2Via2l0
cHkvbGF5b3V0X3Rlc3RzL3BvcnQvYnVpbGRlcnMucHk6CisKIDIwMTItMDMtMTUgIEFkYW0gQmFy
dGggIDxhYmFydGhAd2Via2l0Lm9yZz4KIAogICAgICAgICBUaGUgY29tbWl0LXF1ZXVlIHNob3Vs
ZCBmYXN0LXRyYWNrIHBhdGNoZXMgdGhhdCBoYXZlIGFscmVhZHkgcGFzc2VkIHRoZSB0ZXN0aW5n
IEVXUyBib3RzCkluZGV4OiBUb29scy9TY3JpcHRzL3dlYmtpdHB5L2xheW91dF90ZXN0cy9wb3J0
L2J1aWxkZXJzLnB5Cj09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09
PT09PT09PT09PT09PT09PT09PT09PT0KLS0tIFRvb2xzL1NjcmlwdHMvd2Via2l0cHkvbGF5b3V0
X3Rlc3RzL3BvcnQvYnVpbGRlcnMucHkJKHJldmlzaW9uIDExMDkzNSkKKysrIFRvb2xzL1Njcmlw
dHMvd2Via2l0cHkvbGF5b3V0X3Rlc3RzL3BvcnQvYnVpbGRlcnMucHkJKHdvcmtpbmcgY29weSkK
QEAgLTYwLDYgKzYwLDcgQEAgX2V4YWN0X21hdGNoZXMgPSB7CiAgICAgIlF0IExpbnV4IFJlbGVh
c2UiOiB7InBvcnRfbmFtZSI6ICJxdC1saW51eCIsICJzcGVjaWZpZXJzIjogc2V0KFsid2luIiwg
ImxpbnV4IiwgIm1hYyJdKX0sCiAgICAgIldpbmRvd3MgWFAgRGVidWcgKFRlc3RzKSI6IHsicG9y
dF9uYW1lIjogIndpbi14cCIsICJzcGVjaWZpZXJzIjogc2V0KFsid2luIl0pfSwKICAgICAiV2lu
ZG93cyA3IFJlbGVhc2UgKFdlYktpdDIgVGVzdHMpIjogeyJwb3J0X25hbWUiOiAid2luLWZ1dHVy
ZS13azIiLCAic3BlY2lmaWVycyI6IHNldChbIndrMiJdKX0sCisgICAgIkVGTCBMaW51eCBSZWxl
YXNlIjogeyJwb3J0X25hbWUiOiAiZWZsIiwgInNwZWNpZmllcnMiOiBzZXQoWyJlZmwiXSl9LAog
fQogCiAK
</data>

          </attachment>
      

    </bug>

</bugzilla>