]> sipb.mit.edu Git - ikiwiki.git/commitdiff
fixed, I think?
authorhttp://smcv.pseudorandom.co.uk/ <smcv@web>
Mon, 19 Mar 2012 09:53:49 +0000 (05:53 -0400)
committeradmin <admin@branchable.com>
Mon, 19 Mar 2012 09:53:49 +0000 (05:53 -0400)
doc/plugins/trail/discussion.mdwn

index 2ddf2ce45f6953b4a0f7310c2ab2c7e7f7eb1c0a..e151466f9f30621398522245d080a975e7885dbe 100644 (file)
@@ -67,3 +67,23 @@ t/trail.t .................... 1/?
 > but got: "trail=sorting n=sorting/old p=sorting/end"
 > 
 > Perhaps a legitimate bug? --[[Joey]] 
+
+>> I saw this while developing, but couldn't reproduce it, and assumed
+>> I'd failed to update `blib` before `make test`, or some such.
+>> In fact it's a race condition, I think.
+>>
+>> The change and failure here is that `sorting.mdwn` is modified
+>> to sort its trail in reverse order of title. Previously, it
+>> was sorted by order of directives in the page, and secondarily
+>> by whatever sort order each directive specified (e.g.
+>> new, old and ancient were sorted by increasing age).
+>> `old` appearing between `new` and `ancient`, and `new` appearing
+>> between `end` and `old`, indicates that this re-sorting has not
+>> actually taken effect, and the old sort order is still used.
+>>
+>> I believe this is because the system time (as an integer) remained
+>> the same for the entire test, and mtimes as used in ikiwiki
+>> only have a 1-second resolution. We can either fix this with
+>> utime or sleep; I chose utime, since sleeping for 1 second would
+>> slow down the test significantly. Please merge or cherry-pick
+>> `smcv/trail-test` (there's only one commit). --[[smcv]]