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 --]
prev parent 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.