vcbuild optional packages

Magnus Hagander <magnus@hagander.net> writes:

This patch fixes vcbuild so you can build without OpenSSL and libz
should you want to. This disables the sslinfo and pgcrypto modules
because they require the libraries to build at all.

Applied.

regards, tom lane

On 1/8/07, Magnus Hagander <magnus@hagander.net> wrote:

This patch fixes vcbuild so you can build without OpenSSL and libz
should you want to. This disables the sslinfo and pgcrypto modules
because they require the libraries to build at all.

Both openssl and zlib are optional for pgcrypto. Please
look at the real Makefile. (Put -DDISABLE_ZLIB into CFLAGS.)

--
marko

Marko Kreen wrote:

On 1/8/07, Magnus Hagander <magnus@hagander.net> wrote:

This patch fixes vcbuild so you can build without OpenSSL and libz
should you want to. This disables the sslinfo and pgcrypto modules
because they require the libraries to build at all.

Both openssl and zlib are optional for pgcrypto. Please
look at the real Makefile. (Put -DDISABLE_ZLIB into CFLAGS.)

Ah, I see. Will have to take a look at that later then.

Is there any reason why the check is "backwards" in pgcrypto compared to
the rest of the system? All the other places check for HAVE_ZLIB from
what I can tell.

//Magnus

On 1/14/07, Magnus Hagander <magnus@hagander.net> wrote:

Is there any reason why the check is "backwards" in pgcrypto compared to
the rest of the system? All the other places check for HAVE_ZLIB from
what I can tell.

Eh, just an oversight from my part. I was so focused on Makefile
hacking so I forgot to check if there is already a define
somewhere. And its reverse to have less clutter in command-line
in default case.

Attached patch replaces unnecessary DISABLE_ZLIB define in
pgcrypto with HAVE_LIBZ from core. Result is bit simpler
Makefile which is only good.

--
marko

Marko Kreen wrote:

On 1/14/07, Magnus Hagander <magnus@hagander.net> wrote:

Is there any reason why the check is "backwards" in pgcrypto compared to
the rest of the system? All the other places check for HAVE_ZLIB from
what I can tell.

Eh, just an oversight from my part. I was so focused on Makefile
hacking so I forgot to check if there is already a define
somewhere. And its reverse to have less clutter in command-line
in default case.

Attached patch replaces unnecessary DISABLE_ZLIB define in
pgcrypto with HAVE_LIBZ from core. Result is bit simpler
Makefile which is only good.

Applied.

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