[Top][All Lists]
[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: patch to diff manpage - clarify "-x"
From: |
Tomas Pospisek |
Subject: |
Re: patch to diff manpage - clarify "-x" |
Date: |
Thu, 24 Oct 2002 20:56:25 +0200 (CEST) |
On Thu, 24 Oct 2002, Paul Eggert wrote:
> > Date: Thu, 24 Oct 2002 15:26:25 +0200 (CEST)
> > From: Tomas Pospisek <address@hidden>
> >
> > But wheter you will or not shorten diff --help, the man page should
> > contain a description of all the options.
>
> No, because we don't have time to maintain man pages in addition to
> the regular documentation. The man pages are generated automatically
> from the --help output. The man pages are not intended to be
> complete, and we don't want to spend time worrying about them.
OK.
> > +/* We're excluding things using -x or -X */
> > +static bool excluding = 0;
>
> That patch doesn't look right to me. Among other things, there are
> other options than -x or -X that make sense only when comparing
> subdirectories. If we change diff to exit when any of those options
> used unless directory comparison is also used, I'm worried that we
> might break something else.
OK. Would a warning be OK though ( fprintf(STDERR,"-x bla bla...") )?
But the patch still wont work correctly when doing:
diff -x bla dir1/ dir2/
when either of them is link.
> > The confusion came from the fact that I wanted to test my pattern before
> > letting it rattle through my entire filesystem. So I started off with:
> >
> > $ diff -s bla bla.txt bla.tx
> >
> > which didn't work,
>
> I assume you mean "-x 'bla*'" here instead of "-s bla".
Yes, that's what I tried next.
*t
-----------------------------------------------------------------------
Tomas Pospisek
sourcepole - Linux & Open Source Solutions
http://sourcepole.com
Elestastrasse 18, 7310 Bad Ragaz, Switzerland
Tel:+41 (81) 330 77 13, Fax:+41 (81) 330 77 12
------------------------------------------------------------------------
- Re: patch to diff manpage - clarify "-x", (continued)
- Re: patch to diff manpage - clarify "-x", Paul Eggert, 2002/10/23
- Re: patch to diff manpage - clarify "-x", Tomas Pospisek, 2002/10/23
- Re: patch to diff manpage - clarify "-x", Tomas Pospisek, 2002/10/23
- Re: patch to diff manpage - clarify "-x", Paul Eggert, 2002/10/23
- Re: patch to diff manpage - clarify "-x", Bob Proulx, 2002/10/24
- Re: patch to diff manpage - clarify "-x", Stepan Kasal, 2002/10/24
- Re: patch to diff manpage - clarify "-x", Paul Eggert, 2002/10/24
- Re: patch to diff manpage - clarify "-x", Stepan Kasal, 2002/10/24
- Re: patch to diff manpage - clarify "-x", Tomas Pospisek, 2002/10/24
- Re: patch to diff manpage - clarify "-x", Paul Eggert, 2002/10/24
- Re: patch to diff manpage - clarify "-x",
Tomas Pospisek <=