diff options
| author | Joey Hess <joey@gnu.kitenet.net> | 2009-10-02 15:45:14 -0400 |
|---|---|---|
| committer | Joey Hess <joey@gnu.kitenet.net> | 2009-10-02 15:45:14 -0400 |
| commit | 53a8aeb1d1cfd42ae069e173437b34489d28d157 (patch) | |
| tree | 2c617466e00d4a0cee26e339db22d925bfe56f21 /doc/bugs/transitive_dependencies.mdwn | |
| parent | 4f9c5896b242ac08be181047ad426bd458a0bf49 (diff) | |
| download | ikiwiki-53a8aeb1d1cfd42ae069e173437b34489d28d157.tar ikiwiki-53a8aeb1d1cfd42ae069e173437b34489d28d157.tar.gz | |
implemented one fix in transitive-dependencies branch, but not really happy with it
Diffstat (limited to 'doc/bugs/transitive_dependencies.mdwn')
| -rw-r--r-- | doc/bugs/transitive_dependencies.mdwn | 25 |
1 files changed, 20 insertions, 5 deletions
diff --git a/doc/bugs/transitive_dependencies.mdwn b/doc/bugs/transitive_dependencies.mdwn index c61afe81e..89f0d7085 100644 --- a/doc/bugs/transitive_dependencies.mdwn +++ b/doc/bugs/transitive_dependencies.mdwn @@ -25,9 +25,15 @@ everything uses) will just result in semi-redundant dependency info being recorded. If we make destpage mandatory to fix this, goofing up will lead to this bug coming back. Ugh. -Another approach to fix it could be to say that anything that causes a +---- + +## rebuild = change approach + +[[!template id=gitbranch branch=master/transitive-dependencies author="[[joey]]"]] + +Another approach to fix it is to say that anything that causes a rebuild of B is treated as a change of B. Then when C is changed, B is -rebuilt due to dependencies, and in turn this means A is rebuild because B +rebuilt due to dependencies, and in turn this means A is rebuilt because B "changed". This is essentially what is done with wikilinks now, and why, if a sidebar @@ -40,8 +46,17 @@ here: building contact.mdwn, which depends on sidebar ... -The only downside I can see with this approach is that it involves more work. -Does the dep resolver have to keep looping until no new pages are rebuilt? -Seems worth a try to implement this approach. +Downsides here: + +* Means a minimum of 2x as much time spent resolving dependencies, + at least in my simple implementation, which re-runs the dependency + resolution loop until no new pages are rebuilt. +* Causes extra work for some transitive dependencies that we don't + actually care about. For example, changing index causes + plugins/brokenlinks to update in the first pass; if there's a second + pass, plugins/map is then updated, because it depends on plugins/brokenlinks. + (Of course, this is just a special case of the issue that a real + modification to plugins/brokenlinks causes an unnecessary update of plugins/map, + because we have only one kind of dependency.) --[[Joey]] |