All of lore.kernel.org
 help / color / mirror / Atom feed
From: "Ævar Arnfjörð Bjarmason" <avarab@gmail.com>
To: Philippe Blain <levraiphilippeblain@gmail.com>
Cc: git@vger.kernel.org,
	"Carlo Marcelo Arenas Belón" <carenas@gmail.com>,
	"Ed Maste" <emaste@FreeBSD.org>
Subject: Re: [PATCH] ci: update Cirrus-CI image to FreeBSD 13.0
Date: Tue, 24 May 2022 21:24:14 +0200	[thread overview]
Message-ID: <220524.86czg2wx4a.gmgdl@evledraar.gmail.com> (raw)
In-Reply-To: <20220524165823.18804-1-levraiphilippeblain@gmail.com>


On Tue, May 24 2022, Philippe Blain wrote:

> The FreeBSD CI build (on Cirrus-CI) has been failing in
> 't9001-send-email.sh' for quite some time, with an error from the
> runtime linker relating to the Perl installation:
>
>     ld-elf.so.1: /usr/local/lib/perl5/5.32/mach/CORE/libperl.so.5.32: Undefined symbol "strerror_l@FBSD_1.6"
>
> The first instance is in t9001.6 but it fails similarly in several tests
> in this file.
>
> The FreeBSD image we use is FreeBSD 12.2, which is unsupported since
> March 31st, 2022 [1]. Switching to a supported version, 13.0,
> makes this error disappear [2].
>
> Change the image we use to FreeBSD 13.0.
>
> [1] https://www.freebsd.org/security/unsupported/
> [2] https://lore.kernel.org/git/9cc31276-ab78-fa8a-9fb4-b19266911211@gmail.com/
>
> Signed-off-by: Philippe Blain <levraiphilippeblain@gmail.com>
> ---
>
>     Here is a proper patch following my earlier mail [1].
>     
>     [1] https://lore.kernel.org/git/CAPUEspgdAos4KC-3AwYDd5p+u0hGk73nGocBTFFSR7VB9+M5jw@mail.gmail.com/T/#t
>
>  .cirrus.yml | 2 +-
>  1 file changed, 1 insertion(+), 1 deletion(-)
>
> diff --git a/.cirrus.yml b/.cirrus.yml
> index e114ffee1a..20c85eef81 100644
> --- a/.cirrus.yml
> +++ b/.cirrus.yml
> @@ -9,7 +9,7 @@ freebsd_12_task:
>      DEFAULT_TEST_TARGET: prove
>      DEVELOPER: 1
>    freebsd_instance:
> -    image_family: freebsd-12-2
> +    image_family: freebsd-13-0
>      memory: 2G
>    install_script:
>      pkg install -y gettext gmake perl5
>
> base-commit: 7a3eb286977746bc09a5de7682df0e5a7085e17c

This may or may not be a good idea, I've got no about about this CI
setup & what FreeBSD version we'd prefer.

But that you're seeing this in t9001-send-email.sh in particular
suggests that perl isn't broken in general, as we hard depend on it in a
bunch of places in the test suite.

Rather, it's some more advanced Perl usage.

So aside from this switch perhaps we'd like to have a more specific
prereq for those send-email tests than just "PERL", which is controlled
by our "NO_PERL" build-time setting, i.e. to actually probe if Perl
works.

The error you quote doesn't tell us anything about where it happened,
which aside from anything else would be useful to have in the commit
message. I.e. what command did we run when this failed, and did other
perl commands before that (either in that test, or others) work?

  parent reply	other threads:[~2022-05-24 19:26 UTC|newest]

Thread overview: 9+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2022-05-24 16:58 [PATCH] ci: update Cirrus-CI image to FreeBSD 13.0 Philippe Blain
2022-05-24 19:20 ` Carlo Marcelo Arenas Belón
2022-05-24 20:52   ` Philippe Blain
2022-05-25  4:35     ` Carlo Arenas
2022-05-24 19:24 ` Ævar Arnfjörð Bjarmason [this message]
2022-05-24 20:56   ` Philippe Blain
2022-05-25 10:37 ` Johannes Schindelin
2022-05-25 12:51 ` [PATCH v2] ci: update Cirrus-CI image to FreeBSD 12.3 Philippe Blain
2022-05-25 15:50   ` Junio C Hamano

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=220524.86czg2wx4a.gmgdl@evledraar.gmail.com \
    --to=avarab@gmail.com \
    --cc=carenas@gmail.com \
    --cc=emaste@FreeBSD.org \
    --cc=git@vger.kernel.org \
    --cc=levraiphilippeblain@gmail.com \
    /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.