emacs-devel
[Top][All Lists]
Advanced

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

Re: [ELPA] New package: dape


From: Philip Kaludercic
Subject: Re: [ELPA] New package: dape
Date: Tue, 05 Dec 2023 08:41:53 +0000

Philip Kaludercic <philipk@posteo.net> writes:

> Philip Kaludercic <philipk@posteo.net> writes:
>
>> Daniel Pettersson <daniel@dpettersson.net> writes:
>>
>>> On Wed, Nov 1, 2023 at 7:35 PM Philip Kaludercic <philipk@posteo.net> wrote:
>>>> Did we ever come to a conclusion on this point?
>>>
>>> Sorry I have been putting the naming thing off as this thread sparked
>>> a bit of interest in the package which lead to a lot of great input and
>>> bugfixing. There have been a lot of good suggestions and no consensus
>>> which has made the even task harder.
>>
>> In the end that is your call, it is just currently a blocking issue
>> before we add the package to GNU ELPA.
>
> It seems that the name is here to stay, so I'd go ahead and add the
> package to GNU ELPA.  It just seems that you forgot to add a copyright
> notice to the package header.  Could you apply a change along these
> lines:
>
> diff --git a/dape.el b/dape.el
> index f5de1cd..949049d 100644
> --- a/dape.el
> +++ b/dape.el
> @@ -1,5 +1,7 @@
>  ;;; dape.el --- Debug Adapter Protocol for Emacs -*- lexical-binding: t -*-
>  
> +;; Copyright (C) 2023  Free Software Foundation, Inc.
> +
>  ;; Author: Daniel Pettersson
>  ;; Created: 2023
>  ;; License: GPL-3.0-or-later
>
>
> Followed by a  commit (or in the same one) that bumps the "Version"
> header, indicating a new package release?

I have resolved the issue directly with Daniel.  The package has been
added to GNU ELPA.

Depending on how it is received, it would be nice to see the package be
added to the core in some form or another.  By my book, the naming issue
could be resolved by adding a `start-dap' command (a `start-lsp' alias
for Eglot would also be nice).



reply via email to

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