guix-patches
[Top][All Lists]
Advanced

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

[bug#57426] [PATCH] gnu: Add emacs-consult-org-roam.


From: Nicolas Goaziou
Subject: [bug#57426] [PATCH] gnu: Add emacs-consult-org-roam.
Date: Sat, 27 Aug 2022 10:18:54 +0200
User-agent: Gnus/5.13 (Gnus v5.13) Emacs/28.1 (gnu/linux)

Hello,

Nicolas Graves via Guix-patches via <guix-patches@gnu.org> writes:

> * gnu/packages/emacs-xyz.scm (emacs-consult-org-roam): New variable.

Applied with the changes below. Thank you.

> +(define-public emacs-consult-org-roam
> +  (let* ((commit "9b51aed939054c54934a6969290ad78587051cde")

I used latest commit instead.

> +         (revision "0"))
> +    (package
> +      (name "emacs-consult-org-roam")
> +      (version (git-version "0" revision commit))

Base version is "0.1" according to "Version:" keyword.

> +      (source
> +       (origin
> +         (method git-fetch)
> +         (uri (git-reference
> +               (url "https://github.com/jgru/consult-org-roam.git";)

I removed the ".git" suffix from URL.

> +      (propagated-inputs (list emacs-org-roam emacs-consult))

I sorted inputs alphabetically.

> +      (description
> +       "This is a set of functions to use org-roam with consult.  This
> +packages replaces org-roam's own completing read functions with equivalent
> +versions utilizing consult's internal API.  By doing so, one gains all
> +advantages of consult which enhances Emacs' own completing-read
> +funcionality.")

I removed last sentence, which didn't seem useful in a description.
Also, I used capitalization for project's names (e.g., Consult).

> +      (license license:gpl3))))

License is actually GPL3+.

Regards,
-- 
Nicolas Goaziou





reply via email to

[Prev in Thread] Current Thread [Next in Thread]