]> git.vanrenterghem.biz Git - git.ikiwiki.info.git/blobdiff - doc/plugins/contrib/rsync/discussion.mdwn
Merge commit 'schmonz/master'
[git.ikiwiki.info.git] / doc / plugins / contrib / rsync / discussion.mdwn
index 9b4184e5feb21b325f7d8f82b70b548430e779ab..6bf7a38267d687899bb355ca188ef16aedb10148 100644 (file)
@@ -60,3 +60,18 @@ Revew: --[[Joey]]
   only issue with it is that it is run even when ikiwiki has not
   rendered any updated pages. Bah, I think you do need the new hook, how
   annoying..
+
+> * Depends whether the plugin would be on by default. If yes, then yes.
+>   If the admin has to enable it, I'd think they'd want the error.
+> * Changed the other errors to warnings.
+> * The name might be wrong: there isn't anything rsync-specific about the
+>   plugin, that's just the command I personally need to run. --[[schmonz]]
+
+>> One problem with the error is that it prevents dumping a new setup file with
+>> the plugin enabled, and then editing it to configure. ie:
+
+       joey@gnu:~>ikiwiki -setup .ikiwiki/joeywiki.setup -plugin rsync -dumpsetup new.setup
+       Must specify rsync_command
+
+> rsync seems by far the most likely command, though someone might use something
+> to push via ftp instead. I think calling it rsync is ok. --[[Joey]]