Proper attribution of patches
Harold L Hunt II
huntharo@msu.edu
Tue Dec 23 21:20:00 GMT 2003
Alan Hourihane wrote:
> On Tue, Dec 23, 2003 at 02:02:20PM -0500, Thomas Dickey wrote:
>
>>On Tue, 23 Dec 2003, Harold L Hunt II wrote:
>>
>>
>>>The following CVS commit, made by Thomas Dickey, has no indication that
>>>Thomas was either a) not involved at all in the patch or b) that Thomas
>>>found Ralf Habacker's patch and committed a modified version of that patch.
>>>
>>>The CVS log message says:
>>> fixes for _XtInherit on cygwin.
>>>
>>>The hw/xfree86/CHANGELOG files says:
>>> XFree86 4.3.99.903 (xx December 2003)
>>> + 699. Fixes to build/run on cygwin (Thomas Dickey).
>>>
>>>I know that this patch was based at least in part (if not entirely) on
>>>Ralf Habacker's patch for the same, since it includes a more than twenty
>>>line comment from Ralf along with his name at the bottom:
>>>
>>>http://cvsweb.xfree86.org/cvsweb/xc/lib/Xt/Initialize.c.diff?r1=3.21&r2=3.22&f=h
>>
>>I'm aware of that.
>>
>>Your commit didn't mention this either. Do you have point?
>
>
> Thomas,
>
> If you did get this code directly from Cygwin/X's tree then I'd of
> expected at least the credit to be apportioned to Harold at the very
> least, rather than putting your name against it. Ralf's name could have
> been corrected later, with a follow email from Harold.
>
> It's a simple change to put that right in the CHANGELOG. So I'll do that.
Thanks Alan!
Harold
More information about the Cygwin-xfree
mailing list