[Webkit-unassigned] [Bug 75824] check-webkit-style should warn about missing svn:mime-type for png files

[bugzilla-daemon at webkit.org]

https://bugs.webkit.org/show_bug.cgi?id=75824





--- Comment #4 from Balazs Ankes <bank at inf.u-szeged.hu>  2012-02-28 02:04:19 PST ---
(In reply to comment #3)

Thank you the answer.

> (From update of attachment 129058 [details])
> View in context: https://bugs.webkit.org/attachment.cgi?id=129058&action=review
> 
> I think checking to see if the user's subversion config file is correct is useful, however, that doesn't actually fix this specific bug where certain files might not have the right svn properties set.  I.e., this should probably be a separate bug.
> 
> > Tools/Scripts/webkitpy/style/checkers/png.py:25
> >  """Supports checking WebKit style in png files."""
> 
> This diff is weird.  I don't see png.py in the tree currently.
Because it's a new file, I added to my local git repo.
> 
> > Tools/Scripts/webkitpy/style/checkers/png.py:42
> > +            for line in open(os.path.expanduser("~") + "/.subversion/config", 'r').readlines():
> 
> Does this work with Windows python?  Does this work with Windows svn?
Good questions, I will check these.
> 
> > Tools/Scripts/webkitpy/style/patchreader.py:65
> > +                _log.debug("Using class: " + checker.__class__.__name__)
> 
> I would remove this log line.
Ok.

-- 
Configure bugmail: https://bugs.webkit.org/userprefs.cgi?tab=email
------- You are receiving this mail because: -------
You are the assignee for the bug.