From 2a34df594e90ae844a8a990a54c6acc01861a2aa Mon Sep 17 00:00:00 2001 From: joey Date: Thu, 22 Feb 2007 02:35:31 +0000 Subject: [PATCH] web commit by http://ethan.betacantrips.com/: update on this patch (it still sucks) --- doc/patchqueue/move_page.mdwn | 12 ++++++++++++ 1 file changed, 12 insertions(+) diff --git a/doc/patchqueue/move_page.mdwn b/doc/patchqueue/move_page.mdwn index 3af49bbdb..5329a7a3b 100644 --- a/doc/patchqueue/move_page.mdwn +++ b/doc/patchqueue/move_page.mdwn @@ -3,6 +3,16 @@ This is my first cut at a feature like that requested in [[todo/Moving_Pages]]. A bunch of obvious shortcomings exist: * I'm not sure all the untaints are safe. + + > I've looked them over. + > + > * one is safe because it only allows existing page names through. + > * one is safe because it follows the example of editpage in calling + > titlepage to sanitize. + > * the last one follows the example of rcs_commit in svn.pm, so I + > hope it's safe.. + > + > --Ethan * No precautions whatsoever are made to protect against race conditions or failures in the rcs\_move function. * movepage.tmpl doesn't exist yet. @@ -50,6 +60,8 @@ before it is moved? Bail, or shrug and proceed? > issue that occurs if the page is moved using a regular svn commit too, so > it's not really your concern in a way. :-) +>> I wrote a patch to address this -- it's in [[disappearing_pages]]. --Ethan + Could you elaborate on [[commit-internals]]? Can I assume that ikiwiki's working copy W will always reflect a revision of the master copy M? (That is, nobody changes W and leaves it uncommitted.) I would guess -- 2.44.0