Bug 55468 – New .png files added by commit-queue bots are not marked as binary

• Status: NEW
• Component: Tools / Tests

[Adam Roben (:aroben)]

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.

[Adam Roben (:aroben)]

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

[Adam Roben (:aroben)]

Note that auto-props are respected even when committing via git-svn.

[Eric Seidel (no email)]

Very easy to do.  (Especially if there is a svn config command to do that.)

[Eric Seidel (no email)]

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.

[Eric Seidel (no email)]

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.

[Adam Roben (:aroben)]

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

[Alan Cutter]

bug 107724 addresses this issue and has a patch to amend future bots, pending a cq+.