Changeset 1481 in tailor

Timestamp:

04/01/08 01:30:42 (5 years ago)

Author:

lele@…

Hash name:

20080331233042-97f81-430d82d385698eddafe7991d7c97e0e463a2a1e1

Message:

Added a note about patch-name-format and its implication in potential hash collisions under darcs

File:

• README.rst (modified) (2 diffs)

README.rst

@@ -579 +579 @@
   The prototype may contain ``%(keyword)s`` such as 'author', 'date',
   'revision', 'firstlogline', 'remaininglog' or 'project'.  It
-  defaults to ``[%(project)s @ %(revision)s]``; setting it to the
-  empty string means that tailor will simply use the original
-  changelog.
+  defaults to ``[%(project)s @ %(revision)s]`` [#]_.
 
   When you set it empty, as in
@@ -606 +604 @@
   Off by default, when active tailor reformats every changelog before
   committing on the target system.
+
+.. [#] Modifying the changelog may have subtle consequences!
+       Under darcs, for example, you may hit issue772_ by producing
+       hash collisions, that happens when two distinct patches carry
+       the same "unique" identifier (the hash is computed using
+       *date*, *author*, *changelog* and other details, but **not**
+       the actual content): the default setting, that adds a
+       differentiating prefix, is safer from that point of view.
+
+.. _issue772: http://bugs.darcs.net/issue772