<!DOCTYPE HTML PUBLIC "-//W3C//DTD HTML 4.01 Transitional//EN" "http://www.w3.org/TR/html4/loose.dtd">

<html><head>

    <meta http-equiv="Content-Type" content="text/html; charset=utf-8">

    <meta name="generator" content="Osso Notes">

    <title></title></head>

<body>

<p>----- Oorspronkelijk bericht -----

<br>> On 1/23/2011 9:30 PM, Milian Wolff wrote:

<br>> > What was broken?

<br>> 

<br>> Nothing as far as Commitfilter is concerned. Commitfilter actually

> acts on the X-Commit-Directories headers, not the mail subject.

<br>> This header was actually somewhat broken in the old mails, it has

<br>> been fixed now as part of the hooks rewrite that also brought with

> it the mail subject format change.

<br>> 

<br>> In total the rewrite fixed numerous bugs, many of which were in the

<br>> emails (the old implementation wasn't written by the sysadmin team):

<br>> There were problems with keyword parsing, CCMAIL recipients were not

<br>> all CC'd to the kde-commits mail but rather individual, new mails

<br>> were sent, headers with non-Unicode characters in them weren't en-

<br>> coded properly, authors didn't get CC'd if the hook found something

<br>> unsafe in the diff, license detection for new files was suboptimal,

<br>> etc.

<br>> 

<br>> The rewrite also improved performance by orders of magnitude,

<br>> especially in the auditing steps, and added new features. For

> example the mails now mention who pushed a commit.

<br>> 

<br>> Yet all we hear about is rather aggressive whining about the mail

> subjects ... yep, being a sysadmin is tons of fun.

<br>

Hey, look at it differently. If everyone is talking about the subject as the biggest problem, Ben really, really did a great job. I was expecting a hell of a lot more bugs in the brand new code. People complaining about the subject made me smile actually. It is like building a gigantic big house and the owners are complaing about the color of the doorbell. Ill settle for that.

<br>

<br>Best,

<br>

<br>toma<br></p>

</body>

</html>