diff mbox series

[bug#68386,v1] gnu: Add emacs-vhdl-mode.

Message ID 1116c7d02e4591ed8105146e3c16372cb78a3f3a.1704985017.git.csantosb@inventati.org
State New
Headers show
Series [bug#68386,v1] gnu: Add emacs-vhdl-mode. | expand

Commit Message

Cayetano Santos Jan. 11, 2024, 2:56 p.m. UTC
* gnu/packages/emacs-xyz.scm (emacs-vhdl-mode): New variable.

Signed-off-by: Cayetano Santos <csantosb@inventati.org>
---
 gnu/packages/emacs-xyz.scm | 18 ++++++++++++++++++
 1 file changed, 18 insertions(+)


base-commit: 3dd8bf94a9d1a2eddb6eda9da7d4a3d422b9dfe9

Comments

Maxim Cournoyer Jan. 15, 2024, 2:46 a.m. UTC | #1
Hi,

Cayetano Santos <csantosb@inventati.org> writes:

> * gnu/packages/emacs-xyz.scm (emacs-vhdl-mode): New variable.
>
> Signed-off-by: Cayetano Santos <csantosb@inventati.org>
> ---
>  gnu/packages/emacs-xyz.scm | 18 ++++++++++++++++++
>  1 file changed, 18 insertions(+)
>
> diff --git a/gnu/packages/emacs-xyz.scm b/gnu/packages/emacs-xyz.scm
> index 1d99fb30ff..dcd16caaa9 100644
> --- a/gnu/packages/emacs-xyz.scm
> +++ b/gnu/packages/emacs-xyz.scm
> @@ -7999,6 +7999,24 @@ (define-public emacs-rspec
>  snippets for yasnippet.")
>        (license license:gpl3+))))
>  
> +(define-public emacs-vhdl-mode
> +  (package
> +    (name "emacs-vhdl-mode")
> +    (version "3.39.3")
> +    (source
> +     (origin
> +       (method url-fetch)
> +       (uri (string-append "https://iis-people.ee.ethz.ch/~zimmi/emacs/vhdl-mode-"
> +                           version ".tar.gz"))
> +       (sha256
> +        (base32 "1yvh4gyig8hgf5pahajjgsgp4jpzwgmlprwzsfybqxrcfza6f3yq"))))
> +    (build-system emacs-build-system)
> +    (home-page "https://iis-people.ee.ethz.ch/~zimmi/emacs/vhdl-mode.html")
> +    (synopsis "Major mode for VHDL")
> +    (description
> +     "This Emacs package provides a mode for the VHDL programming language.")
> +    (license license:gpl3+)))

Since there already is a 'vhdl-mode' package shipped with Emacs, I think
it's important to describe how this one differs (what improvements, features,
refinements, etc. does it provide that would make it worth considering
it instead of the default Emacs-provided option?).
Deslauriers, Douglas via Guix-patches" via Jan. 15, 2024, 7:30 a.m. UTC | #2
>dim. 14 janv. 2024 at 21:46, Maxim Cournoyer <maxim.cournoyer@gmail.com> wrote:

> Since there already is a 'vhdl-mode' package shipped with Emacs, I think
> it's important to describe how this one differs (what improvements, features,
> refinements, etc. does it provide that would make it worth considering
> it instead of the default Emacs-provided option?).

There are also many other packages (org, modus-themes, etc.) shipped with
Emacs that we package in its most recent versions (and many others that
pull these as propagated inputs, for some reason, even when already
available with Emacs). Do we also describe how they differ ?

Anyway, the release notes, update history, bug fixes and enhancements of
every single version of vhdl-mode are given in the project home page,
already included along with the package. Where do we need to replicate
this information exactly for the patch to be accepted ?

Thanks,

Cayetano Santos
Maxim Cournoyer Jan. 17, 2024, 2:08 p.m. UTC | #3
Hi,

Cayetano Santos <csantosb@inventati.org> writes:

>>dim. 14 janv. 2024 at 21:46, Maxim Cournoyer <maxim.cournoyer@gmail.com> wrote:
>
>> Since there already is a 'vhdl-mode' package shipped with Emacs, I think
>> it's important to describe how this one differs (what improvements, features,
>> refinements, etc. does it provide that would make it worth considering
>> it instead of the default Emacs-provided option?).
>
> There are also many other packages (org, modus-themes, etc.) shipped with
> Emacs that we package in its most recent versions (and many others that
> pull these as propagated inputs, for some reason, even when already
> available with Emacs). Do we also describe how they differ ?
>
> Anyway, the release notes, update history, bug fixes and enhancements of
> every single version of vhdl-mode are given in the project home page,
> already included along with the package. Where do we need to replicate
> this information exactly for the patch to be accepted ?

Oh, I hadn't noticed that this was the very same package that comes with
Emacs.  I guess it's fine then, but to avoid any misunderstanding,
perhaps the description text could mention it explicitly ("This is the
same @code{vhdl-mode} that comes with Emacs, albeit it may be newer") or
similar wording.
Deslauriers, Douglas via Guix-patches" via Jan. 17, 2024, 4:11 p.m. UTC | #4
>mer. 17 janv. 2024 at 09:08, Maxim Cournoyer <maxim.cournoyer@gmail.com> wrote:

> Hi,
>
> Cayetano Santos <csantosb@inventati.org> writes:
>
>>>dim. 14 janv. 2024 at 21:46, Maxim Cournoyer <maxim.cournoyer@gmail.com> wrote:
>>
>>> Since there already is a 'vhdl-mode' package shipped with Emacs, I think
>>> it's important to describe how this one differs (what improvements, features,
>>> refinements, etc. does it provide that would make it worth considering
>>> it instead of the default Emacs-provided option?).
>>
>> There are also many other packages (org, modus-themes, etc.) shipped with
>> Emacs that we package in its most recent versions (and many others that
>> pull these as propagated inputs, for some reason, even when already
>> available with Emacs). Do we also describe how they differ ?
>>
>> Anyway, the release notes, update history, bug fixes and enhancements of
>> every single version of vhdl-mode are given in the project home page,
>> already included along with the package. Where do we need to replicate
>> this information exactly for the patch to be accepted ?
>
> Oh, I hadn't noticed that this was the very same package that comes with
> Emacs.  I guess it's fine then, but to avoid any misunderstanding,
> perhaps the description text could mention it explicitly ("This is the
> same @code{vhdl-mode} that comes with Emacs, albeit it may be newer") or
> similar wording.

Done.

As an aside note, at some point, it should be decided how do we handle newest
releases of packages already included with emacs: do we need to mention
it explicitly or something else ?

C.
Maxim Cournoyer Jan. 17, 2024, 7:39 p.m. UTC | #5
Hi Cayetano,

Cayetano Santos <csantosb@inventati.org> writes:

>>mer. 17 janv. 2024 at 09:08, Maxim Cournoyer <maxim.cournoyer@gmail.com> wrote:
>
>> Hi,
>>
>> Cayetano Santos <csantosb@inventati.org> writes:
>>
>>>>dim. 14 janv. 2024 at 21:46, Maxim Cournoyer <maxim.cournoyer@gmail.com> wrote:
>>>
>>>> Since there already is a 'vhdl-mode' package shipped with Emacs, I think
>>>> it's important to describe how this one differs (what improvements, features,
>>>> refinements, etc. does it provide that would make it worth considering
>>>> it instead of the default Emacs-provided option?).
>>>
>>> There are also many other packages (org, modus-themes, etc.) shipped with
>>> Emacs that we package in its most recent versions (and many others that
>>> pull these as propagated inputs, for some reason, even when already
>>> available with Emacs). Do we also describe how they differ ?
>>>
>>> Anyway, the release notes, update history, bug fixes and enhancements of
>>> every single version of vhdl-mode are given in the project home page,
>>> already included along with the package. Where do we need to replicate
>>> this information exactly for the patch to be accepted ?
>>
>> Oh, I hadn't noticed that this was the very same package that comes with
>> Emacs.  I guess it's fine then, but to avoid any misunderstanding,
>> perhaps the description text could mention it explicitly ("This is the
>> same @code{vhdl-mode} that comes with Emacs, albeit it may be newer") or
>> similar wording.
>
> Done.
>
> As an aside note, at some point, it should be decided how do we handle newest
> releases of packages already included with emacs: do we need to mention
> it explicitly or something else ?

I think explicit doesn't hurt.
diff mbox series

Patch

diff --git a/gnu/packages/emacs-xyz.scm b/gnu/packages/emacs-xyz.scm
index 1d99fb30ff..dcd16caaa9 100644
--- a/gnu/packages/emacs-xyz.scm
+++ b/gnu/packages/emacs-xyz.scm
@@ -7999,6 +7999,24 @@  (define-public emacs-rspec
 snippets for yasnippet.")
       (license license:gpl3+))))
 
+(define-public emacs-vhdl-mode
+  (package
+    (name "emacs-vhdl-mode")
+    (version "3.39.3")
+    (source
+     (origin
+       (method url-fetch)
+       (uri (string-append "https://iis-people.ee.ethz.ch/~zimmi/emacs/vhdl-mode-"
+                           version ".tar.gz"))
+       (sha256
+        (base32 "1yvh4gyig8hgf5pahajjgsgp4jpzwgmlprwzsfybqxrcfza6f3yq"))))
+    (build-system emacs-build-system)
+    (home-page "https://iis-people.ee.ethz.ch/~zimmi/emacs/vhdl-mode.html")
+    (synopsis "Major mode for VHDL")
+    (description
+     "This Emacs package provides a mode for the VHDL programming language.")
+    (license license:gpl3+)))
+
 (define-public emacs-mode-line-bell
   (package
     (name "emacs-mode-line-bell")