unofficial mirror of guix-patches@gnu.org 
 help / color / mirror / code / Atom feed
From: zamfofex <zamfofex@twdb.moe>
To: "Ludovic Courtès" <ludo@gnu.org>
Cc: Maxime Devos <maximedevos@telenet.be>, 54832@debbugs.gnu.org
Subject: [bug#54832] [patch] update glibc to 2.35
Date: Mon, 6 Jun 2022 09:06:27 -0300 (BRT)	[thread overview]
Message-ID: <818784149.855838.1654517187059@privateemail.com> (raw)
In-Reply-To: <87r14ubhyu.fsf@gnu.org>

> Could you confirm that these patches are no longer needed?

I don’t remember exactly what my thought process was for removing the ‘glibc-dl-cache’ patch except that it wasn’t applicable anymore. At any rate, I don’t fully understand what the patch is actually doing, so it’s a bit difficult to assess whether it’s still necessary to me. (Help would be appreciated!)

Other than that, I did verify the other two patches, and it seems the regexes they were patching have already been fixed upstream!

> Could you explain why the empty .a files need special treatment?  In the end, it seems we’re copying them to the “static” output anyway, so why not let them in ‘files’?

Those files need to be present in both the ‘static’ and ‘out’ outputs, whereas without considering them specially, they would be moved to the ‘static’ output (with ‘rename-file’ as opposed to ‘copy-file’). Is a comment worthwhile? What should I write? I could explain the change in glibc that made those into empty ‘.a’ files and link to the changelog. Is that enough?

> This should be done in a separate patch.

That is fine. Though I will note that the previous version of m4 did not work with the updated glibc, so I think it would make sense to updated it *before* updating glibc in the commit history. Do I need to verify whether the newer version works with the previous glibc too?

> Like Maxime wrote, please start the patch with a short comment explaining what it does, and with a link to the upstream commit or bug report.

I’m still confused about what I should link to. I can write a comment explaining the issues and link to the IRC conversation we held, or maybe even to this thread. But I don’t think there actually is “an upstream commit or bug report” that I could link to.

> One last thing: could you use ‘git format-patch’ and (optionally) ‘git send-email’ to send a revised patch?

I definitely don’t mind investigating using those tools more carefully! I think I can prepare and send another patch once my questions are clarified.

- - -

Apologies to Maxime and Ludovic for the repeated messages once again and to others for taking so long to respond. I’m still getting familiar with sending emails properly!

At any rate, thank you for looking into it!




  reply	other threads:[~2022-06-06 13:25 UTC|newest]

Thread overview: 11+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2022-04-10  1:16 [bug#54832] [patch] update glibc to 2.35 zamfofex
2022-04-10 10:16 ` Maxime Devos
     [not found]   ` <1497835452.18920.1649591936889@privateemail.com>
2022-04-24 14:19     ` [bug#54832] Fwd: " zamfofex
2022-04-10 10:17 ` Maxime Devos
2022-04-10 10:18 ` Maxime Devos
2022-05-15 21:31 ` Ludovic Courtès
2022-06-06 12:06   ` zamfofex [this message]
2022-07-17 10:06     ` Ludovic Courtès
2022-07-19 14:12       ` zamfofex
2022-07-19 19:16         ` Greg Hogan
2022-09-01 22:32           ` bug#54832: " Marius Bakke

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

  List information: https://guix.gnu.org/

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=818784149.855838.1654517187059@privateemail.com \
    --to=zamfofex@twdb.moe \
    --cc=54832@debbugs.gnu.org \
    --cc=ludo@gnu.org \
    --cc=maximedevos@telenet.be \
    /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 public inbox

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

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for read-only IMAP folder(s) and NNTP newsgroup(s).