unofficial mirror of guix-devel@gnu.org 
 help / color / mirror / code / Atom feed
From: Andreas Enge <andreas@enge.fr>
To: Christopher Baines <mail@cbaines.net>
Cc: guix-devel@gnu.org
Subject: Re: Proposed changes to the commit policy (#59513)
Date: Sun, 22 Jan 2023 18:18:24 +0100	[thread overview]
Message-ID: <Y81v4GkdTjo0TROp@jurong> (raw)
In-Reply-To: <87k01k2ef5.fsf@cbaines.net>

Am Wed, Jan 18, 2023 at 11:45:20AM +0000 schrieb Christopher Baines:
> > as a quick concrete question: Do simple package updates still count as
> > trivial, or do they need to go through the patches mailing list?
> My feeling on this is that "simple" package updates are often not
> trivial, or at least doing rigorous testing for these updates isn't
> trivial. A definition of trivial might be "having little value or
> importance", and I don't think that's generally the case for version
> updates, they're often a valuable and important change.

So I tried it once, and find the hassle offputting. It feels like doubling
the effort - after doing the real work, I need to get back to it a week
later and more or less go through the motions again (rebase the commit
and resign it, recompile Guix, build the package again just in case),
plus the debbugs effort.

So expect even less package updates from my part in the future... These
were the kind of instant gratification actions one could do more or less
in the background, and they have become more complex administrative
endeavours with delayed gratification. (I also do not know how to set up
git-sendmail with my remote SMTP server login and password, but this is
a one-time cost of learning which does not matter that much.)

> > In case the answer is that submitting to the patch tracker is required,
> > I would suggest to shorten the waiting period from one week to zero
> > (meaning that it is okay to push when there are no problems detected by qa,
> > without having to wait for human review that has no reason to occur).
> That seems reasonable to me, at least in the case of package
> updates. Given that's such a common change, maybe that needs handling
> specifically in the commit policy.

It would be nice to add this, yes.

Am Wed, Jan 18, 2023 at 04:54:58PM +0000 schrieb Kaelyn:
> On a side note, I'd recently discovered the flag to pass. To have a subject prefix like "[PATCH core-updates]" (as mentioned in the manual for staging and core-updates patches) instead of the default "[PATCH]", one can pass "--subject-prefix="PATCH core-updates" to git format-patch.

This sounds like it could be a useful addition to the QA process.

Andreas



  parent reply	other threads:[~2023-01-22 17:18 UTC|newest]

Thread overview: 16+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2022-12-13 14:10 Proposed changes to the commit policy (#59513) Christopher Baines
2023-01-16 21:47 ` Christopher Baines
2023-01-17 16:35   ` Ludovic Courtès
2023-01-18 11:23   ` Andreas Enge
2023-01-18 11:45     ` Christopher Baines
2023-01-18 16:54       ` Kaelyn
2023-01-22 17:18       ` Andreas Enge [this message]
2023-01-23  9:24         ` Simon Tournier
2023-01-24  9:07           ` Attila Lendvai
2023-01-25 15:03           ` Proposed changes to the commit policy Andreas Enge
2023-01-30 21:40             ` Ludovic Courtès
2023-02-13 11:41               ` Efraim Flashner
2023-01-30 22:03         ` Proposed changes to the commit policy (#59513) Christopher Baines
2023-01-31 11:00           ` Simon Tournier
2023-02-01 13:09           ` Andreas Enge
2023-01-20 11:50     ` Simon Tournier

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=Y81v4GkdTjo0TROp@jurong \
    --to=andreas@enge.fr \
    --cc=guix-devel@gnu.org \
    --cc=mail@cbaines.net \
    /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).