all messages for Guix-related lists mirrored at yhetil.org
 help / color / mirror / code / Atom feed
From: "André Batista" <nandre@riseup.net>
To: Maxim Cournoyer <maxim.cournoyer@gmail.com>
Cc: 61246@debbugs.gnu.org, zimon.toutoune@gmail.com
Subject: [bug#61246] [PATCH] gnu: libgit2: Update to 1.5.1.
Date: Fri, 10 Feb 2023 14:53:32 -0300	[thread overview]
Message-ID: <Y+aEnKdsIjjycJ4h@andel> (raw)
In-Reply-To: <877cwrjz9m.fsf_-_@gmail.com>

Hi Maxim and Zimoun!

qua 08 fev 2023 às 22:25:41 (1675905941), maxim.cournoyer@gmail.com enviou:
> Hello André,
>
> (...)
>
> We do not use Github/Gitlab/etc. auto-generated tarballs has there is no
> guarantee that they'll be bit-for-bit identical upon regeneration and
> this led to broken checksums in the past.  Could you please use the
> tag/commit instead?

Oh I did not know about that. I could use the git tag instead, but after
yours and zimoun's messages I've noticed that:

- I had only check dependencies using libgit2 without any @version numbers
and so I thought that there were fewer dependencies than there are in fact;

- no other package seems to be depending on libgit2-1.1, so it should be
remove instead of updated, as zimoun suggested;

- this patch should probably be on core-updates, not on master.

So I'm inclined to agree with zimoun and just remove libgit2-1.1, what do
you think? I'll work on another version of this patch and get back to
you.

> I noticed about this problem looking at the QA page here:
> https://qa.guix.gnu.org/issue/61246.  I'm not sure what are the other
> problems reported, they look like false positives to me.

I did not know about this QA service, thanks for pointing that out. I
couldn't figure out the other warnings as well, but I only had a quick
look at them.

> Did you rebuild the dependent packages, used that with Guix without
> problems so far?  The QA has yet to answer that question.

I did not rebuild all dependent packages yet, I've rebuild only my local
dependencies on two machines (i686 and x86_64).

Yesterday, however, I've bumped on a build error on libjami that could
be related to this patch, even though it does not seem to be so at a
first glance. The build error occurs at check phase with the following
error:

--- ./test/unitTest/test-suite.log ------------------------------------------

======================================================
   Jami Daemon 13.7.0: test/unitTest/test-suite.log
======================================================

# TOTAL: 15
# PASS:  14
# SKIP:  0
# XFAIL: 0
# FAIL:  1
# XPASS: 0
# ERROR: 0

.. contents:: :depth: 2

FAIL: ut_scheduler
==================

.F


!!!FAILURES!!!
Test Results:
Run:  1   Failures: 1   Errors: 0


1) test: jami::test::SchedulerTest::schedulerTest (F) line: 74 scheduler.cpp
assertion failed
- Expression: cv.wait_for(lk, std::chrono::seconds(3), [&]{ return taskRun == N; })


FAIL ut_scheduler (exit status: 1)


error: in phase 'check': uncaught exception:
%exception #<&invoke-error program: "make" arguments: ("check" "-j" "2" "V=1") exit-status: 2 term-signal: #f stop-signal: #f>
phase `check' failed after 571.8 seconds
command "make" "check" "-j" "2" "V=1" failed with status 2

---

Thanks for helping me out!




  parent reply	other threads:[~2023-02-10 17:55 UTC|newest]

Thread overview: 30+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2023-02-03  3:11 [bug#61246] [PATCH] gnu: libgit2: Update to 1.5.1 André Batista
2023-02-08 15:31 ` Simon Tournier
2023-02-17 19:05   ` [bug#61246] [PATCH v3 0/3] " André Batista
2023-02-17 19:06   ` [bug#61246] [PATCH v3 1/3] gnu: libgit2-1.1: Remove it André Batista
2023-02-17 19:06   ` [bug#61246] [PATCH v3 2/3] doc: Explain how to use local guix repositories André Batista
2023-02-17 19:58     ` Maxim Cournoyer
2023-02-17 23:24       ` André Batista
2023-02-18 17:35         ` Maxim Cournoyer
2023-02-22 18:10           ` André Batista
2023-02-23 21:59             ` bug#61246: " Maxim Cournoyer
2023-02-23 12:43           ` [bug#61246] " André Batista
2023-02-23 22:38             ` Maxim Cournoyer
2023-02-17 20:45     ` Maxim Cournoyer
2023-02-17 23:31       ` André Batista
2023-02-18 17:43         ` Maxim Cournoyer
2023-02-17 19:07   ` [bug#61246] [PATCH v3 3/3] gnu: libgit2: Update to 1.5.1 André Batista
2023-02-08 15:43 ` [bug#61246] [PATCH v2] " André Batista
2023-02-09  3:25   ` [bug#61246] [PATCH] " Maxim Cournoyer
2023-02-09 12:30     ` zimoun
2023-02-10 17:53     ` André Batista [this message]
2023-02-17 19:15       ` André Batista
2023-02-17 19:23         ` André Batista
2023-02-17 19:46         ` Maxim Cournoyer
2023-02-17 23:33           ` André Batista
2023-02-18 18:04           ` Tobias Geerinckx-Rice via Guix-patches via
2023-02-18 18:13             ` Tobias Geerinckx-Rice via Guix-patches via
2023-02-18 20:45               ` Maxim Cournoyer
2023-02-20 10:08                 ` Simon Tournier
2023-02-20 10:05               ` Simon Tournier
2023-02-22 18:17               ` André Batista

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=Y+aEnKdsIjjycJ4h@andel \
    --to=nandre@riseup.net \
    --cc=61246@debbugs.gnu.org \
    --cc=maxim.cournoyer@gmail.com \
    --cc=zimon.toutoune@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.
Code repositories for project(s) associated with this external index

	https://git.savannah.gnu.org/cgit/guix.git

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.