diff mbox

[05/32] package/systemd: add --enable/disable-zlib

Message ID 1456437341-19025-6-git-send-email-gabe@hashrabbit.co
State Accepted
Headers show

Commit Message

Gabe Evans Feb. 25, 2016, 9:55 p.m. UTC
Signed-off-by: Gabe Evans <gabe@hashrabbit.co>
---
 package/systemd/systemd.mk | 7 +++++++
 1 file changed, 7 insertions(+)

Comments

Arnout Vandecappelle Feb. 26, 2016, 12:05 a.m. UTC | #1
On 02/25/16 22:55, Gabe Evans wrote:
> Signed-off-by: Gabe Evans <gabe@hashrabbit.co>

Reviewed-by: Arnout Vandecappelle (Essensium/Mind) <arnout@mind.be>

 Again, for master.

 I wonder: what is zlib used for? Also for compressed journal? If so, why didn't
you update the help text?

 Regards,
 Arnout

> ---
>  package/systemd/systemd.mk | 7 +++++++
>  1 file changed, 7 insertions(+)
> 
> diff --git a/package/systemd/systemd.mk b/package/systemd/systemd.mk
> index 9f15f24..29c764e 100644
> --- a/package/systemd/systemd.mk
> +++ b/package/systemd/systemd.mk
> @@ -97,6 +97,13 @@ else
>  SYSTEMD_CONF_OPTS += --disable-xz
>  endif
>  
> +ifeq ($(BR2_PACKAGE_ZLIB),y)
> +SYSTEMD_DEPENDENCIES += zlib
> +SYSTEMD_CONF_OPTS += --enable-zlib
> +else
> +SYSTEMD_CONF_OPTS += --disable-zlib
> +endif
> +
>  ifeq ($(BR2_PACKAGE_LIBGCRYPT),y)
>  SYSTEMD_DEPENDENCIES += libgcrypt
>  SYSTEMD_CONF_OPTS += \
>
Gabe Evans Feb. 26, 2016, 12:27 a.m. UTC | #2
On Thu, Feb 25, 2016 at 4:05 PM Arnout Vandecappelle <arnout@mind.be> wrote:

> On 02/25/16 22:55, Gabe Evans wrote:
> > Signed-off-by: Gabe Evans <gabe@hashrabbit.co>
>
> Reviewed-by: Arnout Vandecappelle (Essensium/Mind) <arnout@mind.be>
>
>  Again, for master.
>
>  I wonder: what is zlib used for? Also for compressed journal? If so, why
> didn't
> you update the help text?
>

In patch 22, systemd-importd selects libcurl, xz, zlib, bzip2, and
libgcrypt . I didn't add notes for the libraries that weren't needed for
anything else.

Thanks,
Gabe


>
>  Regards,
>  Arnout
>
> > ---
> >  package/systemd/systemd.mk | 7 +++++++
> >  1 file changed, 7 insertions(+)
> >
> > diff --git a/package/systemd/systemd.mk b/package/systemd/systemd.mk
> > index 9f15f24..29c764e 100644
> > --- a/package/systemd/systemd.mk
> > +++ b/package/systemd/systemd.mk
> > @@ -97,6 +97,13 @@ else
> >  SYSTEMD_CONF_OPTS += --disable-xz
> >  endif
> >
> > +ifeq ($(BR2_PACKAGE_ZLIB),y)
> > +SYSTEMD_DEPENDENCIES += zlib
> > +SYSTEMD_CONF_OPTS += --enable-zlib
> > +else
> > +SYSTEMD_CONF_OPTS += --disable-zlib
> > +endif
> > +
> >  ifeq ($(BR2_PACKAGE_LIBGCRYPT),y)
> >  SYSTEMD_DEPENDENCIES += libgcrypt
> >  SYSTEMD_CONF_OPTS += \
> >
>
>
> --
> Arnout Vandecappelle                          arnout at mind be
> Senior Embedded Software Architect            +32-16-286500
> Essensium/Mind                                http://www.mind.be
> G.Geenslaan 9, 3001 Leuven, Belgium           BE 872 984 063 RPR Leuven
> LinkedIn profile: http://www.linkedin.com/in/arnoutvandecappelle
> GPG fingerprint:  7493 020B C7E3 8618 8DEC 222C 82EB F404 F9AC 0DDF
>
Thomas Petazzoni Feb. 29, 2016, 9:48 p.m. UTC | #3
Dear Gabe Evans,

On Thu, 25 Feb 2016 21:55:14 +0000, Gabe Evans wrote:
> Signed-off-by: Gabe Evans <gabe@hashrabbit.co>
> ---
>  package/systemd/systemd.mk | 7 +++++++
>  1 file changed, 7 insertions(+)

Applied to next, thanks.

Thomas
diff mbox

Patch

diff --git a/package/systemd/systemd.mk b/package/systemd/systemd.mk
index 9f15f24..29c764e 100644
--- a/package/systemd/systemd.mk
+++ b/package/systemd/systemd.mk
@@ -97,6 +97,13 @@  else
 SYSTEMD_CONF_OPTS += --disable-xz
 endif
 
+ifeq ($(BR2_PACKAGE_ZLIB),y)
+SYSTEMD_DEPENDENCIES += zlib
+SYSTEMD_CONF_OPTS += --enable-zlib
+else
+SYSTEMD_CONF_OPTS += --disable-zlib
+endif
+
 ifeq ($(BR2_PACKAGE_LIBGCRYPT),y)
 SYSTEMD_DEPENDENCIES += libgcrypt
 SYSTEMD_CONF_OPTS += \