From f81471913f7b1ae2d208f4568c6812d41a8d3054 Mon Sep 17 00:00:00 2001 From: "http://hendry.iki.fi/" Date: Fri, 31 Jul 2009 05:24:02 -0400 Subject: [PATCH 1/7] --- doc/bugs/html5_support.mdwn | 4 ---- 1 file changed, 4 deletions(-) diff --git a/doc/bugs/html5_support.mdwn b/doc/bugs/html5_support.mdwn index 783f5e47c..239474275 100644 --- a/doc/bugs/html5_support.mdwn +++ b/doc/bugs/html5_support.mdwn @@ -37,10 +37,6 @@ This element is poorly supported by browsers. As a workaround, `style.css` needs Internet Explorer will display it as a block, though you can't seem to be able to further control the style. -# Validator complains about no h1-h6 in header - -* [#509](http://bugzilla.validator.nu/show_bug.cgi?id=509) - ## Time element The [time element](http://www.whatwg.org/specs/web-apps/current-work/multipage/text-level-semantics.html#the-time-element) ideally needs the datatime= attribute set by a template variable with what [HTML5 defines as a valid datetime string](http://www.whatwg.org/specs/web-apps/current-work/multipage/infrastructure.html#valid-global-date-and-time-string). From e93636b2f3a3a2678145f20cb26ca31e97e9d545 Mon Sep 17 00:00:00 2001 From: "http://schmonz.livejournal.com/" Date: Fri, 31 Jul 2009 15:21:29 -0400 Subject: [PATCH 2/7] brain gradually engaging --- doc/post-commit/discussion.mdwn | 2 ++ 1 file changed, 2 insertions(+) diff --git a/doc/post-commit/discussion.mdwn b/doc/post-commit/discussion.mdwn index bbe529106..3caa07a85 100644 --- a/doc/post-commit/discussion.mdwn +++ b/doc/post-commit/discussion.mdwn @@ -60,3 +60,5 @@ I was able to work out that when I'm seeing this page save misbehavior, my plugi Can you offer an educated guess what's going wrong here? --[[Schmonz]] > Got `rcs_recentchanges` working, believe it or not, thanks to [cvsps](http://www.cobite.com/cvsps/). If I can figure out this interaction between the post-commit hook and `cvs add` on directories, the CVS plugin is mostly done. Could it be a locking issue? Where should I be looking? Any suggestions appreciated. --[[Schmonz]] + +>> Okay, it is definitely a locking issue. First, on the conjecture that `cvs add ` was triggering the post-commit hook and confusing ikiwiki, I wrapped the ikiwiki post-commit binary with a shell script that exited 0 if the triggering file was a directory. The first half of the conjecture was correct -- my wrapper got triggered -- but the web add of `one/two/three.mdwn` (where `one` and `two` weren't existing CVS-controlled dirs) remained hung as before. There were two ikiwiki processes running. On a whim, I killed the one with the higher PID; `cvs add one` immediately completed successfully, then back to a hang and two ikiwiki processes. I killed the newer one again and then `cvs add one/two` and `cvs add one/two/three.mdwn` completed and the web add was successful. From ba1c3553531893c107d09f4693cacdc0cb29c88c Mon Sep 17 00:00:00 2001 From: "http://smcv.pseudorandom.co.uk/" Date: Fri, 31 Jul 2009 17:14:50 -0400 Subject: [PATCH 3/7] I now realise that one of trail's modes is a bit flawed... --- doc/plugins/contrib/trail.mdwn | 11 +++++++++++ 1 file changed, 11 insertions(+) diff --git a/doc/plugins/contrib/trail.mdwn b/doc/plugins/contrib/trail.mdwn index 52dea52d6..eb233e095 100644 --- a/doc/plugins/contrib/trail.mdwn +++ b/doc/plugins/contrib/trail.mdwn @@ -13,6 +13,17 @@ This plugin can benefit from [[another_of_my_branches|todo/inline_plugin:_specifying_ordered_page_names]] but does not require it. +Bugs: + +* \[[!inline pages="..." trail=yes]] currently tries to work out + what pages are in the trail, and their order, at scan time. That + won't work, because matching a pagespec at scan time is + unreliable - pages we want might not have been scanned yet! I + haven't worked out a solution for this. I think + \[[!inline pagenames="..." trail=yes]] + ([[patch_required|todo/inline_plugin:_specifying_ordered_page_names]]) + would be safe, though. + ---- [[!template id=plugin name=trail author="[[Simon_McVittie|smcv]]"]] From 612505d76bb338b243b9cac1f8b1a4291dd7c3e1 Mon Sep 17 00:00:00 2001 From: "http://smcv.pseudorandom.co.uk/" Date: Fri, 31 Jul 2009 17:18:20 -0400 Subject: [PATCH 4/7] Self-review... --- doc/plugins/contrib/album.mdwn | 7 +++++++ 1 file changed, 7 insertions(+) diff --git a/doc/plugins/contrib/album.mdwn b/doc/plugins/contrib/album.mdwn index f550ca64c..395c99bce 100644 --- a/doc/plugins/contrib/album.mdwn +++ b/doc/plugins/contrib/album.mdwn @@ -65,6 +65,13 @@ The next/previous links are themselves implemented by photo, using a special template (by default `albumnext.tmpl` or `albumprev.tmpl`), in `archive`/`quick` mode. +> With hindsight, using an inline here is wrong - I should just +> run hooks and fill in the template within the album plugin. +> inline has some specialized functionality that's overkill +> here, and its delayed HTML substitution breaks the ability +> to have previous/up/next links both above and below the +> photo, for instance. --[[smcv]] + ## Writing the album The album contains one `\[[!album]]` directive. It may also From faa29aaaf33af916ebb3936d9d2bcad32b8aedd6 Mon Sep 17 00:00:00 2001 From: "http://schmonz.livejournal.com/" Date: Fri, 31 Jul 2009 18:17:53 -0400 Subject: [PATCH 5/7] add working CVS plugin --- doc/plugins/contrib/cvs.mdwn | 31 +++++++++++++++++++++++++++++++ 1 file changed, 31 insertions(+) create mode 100644 doc/plugins/contrib/cvs.mdwn diff --git a/doc/plugins/contrib/cvs.mdwn b/doc/plugins/contrib/cvs.mdwn new file mode 100644 index 000000000..2b51bee2b --- /dev/null +++ b/doc/plugins/contrib/cvs.mdwn @@ -0,0 +1,31 @@ +[[!template id=plugin name=cvs core=0 author="[[schmonz]]"]] + +This plugin allows ikiwiki to use [[!wikipedia desc="CVS" Concurrent Versions System]] as an [[rcs]]. + +* Diffs are against [[3.14159|news/version_3.14159]]. `cvs.pm` started life as a copy of `svn.pm`. +* `IkiWiki.pm` avoids copying CVS metadata into `$DESTDIR`. +* [[ikiwiki-makerepo]]: + * creates a repository, + * imports `$SRCDIR` into top-level module `ikiwiki` (vendor tag IKIWIKI, release tag PRE_CVS), + * creates a small post-commit wrapper to prevent `cvs add ` from being seen by ikiwiki's [[post-commit]] hook, + * configures the wrapper itself as a post-commit hook in `CVSROOT/loginfo`. +* [`cvsps`](http://www.cobite.com/cvsps/) is required (`rcs_recentchanges()` and `rcs_diff()` need it to work). +* CVS multi-directory commits happen separately; the post-commit hook sees only the first directory's changes in time for [[recentchanges|plugins/recentchanges]]. The next run of `ikiwiki --setup` will correctly re-render such a recentchanges entry. It might be possible to solve this problem with scripts like `commit_prep` and `log_accum` from CVS contrib. +* Due to the name of CVS's metadata directories, it's impossible to create `.../CVS/foo.mdwn`. On case-insensitive filesystems it's also impossible to create `.../cvs/foo.mdwn`. +* No testing or special-casing has been done with [[attachments|plugins/attachment]], but they'll probably need `cvs add -kb`. + +Having a `$HOME/.cvsrc` isn't necessary. Sure does make using CVS more livable, though. Here's a good general-purpose one: + + cvs -q + checkout -P + update -dP + diff -u + rdiff -u + +Not knowing how the tests get set up, I blindly attempted to add subversion-like tests to `t/file_pruned.t`. They fail. But the plugin definitely works. :-) + +### Code +* [`cvs.pm`](http://www.netbsd.org/~schmonz/ikiwiki-cvs/cvs.pm) +* [`cvs-IkiWiki.pm.diff`](http://www.netbsd.org/~schmonz/ikiwiki-cvs/cvs-IkiWiki.pm.diff) +* [`cvs-ikiwiki-makerepo.diff`](http://www.netbsd.org/~schmonz/ikiwiki-cvs/cvs-ikiwiki-makerepo.diff) +* [`cvs-t-file_pruned.t.diff`](http://www.netbsd.org/~schmonz/ikiwiki-cvs/cvs-t-file_pruned.t.diff) From 28b390c19aaf10b334236caed11c7a503e5198cc Mon Sep 17 00:00:00 2001 From: "http://schmonz.livejournal.com/" Date: Fri, 31 Jul 2009 18:32:56 -0400 Subject: [PATCH 6/7] voila --- doc/post-commit/discussion.mdwn | 4 +++- 1 file changed, 3 insertions(+), 1 deletion(-) diff --git a/doc/post-commit/discussion.mdwn b/doc/post-commit/discussion.mdwn index 3caa07a85..52642c36b 100644 --- a/doc/post-commit/discussion.mdwn +++ b/doc/post-commit/discussion.mdwn @@ -61,4 +61,6 @@ Can you offer an educated guess what's going wrong here? --[[Schmonz]] > Got `rcs_recentchanges` working, believe it or not, thanks to [cvsps](http://www.cobite.com/cvsps/). If I can figure out this interaction between the post-commit hook and `cvs add` on directories, the CVS plugin is mostly done. Could it be a locking issue? Where should I be looking? Any suggestions appreciated. --[[Schmonz]] ->> Okay, it is definitely a locking issue. First, on the conjecture that `cvs add ` was triggering the post-commit hook and confusing ikiwiki, I wrapped the ikiwiki post-commit binary with a shell script that exited 0 if the triggering file was a directory. The first half of the conjecture was correct -- my wrapper got triggered -- but the web add of `one/two/three.mdwn` (where `one` and `two` weren't existing CVS-controlled dirs) remained hung as before. There were two ikiwiki processes running. On a whim, I killed the one with the higher PID; `cvs add one` immediately completed successfully, then back to a hang and two ikiwiki processes. I killed the newer one again and then `cvs add one/two` and `cvs add one/two/three.mdwn` completed and the web add was successful. +>> Okay, it is definitely a locking issue. First, on the conjecture that `cvs add ` was triggering the post-commit hook and confusing ikiwiki, I wrapped the ikiwiki post-commit binary with a shell script that exited 0 if the triggering file was a directory. The first half of the conjecture was correct -- my wrapper got triggered -- but the web add of `one/two/three.mdwn` (where `one` and `two` weren't existing CVS-controlled dirs) remained hung as before. There were two ikiwiki processes running. On a whim, I killed the one with the higher PID; `cvs add one` immediately completed successfully, then back to a hang and two ikiwiki processes. I killed the newer one again and then `cvs add one/two` and `cvs add one/two/three.mdwn` completed and the web add was successful. --[[Schmonz]] + +>>> Aaaaaand I was wrong about the second half of the conjecture being wrong. The wrapper script wasn't correctly identifying directories; with that fixed, everything works. I've created a [[plugins/contrib/cvs]] plugin page. Thanks for listening. :-) --[[Schmonz]] From 216a349bb92af7d408ec076b6f6f740ac735cdcf Mon Sep 17 00:00:00 2001 From: "http://schmonz.livejournal.com/" Date: Fri, 31 Jul 2009 18:37:58 -0400 Subject: [PATCH 7/7] specify where in IkiWiki.pm CVS metadata is avoided (wiki_file_prune_regexps) --- doc/plugins/contrib/cvs.mdwn | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/doc/plugins/contrib/cvs.mdwn b/doc/plugins/contrib/cvs.mdwn index 2b51bee2b..1ff71d274 100644 --- a/doc/plugins/contrib/cvs.mdwn +++ b/doc/plugins/contrib/cvs.mdwn @@ -3,7 +3,7 @@ This plugin allows ikiwiki to use [[!wikipedia desc="CVS" Concurrent Versions System]] as an [[rcs]]. * Diffs are against [[3.14159|news/version_3.14159]]. `cvs.pm` started life as a copy of `svn.pm`. -* `IkiWiki.pm` avoids copying CVS metadata into `$DESTDIR`. +* `IkiWiki.pm:wiki_file_prune_regexps` avoids copying CVS metadata into `$DESTDIR`. * [[ikiwiki-makerepo]]: * creates a repository, * imports `$SRCDIR` into top-level module `ikiwiki` (vendor tag IKIWIKI, release tag PRE_CVS),