All of lore.kernel.org
 help / color / mirror / Atom feed
From: Alexander Kanavin <alex.kanavin@gmail.com>
To: Peter Kjellerstedt <peter.kjellerstedt@axis.com>
Cc: "openembedded-core@lists.openembedded.org"
	<openembedded-core@lists.openembedded.org>,
	Alexander Kanavin <alex@linutronix.de>,
	Hongxu Jia <hongxu.jia@windriver.com>,
	Richard Purdie <richard.purdie@linuxfoundation.org>
Subject: Re: [OE-core] [PATCH 07/25] openssl: update 1.1.1l -> 3.0.0
Date: Tue, 12 Oct 2021 13:02:37 +0200	[thread overview]
Message-ID: <CANNYZj-BxwBD34yC4W9yOQ-SSTLr0jrEDRZN1q-oKSsZL8ucxA@mail.gmail.com> (raw)
In-Reply-To: <8cecff22cdcf4aa89bac193213219677@axis.com>

[-- Attachment #1: Type: text/plain, Size: 1159 bytes --]

On Mon, 11 Oct 2021 at 20:07, Peter Kjellerstedt <
peter.kjellerstedt@axis.com> wrote:

> > -     mv ${D}${libdir}/ssl-1.1/certs \
> > -        ${D}${libdir}/ssl-1.1/private \
> > -        ${D}${libdir}/ssl-1.1/openssl.cnf \
> > +     mv ${D}${libdir}/ssl-3/certs \
> > +        ${D}${libdir}/ssl-3/private \
> > +        ${D}${libdir}/ssl-3/openssl.cnf \
>
> Given that "${libdir}/ssl-3" is all over the recipe, wouldn't it make
> sense to turn it into a variable? Then it will at least only be one
> place to update the next time they change the version.
>

This changes very rarely and adjusting it was a very minor part of doing
the update, essentially a five second copy-replace. Fixing up everything
that broke due to previously long-deprecated APIs becoming hard errors, and
making the ptests pass again was the bulk of it, and I think we're not
quite done yet as I only checked and fixed oe-core and meta-oe. I'd
appreciate if you test other layers.


> There is a mix of tab and space indentation above (both in the original
> code and in the added lines).
>
> > +
>
> Unnecessary empty line.
>

Can you send a followup patch for these please?

Alex

[-- Attachment #2: Type: text/html, Size: 1813 bytes --]

  reply	other threads:[~2021-10-12 11:02 UTC|newest]

Thread overview: 38+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2021-10-11  9:40 [PATCH 01/25] qemu: install qmp module without hardcoding the python version in oeqa scripts Alexander Kanavin
2021-10-11  9:40 ` [PATCH 02/25] lttng-tools: replace ad hoc ptest fixup with upstream fixes Alexander Kanavin
2021-10-11  9:40 ` [PATCH 03/25] rt-tests: update 2.1 -> 2.2 Alexander Kanavin
2021-10-11  9:40 ` [PATCH 04/25] libgit2: update 1.1.1 -> 1.2.0 Alexander Kanavin
2021-10-11  9:40 ` [PATCH 05/25] libssh2: update 1.9.0 -> 1.10.0 Alexander Kanavin
2021-10-11 16:06   ` Khem Raj
2021-10-11  9:40 ` [PATCH 06/25] python3: update 3.9.6 -> 3.10.0 Alexander Kanavin
2021-10-11  9:40 ` [PATCH 07/25] openssl: update 1.1.1l -> 3.0.0 Alexander Kanavin
2021-10-11 18:07   ` [OE-core] " Peter Kjellerstedt
2021-10-12 11:02     ` Alexander Kanavin [this message]
2021-10-11  9:40 ` [PATCH 08/25] cryptodev-tests: do not use -Werror with openssl 3 Alexander Kanavin
2021-10-11  9:40 ` [PATCH 09/25] serf: add a openssl 3 compatibility fix Alexander Kanavin
2021-10-11  9:40 ` [PATCH 10/25] ruby: disable openssl extension Alexander Kanavin
2021-10-11  9:40 ` [PATCH 11/25] glib-2.0: update 2.68.4 -> 2.70.0 Alexander Kanavin
2021-10-11  9:40 ` [PATCH 12/25] glib-networking: update 2.68.2 " Alexander Kanavin
2021-10-11  9:40 ` [PATCH 13/25] bison: update 3.7.6 -> 3.8.1 Alexander Kanavin
2021-10-11  9:40 ` [PATCH 14/25] libdnf: update 0.63.1 -> 0.64.0 Alexander Kanavin
2021-10-11  9:40 ` [PATCH 15/25] libexif: update 0.6.22 -> 0.6.23 Alexander Kanavin
2021-10-11  9:40 ` [PATCH 16/25] sudo: update 1.9.7p2 -> 1.9.8p1 Alexander Kanavin
2021-10-11  9:40 ` [PATCH 17/25] wget: update 1.21.1 -> 1.21.2 Alexander Kanavin
2021-10-11  9:40 ` [PATCH 18/25] coreutils: update 8.32 -> 9.0 Alexander Kanavin
2021-10-11  9:40 ` [PATCH 19/25] itstool: update 2.0.6 -> 2.0.7 Alexander Kanavin
2021-10-11  9:40 ` [PATCH 20/25] nghttp2: add recipe from meta-oe Alexander Kanavin
2021-10-11  9:40 ` [PATCH 21/25] libsoup: add a 3.x recipe Alexander Kanavin
2021-10-12 10:08   ` [OE-core] " Ross Burton
2021-10-12 10:45     ` Alexander Kanavin
2021-10-11  9:40 ` [PATCH 22/25] webkitgtk: trim down DEPENDS Alexander Kanavin
2021-10-11  9:40 ` [PATCH 23/25] epiphany: " Alexander Kanavin
2021-10-11  9:40 ` [PATCH 24/25] webkitgtk: update 2.32.3 -> 2.34.0 Alexander Kanavin
2021-10-11 16:08   ` Khem Raj
2021-10-26 19:38   ` [OE-core] " Martin Jansa
2021-10-26 20:07     ` Alexander Kanavin
2021-10-26 20:40       ` Martin Jansa
2021-10-26 21:35         ` Alexander Kanavin
2021-10-26 21:51           ` Martin Jansa
2021-10-26 21:59             ` Alexander Kanavin
     [not found]     ` <16B1AD7283CD4D45.19566@lists.openembedded.org>
2021-10-26 20:21       ` Alexander Kanavin
2021-10-11  9:40 ` [PATCH 25/25] epiphany: update 40.3 -> 41.0 Alexander Kanavin

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=CANNYZj-BxwBD34yC4W9yOQ-SSTLr0jrEDRZN1q-oKSsZL8ucxA@mail.gmail.com \
    --to=alex.kanavin@gmail.com \
    --cc=alex@linutronix.de \
    --cc=hongxu.jia@windriver.com \
    --cc=openembedded-core@lists.openembedded.org \
    --cc=peter.kjellerstedt@axis.com \
    --cc=richard.purdie@linuxfoundation.org \
    /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 an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.