Buildroot Archive on lore.kernel.org
 help / color / mirror / Atom feed
From: Peter Seiderer <ps.report@gmx.net>
To: buildroot@busybox.net
Subject: [Buildroot] [PATCH 1/1] postgresql --with-libxml configure option added
Date: Mon, 11 Jun 2018 22:19:09 +0200	[thread overview]
Message-ID: <20180611221909.662ad5ad@gmx.net> (raw)
In-Reply-To: <549ffe67-ad9f-b607-6735-09faa86f0b09@gmx.de>

Hello Daniel,

On Fri, 8 Jun 2018 08:46:42 +0200, Daniel Deterding <daniel.deterding@gmx.de> wrote:

> The postgresql package is now built with --with-libxml if 
> BR2_PACKAGE_LIBXML2 is selected to allow the use of XML functions in 
> postgresql.

Minor nit, maybe add:

  Fixes https://bugs.busybox.net/show_bug.cgi?id=11071

> 
> Signed-off-by: Daniel Deterding <dde@innovaphone.com>

Thanks for the patch, you can add my
Reviewed-by: Peter Seiderer <ps.report@gmx.net>

But I believe your email did not yet hit the mailing list/patchwork
(I did not find it there [1], [2])...

Regards,
Peter

[1] http://lists.busybox.net/pipermail/buildroot/2018-June/date.html
[2] https://patchwork.ozlabs.org/project/buildroot/list/

> ---
>  ?package/postgresql/postgresql.mk | 8 ++++++++
>  ?1 file changed, 8 insertions(+)
> 
> diff --git a/package/postgresql/postgresql.mk 
> b/package/postgresql/postgresql.mk
> index 94b3e60e59..0940873686 100644
> --- a/package/postgresql/postgresql.mk
> +++ b/package/postgresql/postgresql.mk
> @@ -70,6 +70,14 @@ else
>  ?POSTGRESQL_CONF_OPTS += --without-ldap
>  ?endif
> 
> +ifeq ($(BR2_PACKAGE_LIBXML2),y)
> +POSTGRESQL_DEPENDENCIES += libxml2
> +POSTGRESQL_CONF_OPTS += --with-libxml
> +POSTGRESQL_CONF_ENV += XML2_CONFIG=$(STAGING_DIR)/usr/bin/xml2-config
> +else
> +POSTGRESQL_CONF_OPTS += --without-libxml
> +endif
> +
>  ?define POSTGRESQL_USERS
>  ??????? postgres -1 postgres -1 * /var/lib/pgsql /bin/sh - PostgreSQL 
> Server
>  ?endef
> --
> 2.11.0
> 
> 

       reply	other threads:[~2018-06-11 20:19 UTC|newest]

Thread overview: 6+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
     [not found] <549ffe67-ad9f-b607-6735-09faa86f0b09@gmx.de>
2018-06-11 20:19 ` Peter Seiderer [this message]
     [not found] <20180608062231.2345-1-dde@innovaphone.com>
     [not found] ` <F73717A01AEC1F4CB9A3D10E39CBB88F4424CD47@INNO-EX-2010.innovaphone.sifi>
     [not found]   ` <F73717A01AEC1F4CB9A3D10E39CBB88F443662EA@INNO-EX-2010.innovaphone.sifi>
2018-07-27  7:06     ` [Buildroot] [PATCH 1/1] postgresql --with-libxml configure option added Thomas Ehrhardt
2018-07-27  7:31       ` Thomas Petazzoni
2018-07-27  9:55         ` Arnout Vandecappelle
2018-07-27 21:39           ` Adam Duskett
2018-07-27 21:49             ` Thomas Petazzoni

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=20180611221909.662ad5ad@gmx.net \
    --to=ps.report@gmx.net \
    --cc=buildroot@busybox.net \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox