Bug 116016 – [webkitpy] Make FileInfo.repository_name support SVN version > 1.6

• Status: RESOLVED
• Resolution: FIXED
• Component: New Bugs

[Csaba Osztrogonác]

New webkitpy test introduced in r149635 fails on the buildbots.
It fails only in SVN repositories, but passes in GIT repositories. :)

I started digging it, and it seems the problem is somewhere 
near class FileInfo implementation in cpp.py.

from SVN repository:
---------------------
$ Tools/Scripts/test-webkitpy Tools/Scripts/webkitpy/style/checkers/cpp_unittest.py
Suppressing most webkitpy logging while running unit tests.
[1/1] webkitpy.style.checkers.cpp_unittest.CppStyleTest.test_webcore_platform_layering_violation failed:
  Traceback (most recent call last):
    File "/home/oszi/WebKit-svn/Tools/Scripts/webkitpy/style/checkers/cpp_unittest.py", line 251, in test_webcore_platform_layering_violation
      self.assertEqual(1, error_collector.result_list().count(errmsg))
  AssertionError: 1 != 0

Ran 1 test in 0.023s
FAILED (failures=1, errors=0)

from GIT repository:
---------------------
$ Tools/Scripts/test-webkitpy Tools/Scripts/webkitpy/style/checkers/cpp_unittest.py
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)

Ran 1276 tests in 3.063s

OK

[Csaba Osztrogonác]

cpp.py:
----------
def check_for_webcore_platform_layering_violation(filename, clean_lines, line_number, error):
    """Checks for platform-specific code inside WebCore outside of the platform layer."""
    directory = FileInfo(filename).split()[0]
if not match(r'Source/WebCore', directory):
        return
    if match(r'Source/WebCore/platform', directory):
        return


I got the root of the bug. FileInfo(filename).split() returns absolute
path inside SVN repository and relative path inside GIT repository:
- SVN: ('/home/webkitbuildbot/oszi/WebKit-svn/Source/WebCore/loader', 'NavigationAction', '.cpp')
- GIT: ('Source/WebCore/loader', 'NavigationAction', '.cpp')

[Csaba Osztrogonác]

The problem is bigger than I thought before. :-/

The implementation of FileInfo.repository_name is incorrect 
for SVN 1.7, because there isn't .svn directories everywhere, 
only in the root directory.

[Csaba Osztrogonác]

The original implemantation introduced in http://trac.webkit.org/changeset/45857 long long time ago. 

It seems this bug was fixed in the upstream - http://google-styleguide.googlecode.com/svn/trunk/cpplint/cpplint.py

[Csaba Osztrogonác]

Created attachment 201540 [details]
Patch

[WebKit Commit Bot]

Comment on attachment 201540 [details]
Patch

Clearing flags on attachment: 201540

Committed r150028: <http://trac.webkit.org/changeset/150028>

[WebKit Commit Bot]

All reviewed patches have been landed.  Closing bug.