pgsql: Make 'find' syntax consistent; add .git exclusion to make_ctags.

Log Message:
-----------
Make 'find' syntax consistent; add .git exclusion to make_ctags.

Modified Files:
--------------
pgsql/src/tools:
make_ctags (r1.11 -> r1.12)
(http://anoncvs.postgresql.org/cvsweb.cgi/pgsql/src/tools/make_ctags?r1=1.11&r2=1.12)
make_etags (r1.5 -> r1.6)
(http://anoncvs.postgresql.org/cvsweb.cgi/pgsql/src/tools/make_etags?r1=1.5&r2=1.6)

Bruce Momjian wrote:
> Log Message:
> -----------
> Make 'find' syntax consistent; add .git exclusion to make_ctags.

Should this consider the distdir target in /GNUmakefile.in too?

--
Alvaro Herrera http://www.CommandPrompt.com/
PostgreSQL Replication, Consulting, Custom Development, 24x7 support

Alvaro Herrera wrote:
> Bruce Momjian wrote:
> > Log Message:
> > -----------
> > Make 'find' syntax consistent; add .git exclusion to make_ctags.
>
> Should this consider the distdir target in /GNUmakefile.in too?

It only is looking for _directories_ to put 'tags' into.

--
Bruce Momjian <bruce(at)momjian(dot)us> http://momjian.us
EnterpriseDB http://enterprisedb.com

+ If your life is a hard drive, Christ can be your backup. +

Alvaro Herrera wrote:
> Bruce Momjian wrote:
> > Log Message:
> > -----------
> > Make 'find' syntax consistent; add .git exclusion to make_ctags.
>
> Should this consider the distdir target in /GNUmakefile.in too?

Oh, I get it now, 'distdir'. Let me look.

--
Bruce Momjian <bruce(at)momjian(dot)us> http://momjian.us
EnterpriseDB http://enterprisedb.com

+ If your life is a hard drive, Christ can be your backup. +

Alvaro Herrera wrote:
> Bruce Momjian wrote:
> > Log Message:
> > -----------
> > Make 'find' syntax consistent; add .git exclusion to make_ctags.
>
> Should this consider the distdir target in /GNUmakefile.in too?

Uh, I usually run the tag program from the /src directory where there is
no 'distdir', but if you want 'distdir' added, just let me know.

--
Bruce Momjian <bruce(at)momjian(dot)us> http://momjian.us
EnterpriseDB http://enterprisedb.com

+ If your life is a hard drive, Christ can be your backup. +

Bruce Momjian wrote:
> Alvaro Herrera wrote:
> > Bruce Momjian wrote:
> > > Log Message:
> > > -----------
> > > Make 'find' syntax consistent; add .git exclusion to make_ctags.
> >
> > Should this consider the distdir target in /GNUmakefile.in too?
>
> Uh, I usually run the tag program from the /src directory where there is
> no 'distdir', but if you want 'distdir' added, just let me know.

No, I'm just pointing out that the distdir rule has a find invocation,
and it excludes directories named CVS. So if we exclude .git on the
make_[ce]tags scripts, maybe we should on make distdir too.

On the other hand, I'm not sure who (if anyone) runs make distdir, so
maybe this is not an issue.

--
Alvaro Herrera http://www.CommandPrompt.com/
PostgreSQL Replication, Consulting, Custom Development, 24x7 support

Alvaro Herrera wrote:
> Bruce Momjian wrote:
> > Alvaro Herrera wrote:
> > > Bruce Momjian wrote:
> > > > Log Message:
> > > > -----------
> > > > Make 'find' syntax consistent; add .git exclusion to make_ctags.
> > >
> > > Should this consider the distdir target in /GNUmakefile.in too?
> >
> > Uh, I usually run the tag program from the /src directory where there is
> > no 'distdir', but if you want 'distdir' added, just let me know.
>
> No, I'm just pointing out that the distdir rule has a find invocation,
> and it excludes directories named CVS. So if we exclude .git on the
> make_[ce]tags scripts, maybe we should on make distdir too.
>
> On the other hand, I'm not sure who (if anyone) runs make distdir, so
> maybe this is not an issue.

Oh, OK, I didn't realize that. Done.

--
Bruce Momjian <bruce(at)momjian(dot)us> http://momjian.us
EnterpriseDB http://enterprisedb.com

+ If your life is a hard drive, Christ can be your backup. +

Bruce Momjian wrote:
> Alvaro Herrera wrote:
> > Bruce Momjian wrote:
> > > Log Message:
> > > -----------
> > > Make 'find' syntax consistent; add .git exclusion to make_ctags.
> >
> > Should this consider the distdir target in /GNUmakefile.in too?
>
> It only is looking for _directories_ to put 'tags' into.

I have to admit that I find this whole idea a bit weird. Is it common
to symlink the tags file?

What I did was tell my editor that the tags file is to be found on the
root source directory. It seems the most natural way to do this. Am I
alone in this?

--
Alvaro Herrera http://www.CommandPrompt.com/
The PostgreSQL Company - Command Prompt, Inc.

Alvaro Herrera escreveu:
> Bruce Momjian wrote:
>> Alvaro Herrera wrote:
>>> Bruce Momjian wrote:
>>>> Log Message:
>>>> -----------
>>>> Make 'find' syntax consistent; add .git exclusion to make_ctags.
>>> Should this consider the distdir target in /GNUmakefile.in too?
>> It only is looking for _directories_ to put 'tags' into.
>
> I have to admit that I find this whole idea a bit weird. Is it common
> to symlink the tags file?
>
> What I did was tell my editor that the tags file is to be found on the
> root source directory. It seems the most natural way to do this. Am I
> alone in this?
>
No, I do it too.

--
Euler Taveira de Oliveira
http://www.timbira.com/

Euler Taveira de Oliveira wrote:
> Alvaro Herrera escreveu:
> > Bruce Momjian wrote:
> >> Alvaro Herrera wrote:
> >>> Bruce Momjian wrote:
> >>>> Log Message:
> >>>> -----------
> >>>> Make 'find' syntax consistent; add .git exclusion to make_ctags.
> >>> Should this consider the distdir target in /GNUmakefile.in too?
> >> It only is looking for _directories_ to put 'tags' into.
> >
> > I have to admit that I find this whole idea a bit weird. Is it common
> > to symlink the tags file?
> >
> > What I did was tell my editor that the tags file is to be found on the
> > root source directory. It seems the most natural way to do this. Am I
> > alone in this?
> >
> No, I do it too.

Well, I just tell my editor to look for the tags file in the current
directory so I don't have to configure it.

--
Bruce Momjian <bruce(at)momjian(dot)us> http://momjian.us
EnterpriseDB http://enterprisedb.com

+ If your life is a hard drive, Christ can be your backup. +

Bruce Momjian wrote:
> Euler Taveira de Oliveira wrote:
> > Alvaro Herrera escreveu:
> > > Bruce Momjian wrote:
> > >> Alvaro Herrera wrote:
> > >>> Bruce Momjian wrote:
> > >>>> Log Message:
> > >>>> -----------
> > >>>> Make 'find' syntax consistent; add .git exclusion to make_ctags.
> > >>> Should this consider the distdir target in /GNUmakefile.in too?
> > >> It only is looking for _directories_ to put 'tags' into.
> > >
> > > I have to admit that I find this whole idea a bit weird. Is it common
> > > to symlink the tags file?
> > >
> > > What I did was tell my editor that the tags file is to be found on the
> > > root source directory. It seems the most natural way to do this. Am I
> > > alone in this?
> > >
> > No, I do it too.
>
> Well, I just tell my editor to look for the tags file in the current
> directory so I don't have to configure it.

But you had to configure the make_etags script :-) It just seems that
the cruft is in a different place. Don't the tags symlinks show up in
diff output and stuff like that?

--
Alvaro Herrera http://www.CommandPrompt.com/
The PostgreSQL Company - Command Prompt, Inc.

Alvaro Herrera wrote:
> Bruce Momjian wrote:
> > Euler Taveira de Oliveira wrote:
> > > Alvaro Herrera escreveu:
> > > > Bruce Momjian wrote:
> > > >> Alvaro Herrera wrote:
> > > >>> Bruce Momjian wrote:
> > > >>>> Log Message:
> > > >>>> -----------
> > > >>>> Make 'find' syntax consistent; add .git exclusion to make_ctags.
> > > >>> Should this consider the distdir target in /GNUmakefile.in too?
> > > >> It only is looking for _directories_ to put 'tags' into.
> > > >
> > > > I have to admit that I find this whole idea a bit weird. Is it common
> > > > to symlink the tags file?
> > > >
> > > > What I did was tell my editor that the tags file is to be found on the
> > > > root source directory. It seems the most natural way to do this. Am I
> > > > alone in this?
> > > >
> > > No, I do it too.
> >
> > Well, I just tell my editor to look for the tags file in the current
> > directory so I don't have to configure it.
>
> But you had to configure the make_etags script :-) It just seems that
> the cruft is in a different place. Don't the tags symlinks show up in
> diff output and stuff like that?

No because they are not part of CVS.

--
Bruce Momjian <bruce(at)momjian(dot)us> http://momjian.us
EnterpriseDB http://enterprisedb.com

+ If your life is a hard drive, Christ can be your backup. +