GNU bug report logs - #65126
[PATCH] gnu: exiv2: Get rid of reference to GCC.

Previous Next

To add a comment to this bug, you must first unarchive it, by sending
a message to control AT debbugs.gnu.org, with unarchive 65126 in the body.
You can then email your comments to 65126 AT debbugs.gnu.org in the normal way.

Toggle the display of automated, internal messages from the tracker.

Message #5 received at submit <at> debbugs.gnu.org (full text, mbox):

From: iyzsong <at> envs.net
To: guix-patches <at> gnu.org
Cc: 宋文武 <iyzsong <at> member.fsf.org>
Subject: [PATCH] gnu: exiv2: Get rid of reference to GCC.
Date: Mon,  7 Aug 2023 22:03:03 +0800

From: 宋文武 <iyzsong <at> member.fsf.org>

This fixes <https://issues.guix.gnu.org/57677>.

* gnu/packages/image.scm (exiv2)[arguments]: Add phase to
remove _GLIBCXX_ASSERTIONS from compiler flags.
---
 gnu/packages/image.scm | 6 ++++++
 1 file changed, 6 insertions(+)

diff --git a/gnu/packages/image.scm b/gnu/packages/image.scm
index 3336e38852..1dbf3cbdd6 100644
--- a/gnu/packages/image.scm
+++ b/gnu/packages/image.scm
@@ -1511,6 +1511,12 @@ (define-public exiv2
                                "-DEXIV2_ENABLE_BMFF=ON")
        #:phases
        (modify-phases %standard-phases
+         (add-after 'unpack 'patch-gcc-reference
+           (lambda _
+             ;; _GLIBCXX_ASSERTIONS brings reference to GCC.
+             (substitute* "cmake/compilerFlags.cmake"
+               (("add_compile_options[(]-Wp,-D_GLIBCXX_ASSERTIONS[)]")
+                ""))))
          (add-after 'install 'delete-static-libraries
            (lambda* (#:key outputs #:allow-other-keys)
              (let* ((out (assoc-ref outputs "out"))

base-commit: 56667ee55cd7f3368cbff169352fe440f4f93da5
-- 
2.41.0

Message #8 received at 65126 <at> debbugs.gnu.org (full text, mbox):

From: Ludovic Courtès <ludo <at> gnu.org>
To: iyzsong <at> envs.net
Cc: 宋文武 <iyzsong <at> member.fsf.org>, 65126 <at> debbugs.gnu.org
Subject: Re: bug#65126: [PATCH] gnu: exiv2: Get rid of reference to GCC.
Date: Wed, 16 Aug 2023 20:07:50 +0200

Hi!

iyzsong <at> envs.net skribis:

> From: 宋文武 <iyzsong <at> member.fsf.org>
>
> This fixes <https://issues.guix.gnu.org/57677>.

s/This fixes/Fixes/, for consistency.

> +         (add-after 'unpack 'patch-gcc-reference
> +           (lambda _
> +             ;; _GLIBCXX_ASSERTIONS brings reference to GCC.
> +             (substitute* "cmake/compilerFlags.cmake"
> +               (("add_compile_options[(]-Wp,-D_GLIBCXX_ASSERTIONS[)]")
> +                ""))))

Well done!  According to
<https://gcc.gnu.org/onlinedocs/libstdc++/manual/using_macros.html>,
‘_GLIBCXX_ASSERTIONS’ enables various run-time checks; I guess disabling
it is acceptable.

So… LGTM!

Thanks,
Ludo’.

Message #11 received at 65126 <at> debbugs.gnu.org (full text, mbox):

From: Ludovic Courtès <ludo <at> gnu.org>
To: iyzsong <at> envs.net
Cc: 宋文武 <iyzsong <at> member.fsf.org>, 65126 <at> debbugs.gnu.org
Subject: Re: bug#65126: [PATCH] gnu: exiv2: Get rid of reference to GCC.
Date: Wed, 16 Aug 2023 20:08:53 +0200

Ludovic Courtès <ludo <at> gnu.org> skribis:

> So… LGTM!

Well, wait for <https://qa.guix.gnu.org/issue/65126> to be done.

Ludo’.

Message #14 received at 65126 <at> debbugs.gnu.org (full text, mbox):

From: 宋文武 <iyzsong <at> envs.net>
To: Ludovic Courtès <ludo <at> gnu.org>
Cc: 宋文武 <iyzsong <at> member.fsf.org>, 65126 <at> debbugs.gnu.org,
 Christopher Baines <mail <at> cbaines.net>
Subject: Re: bug#65126: [PATCH] gnu: exiv2: Get rid of reference to GCC.
Date: Sun, 20 Aug 2023 16:01:20 +0800

Ludovic Courtès <ludo <at> gnu.org> writes:

> Well, wait for <https://qa.guix.gnu.org/issue/65126> to be done.

Hello, I think CI is stucking here.

On https://qa.guix.gnu.org/patches we get:
  Builds for new patch series suspended as master branch substitute
  availability is low for: i686-linux
In this case should I first fix broken builds for i686-linux?

Also will CI able to automatically rebase the patch upon latest master?
If the patch was built with old base, it won't have substitutes after
merge even it's green.

Message #17 received at 65126 <at> debbugs.gnu.org (full text, mbox):

From: Christopher Baines <mail <at> cbaines.net>
To: 宋文武 <iyzsong <at> envs.net>
Cc: Ludovic Courtès <ludo <at> gnu.org>, 65126 <at> debbugs.gnu.org
Subject: Re: bug#65126: [PATCH] gnu: exiv2: Get rid of reference to GCC.
Date: Sun, 20 Aug 2023 19:27:57 +0100

[Message part 1 (text/plain, inline)]

宋文武 <iyzsong <at> envs.net> writes:

> Ludovic Courtès <ludo <at> gnu.org> writes:
>
>> Well, wait for <https://qa.guix.gnu.org/issue/65126> to be done.
>
> Hello, I think CI is stucking here.

I'm going to answer replacing CI with qa.guix.gnu.org, as I think that
makes the questions make more sense.

> On https://qa.guix.gnu.org/patches we get:
>   Builds for new patch series suspended as master branch substitute
>   availability is low for: i686-linux
> In this case should I first fix broken builds for i686-linux?

I think substitute availability is below 80% for i686-linux because the
bordeaux build farm is still catching up. There's a shepherd issue on
milano-guix-1, so that's reduced the resources available for
x86_64-linux and i686-linux builds.

QA waits until substitute availability is good before submitting builds
as that avoids submitting builds when they probably won't happen for a
while, given the bordeaux build farm will be busy with the master branch
builds instead.

Ideally more changes would be processed by QA and built fully before
being merged, so that substitutes are available when they're merged.

> Also will CI able to automatically rebase the patch upon latest master?
> If the patch was built with old base, it won't have substitutes after
> merge even it's green.

The qa-frontpage does automatically rebase patches, although there's
probably a lot of room for improvement to better detect when a patch
series could do with rebasing.

[signature.asc (application/pgp-signature, inline)]

Message #20 received at 65126 <at> debbugs.gnu.org (full text, mbox):

From: Ludovic Courtès <ludo <at> gnu.org>
To: iyzsong <at> envs.net
Cc: 宋文武 <iyzsong <at> member.fsf.org>, 65126 <at> debbugs.gnu.org
Subject: Re: [bug#65126] [PATCH] gnu: exiv2: Get rid of reference to GCC.
Date: Thu, 19 Oct 2023 21:58:26 +0200

Hi 宋文武,

Ludovic Courtès <ludo <at> gnu.org> skribis:

> Ludovic Courtès <ludo <at> gnu.org> skribis:
>
>> So… LGTM!
>
> Well, wait for <https://qa.guix.gnu.org/issue/65126> to be done.

Looks like it’s all green now, go ahead! :-)

Ludo’.

Message #25 received at 65126-done <at> debbugs.gnu.org (full text, mbox):

From: 宋文武 <iyzsong <at> envs.net>
To: Ludovic Courtès <ludo <at> gnu.org>
Cc: 宋文武 <iyzsong <at> member.fsf.org>,
 65126-done <at> debbugs.gnu.org
Subject: Re: [bug#65126] [PATCH] gnu: exiv2: Get rid of reference to GCC.
Date: Fri, 20 Oct 2023 17:59:15 +0800

Ludovic Courtès <ludo <at> gnu.org> writes:

> Hi 宋文武,
>
> Ludovic Courtès <ludo <at> gnu.org> skribis:
>
>> Ludovic Courtès <ludo <at> gnu.org> skribis:
>>
>>> So… LGTM!
>>
>> Well, wait for <https://qa.guix.gnu.org/issue/65126> to be done.
>
> Looks like it’s all green now, go ahead! :-)

Pushed now, thanks!

Previous Next

GNU bug tracking system
Copyright (C) 1999 Darren O. Benham, 1997,2003 nCipher Corporation Ltd, 1994-97 Ian Jackson.