all messages for Guix-related lists mirrored at yhetil.org
 help / color / mirror / code / Atom feed
* Handling expensive packages
@ 2024-03-11 19:06 Skyler Ferris
  2024-03-12 19:59 ` [bug#55728] " jbranso--- via Guix-patches via
  0 siblings, 1 reply; 5+ messages in thread
From: Skyler Ferris @ 2024-03-11 19:06 UTC (permalink / raw)
  To: guix-devel

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

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

[-- Attachment #2: Type: text/html, Size: 2230 bytes --]

^ permalink raw reply	[flat|nested] 5+ messages in thread

* [bug#55728] Handling expensive packages
  2024-03-11 19:06 Handling expensive packages Skyler Ferris
@ 2024-03-12 19:59 ` jbranso--- via Guix-patches via
  2024-03-12 20:45   ` Peter Polidoro
  0 siblings, 1 reply; 5+ messages in thread
From: jbranso--- via Guix-patches via @ 2024-03-12 19:59 UTC (permalink / raw)
  To: Skyler Ferris, guix-devel, peter, 55728

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
>




^ permalink raw reply	[flat|nested] 5+ messages in thread

* [bug#55728] Handling expensive packages
  2024-03-12 19:59 ` [bug#55728] " jbranso--- via Guix-patches via
@ 2024-03-12 20:45   ` Peter Polidoro
  2024-03-13  0:15       ` [bug#55728] " Skyler Ferris via Guix-patches via
  0 siblings, 1 reply; 5+ messages in thread
From: Peter Polidoro @ 2024-03-12 20:45 UTC (permalink / raw)
  To: jbranso; +Cc: guix-devel, 55728, Skyler Ferris

You can ignore or delete this patch if it is causing problems.

If I remember correctly, that was a patch I submitted a couple of years ago when I was attempting to package some embedded software tools, either the zephyr west tool or platformio, both written in python. That sent me down the rabbit hole of dependency updates, the python-mock package being one of them.

I would still love to have both west and platformio packaged in guix. Perhaps those exist now in Guix or another channel, I have not checked in a while. If not, I will attempt to package them again at some point. 

Thanks!

> On Mar 12, 2024, at 15:59, jbranso@dismail.de wrote:
> 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




^ permalink raw reply	[flat|nested] 5+ messages in thread

* Re: Handling expensive packages
@ 2024-03-13  0:15       ` Skyler Ferris via Guix-patches via
  0 siblings, 0 replies; 5+ messages in thread
From: Skyler Ferris @ 2024-03-13  0:15 UTC (permalink / raw)
  To: Peter Polidoro, jbranso; +Cc: guix-devel, 55728

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

On 3/12/24 13:45, Peter Polidoro wrote:

> If I remember correctly, that was a patch I submitted a couple of years ago when I was attempting to package some embedded software tools, either the zephyr west tool or platformio, both written in python. That sent me down the rabbit hole of dependency updates, the python-mock package being one of them.
>
> I would still love to have both west and platformio packaged in guix. Perhaps those exist now in Guix or another channel, I have not checked in a while. If not, I will attempt to package them again at some point.

Thank you for the information. I see that west is packaged in gnu/packages/embedded.scm, but the only reference to platformio is an emacs plugin which I suspect is not what you are referring to. So it sounds like there is no particular reason to process this patch immediately but it makes sense to leave open because it might be used down the line.

[-- Attachment #2: Type: text/html, Size: 1300 bytes --]

^ permalink raw reply	[flat|nested] 5+ messages in thread

* [bug#55728] Handling expensive packages
@ 2024-03-13  0:15       ` Skyler Ferris via Guix-patches via
  0 siblings, 0 replies; 5+ messages in thread
From: Skyler Ferris via Guix-patches via @ 2024-03-13  0:15 UTC (permalink / raw)
  To: Peter Polidoro, jbranso; +Cc: guix-devel, 55728

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

On 3/12/24 13:45, Peter Polidoro wrote:

> If I remember correctly, that was a patch I submitted a couple of years ago when I was attempting to package some embedded software tools, either the zephyr west tool or platformio, both written in python. That sent me down the rabbit hole of dependency updates, the python-mock package being one of them.
>
> I would still love to have both west and platformio packaged in guix. Perhaps those exist now in Guix or another channel, I have not checked in a while. If not, I will attempt to package them again at some point.

Thank you for the information. I see that west is packaged in gnu/packages/embedded.scm, but the only reference to platformio is an emacs plugin which I suspect is not what you are referring to. So it sounds like there is no particular reason to process this patch immediately but it makes sense to leave open because it might be used down the line.

[-- Attachment #2: Type: text/html, Size: 1300 bytes --]

^ permalink raw reply	[flat|nested] 5+ messages in thread

end of thread, other threads:[~2024-03-13  0:16 UTC | newest]

Thread overview: 5+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2024-03-11 19:06 Handling expensive packages Skyler Ferris
2024-03-12 19:59 ` [bug#55728] " jbranso--- via Guix-patches via
2024-03-12 20:45   ` Peter Polidoro
2024-03-13  0:15     ` Skyler Ferris
2024-03-13  0:15       ` [bug#55728] " Skyler Ferris via Guix-patches via

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.