Ojan, There are also quite a few files with the svn:executable property set (359 in LayoutTests alone, by my count), I think most of which are erroneous.
Philip On Thu, Mar 8, 2012 at 3:05 PM, Ojan Vafai <o...@chromium.org> wrote: > Whoops. A lot of these were from me yesterday. Sorry, I didn't realize I > didn't have my subversion config set correctly. > > I went to fix up my commits from yesterday and realized that a very large > percentage of the pngs in the LayoutTests tree have the wrong > svn:mime-type. Is anyone opposed to me doing a bulk fix for all the pngs? > > find LayoutTests | grep \.png$ | grep -v \.svn | xargs svn ps > svn:mime-type image/png > > > > On Thu, Mar 8, 2012 at 1:52 AM, Ashod Nakashian > <ashodnakash...@yahoo.com>wrote: > >> Please let's also enforce svn:eol-style to LF for all scripts as this is >> breaking (at least) the bash scripts that are checked-out with native style >> on Windows. Some bash versions don't like CR. Please see a (failed) >> attempt at fixing this manually[1]. >> >> I also believe we should mark all (Bash/Perl/Python) scripts as >> executable. It's best to at least automate it, if not also check for >> violations via check-webkit-style. (And for the loving of all that's good, >> would someone please help with this bug[1]?) >> >> [1] https://bugs.webkit.org/show_bug.cgi?id=79509 >> >> -Ash >> >> ------------------------------ >> *From:* Simon Fraser <simon.fra...@apple.com> >> *To:* Eric Seidel <e...@webkit.org> >> *Cc:* WebKit Development <webkit-dev@lists.webkit.org> >> *Sent:* Thursday, March 8, 2012 3:37 AM >> *Subject:* Re: [webkit-dev] Please set the svn:mime-type property on >> binary files before committing >> >> The best way to enforce it would be with a pre-commit hook: >> <https://bugs.webkit.org/show_bug.cgi?id=80548> >> >> Simon >> >> On Mar 7, 2012, at 3:33 PM, Eric Seidel wrote: >> >> > Unless this is enforced by a tool, it's very likely to be forgotten. >> > >> > https://bugs.webkit.org/show_bug.cgi?id=75824 >> > https://bugs.webkit.org/show_bug.cgi?id=75825 >> > >> > >> > On Wed, Mar 7, 2012 at 3:21 PM, Dan Bernstein <m...@apple.com> wrote: >> >> Please set the svn:mime-type property on binary files that you add to >> the >> >> tree, such as *-expected.png, before committing. Otherwise the >> resulting >> >> webkit-changes message will include those files as text, which is >> >> inconvenient. >> >> >> >> Thanks. >> >> >> >> _______________________________________________ >> >> webkit-dev mailing list >> >> webkit-dev@lists.webkit.org >> >> http://lists.webkit.org/mailman/listinfo.cgi/webkit-dev >> >> >> > _______________________________________________ >> > webkit-dev mailing list >> > webkit-dev@lists.webkit.org >> > http://lists.webkit.org/mailman/listinfo.cgi/webkit-dev >> >> _______________________________________________ >> webkit-dev mailing list >> webkit-dev@lists.webkit.org >> http://lists.webkit.org/mailman/listinfo.cgi/webkit-dev >> >> >> >> _______________________________________________ >> webkit-dev mailing list >> webkit-dev@lists.webkit.org >> http://lists.webkit.org/mailman/listinfo.cgi/webkit-dev >> >> > > _______________________________________________ > webkit-dev mailing list > webkit-dev@lists.webkit.org > http://lists.webkit.org/mailman/listinfo.cgi/webkit-dev > >
_______________________________________________ webkit-dev mailing list webkit-dev@lists.webkit.org http://lists.webkit.org/mailman/listinfo.cgi/webkit-dev