Commit messages / was: Re: CVS Update: xc (branch: trunk)

Roland Mainz roland.mainz at nrubsig.org
Tue Nov 2 14:32:56 PST 2004


Daniel Stone wrote:
> 
> On Sun, Oct 31, 2004 at 03:22:08PM -0800, Roland Mainz wrote:
> >   Fix for https://freedesktop.org/bugzilla/show_bug.cgi?id=1743 - Fix build failure on Solaris due "ld: fatal: too many symbols require small PIC references" via adding |#define LargePICTable YES| to the Imakefile.
> 
> Please wrap your commit messages at 80 columns, and clearly indicate
> which files are being changed.  I suggest the following format:
> 
> * whereeveritwas/Imakefile:
> Add '#define LargePICTable YES' to work around Solaris build failure due
> to too many symbols requiring small PIC references.

Yeah, I should stop using the xx@@@!-'-m' option of cvs and switch over
to log files and paste there the Changelog entry.

> When looking through that with 'cvs log', your entry would be largely
> useless to me.

Daniel:
There is always a HTTPS link (e.g. you can click on that in your browser
or email program) to the bugzilla bug which _always_ (the only exception
is when I update the Xprint FAQ as the changes to the generated HTML and
TXT files are useless) has the patch which was commited, 1:1, including
the Changelog diff. And you will even find the output of "cvs commit"
there. I hoped that all these information isn't that useless...

----

Bye,
Roland

-- 
  __ .  . __
 (o.\ \/ /.o) roland.mainz at nrubsig.org
  \__\/\/__/  MPEG specialist, C&&JAVA&&Sun&&Unix programmer
  /O /==\ O\  TEL +49 641 7950090
 (;O/ \/ \O;)



More information about the xorg mailing list