unofficial mirror of guix-patches@gnu.org 
 help / color / mirror / code / Atom feed
From: Suhail Singh <suhailsingh247@gmail.com>
To: Cayetano Santos <csantosb@inventati.org>
Cc: 74231@debbugs.gnu.org
Subject: [bug#74231] QA review for 74231
Date: Thu, 07 Nov 2024 08:32:59 -0500	[thread overview]
Message-ID: <87ed3nyxro.fsf@gmail.com> (raw)
In-Reply-To: <87ed3n1dby.fsf@inventati.org> (Cayetano Santos via Guix-patches via's message of "Thu, 07 Nov 2024 12:40:49 +0100")

Cayetano Santos via Guix-patches via <guix-patches@gnu.org> writes:

> "The Elisp dependencies of Emacs packages are typically provided as
> propagated-inputs when required at run time."
>
> See: https://guix.gnu.org/manual/en/html_node/Emacs-Packages.html.

Yes, please note however, that the dependencies in question were
entirely optional.  The package doesn't have any "required" Elisp
dependencies with the version of Emacs packaged in Guix.  More
importantly, for some of these optional dependencies a user would only
ever want to use one of them at a time (since it's intimately tied to
their personal workflow and MUA etc.).

> To note that this is a completely different beast compared to previous
> package (repo, version and mantainer).

Yes.  Please let me know in case the commit message needs to be revised
(it already does note that we are changing the referenced fork).  The
previous fork hasn't been updated in a couple of years and had a number
of bugs that have since been resolved in the updated fork.

Regards,
-- 
Suhail




  reply	other threads:[~2024-11-07 13:35 UTC|newest]

Thread overview: 8+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2024-11-06 18:37 [bug#74231] [PATCH] gnu: emacs-git-email: Update to 0.5.0 Suhail Singh
2024-11-06 18:37 ` [bug#74231] [PATCH v2] gnu: emacs-git-email: Update to 0.6.0 Suhail Singh
2024-11-15  7:41   ` bug#74231: " Liliana Marie Prikler
2024-11-07 11:40 ` [bug#74231] QA review for 74231 Cayetano Santos via Guix-patches via
2024-11-07 13:32   ` Suhail Singh [this message]
2024-11-07 15:13     ` Cayetano Santos via Guix-patches via
     [not found]       ` <87y11vf3ju.fsf_-_@gmail.com>
2024-11-07 16:20         ` [bug#74231] emacs-git-email: Guix policy for dealing with abandoned packages with active forks Cayetano Santos via Guix-patches via
2024-11-11  2:45           ` Liam Hupfer

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=87ed3nyxro.fsf@gmail.com \
    --to=suhailsingh247@gmail.com \
    --cc=74231@debbugs.gnu.org \
    --cc=csantosb@inventati.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 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).