Thread: FreeBSD Packages/Port for 7.3beta1...

FreeBSD Packages/Port for 7.3beta1...

From
Sean Chittenden
Date:
I've put together some packages for the 7.3beta1 release.  The can be
found here along with a tenative FreeBSD port:
  http://66.250.180.19/postgresql-7.3beta1/

The differences in the files are that the postgresql-7.3b1-O3.tbz has
been compiled with -O3 where as the postgresql-7.3b1.tbz hasn't.  See
my next message for details.

-sc

--
Sean Chittenden

Re: FreeBSD Packages/Port for 7.3beta1...

From
Peter Eisentraut
Date:
Sean Chittenden writes:

> I've put together some packages for the 7.3beta1 release.  The can be
> found here along with a tenative FreeBSD port:
>
>    http://66.250.180.19/postgresql-7.3beta1/

I checked out this port and made some notes that you might find useful.


[Makefile]

You can remove --enable-locale, --enable-syslog, --with-CXX as configure
arguments.  They no longer do anything.

This

LDFLAGS+=       -L${LOCALBASE}/lib -lgnugetopt
CONFIGURE_ENV+= LDFLAGS="${LDFLAGS}"

is redundant.  You already put LOCALBASE/lib into --with-libs.  Also, if
you wish, we can automatically check for -lgnugetopt in configure.  We
already do that for other spellings of the same library.

I would like some details on the following.

# if you want localized messages, make -DWITH_GETTEXT
# WARNING: this seems to require relinking binaries depending on
# libpq.so, including for example mod_php and tcl.

This

CONFIGURE_ENV+= "LIBS=-lintl"
LDFLAGS+=       -L${LOCALBASE}/lib -lintl

is even more redundant, because configure checks for -lintl automatically
(and one of LIBS and LDFLAGS would have sufficed).

Multibyte is no longer an option (it's the default), so you can remove
anything that refers to it.

If you want to strip the binaries, you can use 'gmake install-strip'
instead of 'gmake install'.  It's all automatic.


[patch-ak]

I assume you're going to fix this properly sometime...


[files/patch-al]

Can be removed for beta 2.


[files/dot.*.in]

Do you need the PATH assignment?  Shouldn't the user decide for himself
what he wants in the path?  PostgreSQL certainly doesn't need the path
set, if you're concerned about that.

PGLIB hasn't done anything for several releases...

PGDATESTYLE should now be set in the configuration file, so you can remove
the environment variable assignment.

Setting locales (LC_ALL) is now best done as an option to initdb.  Be sure
to update pkg-message to that effect.  Also, the encoding should be
specified to initdb (rather than configure --enable-multibyte=ENCODING).

Not sure what the TZ assignment is supposed to accomplish.  It certainly
doesn't alter the way the regression tests turn out, as it seems to claim.
Might have been an ancient problem.


[files/patch.configure]

I think you should handle that through the makefiles.  In fact, you
probably shouldn't specify an argument to the krb options if you're
concerned about this.


[files/post-install-notes]

Be sure to revise those, as some of the things are now shipped separately
(such as PgAccess).


[scripts/configure.postgresql]

Remove multibyte option.

PostgreSQL should work with Heimdal now.

-- 
Peter Eisentraut   peter_e@gmx.net



Re: FreeBSD Packages/Port for 7.3beta1...

From
Tatsuo Ishii
Date:
> Setting locales (LC_ALL) is now best done as an option to initdb.  Be sure
> to update pkg-message to that effect.  Also, the encoding should be
> specified to initdb (rather than configure --enable-multibyte=ENCODING).

I guess --enable-multibyte=ENCODING does nothing with 7.3
--
Tatsuo Ishii


Re: FreeBSD Packages/Port for 7.3beta1...

From
Bruce Momjian
Date:
Tatsuo Ishii wrote:
> > Setting locales (LC_ALL) is now best done as an option to initdb.  Be sure
> > to update pkg-message to that effect.  Also, the encoding should be
> > specified to initdb (rather than configure --enable-multibyte=ENCODING).
> 
> I guess --enable-multibyte=ENCODING does nothing with 7.3

I have added to HISTORY:

Always enable multibyte in compile, remove --enable-multibyte option (Tatsuo)
Always enable locale in compile, remove --enable-locale option (Tatsuo)

--  Bruce Momjian                        |  http://candle.pha.pa.us pgman@candle.pha.pa.us               |  (610)
359-1001+  If your life is a hard drive,     |  13 Roberts Road +  Christ can be your backup.        |  Newtown Square,
Pennsylvania19073
 


Re: FreeBSD Packages/Port for 7.3beta1...

From
Bruce Momjian
Date:
Peter Eisentraut wrote:
> LDFLAGS+=       -L${LOCALBASE}/lib -lgnugetopt
> CONFIGURE_ENV+= LDFLAGS="${LDFLAGS}"
>
> is redundant.  You already put LOCALBASE/lib into --with-libs.  Also, if
> you wish, we can automatically check for -lgnugetopt in configure.  We
> already do that for other spellings of the same library.

I have applied the following patch to search for gnugetopt to CVS.
Autoconf updated.

--
  Bruce Momjian                        |  http://candle.pha.pa.us
  pgman@candle.pha.pa.us               |  (610) 359-1001
  +  If your life is a hard drive,     |  13 Roberts Road
  +  Christ can be your backup.        |  Newtown Square, Pennsylvania 19073
Index: configure.in
===================================================================
RCS file: /cvsroot/pgsql-server/configure.in,v
retrieving revision 1.208
diff -c -c -r1.208 configure.in
*** configure.in    11 Sep 2002 04:27:48 -0000    1.208
--- configure.in    16 Sep 2002 20:08:56 -0000
***************
*** 607,613 ****
  AC_CHECK_LIB(gen,      main)
  AC_CHECK_LIB(PW,       main)
  AC_CHECK_LIB(resolv,   main)
! AC_SEARCH_LIBS(getopt_long, [getopt])
  # QNX:
  AC_CHECK_LIB([[unix]], main)
  AC_SEARCH_LIBS(crypt,  crypt)
--- 607,613 ----
  AC_CHECK_LIB(gen,      main)
  AC_CHECK_LIB(PW,       main)
  AC_CHECK_LIB(resolv,   main)
! AC_SEARCH_LIBS(getopt_long, [getopt gnugetopt])
  # QNX:
  AC_CHECK_LIB([[unix]], main)
  AC_SEARCH_LIBS(crypt,  crypt)