All of lore.kernel.org
 help / color / mirror / Atom feed
From: Robbie Harwood <rharwood@redhat.com>
To: Daniel Kiper <dkiper@net-space.pl>
Cc: grub-devel@gnu.org, development@efficientek.com
Subject: Re: [PATCH v5 0/4] Update gnulib and drop some patches
Date: Thu, 24 Feb 2022 14:43:21 -0500	[thread overview]
Message-ID: <jlgv8x4cap2.fsf@redhat.com> (raw)
In-Reply-To: <20220224172844.lgb623oa7vnfeu4y@tomti.i.net-space.pl>

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

Daniel Kiper <dkiper@net-space.pl> writes:

> Robbie Harwood wrote:
>
>> Also, gnulib's position on sign comparison issues is to ignore them,
>> so the "fix" from our end would probably be to pragma them
>
> Huh! Where did you see that? Could you point me a source where they
> say that thing?

"If -Wsign-compare complains, let's disable -Wsign-compare instead of
changing carefully-written signed integers to be unsigned."

from subthread starting here:
https://lists.gnu.org/archive/html/bug-gnulib/2021-12/msg00017.html
on a patch I proposed to fix a bunch of their sign comparison issues
(originally written by pjones).

>> away... but that assumes this is the only issue, which hasn't been
>> checked yet.
>
> I am not sure what you mean here.

I mean that there may be other issues lurking when building with that
gcc version, which there were, but they're fixed in the v6 I sent.

Be well,
--Robbie

[-- Attachment #2: signature.asc --]
[-- Type: application/pgp-signature, Size: 861 bytes --]

      reply	other threads:[~2022-02-24 19:43 UTC|newest]

Thread overview: 12+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2022-02-16 21:41 [PATCH v5 0/4] Update gnulib and drop some patches Robbie Harwood
2022-02-16 21:41 ` [PATCH v5 1/4] Use visual indentation in config.h.in Robbie Harwood
2022-02-16 21:41 ` [PATCH v5 2/4] Drop gnulib fix-base64.patch Robbie Harwood
2022-02-16 21:41 ` [PATCH v5 3/4] Drop gnulib no-abort.patch Robbie Harwood
2022-02-16 21:41 ` [PATCH v5 4/4] Update gnulib version and drop most gnulib patches Robbie Harwood
2022-02-18 16:03 ` [PATCH v5 0/4] Update gnulib and drop some patches Glenn Washburn
2022-02-22 15:21 ` Daniel Kiper
2022-02-22 16:44   ` Robbie Harwood
2022-02-22 17:17     ` Daniel Kiper
2022-02-22 17:35       ` Robbie Harwood
2022-02-24 17:28         ` Daniel Kiper
2022-02-24 19:43           ` Robbie Harwood [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=jlgv8x4cap2.fsf@redhat.com \
    --to=rharwood@redhat.com \
    --cc=development@efficientek.com \
    --cc=dkiper@net-space.pl \
    --cc=grub-devel@gnu.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.