Alexandre,
Can we get the Module name and/or modified files added back to the subject line for CVS? IHMO, this is much more useful than knowing who submitted the patch.
Thankyou, Ron Jensen
On Fri, 2005-12-02 at 09:43 -0600, Alexandre Julliard wrote:
Ron Jensen wino@jentronics.com writes:
Well, I think having some description of what the patch is about is more useful than a truncated file list, but if the general feeling is that the file list is better I can put this back.
On 12/2/05, Alexandre Julliard julliard@winehq.org wrote:
One vote for patch description instead of file list here. The only thing I would like to see that the file list provided is the area affected by the patch. Say one of Rob's OLE patches is committed, then it would be nice to see OLE in the subject somewhere, though it's not essential.
-- James Hawkins
Am Freitag, den 02.12.2005, 21:00 +0100 schrieb Alexandre Julliard:
I would like to see that the Subject-line again start with the Subdir to see the Area, the Patch belongs to. - Easy to filter out the patches, I'm not Interested in. - When find a Patch for an Area that an App has Problems with, it's a good time to test the Functionality of the App again
What about all three informations: a.) author b.) affected directory, like "[dlls/ole32]" c.) beginning of the changelog
An example would be: "Michael Jung: [dlls/shell32] Implemented scrolling during drag&drop for the shellview class"
Another idea would be to replace b.) and c.) by the original patch mail subject. This typically already contains the affected code section, for example "WINED3D: Return the correct parent in IWineD3DPixelShaderImpl_GetParent".
While searching for good examples I good even more comfortable with the second idea: The changelog can be quite long, and only displaying some truncated part of it may not be that good. Using the mail subject instead is better because the patch writer already had the possibility to condense the change into some meaningfull one line-comment.
Regards,
Martin
Am Samstag, den 03.12.2005, 13:43 +0100 schrieb Martin Fuchs:
IMHO, best of all is starting with the affected directory and use the subject for c.
Benefit: Simple sort "by Subject" and the changes for a specific dll/program are all together, no matter, who created the Patch.
"dlls/shell32 [Michael Jung] Implemented scrolling during drag&drop for the shellview class"
---- So, if the part before the colon has no space, try it as a path when not found, try to prefix with "dll/" or "programs/"
When the part is valid, use it as the start of the commit-Subject. When the part is invalid, use the Path of the First affected File as the start of the commit-Subject. ---- Your Example: "WINED3D [Creator] Return the correct parent in IWineD3DPixelShaderImpl_GetParent"
Martin Fuchs fuchs.martin@gmail.com writes:
Yes, I think that's the way to go.
-
Alexandre Julliard -
Detlef Riekenberg -
James Hawkins -
Martin Fuchs -
Ron Jensen