On 12 January 2015 at 18:39, Julian Foad <julianf...@btopenworld.com> wrote: > Ivan Zhakov wrote: >>>>> URL: http://svn.apache.org/r1649029 >>>>> Log: >>>>> Sync FSX with FSFS: Merge DAG-related sub-pool introduction >>>>> patches r1647905, r1648243, r1648253 and r1648272 from FSFS >>>>> and resolve the usual text conflicts due to naming differences. >>>>> Block revisions r1648230, r1648238, r1648241, r1648242 and r1648532. > >> Also, it seems that this commit has another problems: >> 1. It doesn't have proper log message [1]. >> 2. It mixes different unrelated changes in one commit, while it will >> be much easier to review them separately. > > There is nothing wrong here. The log message says what's merged, and that is > four revisions all described as "DAG-related sub-pool introduction", which > means they are all parts of the same change - not unrelated changes. >
Subversion community guide is very specific about proper log messages and I don't understand why this commit (and other previous FSX "merge" commits") does not follow it. Quoting Subversion community guide [1] again: [[[ The log message should name every affected function, variable, macro, makefile target, grammar rule, etc, including the names of symbols that are being removed in this commit. This helps people searching through the logs later. Don't hide names in wildcards, because the globbed portion may be what someone searches for later. ]]] There is very good reason that every affected identifier should be mentioned in log message. [1] https://subversion.apache.org/docs/community-guide/conventions.html#log-messages -- Ivan Zhakov
