GNU bug report logs - #64863
[PATCH] gnu: Add emacs-lexic.

Previous Next

Full log

View this message in rfc822 format

From: help-debbugs <at> gnu.org (GNU bug Tracking System)
To: Clément Lassieur <clement <at> lassieur.org>
Cc: tracker <at> debbugs.gnu.org
Subject: bug#64863: closed ([PATCH] gnu: Add emacs-lexic.)
Date: Fri, 20 Oct 2023 16:20:02 +0000

[Message part 1 (text/plain, inline)]

Your message dated Fri, 20 Oct 2023 18:19:17 +0200
with message-id <87ttqltgfu.fsf <at> lassieur.org>
and subject line Re: bug#64863: [PATCH] gnu: Add emacs-lexic.
has caused the debbugs.gnu.org bug report #64863,
regarding [PATCH] gnu: Add emacs-lexic.
to be marked as done.

(If you believe you have received this mail in error, please contact
help-debbugs <at> gnu.org.)


-- 
64863: https://debbugs.gnu.org/cgi/bugreport.cgi?bug=64863
GNU Bug Tracking System
Contact help-debbugs <at> gnu.org with problems

[Message part 2 (message/rfc822, inline)]

From: pinoaffe <pinoaffe <at> gmail.com>
Subject: [PATCH] gnu: Add emacs-lexic.
Date: Tue, 25 Jul 2023 23:06:07 +0200

* gnu/packages/emacs-xyz.scm (emacs-lexic): New variable.
---
 gnu/packages/emacs-xyz.scm | 36 ++++++++++++++++++++++++++++++++++++
 1 file changed, 36 insertions(+)

diff --git a/gnu/packages/emacs-xyz.scm b/gnu/packages/emacs-xyz.scm
index d7e45fffc6..10b8923ada 100644
--- a/gnu/packages/emacs-xyz.scm
+++ b/gnu/packages/emacs-xyz.scm
@@ -36508,6 +36508,42 @@ (define-public emacs-multitran
 to the @url{https://multitran.com} online dictionary.")
     (license license:gpl3+)))
 
+(define-public emacs-lexic
+  (let ((release "0")
+        (revision "0")
+        (commit "f9b3de4d9c2dd1ce5022383e1a504b87bf7d1b09"))
+    (package
+      (name "emacs-lexic")
+      (version (git-version release revision commit))
+      (source (origin
+                (method git-fetch)
+                (uri (git-reference
+                      (url "https://git.tecosaur.net/tec/lexic")
+                      (commit commit)))
+                (file-name (git-file-name name version))
+                (sha256
+                 (base32
+                  "02iz8dh18gb1q97v8ghcd3lavkf28sqbrk0bx6jzzryp69ickk4h"))))
+      (build-system emacs-build-system)
+      (inputs (list sdcv))
+      (arguments
+       `(#:phases
+         (modify-phases %standard-phases
+           (add-after 'unpack 'set-sdcv-path
+             (lambda* (#:key inputs outputs #:allow-other-keys)
+               (emacs-substitute-variables "lexic.el"
+                 ("lexic-program-path"
+                  (string-append (assoc-ref inputs "sdcv")
+                                 "/bin/sdcv"))))))))
+      (propagated-inputs (list sdcv))
+      (home-page "https://git.tecosaur.net/tec/lexic")
+      (synopsis "Find out more about words using Stardict dictionaries")
+      (description "This provides a major mode to view the output of
+dictionary tools, and utilities that perform searches and nicely format the
+results.  Currently tied to sdcv, but this is intended to be changed in the
+future.")
+      (license license:gpl2+))))
+
 (define-public emacs-blacken
   (package
     (name "emacs-blacken")

base-commit: 005912c595bf7a3329f8aa51a4ccb1d91b6ecd9e
-- 
2.41.0

[Message part 3 (message/rfc822, inline)]

From: Clément Lassieur <clement <at> lassieur.org>
To: pinoaffe <pinoaffe <at> gmail.com>
Cc: 64863-done <at> debbugs.gnu.org
Subject: Re: bug#64863: [PATCH] gnu: Add emacs-lexic.
Date: Fri, 20 Oct 2023 18:19:17 +0200

LGTM, pushed with a few modifications:

- removing the "release" local binding.  The reason for the "revision"
  local binding is, I believe, to make sure people increment it upon
  modifying the package.  But there is no need to increment the
  "release" number if it's not incremented upstream.

- removed sdcv as propagated input.  It's only an input, I don't think
  it needs to be installed since the full path is written within the .el
  file.

- changed the license to gpl3+

Thanks and sorry for the huge delay.
Clément


pinoaffe <pinoaffe <at> gmail.com> writes:

> * gnu/packages/emacs-xyz.scm (emacs-lexic): New variable.
> ---
>  gnu/packages/emacs-xyz.scm | 36 ++++++++++++++++++++++++++++++++++++
>  1 file changed, 36 insertions(+)
>
> diff --git a/gnu/packages/emacs-xyz.scm b/gnu/packages/emacs-xyz.scm
> index d7e45fffc6..10b8923ada 100644
> --- a/gnu/packages/emacs-xyz.scm
> +++ b/gnu/packages/emacs-xyz.scm
> @@ -36508,6 +36508,42 @@ (define-public emacs-multitran
>  to the @url{https://multitran.com} online dictionary.")
>      (license license:gpl3+)))
>  
> +(define-public emacs-lexic
> +  (let ((release "0")
> +        (revision "0")
> +        (commit "f9b3de4d9c2dd1ce5022383e1a504b87bf7d1b09"))
> +    (package
> +      (name "emacs-lexic")
> +      (version (git-version release revision commit))
> +      (source (origin
> +                (method git-fetch)
> +                (uri (git-reference
> +                      (url "https://git.tecosaur.net/tec/lexic")
> +                      (commit commit)))
> +                (file-name (git-file-name name version))
> +                (sha256
> +                 (base32
> +                  "02iz8dh18gb1q97v8ghcd3lavkf28sqbrk0bx6jzzryp69ickk4h"))))
> +      (build-system emacs-build-system)
> +      (inputs (list sdcv))
> +      (arguments
> +       `(#:phases
> +         (modify-phases %standard-phases
> +           (add-after 'unpack 'set-sdcv-path
> +             (lambda* (#:key inputs outputs #:allow-other-keys)
> +               (emacs-substitute-variables "lexic.el"
> +                 ("lexic-program-path"
> +                  (string-append (assoc-ref inputs "sdcv")
> +                                 "/bin/sdcv"))))))))
> +      (propagated-inputs (list sdcv))
> +      (home-page "https://git.tecosaur.net/tec/lexic")
> +      (synopsis "Find out more about words using Stardict dictionaries")
> +      (description "This provides a major mode to view the output of
> +dictionary tools, and utilities that perform searches and nicely format the
> +results.  Currently tied to sdcv, but this is intended to be changed in the
> +future.")
> +      (license license:gpl2+))))
> +
>  (define-public emacs-blacken
>    (package
>      (name "emacs-blacken")
>
> base-commit: 005912c595bf7a3329f8aa51a4ccb1d91b6ecd9e

Previous Next

GNU bug tracking system
Copyright (C) 1999 Darren O. Benham, 1997,2003 nCipher Corporation Ltd, 1994-97 Ian Jackson.