Re: [ODBC] MD5 support for ODBC - Mailing list pgsql-patches

From Peter Eisentraut
Subject Re: [ODBC] MD5 support for ODBC
Date
Msg-id Pine.LNX.4.30.0111131953130.17319-100000@peter.localdomain
Whole thread Raw
In response to Re: [ODBC] MD5 support for ODBC  (Bruce Momjian <pgman@candle.pha.pa.us>)
Responses Re: [ODBC] MD5 support for ODBC
List pgsql-patches
Bruce Momjian writes:

> > Nope, this doesn't work.  The source directory is not necessarily the
> > current directory.
>
> OK, I will remove it.  However, many of our Makefiles that do ../..

The *build* directory is always the current directory, so '..' stuff is
okay to refer to built files.  But to reference source files you need to
start with $(srcdir) or $(top_srcdir), whichever is convenient.

> Also, should we add such checks to the other files that must remain
> identical?

I don't think we have any others that must remain identical per se.

In general, checks for source code sanity are okay, but if you want them
in a makefile they should be under the maintainer-check target, not in the
build path.  However, since there is very little that merits an automated
check of this sort, I doubt very many people would actually run this
target.

> Yes, I already do a symlink for md5.c in libpq.  The problem is that
> ODBC has to compile stand-alone, with no reference to any outside
> directory.

Okay, that's reasonable.  I think making a note in the file under odbc/
should be sufficient.  Possibly include which CVS revision of the original
the copy is based on, so one can see when it needs updating.  Also, a note
in the venerable RELEASE_CHANGES file seems appropriate.

--
Peter Eisentraut   peter_e@gmx.net


pgsql-patches by date:

Previous
From: Tom Lane
Date:
Subject: Re: Patch to add Heimdal kerberos support
Next
From: Bruce Momjian
Date:
Subject: Re: [ODBC] MD5 support for ODBC