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.

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.

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

for rcs_remove(), there's no new file

comment - index attachments

Merge branch 'master' into cvs

clean example blog

This reverts commit 1d1447323474a00d4d479fcdfd4daa85a6e0eecf

oops!

This reverts commit efbae4b0f93e53cf2bbe3e0dcac66895ffc83a0c

some test records

тест

Added a comment: apache module?

removed

Added a comment: apache module?

response

Added a comment

can the 'add comment' form be generated and returned even if the srcdir is locked?

Added a comment: just my 2 cents

fix underlay link

Added a comment: updating setup file

Added a comment: Great! It worked!

removed

mdwn: Added nodiscount setting, which can be used to avoid using the markdown discount engine, when maximum compatability is needed.

Added a comment: thats cool

Added a comment

oops typo

new plugin "newpage"

correction to username

reply - search attachments

add news item for ikiwiki 3.20120115

releasing version 3.20120115

attachment: Fix utf-8 display bug.

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

releasing version 3.20120115

Workaround discount's eliding of <style> blocks.

Added a comment

Added a comment

Added a comment

removed

Added a comment

Added a comment

Make backlink(.) work. Thanks, Giuseppe Bilotta.

backlink(.) should behave like backlink(<current page>)

Since commit c4d4cad3befbbd444d094cbeb0b6ebba3910a025, the single dot in
a pagespec can be used to mean the current page. While this worked
correctly in link() it didn't work in backlink(). Fix this by explicitly
checking the testpage in backlink against . and replacing it with the
current location if necessary.

Propose patch to fix backlink(.)

simplify bug report

test doesn't work live. damnit. try varying pagespec

backlink(.) doesn't work

new forum post about search in attachments

add news item for ikiwiki 3.20120109

releasing version 3.20120109

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