Opened 10 years ago

Closed 10 years ago

#496 closed task (fixed)

copyright-string is not included in cache hash

Reported by: Peter Johansson Owned by: Peter Johansson
Priority: minor Milestone: svndigest 0.10
Component: core Version: trunk
Keywords: Cc:

Description

The config value for copyright-string is not included in the cache hash. The purpose of the cache hash is to decide whether the configuration used to generate the cache would generate the same stats as the current configuration. Changing the copyright-string would likely change the stats (most likely flip lines between copyright and docs). Is it worth including this in cache hash? If we do we should think about how to be backward compatible, in other words, we don't want all cache files to be outdated just because this is missing in the hash. At minimum the hash should be valid when copyright-string is missing in hash and default value is used in current configuration, Copyright (C). Perhaps the value should only be written into hash when it's non-default?

Change History (2)

comment:1 Changed 10 years ago by Peter Johansson

Milestone: svndigest 0.x+svndigest 0.10
Owner: changed from Jari Häkkinen to Peter Johansson
Status: newassigned

comment:2 Changed 10 years ago by Peter Johansson

Resolution: fixed
Status: assignedclosed

(In [1348]) include 'copyright string' in cache. fixes #496

Note: See TracTickets for help on using tickets.