[PATCH] Extend the Hacker's Guide to explain how to log property changes

[Jeremy Whitlock <jc...@gmail.com>]

Hi All,
    Attached, and pasted below, is a patch that adds a little content
to the Hacker's Guide that suggests how to write log messages for
property changes.  Please feel free to let me know what changes, if
any, need to happen to get this approved.

-- 
Take care,

Jeremy Whitlock
http://www.thoughtspark.org




[[[
* www/hacking.html: Added content on how to log property changes.
]]]

Index: www/hacking.html
===================================================================
--- www/hacking.html	(revision 33188)
+++ www/hacking.html	(working copy)
@@ -1572,6 +1572,17 @@
 `handle_parser_warning', but only because the two structures were
 mentioned by full name elsewhere in the log entry.</p>

+<p>You should also include property changes in your log messages.
+For example, if you were to modify the "svn:ignore" property on
+the trunk, you might put something like this in your log:</p>
+
+<pre>
+   * trunk/ (svn:ignore): Ignore build.
+</pre>
+
+<p>The above only applies to properties you maintain, not those
+maintained by subversion like "svn:mergeinfo".</p>
+
 <p>Note how each file gets its own entry prefixed with an "*", and the
 changes within a file are grouped by symbol, with the symbols listed
 in parentheses followed by a colon, followed by text describing the

Re: [PATCH] Extend the Hacker's Guide to explain how to log property changes

[Karl Fogel <kf...@red-bean.com>]

"Jeremy Whitlock" <jc...@gmail.com> writes:
>     Attached, and pasted below, is a patch that adds a little content
> to the Hacker's Guide that suggests how to write log messages for
> property changes.  Please feel free to let me know what changes, if
> any, need to happen to get this approved.

Looks good to me; +1 to commit.

---------------------------------------------------------------------
To unsubscribe, e-mail: dev-unsubscribe@subversion.tigris.org
For additional commands, e-mail: dev-help@subversion.tigris.org