Re: [PATCHES] build/install xml2 when configured with libxml

2007-05-20 Thread Nikolay Samokhvalov

On 4/15/07, Peter Eisentraut [EMAIL PROTECTED] wrote:

Well, if we're going to make libxslt an explicit thing, then it'd be
trivial to add an xslt transformation function into the core, and then
I think we can claim equivalent support.  But we'll have to check the
details, of course.

I have been thinking, however, that I don't want to add more and more
library dependencies into the server.  libxml2 was necessary to some
extent.  But xslt functionality could easily be provided as a module.
This would be easy to do and might be useful even for 8.3.  But I don't
really know how to label that.  Having a contrib/xslt alongside
contrib/xml2 would probably be confusing.  Ideas?


The current CVS' configure is really confusing: it has --with-xslt
option, while there is no XSLT support in the core. At least let's
change the option's comment to smth like build with XSLT support (now
it is used for contrib/xml2 only)...

--
Best regards,
Nikolay

---(end of broadcast)---
TIP 7: You can help support the PostgreSQL project by donating at

   http://www.postgresql.org/about/donate


Re: [PATCHES] build/install xml2 when configured with libxml

2007-05-20 Thread Andrew Dunstan



Nikolay Samokhvalov wrote:


The current CVS' configure is really confusing: it has --with-xslt
option, while there is no XSLT support in the core. At least let's
change the option's comment to smth like build with XSLT support (now
it is used for contrib/xml2 only)...



contrib is a misnomer at best. When 8.3 branches I intend to propose 
that we abandon it altogether, in line with some previous discussions.


We can change the configure help text if people think it matters that 
much - which seems to me much more potentially useful than changing 
comments.


cheers

andrew

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


Re: [PATCHES] build/install xml2 when configured with libxml

2007-05-20 Thread Nikolay Samokhvalov

On 5/20/07, Andrew Dunstan [EMAIL PROTECTED] wrote:


contrib is a misnomer at best. When 8.3 branches I intend to propose
that we abandon it altogether, in line with some previous discussions.

We can change the configure help text if people think it matters that
much - which seems to me much more potentially useful than changing
comments.


Actually, I meant configure help text, not any comment in the code :-)

--
Best regards,
Nikolay

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


Re: [PATCHES] build/install xml2 when configured with libxml

2007-04-15 Thread Peter Eisentraut
Andrew Dunstan wrote:
 I don't think it should go away until we provide for equivalents in
 core, at least optionally.

Well, if we're going to make libxslt an explicit thing, then it'd be 
trivial to add an xslt transformation function into the core, and then 
I think we can claim equivalent support.  But we'll have to check the 
details, of course.

I have been thinking, however, that I don't want to add more and more 
library dependencies into the server.  libxml2 was necessary to some 
extent.  But xslt functionality could easily be provided as a module.  
This would be easy to do and might be useful even for 8.3.  But I don't 
really know how to label that.  Having a contrib/xslt alongside 
contrib/xml2 would probably be confusing.  Ideas?

-- 
Peter Eisentraut
http://developer.postgresql.org/~petere/

---(end of broadcast)---
TIP 7: You can help support the PostgreSQL project by donating at

http://www.postgresql.org/about/donate


Re: [PATCHES] build/install xml2 when configured with libxml

2007-04-14 Thread Peter Eisentraut
Andrew Dunstan wrote:
 The attached patch adds a test for libxslt/xslt.h and only builds
 contrib/xml2 if it's found,

But the policy is that the presence of features in the final build 
should not depend on the incidental presence of features in the build 
environment.  Either you select a feature, then it's built, or you 
don't, then it's not.  So the only options we really have are adding 
another switch for libxslt, or including it with libxml.  I'm not sure 
which is better.

-- 
Peter Eisentraut
http://developer.postgresql.org/~petere/

---(end of broadcast)---
TIP 9: In versions below 8.0, the planner will ignore your desire to
   choose an index scan if your joining column's datatypes do not
   match


Re: [PATCHES] build/install xml2 when configured with libxml

2007-04-14 Thread Andrew Dunstan



Tom Lane wrote:

Andrew Dunstan [EMAIL PROTECTED] writes:
  
The attached patch adds a test for libxslt/xslt.h and only builds 
contrib/xml2 if it's found, which I think should handle Peter's 
objection, as well as unbreak the buildfarm. (The patch is large because 
cvs diff seems to have behaved a bit oddly with the configure script - 
but otherwise it's fairly minimal).



[squint...] Are you using the same autoconf version as the rest of us?


  


Yes. Anyway, it looks like we need to do something else.

cheers

andrew

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


Re: [PATCHES] build/install xml2 when configured with libxml

2007-04-14 Thread Andrew Dunstan



Peter Eisentraut wrote:

Andrew Dunstan wrote:
  

The attached patch adds a test for libxslt/xslt.h and only builds
contrib/xml2 if it's found,



But the policy is that the presence of features in the final build 
should not depend on the incidental presence of features in the build 
environment.  Either you select a feature, then it's built, or you 
don't, then it's not.  So the only options we really have are adding 
another switch for libxslt, or including it with libxml.  I'm not sure 
which is better.


  


Then let's add a switch for libxslt. I have a strong suspicion we'll 
want it before long anyway, and it will mean we can do this in a less 
kludgy way.


cheers

andrew

---(end of broadcast)---
TIP 7: You can help support the PostgreSQL project by donating at

   http://www.postgresql.org/about/donate


Re: [PATCHES] build/install xml2 when configured with libxml

2007-04-14 Thread Andrew Dunstan



Bruce Momjian wrote:

Andrew Dunstan wrote:
  

Peter Eisentraut wrote:

And if it is, then you have several options: 
. don't configure with libxml, or

. don't build contrib modules from the contrib Makefile (use the
individual module Makefiles instead), or
. change the xml2 Makefile


Any of these could be worth considering, but it needs to be thought 
through first.


  
  

Well, I'm happy to receive suggestions.



Andrew has enabled /contrib/xml2 builds.

  


I reverted the contrib/Makefile portion of this until we find a better way.

cheers

andrew

---(end of broadcast)---
TIP 7: You can help support the PostgreSQL project by donating at

   http://www.postgresql.org/about/donate


Re: [PATCHES] build/install xml2 when configured with libxml

2007-04-14 Thread Tom Lane
Andrew Dunstan [EMAIL PROTECTED] writes:
 Peter Eisentraut wrote:
 But the policy is that the presence of features in the final build 
 should not depend on the incidental presence of features in the build 
 environment.  Either you select a feature, then it's built, or you 
 don't, then it's not.  So the only options we really have are adding 
 another switch for libxslt, or including it with libxml.  I'm not sure 
 which is better.

 Then let's add a switch for libxslt.

+1 --- the fact that so many buildfarm members only have one of the two
libraries is pretty suggestive that that's common.  We shouldn't require
both libraries to build the core xml features, if only because
contrib/xml2 is expected to go away eventually, no?

regards, tom lane

---(end of broadcast)---
TIP 9: In versions below 8.0, the planner will ignore your desire to
   choose an index scan if your joining column's datatypes do not
   match


Re: [PATCHES] build/install xml2 when configured with libxml

2007-04-14 Thread Stefan Kaltenbrunner
Tom Lane wrote:
 Andrew Dunstan [EMAIL PROTECTED] writes:
 Peter Eisentraut wrote:
 But the policy is that the presence of features in the final build 
 should not depend on the incidental presence of features in the build 
 environment.  Either you select a feature, then it's built, or you 
 don't, then it's not.  So the only options we really have are adding 
 another switch for libxslt, or including it with libxml.  I'm not sure 
 which is better.
 
 Then let's add a switch for libxslt.
 
 +1 --- the fact that so many buildfarm members only have one of the two
 libraries is pretty suggestive that that's common.  We shouldn't require
 both libraries to build the core xml features, if only because
 contrib/xml2 is expected to go away eventually, no?

well from a buildfarm maintainer perspective - I only have (or had on
some boxes) only libxml(or rather libxm-dev) installed because that's
what was required when we got the initial XML support.
Most of those boxes would have neither of those two - it's the buildfarm
itself that resulted in that library getting installed (and only that
one because it was sufficient at the time).


Stefan

---(end of broadcast)---
TIP 3: Have you checked our extensive FAQ?

   http://www.postgresql.org/docs/faq


Re: [PATCHES] build/install xml2 when configured with libxml

2007-04-14 Thread Andrew Dunstan



Tom Lane wrote:

Andrew Dunstan [EMAIL PROTECTED] writes:
  

Peter Eisentraut wrote:

But the policy is that the presence of features in the final build 
should not depend on the incidental presence of features in the build 
environment.  Either you select a feature, then it's built, or you 
don't, then it's not.  So the only options we really have are adding 
another switch for libxslt, or including it with libxml.  I'm not sure 
which is better.
  


  

Then let's add a switch for libxslt.



+1 --- the fact that so many buildfarm members only have one of the two
libraries is pretty suggestive that that's common.  We shouldn't require
both libraries to build the core xml features, if only because
contrib/xml2 is expected to go away eventually, no?


  


I don't think it should go away until we provide for equivalents in 
core, at least optionally.


Anyway, here's the patch.

cheers

andrew
Index: configure
===
RCS file: /cvsroot/pgsql/configure,v
retrieving revision 1.543
diff -c -r1.543 configure
*** configure	13 Apr 2007 18:50:01 -	1.543
--- configure	14 Apr 2007 19:23:54 -
***
*** 314,320 
  # include unistd.h
  #endif
  
! ac_subst_vars='SHELL PATH_SEPARATOR PACKAGE_NAME PACKAGE_TARNAME PACKAGE_VERSION PACKAGE_STRING PACKAGE_BUGREPORT exec_prefix prefix program_transform_name bindir sbindir libexecdir datadir sysconfdir sharedstatedir localstatedir libdir includedir oldincludedir infodir mandir build_alias host_alias target_alias DEFS ECHO_C ECHO_N ECHO_T LIBS configure_args build build_cpu build_vendor build_os host host_cpu host_vendor host_os PORTNAME docdir enable_nls WANTED_LANGUAGES default_port enable_shared enable_rpath enable_debug enable_profiling DTRACE DTRACEFLAGS enable_dtrace CC CFLAGS LDFLAGS CPPFLAGS ac_ct_CC EXEEXT OBJEXT CPP GCC TAS autodepend INCLUDES enable_thread_safety with_tcl with_perl with_python with_krb5 krb_srvtab with_pam with_ldap with_bonjour with_openssl XML2_CONFIG with_libxml with_zlib EGREP ELF_SYS LDFLAGS_SL AWK FLEX FLEXFLAGS LN_S LD with_gnu_ld ld_R_works RANLIB ac_ct_RANLIB TAR STRIP ac_ct_STRIP STRIP_STATIC_LIB STRIP_SHARED_LIB YACC YFLAGS PERL perl_archlibexp perl_privlibexp perl_useshrplib perl_embed_ldflags PYTHON python_version python_configdir python_includespec python_libdir python_libspec python_additional_libs HAVE_IPV6 LIBOBJS acx_pthread_config PTHREAD_CC PTHREAD_LIBS PTHREAD_CFLAGS LDAP_LIBS_FE LDAP_LIBS_BE HAVE_POSIX_SIGNALS MSGFMT MSGMERGE XGETTEXT localedir TCLSH TCL_CONFIG_SH TCL_INCLUDE_SPEC TCL_LIB_FILE TCL_LIBS TCL_LIB_SPEC TCL_SHARED_BUILD TCL_SHLIB_LD_LIBS NSGMLS JADE have_docbook DOCBOOKSTYLE COLLATEINDEX SGMLSPL vpath_build LTLIBOBJS'
  ac_subst_files=''
  
  # Initialize some variables set by options.
--- 314,320 
  # include unistd.h
  #endif
  
! ac_subst_vars='SHELL PATH_SEPARATOR PACKAGE_NAME PACKAGE_TARNAME PACKAGE_VERSION PACKAGE_STRING PACKAGE_BUGREPORT exec_prefix prefix program_transform_name bindir sbindir libexecdir datadir sysconfdir sharedstatedir localstatedir libdir includedir oldincludedir infodir mandir build_alias host_alias target_alias DEFS ECHO_C ECHO_N ECHO_T LIBS configure_args build build_cpu build_vendor build_os host host_cpu host_vendor host_os PORTNAME docdir enable_nls WANTED_LANGUAGES default_port enable_shared enable_rpath enable_debug enable_profiling DTRACE DTRACEFLAGS enable_dtrace CC CFLAGS LDFLAGS CPPFLAGS ac_ct_CC EXEEXT OBJEXT CPP GCC TAS autodepend INCLUDES enable_thread_safety with_tcl with_perl with_python with_krb5 krb_srvtab with_pam with_ldap with_bonjour with_openssl XML2_CONFIG with_libxml with_libxslt with_zlib EGREP ELF_SYS LDFLAGS_SL AWK FLEX FLEXFLAGS LN_S LD with_gnu_ld ld_R_works RANLIB ac_ct_RANLIB TAR STRIP ac_ct_STRIP STRIP_STATIC_LIB STRIP_SHARED_LIB YACC YFLAGS PERL perl_archlibexp perl_privlibexp perl_useshrplib perl_embed_ldflags PYTHON python_version python_configdir python_includespec python_libdir python_libspec python_additional_libs HAVE_IPV6 LIBOBJS acx_pthread_config PTHREAD_CC PTHREAD_LIBS PTHREAD_CFLAGS LDAP_LIBS_FE LDAP_LIBS_BE HAVE_POSIX_SIGNALS MSGFMT MSGMERGE XGETTEXT localedir TCLSH TCL_CONFIG_SH TCL_INCLUDE_SPEC TCL_LIB_FILE TCL_LIBS TCL_LIB_SPEC TCL_SHARED_BUILD TCL_SHLIB_LD_LIBS NSGMLS JADE have_docbook DOCBOOKSTYLE COLLATEINDEX SGMLSPL vpath_build LTLIBOBJS'
  ac_subst_files=''
  
  # Initialize some variables set by options.
***
*** 896,901 
--- 896,902 
--without-readline  do not use GNU Readline nor BSD Libedit for editing
--with-libedit-preferred  prefer BSD Libedit over GNU Readline
--with-libxml   build with XML support
+   --with-libxslt   build with XSLT support
--without-zlib  do not use Zlib
--with-gnu-ld   assume the C compiler uses GNU ld [default=no]
  
***
*** 4333,4338 
--- 4334,4377 
  
  
  #
+ # XSLT
+ #
+ 
+ pgac_args=$pgac_args 

Re: [PATCHES] build/install xml2 when configured with libxml

2007-04-13 Thread Bruce Momjian
Andrew Dunstan wrote:
 Peter Eisentraut wrote:
  And if it is, then you have several options: 
  . don't configure with libxml, or
  . don't build contrib modules from the contrib Makefile (use the
  individual module Makefiles instead), or
  . change the xml2 Makefile
  
 
  Any of these could be worth considering, but it needs to be thought 
  through first.
 

 
 Well, I'm happy to receive suggestions.

Andrew has enabled /contrib/xml2 builds.

-- 
  Bruce Momjian  [EMAIL PROTECTED]  http://momjian.us
  EnterpriseDB   http://www.enterprisedb.com

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

---(end of broadcast)---
TIP 1: if posting/reading through Usenet, please send an appropriate
   subscribe-nomail command to [EMAIL PROTECTED] so that your
   message can get through to the mailing list cleanly


Re: [PATCHES] build/install xml2 when configured with libxml

2007-04-13 Thread Andrew Dunstan

Bruce Momjian wrote:

Andrew Dunstan wrote:
  

Peter Eisentraut wrote:

And if it is, then you have several options: 
. don't configure with libxml, or

. don't build contrib modules from the contrib Makefile (use the
individual module Makefiles instead), or
. change the xml2 Makefile


Any of these could be worth considering, but it needs to be thought 
through first.


  
  

Well, I'm happy to receive suggestions.



Andrew has enabled /contrib/xml2 builds.

  


And in turn we have found some warnings we should look at cleaning up, e.g.:

ccache gcc -no-cpp-precomp -I/opt/libxml2-2.6.27_20070107/include/libxml2  -I. 
-I../../src/include -I/opt/libxml2-2.6.27_20070107/include/libxml2   -c -o 
xpath.o xpath.c -MMD -MP -MF .deps/xpath.Po
xpath.c: In function 'xml_encode_special_chars':
xpath.c:212: warning: pointer targets in passing argument 1 of 'strlen' differ 
in signedness
xpath.c: In function 'pgxmlNodeSetToText':
xpath.c:268: warning: pointer targets in passing argument 2 of 
'xmlBufferWriteChar' differ in signedness
xpath.c: In function 'pgxml_result_to_text':
xpath.c:607: warning: pointer targets in passing argument 1 of 'xmlStrdup' 
differ in signedness
xpath.c:612: warning: pointer targets in passing argument 1 of 'strlen' differ 
in signedness
xpath.c: In function 'xpath_table':
xpath.c:663: warning: pointer targets in initialization differ in signedness
xpath.c:738: warning: pointer targets in assignment differ in signedness
xpath.c:742: warning: pointer targets in passing argument 1 of 'strstr' differ 
in signedness
xpath.c:742: warning: pointer targets in passing argument 2 of 'strstr' differ 
in signedness
xpath.c:742: warning: pointer targets in assignment differ in signedness
xpath.c:896: warning: pointer targets in passing argument 1 of 'xmlStrdup' 
differ in signedness
xpath.c:904: warning: pointer targets in assignment differ in signedness
ccache gcc -no-cpp-precomp -I/opt/libxml2-2.6.27_20070107/include/libxml2  -I. 
-I../../src/include -I/opt/libxml2-2.6.27_20070107/include/libxml2   -c -o 
xslt_proc.o xslt_proc.c -MMD -MP -MF .deps/xslt_proc.Po
xslt_proc.c: In function 'xslt_process':
xslt_proc.c:105: warning: pointer targets in passing argument 1 of 
'xsltParseStylesheetFile' differ in signedness



cheers

andrew



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


Re: [PATCHES] build/install xml2 when configured with libxml

2007-04-13 Thread Andrew Dunstan



Bruce Momjian wrote:

Andrew Dunstan wrote:
  

Peter Eisentraut wrote:

And if it is, then you have several options: 
. don't configure with libxml, or

. don't build contrib modules from the contrib Makefile (use the
individual module Makefiles instead), or
. change the xml2 Makefile


Any of these could be worth considering, but it needs to be thought 
through first.


  
  

Well, I'm happy to receive suggestions.



Andrew has enabled /contrib/xml2 builds.

  



The attached patch adds a test for libxslt/xslt.h and only builds 
contrib/xml2 if it's found, which I think should handle Peter's 
objection, as well as unbreak the buildfarm. (The patch is large because 
cvs diff seems to have behaved a bit oddly with the configure script - 
but otherwise it's fairly minimal).


cheers

andrew
Index: configure
===
RCS file: /cvsroot/pgsql/configure,v
retrieving revision 1.543
diff -c -r1.543 configure
*** configure	13 Apr 2007 18:50:01 -	1.543
--- configure	14 Apr 2007 03:44:24 -
***
*** 314,320 
  # include unistd.h
  #endif
  
! ac_subst_vars='SHELL PATH_SEPARATOR PACKAGE_NAME PACKAGE_TARNAME PACKAGE_VERSION PACKAGE_STRING PACKAGE_BUGREPORT exec_prefix prefix program_transform_name bindir sbindir libexecdir datadir sysconfdir sharedstatedir localstatedir libdir includedir oldincludedir infodir mandir build_alias host_alias target_alias DEFS ECHO_C ECHO_N ECHO_T LIBS configure_args build build_cpu build_vendor build_os host host_cpu host_vendor host_os PORTNAME docdir enable_nls WANTED_LANGUAGES default_port enable_shared enable_rpath enable_debug enable_profiling DTRACE DTRACEFLAGS enable_dtrace CC CFLAGS LDFLAGS CPPFLAGS ac_ct_CC EXEEXT OBJEXT CPP GCC TAS autodepend INCLUDES enable_thread_safety with_tcl with_perl with_python with_krb5 krb_srvtab with_pam with_ldap with_bonjour with_openssl XML2_CONFIG with_libxml with_zlib EGREP ELF_SYS LDFLAGS_SL AWK FLEX FLEXFLAGS LN_S LD with_gnu_ld ld_R_works RANLIB ac_ct_RANLIB TAR STRIP ac_ct_STRIP STRIP_STATIC_LIB STRIP_SHARED_LIB YACC YFLAGS PERL perl_archlibexp perl_privlibexp perl_useshrplib perl_embed_ldflags PYTHON python_version python_configdir python_includespec python_libdir python_libspec python_additional_libs HAVE_IPV6 LIBOBJS acx_pthread_config PTHREAD_CC PTHREAD_LIBS PTHREAD_CFLAGS LDAP_LIBS_FE LDAP_LIBS_BE HAVE_POSIX_SIGNALS MSGFMT MSGMERGE XGETTEXT localedir TCLSH TCL_CONFIG_SH TCL_INCLUDE_SPEC TCL_LIB_FILE TCL_LIBS TCL_LIB_SPEC TCL_SHARED_BUILD TCL_SHLIB_LD_LIBS NSGMLS JADE have_docbook DOCBOOKSTYLE COLLATEINDEX SGMLSPL vpath_build LTLIBOBJS'
  ac_subst_files=''
  
  # Initialize some variables set by options.
--- 314,320 
  # include unistd.h
  #endif
  
! ac_subst_vars='SHELL PATH_SEPARATOR PACKAGE_NAME PACKAGE_TARNAME PACKAGE_VERSION PACKAGE_STRING PACKAGE_BUGREPORT exec_prefix prefix program_transform_name bindir sbindir libexecdir datadir sysconfdir sharedstatedir localstatedir libdir includedir oldincludedir infodir mandir build_alias host_alias target_alias DEFS ECHO_C ECHO_N ECHO_T LIBS configure_args build build_cpu build_vendor build_os host host_cpu host_vendor host_os PORTNAME docdir enable_nls WANTED_LANGUAGES default_port enable_shared enable_rpath enable_debug enable_profiling DTRACE DTRACEFLAGS enable_dtrace CC CFLAGS LDFLAGS CPPFLAGS ac_ct_CC EXEEXT OBJEXT CPP GCC TAS autodepend INCLUDES enable_thread_safety with_tcl with_perl with_python with_krb5 krb_srvtab with_pam with_ldap with_bonjour with_openssl XML2_CONFIG EGREP with_libxml have_libxslt with_zlib ELF_SYS LDFLAGS_SL AWK FLEX FLEXFLAGS LN_S LD with_gnu_ld ld_R_works RANLIB ac_ct_RANLIB TAR STRIP ac_ct_STRIP STRIP_STATIC_LIB STRIP_SHARED_LIB YACC YFLAGS PERL perl_archlibexp perl_privlibexp perl_useshrplib perl_embed_ldflags PYTHON python_version python_configdir python_includespec python_libdir python_libspec python_additional_libs HAVE_IPV6 LIBOBJS acx_pthread_config PTHREAD_CC PTHREAD_LIBS PTHREAD_CFLAGS LDAP_LIBS_FE LDAP_LIBS_BE HAVE_POSIX_SIGNALS MSGFMT MSGMERGE XGETTEXT localedir TCLSH TCL_CONFIG_SH TCL_INCLUDE_SPEC TCL_LIB_FILE TCL_LIBS TCL_LIB_SPEC TCL_SHARED_BUILD TCL_SHLIB_LD_LIBS NSGMLS JADE have_docbook DOCBOOKSTYLE COLLATEINDEX SGMLSPL vpath_build LTLIBOBJS'
  ac_subst_files=''
  
  # Initialize some variables set by options.
***
*** 4275,4280 
--- 4275,4281 
  fi;
  
  
+ have_libxslt=no
  if test $with_libxml = yes ; then
for ac_prog in xml2-config
  do
***
*** 4328,4334 
--- 4329,4740 
esac
  done
fi
+ 
+ echo $as_me:$LINENO: checking for egrep 5
+ echo $ECHO_N checking for egrep... $ECHO_C 6
+ if test ${ac_cv_prog_egrep+set} = set; then
+   echo $ECHO_N (cached) $ECHO_C 6
+ else
+   if echo a | (grep -E '(a|b)') /dev/null 21
+ then ac_cv_prog_egrep='grep -E'
+ else ac_cv_prog_egrep='egrep'
+ fi
+ fi
+ echo $as_me:$LINENO: result: $ac_cv_prog_egrep 5
+ echo 

Re: [PATCHES] build/install xml2 when configured with libxml

2007-04-13 Thread Tom Lane
Andrew Dunstan [EMAIL PROTECTED] writes:
 The attached patch adds a test for libxslt/xslt.h and only builds 
 contrib/xml2 if it's found, which I think should handle Peter's 
 objection, as well as unbreak the buildfarm. (The patch is large because 
 cvs diff seems to have behaved a bit oddly with the configure script - 
 but otherwise it's fairly minimal).

[squint...] Are you using the same autoconf version as the rest of us?

regards, tom lane

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


[PATCHES] build/install xml2 when configured with libxml

2007-04-04 Thread Andrew Dunstan


Since contrib/xml2 seems to be staying with us at least for now, this 
small patch enables it to be built and installed from the contrib 
Makefile when --with-libxml is given to configure.


If there's no objection I'll apply in a day or two.

cheers

andrew
Index: configure
===
RCS file: /cvsroot/pgsql/configure,v
retrieving revision 1.540
diff -c -r1.540 configure
*** configure	29 Mar 2007 15:30:51 -	1.540
--- configure	4 Apr 2007 19:05:01 -
***
*** 314,320 
  # include unistd.h
  #endif
  
! ac_subst_vars='SHELL PATH_SEPARATOR PACKAGE_NAME PACKAGE_TARNAME PACKAGE_VERSION PACKAGE_STRING PACKAGE_BUGREPORT exec_prefix prefix program_transform_name bindir sbindir libexecdir datadir sysconfdir sharedstatedir localstatedir libdir includedir oldincludedir infodir mandir build_alias host_alias target_alias DEFS ECHO_C ECHO_N ECHO_T LIBS configure_args build build_cpu build_vendor build_os host host_cpu host_vendor host_os PORTNAME docdir enable_nls WANTED_LANGUAGES default_port enable_shared enable_rpath enable_debug enable_profiling DTRACE DTRACEFLAGS enable_dtrace CC CFLAGS LDFLAGS CPPFLAGS ac_ct_CC EXEEXT OBJEXT CPP GCC TAS autodepend INCLUDES enable_thread_safety with_tcl with_perl with_python with_krb5 krb_srvtab with_pam with_ldap with_bonjour with_openssl XML2_CONFIG with_zlib EGREP ELF_SYS LDFLAGS_SL AWK FLEX FLEXFLAGS LN_S LD with_gnu_ld ld_R_works RANLIB ac_ct_RANLIB TAR STRIP ac_ct_STRIP STRIP_STATIC_LIB STRIP_SHARED_LIB YACC YFLAGS PERL perl_archlibexp perl_privlibexp perl_useshrplib perl_embed_ldflags PYTHON python_version python_configdir python_includespec python_libdir python_libspec python_additional_libs HAVE_IPV6 LIBOBJS acx_pthread_config PTHREAD_CC PTHREAD_LIBS PTHREAD_CFLAGS LDAP_LIBS_FE LDAP_LIBS_BE HAVE_POSIX_SIGNALS MSGFMT MSGMERGE XGETTEXT localedir TCLSH TCL_CONFIG_SH TCL_INCLUDE_SPEC TCL_LIB_FILE TCL_LIBS TCL_LIB_SPEC TCL_SHARED_BUILD TCL_SHLIB_LD_LIBS NSGMLS JADE have_docbook DOCBOOKSTYLE COLLATEINDEX SGMLSPL vpath_build LTLIBOBJS'
  ac_subst_files=''
  
  # Initialize some variables set by options.
--- 314,320 
  # include unistd.h
  #endif
  
! ac_subst_vars='SHELL PATH_SEPARATOR PACKAGE_NAME PACKAGE_TARNAME PACKAGE_VERSION PACKAGE_STRING PACKAGE_BUGREPORT exec_prefix prefix program_transform_name bindir sbindir libexecdir datadir sysconfdir sharedstatedir localstatedir libdir includedir oldincludedir infodir mandir build_alias host_alias target_alias DEFS ECHO_C ECHO_N ECHO_T LIBS configure_args build build_cpu build_vendor build_os host host_cpu host_vendor host_os PORTNAME docdir enable_nls WANTED_LANGUAGES default_port enable_shared enable_rpath enable_debug enable_profiling DTRACE DTRACEFLAGS enable_dtrace CC CFLAGS LDFLAGS CPPFLAGS ac_ct_CC EXEEXT OBJEXT CPP GCC TAS autodepend INCLUDES enable_thread_safety with_tcl with_perl with_python with_krb5 krb_srvtab with_pam with_ldap with_bonjour with_openssl XML2_CONFIG with_libxml with_zlib EGREP ELF_SYS LDFLAGS_SL AWK FLEX FLEXFLAGS LN_S LD with_gnu_ld ld_R_works RANLIB ac_ct_RANLIB TAR STRIP ac_ct_STRIP STRIP_STATIC_LIB STRIP_SHARED_LIB YACC YFLAGS PERL perl_archlibexp perl_privlibexp perl_useshrplib perl_embed_ldflags PYTHON python_version python_configdir python_includespec python_libdir python_libspec python_additional_libs HAVE_IPV6 LIBOBJS acx_pthread_config PTHREAD_CC PTHREAD_LIBS PTHREAD_CFLAGS LDAP_LIBS_FE LDAP_LIBS_BE HAVE_POSIX_SIGNALS MSGFMT MSGMERGE XGETTEXT localedir TCLSH TCL_CONFIG_SH TCL_INCLUDE_SPEC TCL_LIB_FILE TCL_LIBS TCL_LIB_SPEC TCL_SHARED_BUILD TCL_SHLIB_LD_LIBS NSGMLS JADE have_docbook DOCBOOKSTYLE COLLATEINDEX SGMLSPL vpath_build LTLIBOBJS'
  ac_subst_files=''
  
  # Initialize some variables set by options.
***
*** 4331,4336 
--- 4331,4337 
  fi
  
  
+ 
  #
  # Zlib
  #
***
*** 24091,24096 
--- 24092,24098 
  s,@with_bonjour@,$with_bonjour,;t t
  s,@with_openssl@,$with_openssl,;t t
  s,@XML2_CONFIG@,$XML2_CONFIG,;t t
+ s,@with_libxml@,$with_libxml,;t t
  s,@with_zlib@,$with_zlib,;t t
  s,@EGREP@,$EGREP,;t t
  s,@ELF_SYS@,$ELF_SYS,;t t
Index: configure.in
===
RCS file: /cvsroot/pgsql/configure.in,v
retrieving revision 1.507
diff -c -r1.507 configure.in
*** configure.in	29 Mar 2007 15:30:51 -	1.507
--- configure.in	4 Apr 2007 19:05:05 -
***
*** 577,582 
--- 577,583 
fi
  fi
  
+ AC_SUBST(with_libxml)
  
  #
  # Zlib
Index: contrib/Makefile
===
RCS file: /cvsroot/pgsql/contrib/Makefile,v
retrieving revision 1.71
diff -c -r1.71 Makefile
*** contrib/Makefile	8 Feb 2007 15:09:47 -	1.71
--- contrib/Makefile	4 Apr 2007 19:05:05 -
***
*** 37,45 
  WANTED_DIRS += sslinfo
  endif
  
  # Missing:
  #		start-scripts	\ (does not have a makefile)
- #		xml2		\ (requires libxml installed)
  
  
  all 

Re: [PATCHES] build/install xml2 when configured with libxml

2007-04-04 Thread Peter Eisentraut
Andrew Dunstan wrote:
 Since contrib/xml2 seems to be staying with us at least for now, this
 small patch enables it to be built and installed from the contrib
 Makefile when --with-libxml is given to configure.

contrib/xml2 also requires libxslt.

-- 
Peter Eisentraut
http://developer.postgresql.org/~petere/

---(end of broadcast)---
TIP 3: Have you checked our extensive FAQ?

   http://www.postgresql.org/docs/faq


Re: [PATCHES] build/install xml2 when configured with libxml

2007-04-04 Thread Andrew Dunstan

Peter Eisentraut wrote:

Andrew Dunstan wrote:
  

Since contrib/xml2 seems to be staying with us at least for now, this
small patch enables it to be built and installed from the contrib
Makefile when --with-libxml is given to configure.



contrib/xml2 also requires libxslt.

  
Well, how often is libxslt missing when libxml is present, in practice? 
And if it is, then you have several options:

. don't configure with libxml, or
. don't build contrib modules from the contrib Makefile (use the 
individual module Makefiles instead), or

. change the xml2 Makefile

My main purpose is to complete buildfarm build coverage.

cheers

andrew



---(end of broadcast)---
TIP 7: You can help support the PostgreSQL project by donating at

   http://www.postgresql.org/about/donate


Re: [PATCHES] build/install xml2 when configured with libxml

2007-04-04 Thread Peter Eisentraut
Andrew Dunstan wrote:
 Well, how often is libxslt missing when libxml is present, in
 practice?

A local sample shows a probability of 100%.

 And if it is, then you have several options: 
 . don't configure with libxml, or
 . don't build contrib modules from the contrib Makefile (use the
 individual module Makefiles instead), or
 . change the xml2 Makefile

Any of these could be worth considering, but it needs to be thought 
through first.

-- 
Peter Eisentraut
http://developer.postgresql.org/~petere/

---(end of broadcast)---
TIP 4: Have you searched our list archives?

   http://archives.postgresql.org


Re: [PATCHES] build/install xml2 when configured with libxml

2007-04-04 Thread Andrew Dunstan

Peter Eisentraut wrote:
And if it is, then you have several options: 
. don't configure with libxml, or

. don't build contrib modules from the contrib Makefile (use the
individual module Makefiles instead), or
. change the xml2 Makefile



Any of these could be worth considering, but it needs to be thought 
through first.


  


Well, I'm happy to receive suggestions.

cheers

andrew


---(end of broadcast)---
TIP 1: if posting/reading through Usenet, please send an appropriate
  subscribe-nomail command to [EMAIL PROTECTED] so that your
  message can get through to the mailing list cleanly