unofficial mirror of guix-devel@gnu.org 
 help / color / mirror / code / Atom feed
* python-dbus-python changes triggered many rebuilds
@ 2024-11-02  0:20 Vagrant Cascadian
  2024-11-02  2:13 ` Ian Eure
  2024-11-02  8:16 ` Sharlatan Hellseher
  0 siblings, 2 replies; 4+ messages in thread
From: Vagrant Cascadian @ 2024-11-02  0:20 UTC (permalink / raw)
  To: guix-devel; +Cc: Sharlatan Hellseher

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

A large rebuild was triggered by:

commit a9abf9a7b30f6801e122cae759df87b44c458773
Author: Sharlatan Hellseher <sharlatanus@gmail.com>
Date:   Fri Nov 1 21:10:04 2024 +0000

    gnu: python-dbus-python: Fix indentation.

    * gnu/packages/python-xyz.scm (python-dbus-python): Fix indentation,
    adjust order of fields, sort inputs alphabetically.

    Change-Id: I895518f041bd2cfc9c2f94774a9d1db47b26ffc3

Guix refresh claims this would trigger 3987 builds on x86_64-linux, and
ci is cranking away at over 13000 builds across several architectures:

   https://ci.guix.gnu.org/eval/1772855

Anyone able to cancel that evaluation?


I pushed a commit reverting the ordering changes, which I think appears
to not trigger the rebuild:

commit ea11d3608566174c4bae70faa4f9d0c67748d2db
Author: Vagrant Cascadian <vagrant@debian.org>
Date:   Fri Nov 1 16:55:02 2024 -0700

    gnu: python-dbus-python: Revert ordering change on native-inputs.

    A large number of rebuilds (3987 according to guix refresh) was triggered by:

      a9abf9a7b30f6801e122cae759df87b44c458773 gnu: python-dbus-python: Fix
      indentation.

    Reverting the ordering changes does not trigger any rebuilds.

    * gnu/packages/python-xyz.scm (python-dbus-python): Unsort native-inputs.


Hopefully that was the right thing to do!


live well,
  vagrant

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

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

* Re: python-dbus-python changes triggered many rebuilds
  2024-11-02  0:20 python-dbus-python changes triggered many rebuilds Vagrant Cascadian
@ 2024-11-02  2:13 ` Ian Eure
  2024-11-02  9:15   ` Ricardo Wurmus
  2024-11-02  8:16 ` Sharlatan Hellseher
  1 sibling, 1 reply; 4+ messages in thread
From: Ian Eure @ 2024-11-02  2:13 UTC (permalink / raw)
  To: guix-devel

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

Would it make sense to sort package inputs when computing derivations to prevent this sort of unintentional change?  I don't think the input order is important for the build, so this seems like it could be relatively simple to implement & avoid this recurring.

On November 1, 2024 5:20:51 PM PDT, Vagrant Cascadian <vagrant@debian.org> wrote:
>A large rebuild was triggered by:
>
>commit a9abf9a7b30f6801e122cae759df87b44c458773
>Author: Sharlatan Hellseher <sharlatanus@gmail.com>
>Date:   Fri Nov 1 21:10:04 2024 +0000
>
>    gnu: python-dbus-python: Fix indentation.
>
>    * gnu/packages/python-xyz.scm (python-dbus-python): Fix indentation,
>    adjust order of fields, sort inputs alphabetically.
>
>    Change-Id: I895518f041bd2cfc9c2f94774a9d1db47b26ffc3
>
>Guix refresh claims this would trigger 3987 builds on x86_64-linux, and
>ci is cranking away at over 13000 builds across several architectures:
>
>   https://ci.guix.gnu.org/eval/1772855
>
>Anyone able to cancel that evaluation?
>
>
>I pushed a commit reverting the ordering changes, which I think appears
>to not trigger the rebuild:
>
>commit ea11d3608566174c4bae70faa4f9d0c67748d2db
>Author: Vagrant Cascadian <vagrant@debian.org>
>Date:   Fri Nov 1 16:55:02 2024 -0700
>
>    gnu: python-dbus-python: Revert ordering change on native-inputs.
>
>    A large number of rebuilds (3987 according to guix refresh) was triggered by:
>
>      a9abf9a7b30f6801e122cae759df87b44c458773 gnu: python-dbus-python: Fix
>      indentation.
>
>    Reverting the ordering changes does not trigger any rebuilds.
>
>    * gnu/packages/python-xyz.scm (python-dbus-python): Unsort native-inputs.
>
>
>Hopefully that was the right thing to do!
>
>
>live well,
>  vagrant

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

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

* Re: python-dbus-python changes triggered many rebuilds
  2024-11-02  0:20 python-dbus-python changes triggered many rebuilds Vagrant Cascadian
  2024-11-02  2:13 ` Ian Eure
@ 2024-11-02  8:16 ` Sharlatan Hellseher
  1 sibling, 0 replies; 4+ messages in thread
From: Sharlatan Hellseher @ 2024-11-02  8:16 UTC (permalink / raw)
  To: Vagrant Cascadian; +Cc: guix-devel

Hi,

I've canceled the pending builds.

Sorry it was me, not expected that order of the same inputs would trigger the
build.

The reasoning was to refactor the indentation and higlight that
python-meson-python has to be replaced by meson-python.

Thanks,
Oleg

On Sat, 2 Nov 2024 at 00:20, Vagrant Cascadian <vagrant@debian.org> wrote:
>
> A large rebuild was triggered by:
>
> commit a9abf9a7b30f6801e122cae759df87b44c458773
> Author: Sharlatan Hellseher <sharlatanus@gmail.com>
> Date:   Fri Nov 1 21:10:04 2024 +0000
>
>     gnu: python-dbus-python: Fix indentation.
>
>     * gnu/packages/python-xyz.scm (python-dbus-python): Fix indentation,
>     adjust order of fields, sort inputs alphabetically.
>
>     Change-Id: I895518f041bd2cfc9c2f94774a9d1db47b26ffc3
>
> Guix refresh claims this would trigger 3987 builds on x86_64-linux, and
> ci is cranking away at over 13000 builds across several architectures:
>
>    https://ci.guix.gnu.org/eval/1772855
>
> Anyone able to cancel that evaluation?
>
>
> I pushed a commit reverting the ordering changes, which I think appears
> to not trigger the rebuild:
>
> commit ea11d3608566174c4bae70faa4f9d0c67748d2db
> Author: Vagrant Cascadian <vagrant@debian.org>
> Date:   Fri Nov 1 16:55:02 2024 -0700
>
>     gnu: python-dbus-python: Revert ordering change on native-inputs.
>
>     A large number of rebuilds (3987 according to guix refresh) was triggered by:
>
>       a9abf9a7b30f6801e122cae759df87b44c458773 gnu: python-dbus-python: Fix
>       indentation.
>
>     Reverting the ordering changes does not trigger any rebuilds.
>
>     * gnu/packages/python-xyz.scm (python-dbus-python): Unsort native-inputs.
>
>
> Hopefully that was the right thing to do!
>
>
> live well,
>   vagrant



-- 
VCS: https://github.incerto.xyz/; https://git.sr.ht/~hellseher/
GPG: 9847 81DE 689C 21C2 6418 0867 76D7 27BF F62C D2B5

… наш разум - превосходная объяснительная машина которая способна
найти смысл почти в чем угодно, истолковать любой феномен, но
совершенно не в состоянии принять мысль о непредсказуемости.


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

* Re: python-dbus-python changes triggered many rebuilds
  2024-11-02  2:13 ` Ian Eure
@ 2024-11-02  9:15   ` Ricardo Wurmus
  0 siblings, 0 replies; 4+ messages in thread
From: Ricardo Wurmus @ 2024-11-02  9:15 UTC (permalink / raw)
  To: Ian Eure; +Cc: guix-devel

Ian Eure <ian@retrospec.tv> writes:

> Would it make sense to sort package inputs when computing 
> derivations to prevent this sort of unintentional
> change?

The order matters, e.g. when we add a different gcc to shadow the 
existing gcc.

-- 
Ricardo


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

end of thread, other threads:[~2024-11-02  9:16 UTC | newest]

Thread overview: 4+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2024-11-02  0:20 python-dbus-python changes triggered many rebuilds Vagrant Cascadian
2024-11-02  2:13 ` Ian Eure
2024-11-02  9:15   ` Ricardo Wurmus
2024-11-02  8:16 ` Sharlatan Hellseher

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).