unofficial mirror of guix-devel@gnu.org 
 help / color / mirror / code / Atom feed
From: jbranso--- via Guix-patches via <guix-patches@gnu.org>
To: "Skyler Ferris" <skyvine@protonmail.com>,
	guix-devel@gnu.org, peter@polidoro.io, 55728@debbugs.gnu.org
Subject: [bug#55728] Handling expensive packages
Date: Tue, 12 Mar 2024 19:59:04 +0000	[thread overview]
Message-ID: <4d42f1c112461bd6bb81420b9794c027d1f2e139@dismail.de> (raw)
In-Reply-To: <c6b78b38-8999-4821-81ab-b32ebd704c3c@protonmail.com>

Hey Peter!

Your patch found its way on onto the guix devel list!

Thanks for contributing to guix!  We want to add your patch, and we are wondering what your motivation was for sending this patch?  Is your patch a prerequisite for a package you want to add?  Or do you have some other reason?

Thanks,

Joshua
https://gnucode.me

March 11, 2024 at 3:06 PM, "Skyler Ferris" <skyvine@protonmail.com> wrote:



> 
> Hello,
>  
>  I am looking through the backlog of open patch submissions https://issues.guix.gnu.org/search?query=is%3Aopen+tag%3Apatch  to see if any are actionable on my end. One such patch is issue 55728 which updates python-mock https://issues.guix.gnu.org/55728 . Based on the output of `guix refresh --list-dependent python-mock | wc`, this will impact more than 2000 packages. While this submission is very old, neither the master nor python-team branches have updated this package yet. In section 22.8.2 "Managing Patches and Branches" https://guix.gnu.org/en/manual/devel/en/html_node/Managing-Patches-and-Branches.html , there is a recommendation that changes which effect more than 300 dependents are added to a different branch for testing.
>  
>  These dependents presumably still work, as there are not 2000 build failures or a flood of related bug reports. So I think it would make sense to first ask the submitter for their motivation for sending the patch (for example, it might be a prerequisite for a package they want to add and they did not send it as a series for some reason). Depending on their response it might make sense to do something other than apply the update as given (for example, by providing both versions of the package so that a new package can be added without impacting existing branches). But there also might be some reason why it makes sense to apply the update everywhere (for example, if significant optimizations in the update reduces build times for all of the dependent packages).
>  
>  So my main question is whether or not people agree that it makes sense to ask the submitter for more information and take no other action at this time. And as a secondary question, if it does make sense to update the package everywhere is there anything actionable on my end?
>  
>  Regards,
>  Skyler
>




  reply	other threads:[~2024-03-12 19:59 UTC|newest]

Thread overview: 5+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2024-03-11 19:06 Handling expensive packages Skyler Ferris
2024-03-12 19:59 ` jbranso--- via Guix-patches via [this message]
2024-03-12 20:45   ` [bug#55728] " Peter Polidoro
2024-03-13  0:15     ` Skyler Ferris
2024-03-13  0:15       ` [bug#55728] " Skyler Ferris via Guix-patches via

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=4d42f1c112461bd6bb81420b9794c027d1f2e139@dismail.de \
    --to=guix-patches@gnu.org \
    --cc=55728@debbugs.gnu.org \
    --cc=guix-devel@gnu.org \
    --cc=jbranso@dismail.de \
    --cc=peter@polidoro.io \
    --cc=skyvine@protonmail.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 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).