all messages for Guix-related lists mirrored at yhetil.org
 help / color / mirror / code / Atom feed
From: Nicolas Goaziou <mail@nicolasgoaziou.fr>
To: Maxime Devos <maximedevos@telenet.be>
Cc: 54252@debbugs.gnu.org, Jai Vetrivelan <jaivetrivelan@gmail.com>,
	Maxim Cournoyer <maxim.cournoyer@gmail.com>
Subject: [bug#54252] [PATCH] gnu: lemonbar: Update to 1.4.
Date: Sat, 12 Mar 2022 00:14:37 +0100	[thread overview]
Message-ID: <874k44ytcy.fsf@nicolasgoaziou.fr> (raw)
In-Reply-To: <8e6940de1f48cc2f285f52c0ee1d4ec7f9f7ae7c.camel@telenet.be> (Maxime Devos's message of "Fri, 11 Mar 2022 23:30:23 +0100")

Hello,

Maxime Devos <maximedevos@telenet.be> writes:

> However, this seems a bit irrelevant to the issue of the patch being
> committed before this issue was reported -- basically, I make the
> following assumptions:
>
>   1. there is a known issue with the package
>   2. known issues have to be reported upstream in order for a package
>      to be included in Guix

I understand these are your assumptions, but I don't see anything like
that in the packaging guidelines. It may be our duty to report them, as
Maxime Cournoyer wrote, but I don't see this as a blocker either.

Besides, sorry for being bold, but Jai Vetrivelan is not packaging
anything here; they are just bumping a version. The issue with the
package itself is not really of their concern.

>   3. the packager did not report the issue
>   4. a reviewer or the committer can report the issue

Anyone can. 

As a data point, even if I understand the importance of working with
upstream, I consider making that report is way above my pay grade.

> If no-one reported the issue, then from (1) and (2) it follows that the
> package cannot (yet) be included in Guix.

Yes, according to your own assumptions, this is correct.

> Now, when can it be included in Guix?

It is already included: this is a version bump. I assume you're talking
about a hypothetical new package.

> My conclusion is: to include a package in Guix, the issue must first be
> reported upstream.

So your conclusion is your initial assumption? Sorry, but you lost me.

> Additionally, if the packager did not report the
> issue, that does not override the (undocumented?) requirement; if the
> packager did not report it, that merely implies the reviewers or
> committer will have to do it. 

If they have the will, the time, and the skill to do so, absolutely.

However, please remember everyone—including you, of course—is doing
their best, as a benevolent. In this context, I consider "will have to"
to be strong words.

This is all personal but I don't think putting more pressure on
committers—or reviewers for that matter—would be doing any good to the
patch reviewing process, either.

Regards,
-- 
Nicolas Goaziou




  reply	other threads:[~2022-03-11 23:15 UTC|newest]

Thread overview: 13+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2022-03-05  6:05 [bug#54252] [PATCH] gnu: lemonbar: Update to 1.4 Jai Vetrivelan
2022-03-05  9:59 ` Maxime Devos
     [not found]   ` <87h78c4knq.fsf@gmail.com>
2022-03-05 15:32     ` Maxime Devos
2022-03-11 12:32   ` Maxime Devos
2022-03-05 10:01 ` Maxime Devos
2022-03-11  0:47 ` bug#54252: " Nicolas Goaziou
2022-03-11  7:28   ` [bug#54252] " Maxime Devos
2022-03-11  8:31     ` Nicolas Goaziou
2022-03-11 19:15       ` Maxim Cournoyer
2022-03-11 22:09         ` Nicolas Goaziou
2022-03-11 22:30           ` Maxime Devos
2022-03-11 23:14             ` Nicolas Goaziou [this message]
2022-03-11 23:26               ` Maxime Devos

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=874k44ytcy.fsf@nicolasgoaziou.fr \
    --to=mail@nicolasgoaziou.fr \
    --cc=54252@debbugs.gnu.org \
    --cc=jaivetrivelan@gmail.com \
    --cc=maxim.cournoyer@gmail.com \
    --cc=maximedevos@telenet.be \
    /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.