Merge branch 'master' into cvs

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

fix typo in LC_TIME locale lookup

Express rcs_recentchanges() tests via utility functions, in preparation
for writing more thorough tests, in order to safely extract the
cvsps output-parsing routine for reuse by an upcoming rcs_revert().

Fail a test if an rcs_receive() hook is ever defined for CVS.

Extract add_and_commit(), a common test action.

clean example blog

This reverts commit e8dfb79a80103c70371b8c741ecac1e80936a63c

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

A comment.

Merge branch 'master' into cvs

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

Merge branch 'master' of github.com:schmonz/ikiwiki

Merge remaining TODO items into CVS page, and delete TODO.cvs.

agreed

there are actually reasons, yes

remove non-ikiwiki site

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

remove stealthy spam

Added a comment

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

fix typo (doubled quote)

Added a comment

changelog

Merge remote-tracking branch 'schmonz/portability'

cleanup

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

fix

document a bug fix

Original bug was introduced 24 Nov 2011.

calendar, prettydate: Fix strftime encoding bug

strftime is a C function, it does not return decoded utf8.
Several places in ikiwiki manually decoded it, but at least two
forgot to.

Also, strftime might not return even encoded utf8, if LC_TIME is set
to a non-utf8 value. Went ahead and supported decoding whatever encoding
it uses.

The remaining direct calls to strftime() are all ones that first set
LC_TIME=C, in order to get times that are not for human display.

comment on haskell rebuild

merged

changelog

nuke

query

Revert "ignore a file that always clutters status on OS X (no known fix)"

This reverts commit 662d256c821d85d041a7620aa0bac6144ff95af4.

I sympathize, but this would prevent commits to that file everywhere else.

Merge remote-tracking branch 'schmonz/cvs'

comments

functional changes are underway

even further thought

further thoughts

comment about Haskell

response

Switch to YAML::XS to work around insanity in YAML::Mo. Closes: #657533

https://rt.cpan.org/Ticket/Display.html?id=74487

Gave up trying to support multiple YAML backends. The XS one requires ugly
manual encoding to get unicode right, and doesn't allow dumping yaml
fragments w/o the yaml header, but at least it doesn't randomly crash
on import like YAML::Mo has started to.

cleanup

why do I always forget something?

new plugins, anyone?

new jssearchfield directive

new plugin "jssearchfield"

Clearer (I hope) bug report

New bug: encoding problem in calendar plugin

Oups

Encoding problem in french with ikiwiki-calendar

which came first, the chicken or the egg?

comment to multi-threading discussion

unofficial opinion

Move cvs_keyword_subst_args() down with the other internal support routines.

Uncomment the CVS keyword substitution mode tests, and make them pass:

Extract cvs_keyword_subst_args() and ensure it runs in $config{srcdir}.
Using Perl's -T operator appears to work equally well, perhaps switch?

Mollify strictures about barewords.

Fix wikipedia URL: wikipedia.org, not wikimedia.org

Merge branch 'cvs' of github.com:schmonz/ikiwiki into cvs

Merge branch 'master' of git://git.ikiwiki.info into cvs

Move tests that simulate web commits into test_rcs_add(), then add
a bunch more tests (that wind up exercising rcs_commit(),
rcs_commit_staged(), and rcs_recentchanges()). Extract some support
routines for brevity. Most is_in_keyword_substitution_mode() tests
are commented out because there's a bug -- non-binary files are
being added with "cvs add -kb".

Move tests that inspect recentchanges after direct CVS operations
into test_rcs_recentchanges().

Merge branch 'cvs' of github.com:schmonz/ikiwiki into cvs

Merge branch 'master' of git://git.ikiwiki.info into cvs

recentchangesdiff: truncate extremely large diffs

A diff was already truncated after 200 lines. But it could still be
arbitrarily enormous, if a spammer or other random noise source likes long
lines. That could use a lot of memory to html encode etc the diff and fill
it into the template. Truncating after 100kb seems sufficient; it allows
for 200 lines of up to 512 characters each.

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

fix the nodiscount setting

Merge branch 'master' into cvs

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

pull request (no functional change, just worried I broke easy merging)

Merge branch 'master' into cvs

pull request (no functional change, just worried I broke easy merging)

Merge branch 'master' into cvs

ignore a file that always clutters status on OS X (no known fix)

Define required programs and modules at top for easy spotting.

Merge branch 'cvs' of github.com:schmonz/ikiwiki into cvs

Conflicts:
TODO.cvs
t/cvs.t

update

No functional change: sort sub definitions.

In the code:
* general plugin API calls (in plugins/write order),
* VCS plugin API calls (in plugins/write order), then
* internal support routines (in alphabetical order).

In the tests:
* general meta-behavior (in no particular order, yet),
* general plugin API calls (in plugins/write order),
* VCS plugin API calls (in plugins/write order), then
* internal support routines (in semi-logical order).

Test::Class runs test subs in alphabetical order, so we do too.

Ape xUnit more closely to remove ordering constraints on test subs:
* Add setup and teardown methods, called before and after every test sub.
* In setup, make a fresh repo; in teardown, throw it out.
* Extract runtests method and define default test methods at top.
* Move reflection routines near the xUnit-style subs they support.

Adapt existing test subs to run independently:
* In test_manual_add_and_commit(), assume a fresh repo.

While here, plan a bit better:
* Check for all modules used by cvs.pm.
* Check for program existence more generally.
* Check that we can rmdir after mkdir.

Describe the needed test cases, and implement a couple Test::Class features.

* Run all subs matching /^test_*/ (for which we can plan)...
* Unless TEST_METHOD is set, in which case run matching subs (sans plan).
* Define total number of tests very near 'use Test::More', where expected.
* Define test tempdir where it's declared, no longer any reason why not.
* Move most comments from TODO.cvs into t/cvs.t.
* Add a whole bunch more comments describing the needed test cases.

XXX existing tests are order-dependent, but currently happen to pass

Fix regression: failing to make the temp dir should nix the test plan.
While here, nitpick style and wrap long lines.

Use config parameters directly and shorten long lines.

* Define expected number of tests near the top.
* Call readfile() directly from writefile().
* Parameterize commit message for the web-commit case.
* Describe intent of test cases.
* Rename test subs to match what they actually do.
* To prove extra path slashes don't cause trouble, instead of running
  the same tests a second time, just assert that checkconfig()
  strips the slashes.

Fix a typo in the web commit test.

Simplify startup: use Test::More unconditionally, then determine
the test plan at runtime. Use IkiWiki unconditionally too (as that's
not what I'm testing here) to avoid the TAP error of printing a
test result before having printed the plan.

Extract subs: startup/shutdown, and tests for web and manual commits.

No functional change intended.

add TODO notes

point to my branch, wrap lines, working on test coverage now

Merge branch 'cvs' of github.com:schmonz/ikiwiki into cvs

Conflicts:
TODO.cvs

Describe the needed test cases, and implement a couple Test::Class features.

* Run all subs matching /^test_*/ (for which we can plan)...
* Unless TEST_METHOD is set, in which case run matching subs (sans plan).
* Define total number of tests very near 'use Test::More', where expected.
* Define test tempdir where it's declared, no longer any reason why not.
* Move most comments from TODO.cvs into t/cvs.t.
* Add a whole bunch more comments describing the needed test cases.

XXX existing tests are order-dependent, but currently happen to pass

Fix regression: failing to make the temp dir should nix the test plan.
While here, nitpick style and wrap long lines.

Use config parameters directly and shorten long lines.

* Define expected number of tests near the top.
* Call readfile() directly from writefile().
* Parameterize commit message for the web-commit case.
* Describe intent of test cases.
* Rename test subs to match what they actually do.
* To prove extra path slashes don't cause trouble, instead of running
  the same tests a second time, just assert that checkconfig()
  strips the slashes.