dbTalk Databases Forums  

[BUGS] Some patches to enhance the contrib build

mailing.database.pgsql-bugs mailing.database.pgsql-bugs


Discuss [BUGS] Some patches to enhance the contrib build in the mailing.database.pgsql-bugs forum.



Reply
 
Thread Tools Display Modes
  #1  
Old   
Martin Pitt
 
Posts: n/a

Default [BUGS] Some patches to enhance the contrib build - 10-31-2004 , 05:56 PM







--BXVAT5kNtrzKuDFl
Content-Type: text/plain; charset=us-ascii
Content-Disposition: inline
Content-Transfer-Encoding: quoted-printable

Hi PostgreSQL developers!

The current Debian package has some patches that tweak the building of
contrib modules. I think they would be interesting for other
distributions, too. Most of this stuff was contributed by users who
actually use these modules.

25contrib-dbmirror:=20
- additionally install some useful scripts and data files

25contrib-enablemysql:
- enable mysql module build
- add Makefile
- fix hashbang of mysql2pgsql

25contrib-enableoracle:
- enable oracle module build
- add Makefile
- patch ora2pg.pl to look for Ora2Pg.pm additionally in the path
where ora2pg.pl is installed

25contrib-enablexml:
- enable xml module build
- Makefile: add CFLAGS for "-I/usr/include/libxml2" to allow building

26dbf2pg-errorcheck:
- add better error checking to dbf2pg

27dbf2pg-textfield:
- add dbf2pg support for the dbase field 'M' which corresponds to
PostgreSQL's TEXT field

Do you consider adopting them in the official version?

Thanks a lot and have a nice day!

Martin
--=20
Martin Pitt http://www.piware.de
Ubuntu Developer http://www.ubuntulinux.org
Debian GNU/Linux Developer http://www.debian.org

--BXVAT5kNtrzKuDFl
Content-Type: application/pgp-signature; name="signature.asc"
Content-Description: Digital signature
Content-Disposition: inline

-----BEGIN PGP SIGNATURE-----
Version: GnuPG v1.2.4 (GNU/Linux)

iD8DBQFBhXoBDecnbV4Fd/IRAi89AJoDEcCPwyjmk4rkuKMlSUlFMHbYLgCfYMjA
M50Dke8kRxygHXmDbh9bAyQ=
=50/o
-----END PGP SIGNATURE-----

--BXVAT5kNtrzKuDFl--

Reply With Quote
  #2  
Old   
Neil Conway
 
Posts: n/a

Default Re: [BUGS] Some patches to enhance the contrib build - 10-31-2004 , 09:17 PM






On Mon, 2004-11-01 at 10:49, Martin Pitt wrote:
Quote:
The current Debian package has some patches that tweak the building of
contrib modules. I think they would be interesting for other
distributions, too. Most of this stuff was contributed by users who
actually use these modules.
Thanks for letting us know.

BTW, the information in postgresql-7.4.6/debian/copyright is out of date
(old download URL, old PostgreSQL copyright dates & the regex copyright
should be updated per the license in src/backend/regex/COPYRIGHT).

20pg_dump_datestyle seems redundant (we already set DateStyle to ISO in
pg_dump).

Quote:
25contrib-dbmirror:
- additionally install some useful scripts and data files
Makes sense to me, although I won't claim to be a dbmirror user. I'll
apply this to HEAD unless anyone objects.

Quote:
25contrib-enablexml:
- enable xml module build
- Makefile: add CFLAGS for "-I/usr/include/libxml2" to allow building
ISTM the right fix is to use xml2-config.

Quote:
25contrib-enablemysql:
- enable mysql module build
- add Makefile
- fix hashbang of mysql2pgsql

25contrib-enableoracle:
- enable oracle module build
- add Makefile
- patch ora2pg.pl to look for Ora2Pg.pm additionally in the path
where ora2pg.pl is installed

26dbf2pg-errorcheck:
- add better error checking to dbf2pg

27dbf2pg-textfield:
- add dbf2pg support for the dbase field 'M' which corresponds to
PostgreSQL's TEXT field
I can't find these patches in this diff:

http://ftp.debian.org/debian/pool/ma....4.6-2.diff.gz

could you tell me where to find them?

-Neil



---------------------------(end of broadcast)---------------------------
TIP 8: explain analyze is your friend


Reply With Quote
  #3  
Old   
Tom Lane
 
Posts: n/a

Default Re: [BUGS] Some patches to enhance the contrib build - 10-31-2004 , 09:48 PM



Neil Conway <neilc (AT) samurai (DOT) com> writes:
Quote:
On Mon, 2004-11-01 at 10:49, Martin Pitt wrote:
25contrib-dbmirror:
- additionally install some useful scripts and data files

Makes sense to me, although I won't claim to be a dbmirror user. I'll
apply this to HEAD unless anyone objects.
Steve Singer, the author and maintainer of dbmirror, is still around...
so it would be polite to get his concurrence first.

regards, tom lane

---------------------------(end of broadcast)---------------------------
TIP 4: Don't 'kill -9' the postmaster


Reply With Quote
  #4  
Old   
Martin Pitt
 
Posts: n/a

Default Re: [BUGS] Some patches to enhance the contrib build - 11-01-2004 , 02:27 AM




--GPJrCs/72TxItFYR
Content-Type: text/plain; charset=us-ascii
Content-Disposition: inline
Content-Transfer-Encoding: quoted-printable

Hi again!

Martin Pitt [2004-11-01 9:04 +0100]:
Quote:
Neil Conway [2004-11-01 14:06 +1100]:
20pg_dump_datestyle seems redundant (we already set DateStyle to ISO in
pg_dump).
I will look at this and remove it if appropriate. Thanks for the hint.

Quote:
ISTM the right fix is to use xml2-config.
=20
Right, replacing the -I option with `xml2-config --cflags` works fine
and seems to be more portable. Fixed in arch head.
BTW, I actually used

CFLAGS +=3D $(shell xml2-config --cflags)

in the current patch. I do not really know which is more portable,
using backticks to call xml2-config at compiler invocation time, or
using the $(shell ) syntax which seems GNU make specific. Both wors,
your choice :-)

Martin
--=20
Martin Pitt http://www.piware.de
Ubuntu Developer http://www.ubuntulinux.org
Debian GNU/Linux Developer http://www.debian.org

--GPJrCs/72TxItFYR
Content-Type: application/pgp-signature; name="signature.asc"
Content-Description: Digital signature
Content-Disposition: inline

-----BEGIN PGP SIGNATURE-----
Version: GnuPG v1.2.4 (GNU/Linux)

iD8DBQFBhe+zDecnbV4Fd/IRAhVMAKDULOtGcqYbCa+0kf5JZg6QV7JE6ACdETF3
cIofKk5CL+lP6R+a6bTcZDc=
=TVmh
-----END PGP SIGNATURE-----

--GPJrCs/72TxItFYR--


Reply With Quote
  #5  
Old   
Martin Pitt
 
Posts: n/a

Default Re: [BUGS] Some patches to enhance the contrib build - 11-01-2004 , 02:32 AM




--mSxgbZZZvrAyzONB
Content-Type: text/plain; charset=us-ascii
Content-Disposition: inline
Content-Transfer-Encoding: quoted-printable

(Sorry if you get thise message twice; the first time I accidentially
used my @debian.org address, which is not subscribed).

Hi Neil!

Neil Conway [2004-11-01 14:06 +1100]:
Quote:
BTW, the information in postgresql-7.4.6/debian/copyright is out of date
(old download URL, old PostgreSQL copyright dates & the regex copyright
should be updated per the license in src/backend/regex/COPYRIGHT).
Oh, thanks for that hint. Fixed in arch head.

Quote:
20pg_dump_datestyle seems redundant (we already set DateStyle to ISO in
pg_dump).
=20
25contrib-dbmirror:=20
- additionally install some useful scripts and data files
=20
Makes sense to me, although I won't claim to be a dbmirror user. I'll
apply this to HEAD unless anyone objects.
I do not use it myself, but adding a few scripts to the installation
does not break anything and it was found useful by an user.

Quote:
25contrib-enablexml:
- enable xml module build
- Makefile: add CFLAGS for "-I/usr/include/libxml2" to allow building
=20
ISTM the right fix is to use xml2-config.
Right, replacing the -I option with `xml2-config --cflags` works fine
and seems to be more portable. Fixed in arch head.

Quote:
25contrib-enablemysql:
- enable mysql module build
- add Makefile
- fix hashbang of mysql2pgsql
=20
25contrib-enableoracle:
- enable oracle module build
- add Makefile
- patch ora2pg.pl to look for Ora2Pg.pm additionally in the path
where ora2pg.pl is installed
=20
26dbf2pg-errorcheck:
- add better error checking to dbf2pg
=20
27dbf2pg-textfield:
- add dbf2pg support for the dbase field 'M' which corresponds to
PostgreSQL's TEXT field
=20
I can't find these patches in this diff:>=20
http://ftp.debian.org/debian/pool/ma...ql_7.4.6-2.di=
ff.gz
=20
could you tell me where to find them?
Right now only in arch head:
http://arch.debian.org/arch/pkg-postgresql/postgresql/

The three 25contrib* were split out of the 25contrib patch currently
in 7.4.6-2. The dbf2pg patches were added by me yesterday. I will
probably make a new upload soon, though.

Thanks for considering and have a nice day!

Martin

--=20
Martin Pitt http://www.piware.de
Ubuntu Developer http://www.ubuntulinux.org
Debian GNU/Linux Developer http://www.debian.org

--mSxgbZZZvrAyzONB
Content-Type: application/pgp-signature; name="signature.asc"
Content-Description: Digital signature
Content-Disposition: inline

-----BEGIN PGP SIGNATURE-----
Version: GnuPG v1.2.4 (GNU/Linux)

iD8DBQFBhfEiDecnbV4Fd/IRAnmeAKCdXN7NuZI3SyOTeufV5b4wdZB6ygCgqAdW
/ALQLNoycx37OD5hrrVinEA=
=GYbm
-----END PGP SIGNATURE-----

--mSxgbZZZvrAyzONB--


Reply With Quote
Reply




Thread Tools
Display Modes

Posting Rules
You may not post new threads
You may not post replies
You may not post attachments
You may not edit your posts

vB code is On
Smilies are On
[IMG] code is On
HTML code is Off



Powered by vBulletin Version 3.5.3
Copyright ©2000 - 2012, Jelsoft Enterprises Ltd.