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
next prev parent 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
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=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 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).