[darcs-devel] darcs patch: use impossible to document impossible case in Repair.

2006-12-04 Thread David Roundy
A simple fix, documenting that this really is considered an impossible
case, rather than just an unhandled error.  In Repair.

David

Mon Dec  4 07:28:54 PST 2006  David Roundy [EMAIL PROTECTED]
  * use impossible to document impossible case in Repair.


New patches:

[use impossible to document impossible case in Repair.
David Roundy [EMAIL PROTECTED]**20061204152854] 

 {
hunk ./Repair.lhs 40
 import Lock( withTempDir )
 import Check ( check_uniqueness )
 import Printer ( putDocLn, text )
+#include impossible.h
 \end{code}
 
 \options{repair}
hunk ./Repair.lhs 97
 apply_patches_with_feedback [] False feedback putInfo $
 (chtg, Just chk)
 : reverse (concat $ get_patches_beyond_tag chtg patches)
-Nothing - fail Bug in repair_cmd, please report.
+Nothing - impossible -- checkpoint is always a tag
 Nothing - apply_patches_with_feedback [] False feedback putInfo
$ reverse $ concat patches
 -- withTempDir ignores error on delete -- hence the hack below.
}

Context:

[Remove raw_mode functions from atomic_create.h.
Eric Kow [EMAIL PROTECTED]**20061008202738
 
 It seems these were once implemented in compat.c and have since been
 reimplemented in Haskell by Ian Lynagh on 2005-07-30.  These appear to
 just be leftover declarations in the C header.
 
] 
[resolve conflicts
Tommy Pettersson [EMAIL PROTECTED]**20061117222757
 between 'clean up unrevert and pending handling'
 and 'ignore failure from hSetBufferin'
] 
[ignore failure from hSetBuffering
Tommy Pettersson [EMAIL PROTECTED]**20061117221424
 This affects:
   issue41	Doesn't like pasted text.
   issue94	Crash on bogus input
   issue146	hSetBuffering: invalid argument
   issue318	buffering error of darcs record under bash/cmd.exe
 It doesn't necessarily fix anything. It prevents darcs from quiting,
 instead continuing with perhaps an undesirable buffering mode, which may or
 may not be better ... or worse.
] 
[Fix curses stuff, especially on Solaris 10.
Dave Love [EMAIL PROTECTED]**20061120171211] 
[TAG 1.0.9rc2
Tommy Pettersson [EMAIL PROTECTED]**20061116140351] 
[add some changelog entries
Tommy Pettersson [EMAIL PROTECTED]**20061116135447] 
[Define infodepspatch locally in AmendRecord instead of exporting it from Patch
[EMAIL PROTECTED]
 
] 
[Make libcurl use any http authentication.
Tobias Gruetzmacher [EMAIL PROTECTED]**20061118230406
 This let darcs use repositories protected with digest authentication.
] 
[Redirect stderr to Null when exiting SSH control master.
Eric Kow [EMAIL PROTECTED]**20061118212115
 
 This suppresses the output
 * Pseudo-terminal will not be allocated because stdin is not a terminal.
   (result of redirecting stdin from /dev/null)
 * Exit request sent.
   (seems to be normal output. Seems also that there is no way to suppress
this; -q does not do the job, for example)
 
] 
[Overhaul and improve automation of ssh_test.
Eric Kow [EMAIL PROTECTED]**20061121141802
 
 * Now quits if you don't supply REMOTE; does not have any
   silly default values
 * Options now passed in through environment variables, so:
 NO_CONTROL_MASTER=1 [EMAIL PROTECTED] ./ssh_test
 * Performs some automated success checks (which means that
   it should be possible to use this from the harness if you
   have ssh-agent running)
 * Performs darcs send test
 * Does not try to pass darcs-ssh flags (like --disable-ssh-cm)
   to non-ssh-using commands like record
 
] 
[Rename ssh_test to ssh.sh (for shell harness).
Eric Kow [EMAIL PROTECTED]**20061121141101
 
 Note that you must set environment variables for it do anything
 useful (namely [EMAIL PROTECTED]); something like the following
 should work:
   [EMAIL PROTECTED] make test
 
 You need to be using public key authentication to have a fully
 automated test.
 
] 
[Support darcs send --disable-ssh-cm.
Eric Kow [EMAIL PROTECTED]**20061121134158] 
[Canonize Edwin Thomson.
Eric Kow [EMAIL PROTECTED]**20061118174454] 
[Make Exec.lhs not import unneeded Control.Exception functions when compiling on Windows.
[EMAIL PROTECTED]
 
] 
[Annotate various boring patterns.
Dave Love [EMAIL PROTECTED]**20061113225701] 
[Add make rules for tags files.
Dave Love [EMAIL PROTECTED]**20061113213923] 
[Amending a patch doesn't remove explicit dependencies
[EMAIL PROTECTED] 
[look for --disable-ssh-cm in defaults files (issue351)
Tommy Pettersson [EMAIL PROTECTED]**20061117180942] 
[configure should fail if a required module isn't present.
David Roundy [EMAIL PROTECTED]**20061128024557] 
[refactor is_malicious_path (for easier reading)
Tommy Pettersson [EMAIL PROTECTED]**20061113191040] 
[really use new malicious file path check in pull (not in changes)
Tommy Pettersson [EMAIL PROTECTED]**20061113182628] 
[Add a semi-automated test for SSH-related things.
Eric Kow [EMAIL PROTECTED]**20061110110801
 
 Testing SSH stuff is tricky in that (1) you need some place to connect
 to and (2) you 

[darcs-devel] summary of my recent spurt of patches, and request for suggestions

2006-12-04 Thread David Roundy
Hi all,

This is intended to be a summary of my activity over the weekend, for those
who are overwhelmed by the number of darcs sends I made--I kept thinking I
was done for the weekend.

I've been working hard on getting support for the new hashed inventory
format into good shape.  If you aren't familiar with the benefits of the
new format (which I've talked about with at least some of you in person),
suffice to say that I see it as a precursor to working out the new way of
dealing with conflicts.

I've gotten most of the tests to pass when run with hashed inventories, and
soon hope to get the rest under control.  In the process of generalizing to
a new repository format, I've been moving the code over to a more generic
framework (first introduced by Juliusz, but expanded by myself) which
abstracts out some of the details of the repository itself.

There are still several tests that fail, which I intend to track down:

TESTS FAILED!
obliterate.sh
partial.sh
unrecord.sh

and 

Failed Test   Stat Wstat Total Fail  Failed  List of Failed
---
optimize.pl  2   512 42  50.00%  3-4
repodir.pl   1   256161   6.25%  10
set_scripts_executable.pl1   256251   4.00%  20

This is pretty small.  I don't know why the obliterate and unrecord
commands fail, but partial hashed inventory repositories are still on my
list to implement, so that's an understood failure, and optimize also
hasn't been updated to the new code.  Repair will also fail, but doesn't
seem to be in the tests--we should add a repair testcase or two.  I'm not
sure what causes the set_scripts_executable.pl error, but know where to
look.

Given these caveats, I'd appreciate it if interested persons could play
with hashed inventories, and see if they have any problems.  Patches to the
hashed_inventory.sh test would be most welcome, as I've mostly been running
tests with *all* hashed inventories, but am sure that some interesting bugs
will come out in the interaction between hashed inventory repositories and
old-fashioned repositories.

This is also the time to suggest changes to the actual format of the hashed
inventories.  It might be nice to add subdirectories to the patches/
directory, either by date or by the first couple characters of the hash
itself.  And I'm open to suggestions to make the format more parseable by
third party tools.  I just hacked this one together in... well quite a
short period of time, and since then have been focusing on the rest of the
code (which really doesn't depend on the inventory format in any
interesting way).

I guess that's all.  The term is over (which is why I've got a bit more
time and energy), but I've still got a final to give, and need to get
cracking on research (which was delayed by my class), so I'm not sure how
much darcs time I'll have in the upcoming weeks.  But it's likely to be
more than I had in the past couple of months.
-- 
David Roundy
Department of Physics
Oregon State University

___
darcs-devel mailing list
darcs-devel@darcs.net
http://www.abridgegame.org/cgi-bin/mailman/listinfo/darcs-devel


Re: [darcs-devel] summary of my recent spurt of patches, and request for suggestions

2006-12-04 Thread Eric Y. Kow
Hi David,

On Mon, Dec 04, 2006 at 07:53:03 -0800, David Roundy wrote:
 I've been working hard on getting support for the new hashed inventory
 format into good shape.  If you aren't familiar with the benefits of the
 new format (which I've talked about with at least some of you in person),
 suffice to say that I see it as a precursor to working out the new way of
 dealing with conflicts.

Good stuff.  Just wanted to say that I'm now back online and will be
looking over the spurt, most likely on the weekend.

Best,

-- 
Eric Kow http://www.loria.fr/~kow
PGP Key ID: 08AC04F9 Merci de corriger mon français.


pgpc5xhRq6au4.pgp
Description: PGP signature
___
darcs-devel mailing list
darcs-devel@darcs.net
http://www.abridgegame.org/cgi-bin/mailman/listinfo/darcs-devel


[darcs-devel] Re: darcs patch: use System.Process on Windows, and a few more

2006-12-04 Thread Simon Marlow

Tommy Pettersson wrote:

I've reviewed and tested these patches on linux with ghc 6.4.


| Wed Nov 29 17:01:44 CET 2006  Simon Marlow [EMAIL PROTECTED]
|   * add explicit import list

| Wed Nov 29 17:03:42 CET 2006  Simon Marlow [EMAIL PROTECTED]
|   * hFlush after waiting for lock message

| Wed Nov 29 17:06:20 CET 2006  Simon Marlow [EMAIL PROTECTED]
|   * catch exceptions in stdout_is_a_pipe

These three looks good and works fine.



| Wed Nov 29 17:07:10 CET 2006  Simon Marlow [EMAIL PROTECTED]
|   * Use System.Process on Windows

This one I couldn't compile, but the following changes fixed it:

  hunk ./Exec.lhs 25
  -import System.Exit  ( ExitCode )
  +import System.Exit  ( ExitCode(..) )
  hunk ./Exec.lhs 39
  -import System
  +import System.IO ( stdin )
  +import Control.Exception ( bracket )

I can not test this patch since it's windows only, but I tried
to compile with #define WIN32, and got:

  Exec.lhs:33:27: Module `Control.Exception' does not export `bracketOnError'

Don't know if that's important, but I thought I'd mention it.


Thanks for taking a look.

These are due to changes in libraries between versions of GHC; bracketOnError 
only appeared in GHC 6.6, it seems.  So we need some configure tests to 
straighten things out.  Can I leave these up to you guys to sort out?


Cheers,
Simon


___
darcs-devel mailing list
darcs-devel@darcs.net
http://www.abridgegame.org/cgi-bin/mailman/listinfo/darcs-devel