Bug 134954 - [EFL] Remove EFL WK1 directory from the checker.py
Summary: [EFL] Remove EFL WK1 directory from the checker.py
Status: RESOLVED FIXED
Alias: None
Product: WebKit
Classification: Unclassified
Component: WebKit EFL (show other bugs)
Version: 528+ (Nightly build)
Hardware: Unspecified Unspecified
: P2 Normal
Assignee: Ryuan Choi
URL:
Keywords:
Depends on:
Blocks: 134087
  Show dependency treegraph
 
Reported: 2014-07-15 18:00 PDT by Ryuan Choi
Modified: 2014-07-16 03:07 PDT (History)
5 users (show)

See Also:


Attachments
Patch (1.52 KB, patch)
2014-07-15 18:05 PDT, Ryuan Choi
no flags Details | Formatted Diff | Diff

Note You need to log in before you can comment on or make changes to this bug.
Description Ryuan Choi 2014-07-15 18:00:32 PDT
SSIA
Comment 1 Ryuan Choi 2014-07-15 18:05:15 PDT
Created attachment 234968 [details]
Patch
Comment 2 Gyuyoung Kim 2014-07-15 21:15:15 PDT
Comment on attachment 234968 [details]
Patch

LGTM
Comment 3 WebKit Commit Bot 2014-07-15 21:34:04 PDT
Comment on attachment 234968 [details]
Patch

Clearing flags on attachment: 234968

Committed r171137: <http://trac.webkit.org/changeset/171137>
Comment 4 WebKit Commit Bot 2014-07-15 21:34:09 PDT
All reviewed patches have been landed.  Closing bug.
Comment 5 Carlos Alberto Lopez Perez 2014-07-16 03:03:11 PDT
This broke 1 webkitpy test

$ Tools/Scripts/test-webkitpy 
Suppressing most webkitpy logging while running unit tests.
Skipping tests in the following modules or packages because they are really, really, slow:
    webkitpy.common.checkout.scm.scm_unittest
    (https://bugs.webkit.org/show_bug.cgi?id=31818; use --all to include)

Checking imports .../home/clopez/webkit/webkit
[831/1424] webkitpy.style.checker_unittest.GlobalVariablesTest.test_path_rules_specifier failed:                                      
  Traceback (most recent call last):
    File "/home/clopez/webkit/webkit/Tools/Scripts/webkitpy/style/checker_unittest.py", line 225, in test_path_rules_specifier
      "readability/naming")
    File "/home/clopez/webkit/webkit/Tools/Scripts/webkitpy/style/checker_unittest.py", line 212, in assertNoCheck
      self.assertFalse(config.should_check(category, path), message)
  AssertionError: True is not False : Should not check category "readability/naming" for path "Source/WebKit/efl/ewk/ewk_view.h".
  
Ran 1424 tests in 6.008s                                                                                                                  
FAILED (failures=1, errors=0)



http://build.webkit.org/builders/Apple%20Mavericks%20Release%20WK1%20%28Tests%29/builds/7381
Comment 6 Gyuyoung Kim 2014-07-16 03:04:04 PDT
(In reply to comment #5)
> This broke 1 webkitpy test
> 
> $ Tools/Scripts/test-webkitpy 
> Suppressing most webkitpy logging while running unit tests.
> Skipping tests in the following modules or packages because they are really, really, slow:
>     webkitpy.common.checkout.scm.scm_unittest
>     (https://bugs.webkit.org/show_bug.cgi?id=31818; use --all to include)
> 
> Checking imports .../home/clopez/webkit/webkit
> [831/1424] webkitpy.style.checker_unittest.GlobalVariablesTest.test_path_rules_specifier failed:                                      
>   Traceback (most recent call last):
>     File "/home/clopez/webkit/webkit/Tools/Scripts/webkitpy/style/checker_unittest.py", line 225, in test_path_rules_specifier
>       "readability/naming")
>     File "/home/clopez/webkit/webkit/Tools/Scripts/webkitpy/style/checker_unittest.py", line 212, in assertNoCheck
>       self.assertFalse(config.should_check(category, path), message)
>   AssertionError: True is not False : Should not check category "readability/naming" for path "Source/WebKit/efl/ewk/ewk_view.h".
> 
> Ran 1424 tests in 6.008s                                                                                                                  
> FAILED (failures=1, errors=0)
> 
> 
> 
> http://build.webkit.org/builders/Apple%20Mavericks%20Release%20WK1%20%28Tests%29/builds/7381

I filed a bug for it - https://bugs.webkit.org/show_bug.cgi?id=134967

Could you review it ?
Comment 7 Carlos Alberto Lopez Perez 2014-07-16 03:07:27 PDT
> I filed a bug for it - https://bugs.webkit.org/show_bug.cgi?id=134967
> 
> Could you review it ?

I'm not a reviewer.

Maybe asking on the IRC would help.

Thanks.