mailarchive of the ptxdist mailing list
 help / color / mirror / Atom feed
From: Alexander Dahl <ada@thorsis.com>
To: Lars Pedersen <lapeddk@gmail.com>
Cc: ptxdist@pengutronix.de
Subject: Re: [ptxdist] [PATCH] strongswan: version bump 5.9.2 -> 5.9.6
Date: Thu, 16 Jun 2022 09:53:23 +0200	[thread overview]
Message-ID: <Yqrhc39xY/y/ZkCq@ada.ifak-system.com> (raw)
In-Reply-To: <CAKd8=GtPA8q92jZ3k1S_oG56hDTyQ7mc9Bbm-4mSS545RgDpmw@mail.gmail.com>

Hello Lars,

Am Thu, Jun 16, 2022 at 09:25:59AM +0200 schrieb Lars Pedersen:
> Hello Alex,
> 
> On Thu, 16 Jun 2022 at 08:48, Alexander Dahl <ada@thorsis.com> wrote:
> >
> > Hello Lars,
> >
> > Am Thu, Jun 16, 2022 at 08:42:35AM +0200 schrieb Lars Pedersen:
> > > New configure options --with-selinux and --disable-kdf
> >
> > That first option is not considered in the patch below, is it?
> 
> --with-selinux is implicitly disabled (without). Should I explicitly
> set it --without-selinux?

Christian already suggested something, but let me answer this
question, because we just discuss the same issue in the RAUC patchset
thread: yes, configure options should be explicitly set in ptxdist.

The reason is package builds should be reproducible:

https://www.ptxdist.org/doc/contributing.html#package-builds-should-be-reproducible

And:

https://www.ptxdist.org/doc/contributing.html#configure-helper

Greets
Alex

> 
> >
> > Greets
> > Alex
> >
> > > Enabled generic modularized KDF (prf/prf+) implementation
> > >
> > > Signed-off-by: Lars Pedersen <lapeddk@gmail.com>
> > > ---
> > >  rules/strongswan.make | 6 ++++--
> > >  1 file changed, 4 insertions(+), 2 deletions(-)
> > >
> > > diff --git a/rules/strongswan.make b/rules/strongswan.make
> > > index a64e67cc8..2bea6c839 100644
> > > --- a/rules/strongswan.make
> > > +++ b/rules/strongswan.make
> > > @@ -15,8 +15,8 @@ PACKAGES-$(PTXCONF_STRONGSWAN) += strongswan
> > >  #
> > >  # Paths and names
> > >  #
> > > -STRONGSWAN_VERSION   := 5.9.2
> > > -STRONGSWAN_MD5               := 8918e6675e1be3784817641f07eadeb8
> > > +STRONGSWAN_VERSION   := 5.9.6
> > > +STRONGSWAN_MD5               := 0eeb13eda09fb34e9ab5e2bfcfab1211
> > >  STRONGSWAN           := strongswan-$(STRONGSWAN_VERSION)
> > >  STRONGSWAN_SUFFIX    := tar.bz2
> > >  STRONGSWAN_URL               := https://download.strongswan.org/$(STRONGSWAN).$(STRONGSWAN_SUFFIX)
> > > @@ -221,6 +221,7 @@ STRONGSWAN_CONF_OPT       := \
> > >       --disable-log-thread-ids \
> > >       --disable-monolithic \
> > >       --disable-defaults \
> > > +     --enable-kdf \
> > >       --enable-dependency-tracking \
> > >       --enable-shared \
> > >       --$(call ptx/endis, PTXCONF_STRONGSWAN_SWANCTL)-swanctl \
> > > @@ -243,6 +244,7 @@ STRONGSWAN_PLUGINS := \
> > >       libstrongswan-gcm.so \
> > >       libstrongswan-gmp.so \
> > >       libstrongswan-hmac.so \
> > > +     libstrongswan-kdf.so \
> > >       libstrongswan-kernel-netlink.so \
> > >       libstrongswan-nonce.so \
> > >       libstrongswan-pem.so \
> > > --
> > > 2.36.1
> > >
> > >
> 



      parent reply	other threads:[~2022-06-16  7:53 UTC|newest]

Thread overview: 5+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2022-06-16  6:42 Lars Pedersen
2022-06-16  6:48 ` Alexander Dahl
2022-06-16  7:25   ` Lars Pedersen
2022-06-16  7:28     ` Christian Melki
2022-06-16  7:53     ` Alexander Dahl [this message]

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=Yqrhc39xY/y/ZkCq@ada.ifak-system.com \
    --to=ada@thorsis.com \
    --cc=lapeddk@gmail.com \
    --cc=ptxdist@pengutronix.de \
    /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