follow-up: sorry, I appreciate that wasn't as useful as it could have been

split done and not done patch list, remove patches from the other lists

explicitely tag this as a patch

Gentle ping on proposed branch

cannot reproduce new bug

Update changelog with schmonz's changes

bug remains for inline table case

bug report

Fix build with po4a 0.56 via consistent "\n" use.

Errors look like so:

PERL5LIB=.. ./po2wiki underlay.setup
Invalid po file
/var/folders/y5/k8bh4xsj75g4j1n631f2gqz80000gp/T/ikiwiki-po-filter-in.OPXJxZ0juW:
/var/folders/y5/k8bh4xsj75g4j1n631f2gqz80000gp/T/ikiwiki-po-filter-in.OPXJxZ0juW:293:
'msgid' and 'msgstr' entries do not both end with '\n'
/var/folders/y5/k8bh4xsj75g4j1n631f2gqz80000gp/T/ikiwiki-po-filter-in.OPXJxZ0juW:589:
'msgid' and 'msgstr' entries do not both end with '\n'
msgfmt: found 2 fatal errors

make[1]: *** [po2wiki_stamp] Error 255

not sure this is necessary

add link to auto-numbering on CSS

MultiMarkdown footnote un-inversion: done

Fix inverted footnote config with MultiMarkdown.

Bug spotted and fix from Giuseppe Bilotta <giuseppe.bilotta@gmail.com>.

Extend mdwn tests to cover MultiMarkdown, where applicable.

Please test sudo-mass-rebuild branch on Debian

Response: am trying sudo

rename bugs/MultiMarkdown_footnot_support_seems_to_be_broken.mdwn to bugs/MultiMarkdown_footnote_support_seems_to_be_broken.mdwn

Directive fix

I've not looked at Shake yet, thanks, I'll give it a look! — [[Jon]]

Suggest shake

some beginning code to migrate an ikiwiki to hakyll

lol

saw typesetter-css and was reminded of my appeal against the anti-theme as default

Adding my work-in-progress blog to the list

No longer hosting these on Branchable.

Merge branch 'master' of ssh://git.ikiwiki.info

Updated German program translation from Sebastian Kuhnert.

Fix link in previous.

Reply to fancypodcast question.

Updated German basewiki and directives translation from Sebastian Kuhnert.

Open up the discussion

Catch up to highlight 3.51 API change.

As of 3.51, searchFile() is no longer provided in highlight's Perl
bindings (at least on NetBSD and OS X, as built from pkgsrc). This
leaves us falling through to getConfDir(), which has been gone
rather longer.

From highlight git, it appears searchFile() and getFiletypesConfPath()
both originated in the 3.14 release. The latter is still available in
3.51, and returns the same result searchFile() used to. Switch to it.

Add my ikiwiki/blog

Response to jsewell's bug report.

add gitbranch template, prompted by Integeri. Thanks

Fix syntax.

Submit branch for review.

update my comment to reflect new commits on that branch

initial implementation (I was unable to use untrusted git push to add this comment)

Merge remote-tracking branch 'origin/master'

Announce 3.20190228 and 3.20170111.1

Add an anchor for /security/#cve-2019-9187

Prepare 3.20190228 for future release

doc: Document security issues involving LWP::UserAgent

Recommend the LWPx::ParanoidAgent module where appropriate.
It is particularly important for openid, since unauthenticated users
can control which URLs that plugin will contact. Conversely, it is
non-critical for blogspam, since the URL to be contacted is under
the wiki administrator's control.

Signed-off-by: Simon McVittie <smcv@debian.org>

useragent: Automatically choose whether to use LWPx::ParanoidAgent

The simple implementation of this, which I'd prefer to use, would be:
if we can import LWPx::ParanoidAgent, use it; otherwise, use
LWP::UserAgent.

However, aggregate has historically worked with proxies, and
LWPx::ParanoidAgent quite reasonably refuses to work with proxies
(because it can't know whether those proxies are going to do the same
filtering that LWPx::ParanoidAgent would).

Signed-off-by: Simon McVittie <smcv@debian.org>

useragent: Don't allow non-HTTP protocols to be used

This prevents the aggregate plugin from being used to read the contents
of local files via file:/// URLs.

Signed-off-by: Simon McVittie <smcv@debian.org>

This reverts commit 727147aa6e50229178fb853f26c9c340fa789799

useragent: Raise an exception if the LWP module can't be loaded

Signed-off-by: Simon McVittie <smcv@debian.org>

po: Always filter .po files

The input to filter hooks is meant to be the content of a source file
on disk. If we only filter once per (page, destpage) pair, and a page
is inlined into the same destpage more than once, then the second
occurrence will render as the result of htmlizing .po source as if
it was Markdown (or whatever the type of the corresponding master page
is), which is never going to end well.

The alreadyfiltered mechanism was added in commit 1e874b3f to avoid
preprocessing loops, but I'm not sure where it could lead to a loop:
filter hooks are only called from IkiWiki::filter, which is only called
on page content from disk or on proposed content being previewed.
According to <https://bugs.debian.org/911356#41>, deleting the
alreadyfiltered mechanism resolves the problem, as well as simplifying
the code.

Closes: #911356
Tested-by: intrigeri

Recommend against cvsps3 (haven't tried it).

Announce v3.20190207

Signed-off-by: Simon McVittie <smcv@debian.org>

Prepare new release

Signed-off-by: Simon McVittie <smcv@debian.org>

review

comment

tag as reviewed

respond

Exclude reviewed patches from this list

close

append javascript after CSS

Javascript resources should be presented to browsers after CSS, and
"after the fold" (ATF) according to the best practices:

https://developers.google.com/speed/docs/insights/mobile#PutStylesBeforeScripts

This change allows the browser to download Javascript files in
parallel, by including Javascript on the *closing* </body> tag instead
of the opening tag.

We also improve the regex to tolerate spaces before the body tag, as
some templates have (proper) indentation for the tag.

Remove unreachable git repositories

Signed-off-by: Simon McVittie <smcv@debian.org>

Mark as applied

Signed-off-by: Simon McVittie <smcv@debian.org>

remove the "add comment" button from printed media

Add a missing changelog entry.

Update changelog

Signed-off-by: Simon McVittie <smcv@debian.org>

Allow Breezy as alternative to Bazaar.

(cherry picked from commit a07f048d9fc99928ebbb74b34f5d1932ff3d7884)

comments.t: Assert that comments get permalink metadata

Signed-off-by: Simon McVittie <smcv@debian.org>

comments.t: Exercise post-2009 comment naming

Since commit 6af6c89d, comments are in files whose names contain a hash.

Signed-off-by: Simon McVittie <smcv@debian.org>

old regexp would have failed for old comment page names

git-cgi.t: Exercise an alphanumeric, but non-ASCII, root page

My previous attempt to reproduce this bug used a non-alphanumeric
ASCII character. This is not currently considered to be a valid
value for rootpage, although for a "do what I mean" approach, perhaps
we should accept it and pass it through titlepage() or linkpage().

Using Chinese characters (which are considered to match [[:alnum:]]
even though the Chinese script is not, strictly speaking, an alphabet),
as in the original bug report, reproduces the bug.

Signed-off-by: Simon McVittie <smcv@debian.org>

Fix inline plugin for non-ASCII rootpage

t: Exercise Chinese and Cyrillic page titles

Signed-off-by: Simon McVittie <smcv@debian.org>

trail: Allow unescaped punctuation in pagenames

By processing the pagenames through linkpage, we let users specify
page names that contain non-alphanumerics in a more natural way.

Signed-off-by: Simon McVittie <smcv@debian.org>

trail.t: Exercise numeric escapes in pagenames parameter

Signed-off-by: Simon McVittie <smcv@debian.org>

linkpage.t: Assert we can link to pages with literal underscore

Signed-off-by: Simon McVittie <smcv@debian.org>

t: Consistently remove temp directory before testing, not after

When a test fails, it's useful to be able to inspect the output.

Signed-off-by: Simon McVittie <smcv@debian.org>

build: Add `make reset-generated`

Signed-off-by: Simon McVittie <smcv@debian.org>

pagetitle.t, linkpage.t, titlepage.t: Exercise Unicode more

Signed-off-by: Simon McVittie <smcv@debian.org>

link to recently-added tests

I'm confused about what the bug is, and what's being fixed. Can you give a complete test or example?

close bug

Update changelog

Add a simple test for non-ASCII in tables

Signed-off-by: Simon McVittie <smcv@debian.org>

Fix table plugin can handle UTF-8 csv format

respond

Fix syntax and escaping

git-cgi.t: Add a failing test for a blog with a non-ASCII rootpage

This is one of several possible bug reports on
"doc/bugs/About %2F problem" (I'm not sure what the actual bug being
reported is).

Signed-off-by: Simon McVittie <smcv@debian.org>

git-cgi.t: Add a simple test for blog posts from a root page

This is the working base case for "doc/bugs/About %2F problem".

Signed-off-by: Simon McVittie <smcv@debian.org>

git-cgi.t: Print query string as a TAP diagnostic

Signed-off-by: Simon McVittie <smcv@debian.org>