Bug 55468 - New .png files added by commit-queue bots are not marked as binary
Summary: New .png files added by commit-queue bots are not marked as binary
Status: NEW
Alias: None
Product: WebKit
Classification: Unclassified
Component: Tools / Tests (show other bugs)
Version: 528+ (Nightly build)
Hardware: All All
: P2 Normal
Assignee: Nobody
URL:
Keywords:
Depends on: 107724
Blocks:
  Show dependency treegraph
 
Reported: 2011-03-01 07:37 PST by Adam Roben (:aroben)
Modified: 2013-02-28 15:50 PST (History)
5 users (show)

See Also:


Attachments

Note You need to log in before you can comment on or make changes to this bug.
Description Adam Roben (:aroben) 2011-03-01 07:37:08 PST
Whenever the commit-queue commits a patch that adds new .png files, the .pngs aren't marked as binary. The effect of this most noticeable to me is that the webkit-changes email generated for these patches includes a bunch of PNG-as-ASCII junk which significantly bloats the email.
Comment 1 Adam Roben (:aroben) 2011-03-01 07:38:09 PST
One way to fix this would be to turn on Subversion auto-props on the commit-queue bots. To do that, you just need to uncomment the following lines in ~/.subversion/config

enable-auto-props = yes

...and later...

*.png = svn:mime-type=image/png
Comment 2 Adam Roben (:aroben) 2011-03-01 07:38:42 PST
Note that auto-props are respected even when committing via git-svn.
Comment 3 Eric Seidel (no email) 2011-03-01 09:45:01 PST
Very easy to do.  (Especially if there is a svn config command to do that.)
Comment 4 Eric Seidel (no email) 2011-03-01 09:48:04 PST
I suspect we should create the perfect config file for the commit-queue, and then copy it into the svn repository.  Then the commit bots can either install it automatically, or use --config-dir.

It doesn't appear that svn has a "config" command like git does for editing its config files.
Comment 5 Eric Seidel (no email) 2011-03-01 09:49:14 PST
Basically if someone tells me exactly what commands to run on the bots, I'm happy to make that happen.  Even better would be pointing towards the exact desired config file, which I'm happy to make sure all of the bots use.
Comment 6 Adam Roben (:aroben) 2011-03-01 10:15:12 PST
I think putting this in ~/.subversion/config will do what we want (and nothing more):

[miscellany]
enable-auto-props = yes

[auto-props]
*.png = svn:mime-type=image/png
Comment 7 Alan Cutter 2013-02-28 15:44:50 PST
bug 107724 addresses this issue and has a patch to amend future bots, pending a cq+.