aboutsummaryrefslogtreecommitdiff
diff options
context:
space:
mode:
authorhttps://id.koumbit.net/anarcat <https://id.koumbit.net/anarcat@web>2013-09-07 18:51:28 -0400
committeradmin <admin@branchable.com>2013-09-07 18:51:28 -0400
commit25779dd00f4201d72bb49a191534d2525b65a70c (patch)
tree12df1fb3d4dad738e0cc6bbc415c717919c79a72
parentf99bc3a273d86f86c06635a0ed57c0af5d1c0e55 (diff)
downloadikiwiki-25779dd00f4201d72bb49a191534d2525b65a70c.tar
ikiwiki-25779dd00f4201d72bb49a191534d2525b65a70c.tar.gz
whee, working patch *and* unit test
-rw-r--r--doc/bugs/syslog_fails_with_non-ASCII_wikinames.mdwn22
1 files changed, 2 insertions, 20 deletions
diff --git a/doc/bugs/syslog_fails_with_non-ASCII_wikinames.mdwn b/doc/bugs/syslog_fails_with_non-ASCII_wikinames.mdwn
index f2e3849f4..712205b7a 100644
--- a/doc/bugs/syslog_fails_with_non-ASCII_wikinames.mdwn
+++ b/doc/bugs/syslog_fails_with_non-ASCII_wikinames.mdwn
@@ -14,26 +14,8 @@ My guess is this sits somewhere here:
Yet I am not sure how to fix that kind of problem in Perl... --[[anarcat]]
-I tried to make this unit test:
-
-[[!format perl """
-#!/usr/bin/perl
-use warnings;
-use strict;
-use Test::More tests => 5;
-
-BEGIN { use_ok("IkiWiki"); }
-
-$IkiWiki::config{verbose} = 1;
-$IkiWiki::config{syslog} = 1;
-$IkiWiki::config{wikiname} = 'ascii';
-ok(debug('test'));
-$IkiWiki::config{wikiname} = 'not â’¶SCII';
-ok(debug('test'));
-"""]]
-
-And I couldn't reproduce the problem. Therefore, I suspect the problem may be somewhere else - maybe in the options parsing? One thing that's for sure: when I put that funky character in the title, it breaks logging! If I remove the "eval" above, I get:
+If I remove the "eval" above, I get:
Error: Wide character in syswrite at /usr/lib/perl/5.14/Sys/Syslog.pm line 485.
-I have improved a little the error handling in log_message() so that we see *something* when syslog fails, see the branch documented above. I can also confirm that reverting [[todo/syslog_should_show_wiki_name]] fixes the bug.
+I have improved a little the error handling in log_message() so that we see *something* when syslog fails, see the branch documented above. I can also confirm that reverting [[todo/syslog_should_show_wiki_name]] fixes the bug. Finally, I have a unit test that reproduces the problem in git, and a working [[!taglink patch]] for the bug, again in git.