[cairo] [PATCH] Restore glitz-surface-source test

Chris Wilson chris at chris-wilson.co.uk
Wed Nov 26 09:09:09 PST 2008


On Wed, 2008-11-26 at 16:12 +0100, Paolo Bonzini wrote:
> Hi all,
> 
> the glitz-surface-source test is currently disabled.  The attached patch
> adds support for the AGL backend (in theory you'd have separate tests
> for each backend) and adds the missing PostScript reference images that
> are needed for it to pass.  It can be applied with "git am" (it includes
> file copies).
> 
> Ok?  Can it be committed for me?

Pushed all 3 patches - though I had to add commentary for the 'why' part
of the commit log.

Carl's guidelines for the change log is a single line header to describe
what changed followed by a paragraph (or more depending upon the
complexity) of why. Even for a simple compile fix, taking the time to
explain why the fix is needed is useful for other developers reading the
commit log. And the second aspect of the change log is that in a few
months time, Carl will need to read through the changes and produce a
summary of our brilliance for the NEWS and release announcements.

An addendum to those basic principles, we have evolved a system of
using:
[subsystem] Summary of change
for the single line header, which helps when scanning through the logs. 
-- 
Chris



More information about the cairo mailing list