David Roundy wrote:
Were your tests done by  any chance with a darcs compiled with ghc
6.8.2 or earlier (but post ghc 6.6)? If so, you may suffering from a
ghc bug.

I'm afraid it's not compiled with 6.8.2 or earlier. It's the "version 2.0.2 + 75 patches" one from <http://wiki.darcs.net/DarcsWiki/CategoryBinaries>. I've attached its --exact-version.

# darcs2 +RTS --info | find "version"
 ,("GHC version", "6.8.3")

Shortest testcase:
# mkdir test
# cd test
# darcs2 init --darcs-2
# echo a >a
---wait a while---
# darcs2 record --look-for-adds -a -A "test" -m "test"
Finished recording patch 'test'

After that, the modified time of /test/a is 11:40:25 and the modified times of /test/_darcs/hashed_inventory and the files it directly or indirectly references are all 11:41:32. There's an unreferenced file in /test/_darcs/pristine.hashed/, but that has a modified time of 11:38:50 (which seems to match the creation time for the repo).

I get the same behaviour when a file is simply modified, rather than being created. I also get the same behaviour when I use --hashed rather than --darcs-2 (and also, again, when the file is simply modified).

Or maybe there's a bug in the windows-specific code.  I don't know.
But what you're describing is definitely a bug, since darcs does
track time stamps.

So I should raise a bug for this? Either way, now might be a good time for me (or a colleague) to try to get a working Windows build environment.

Regards,
Grant.
darcs compiled on Aug 19 2008, at 14:49:05
# configured Mon Aug 18 15:02:55 GMTDT 2008
./configure /usr/local/share/config.site /usr/local/etc/config.site

Context:

[Fix Windows build
Simon Marlow <[EMAIL PROTECTED]>**20080819134252
 On Windows, System.Posix.Types.FileOffset is not the same as the type
 of the st_size field of the stat structure: the latter is Int64,
 whereas COff == Int32.
 
 This is almost ceratinly not the right fix, but it gets the build
 going.
 
 In general I don't recommend using System.Posix.* on Windows.  The
 right way is to either use the official platform-independent libraries
 like System.IO, System.Directory or System.Process, or to use
 System.Win32 directly.
] 

[Resolve issue823: do not exit on keyboard interrupt when getting patches.
Dmitry Kurochkin <[EMAIL PROTECTED]>**20080815070943
 And give a chance for go_to_chosen_version to run.
] 

[fix buggy comments in bugs/identical-patches.sh.
David Roundy <[EMAIL PROTECTED]>**20080814135322] 

[Add Ian's identical-patch test case.
Eric Kow <[EMAIL PROTECTED]>**20080813171032] 

[URL.hs: store only URL in waitToStart queue.
Dmitry Kurochkin <[EMAIL PROTECTED]>**20080813122246] 

[Add (failing) test for issue944.
Eric Kow <[EMAIL PROTECTED]>**20080814055903
 
 This appears to be a reggression from darcs 1.0.9, and was submitted by
 Wolfgang Lux on the bug tracker.  Interestingly, only the old format
 repositories are affected, not the hashed ones.
] 

[add type witnesses to TouchesFiles
Jason Dagit <[EMAIL PROTECTED]>**20080810063403] 

[add type witnesses to Patch/Choices.lhs
Jason Dagit <[EMAIL PROTECTED]>**20080809000237] 

[Split Cache mostly out of Darsc/Repository/Prefs into its own file (take 2)
[EMAIL PROTECTED] 

[Make Darcs.Repository.Prefs export the cache hash function
[EMAIL PROTECTED] 

[remove a few unsightly functions
Jason Dagit <[EMAIL PROTECTED]>**20080813061256] 

[Fix URL module bug with pipelining enabled.
Dmitry Kurochkin <[EMAIL PROTECTED]>**20080813081218] 

[Minor change to URL module.
Dmitry Kurochkin <[EMAIL PROTECTED]>**20080813074218] 

[Enable pipelining by default, add --disable-pipelining option (issue838).
Dmitry Kurochkin <[EMAIL PROTECTED]>**20080813011342] 

[Generalize HashRepo.clean_pristine to HashIO.clean_hashdir.
[EMAIL PROTECTED] 

[Add writeSlurpy to roll out a copy of slurpy into a filesystem.
[EMAIL PROTECTED] 

[fix breakage in URL.
David Roundy <[EMAIL PROTECTED]>**20080812141220] 

[Parametrize "pristine.hashed" in a bunch of functions.
[EMAIL PROTECTED] 

[Rework URL module for multi threading.
Dmitry Kurochkin <[EMAIL PROTECTED]>**20080811221209] 

[Add thread synchronization to URL module and resume select() if interrupted by 
signal in curl module.
Dmitry Kurochkin <[EMAIL PROTECTED]>**20080810092810] 

[Handle error case with empty URL in URL.waitNextUrl function.
Dmitry Kurochkin <[EMAIL PROTECTED]>**20080809221755] 

[Add --debug-http flag to enable curl and libwww debug at run-time instead of 
compile-time.
Dmitry Kurochkin <[EMAIL PROTECTED]>**20080809154834] 

[Print a warning when the remote end does not have darcs 2.
Eric Kow <[EMAIL PROTECTED]>**20080811100933
 
 Two reasons:
 (1) right now people get a scary warning from ssh when it can't fetch
     some non-essential files (it used to be that we would send stderr from ssh
     to /dev/null, but that has other problems...)
 (2) darcs transfer-mode more widely deployed could help a lot of people
     wrt darcs performance
] 

[Added a beware note to the unrecord command
[EMAIL PROTECTED] 

[Fixed typo
[EMAIL PROTECTED] 

[Better debug messages in URL module.
Dmitry Kurochkin <[EMAIL PROTECTED]>**20080809215247] 

[make Convert.lhs compile.
David Roundy <[EMAIL PROTECTED]>**20080810201725] 

[improve type safety of Darcs.Repository.Internal.
Jason Dagit <[EMAIL PROTECTED]>**20080810051109] 

[Refactor `darcs convert' warning at kowey's request.
Trent W. Buck <[EMAIL PROTECTED]>**20080810110014] 

[Expand formats text based in part on suggestions from darcs-users
Max Battcher <[EMAIL PROTECTED]>**20080809184043] 

[Fixes to global cache text based on darcs-users suggestions
Max Battcher <[EMAIL PROTECTED]>**20080809181424] 

[Add user-focused documentation of repository format options
Max Battcher <[EMAIL PROTECTED]>**20080807195429] 

[Highlight the global cache as a best practice
Max Battcher <[EMAIL PROTECTED]>**20080807193918] 

[Describe best practice in `darcs convert --help'.
Trent W. Buck <[EMAIL PROTECTED]>**20080810110615] 

[add type witnesses to Population
Jason Dagit <[EMAIL PROTECTED]>**20080808053252] 

[add type witnesses to CommandsAux
Jason Dagit <[EMAIL PROTECTED]>**20080808052738] 

[Add type witnesses to more modules, rounding out Darcs/Repository/*
Jason Dagit <[EMAIL PROTECTED]>**20080808050947] 

[fixed a bug in identity_commutes property
Jason Dagit <[EMAIL PROTECTED]>**20080808023025
 In the right identity check the patch order should have gone from
 (identity :> p) to (p2 :> i2).  I added a rigid type context too
 so that ghc 6.8 and newer would type the definition.
] 

[Make Darcs.Repository.Internal compile with type witnesses.
Jason Dagit <[EMAIL PROTECTED]>**20080808015343] 

[UF8.lhs: remove unusued functions/imports/docs
[EMAIL PROTECTED] 

[Resolve issue974 : do not pass both -optc-g and -opta-g to GHC
Eric Kow <[EMAIL PROTECTED]>**20080807073620] 

[make this test more cross-platform
Simon Michael <[EMAIL PROTECTED]>**20080807103433] 

[document how to run unit tests
Simon Michael <[EMAIL PROTECTED]>**20080807030416] 

[move (most) failing tests to bugs for clean test output
Simon Michael <[EMAIL PROTECTED]>**20080806191336] 

[fix an old spelling error
Simon Michael <[EMAIL PROTECTED]>**20080806170432] 

[make searching for "test:" in makefile work
Simon Michael <[EMAIL PROTECTED]>**20080805222241] 

[run only normal (expected to pass) tests by default
Simon Michael <[EMAIL PROTECTED]>**20080805222108] 

[Downplay quantum mechanics link.
Eric Kow <[EMAIL PROTECTED]>**20080806124109
 Besides, darcs has far more than 3 users by now.
] 

[Make patch theory intro more inviting to math people.
Eric Kow <[EMAIL PROTECTED]>**20080806123411] 

[cleanup and slight rewrite of the test docs
Simon Michael <[EMAIL PROTECTED]>**20080806165949] 

[make order of running tests consistent
Simon Michael <[EMAIL PROTECTED]>**20080806172123] 

[small makefile refactoring: allow just the normal tests to be run, without 
bugs/*
Simon Michael <[EMAIL PROTECTED]>**20080805203242] 

[Rectify dist help
[EMAIL PROTECTED]
 Removed the "make dist" suggestion, the manual is a better place for that.
 Instead, make clear that it operates on a clean copy of the tree, and
 mention the "predist" functionality.
] 

[website: explain that darcs 2 is required to get the darcs source.
Simon Michael <[EMAIL PROTECTED]>**20080803181216] 

[Canonize Gaetan Lehmann and Daniel Buenzli.
Eric Kow <[EMAIL PROTECTED]>**20080730104357
 (for Daniel B, avoid an accent in his name)
] 

[configure: check for packages needed with split base.
Eric Kow <[EMAIL PROTECTED]>**20080730103840
 Now that all packages must be used explicitly.
] 

[fix type witness compile errors specific to ghc 6.8
Jason Dagit <[EMAIL PROTECTED]>**20080722182729] 

[avoid import of unused function fromMaybe.
David Roundy <[EMAIL PROTECTED]>**20080729172825] 

[configure: suggest regex-compat before text
Eric Kow <[EMAIL PROTECTED]>**20080725095336] 

[configure: mention Haskell in 'try installing' suggestion
Eric Kow <[EMAIL PROTECTED]>**20080725095015] 

[Typo (Text.Regex)
Eric Kow <[EMAIL PROTECTED]>**20080715121708] 

[Use haskeline to have a readline-like behavior when asking something to the 
user
[EMAIL PROTECTED]
 Unlike the implementations using readline or editline packages, this code
 code doesn't break the Ctrl-C behavior.
] 

[Improve generic rules for English plurals. 
Eric Kow <[EMAIL PROTECTED]>**20080604123728] 

[add configure check for Network.URI.
David Roundy <[EMAIL PROTECTED]>**20080711011914] 

[add -hide-all-packages to default GHCFLAGS.
David Roundy <[EMAIL PROTECTED]>**20080711010952] 

[add support for outputting patch numbers in darcs changes.
David Roundy <[EMAIL PROTECTED]>**20080710011211] 

[add support for matching single patches by index.
David Roundy <[EMAIL PROTECTED]>**20080710004512] 

[add support for matching ranges of patches (counting back from present).
David Roundy <[EMAIL PROTECTED]>**20080710003225] 

[Better avoid silly manpage error.
Trent W. Buck <[EMAIL PROTECTED]>**20080704024920
 
 It turned out only initialize's help string used 'quotes', so just
 remove them.  This makes init's docstring consistent with the others.
] 

[Missing period at end of sentence.
Trent W. Buck <[EMAIL PROTECTED]>**20080704024232] 

[darcs --overview no longer works, so don't document it.
Trent W. Buck <[EMAIL PROTECTED]>**20080704030804] 

[Avoid silly manpage error.
Trent W. Buck <[EMAIL PROTECTED]>**20080703010733
 man (nroff) treats an apostrophe in the first column specially,
 resulting in a syntax error without this patch.
 
 Ideally, all cases of 'foo' in the manpage (i.e. docstrings) should
 become `foo', since man -Tps turns ` and ' into left and right single
 quotes respectively.
] 

[obliterate whitespace in Darcs.Commands.Get
[EMAIL PROTECTED]
 'twas causing lhs/haddock difficulties where a \end{code} wasn't getting 
recognized.
] 

[rm haddock CPP business
[EMAIL PROTECTED]
 Try as I might, I can't see any reason to special-case some Haddock CPP logic 
to deal with some *commented-out guards*, unless CPP magically restores and 
uncomments the code if Haddock isn't being run.
] 

[make pull less verbose when --verbose flag is given.
David Roundy <[EMAIL PROTECTED]>**20080624170035] 

[fix makefile to remember to regenerate version information after running 
configure.
David Roundy <[EMAIL PROTECTED]>**20080624170001] 

[TAG 2.0.2
David Roundy <[EMAIL PROTECTED]>**20080624012041] 

_______________________________________________
darcs-users mailing list
darcs-users@darcs.net
http://lists.osuosl.org/mailman/listinfo/darcs-users

Reply via email to