all messages for Guix-related lists mirrored at yhetil.org
 help / color / mirror / code / Atom feed
From: Ian Eure <ian@retrospec.tv>
To: guix-devel@gnu.org
Subject: Re: python-dbus-python changes triggered many rebuilds
Date: Fri, 01 Nov 2024 19:13:12 -0700	[thread overview]
Message-ID: <1B875A30-5C23-46B7-989B-F9D4365EA4AE@retrospec.tv> (raw)
In-Reply-To: <874j4q7afw.fsf@wireframe>

[-- 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 --]

      reply	other threads:[~2024-11-02  2:14 UTC|newest]

Thread overview: 2+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2024-11-02  0:20 python-dbus-python changes triggered many rebuilds Vagrant Cascadian
2024-11-02  2:13 ` Ian Eure [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=1B875A30-5C23-46B7-989B-F9D4365EA4AE@retrospec.tv \
    --to=ian@retrospec.tv \
    --cc=guix-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.
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.