[darcs-devel] darcs patch: Add some rudimentary amend-record + meta... (and 19 more)

[David Roundy]

On Thu, Aug 02, 2007 at 11:14:23PM +0200, Eric Y. Kow wrote:
> On Thu, Aug 02, 2007 at 14:03:24 -0700, David Roundy wrote:
> > > +               apply_patches_to_some_files repository path_list $ get_nonrange_match_s opts
> 
> > I wonder if it might be better in terms of efficiency to apply patches to
> > Slurpies.
> 
> Yeah, that was actually what I was aiming for.  I mostly copied the
> Diff code.  This function (from Edwin) applies to slurpy and dumps
> just the stuff you want to disk.  This makes sense if you want to
> call diff or some other program on it, but not if you just want to
> show contents.  So we're being silly, only moderately so.

Right.

> Now watch somebody say that they want to be able to call an external
> viewer from show contents.  Actually, another thought I had was that you
> might want to supply an output directory with -o (in which case the
> above code might make more sense), and links up with what you say below.

I think you misunderstood, I meant that if I call

darcs show contents src/

I'd almost certainly want darcs to spit out:

atomic_create.c
atomic_create.h
...
Workaround.hs

so that this command would display the contents of either files or
directories.  But this sort of overloading might be considered
undesireable.

> > Also, I noticed that the documentation said show contents accepted [FILE or
> > DIRECTORY] arguments, but only files actually work (as far as I can see).
> > I think it's reasonable to give directory arguments and expect a listing of
> > files in that directory... but maybe better to keep this simple.
> 
> Well, I think I'll leave that to others :-).  Will submit a patch to fix
> the documentation in the meantime.
-- 
David Roundy
Department of Physics
Oregon State University
-------------- next part --------------
A non-text attachment was scrubbed...
Name: not available
Type: application/pgp-signature
Size: 189 bytes
Desc: Digital signature
Url : http://lists.osuosl.org/pipermail/darcs-devel/attachments/20070802/064f1947/attachment.pgp