unofficial mirror of emacs-devel@gnu.org 
 help / color / mirror / code / Atom feed
* [ELPA] New package devicetree-ts-mode
@ 2023-12-21 17:47 Aleksandr Vityazev
  2023-12-22  7:36 ` Philip Kaludercic
  2023-12-22 16:02 ` Stefan Kangas
  0 siblings, 2 replies; 12+ messages in thread
From: Aleksandr Vityazev @ 2023-12-21 17:47 UTC (permalink / raw)
  To: emacs-devel

[-- Attachment #1: Type: text/plain, Size: 203 bytes --]


Hello,

I'd like to submit devicetree-ts-mode [1] to GNU ELPA.
This is tree-sitter major mode for Devicetree [2] files.

[1] https://git.sr.ht/~akagi/devicetree-ts-mode/
[2] https://www.devicetree.org/

[-- Warning: decoded text below may be mangled, UTF-8 assumed --]
[-- Attachment #2: patch --]
[-- Type: text/x-patch, Size: 990 bytes --]

From b69687be8232cfd305893b7c0b6999e6667d6dd8 Mon Sep 17 00:00:00 2001
Message-ID: <b69687be8232cfd305893b7c0b6999e6667d6dd8.1703075351.git.avityazew@gmail.com>
From: Aleksandr Vityazev <avityazew@gmail.com>
Date: Wed, 20 Dec 2023 15:28:35 +0300
Subject: [PATCH] * elpa-packages (devicetree-ts-mode): New package

---
 elpa-packages | 2 ++
 1 file changed, 2 insertions(+)

diff --git a/elpa-packages b/elpa-packages
index 612bc676cd..6f86a59cfd 100644
--- a/elpa-packages
+++ b/elpa-packages
@@ -207,6 +207,8 @@
   :news "CHANGELOG.org"
   :readme "README.md")
  (devdocs		:url "https://github.com/astoff/devdocs.el")
+ (devicetree-ts-mode	:url "https://sr.ht/~akagi/devicetree-ts-mode"
+  :ignored-files ("LICENSE"))
  (dict-tree		:url nil) ;"http://www.dr-qubit.org/git/predictive.git"
  (diff-hl		:url "https://github.com/dgutov/diff-hl.git")
  (diffview            	:url "https://github.com/mgalgs/diffview-mode.git")

base-commit: b7bbd439862f2a58151eacacebc1815b7ddf3322
-- 
2.41.0


[-- Attachment #3: Type: text/plain, Size: 38 bytes --]


-- 
Best regards,
Aleksandr Vityazev

^ permalink raw reply related	[flat|nested] 12+ messages in thread

* Re: [ELPA] New package devicetree-ts-mode
  2023-12-21 17:47 [ELPA] New package devicetree-ts-mode Aleksandr Vityazev
@ 2023-12-22  7:36 ` Philip Kaludercic
  2023-12-22 20:08   ` Aleksandr Vityazev
  2023-12-22 16:02 ` Stefan Kangas
  1 sibling, 1 reply; 12+ messages in thread
From: Philip Kaludercic @ 2023-12-22  7:36 UTC (permalink / raw)
  To: Aleksandr Vityazev; +Cc: emacs-devel

[-- Attachment #1: Type: text/plain, Size: 291 bytes --]

Aleksandr Vityazev <avityazew@gmail.com> writes:

> Hello,
>
> I'd like to submit devicetree-ts-mode [1] to GNU ELPA.
> This is tree-sitter major mode for Devicetree [2] files.
>
> [1] https://git.sr.ht/~akagi/devicetree-ts-mode/
> [2] https://www.devicetree.org/

Here are a few comments:


[-- Attachment #2: Type: text/plain, Size: 3397 bytes --]

diff --git a/devicetree-ts-mode.el b/devicetree-ts-mode.el
index 1d9f72c..d26937c 100644
--- a/devicetree-ts-mode.el
+++ b/devicetree-ts-mode.el
@@ -33,6 +33,7 @@
 ;; * IMenu
 ;; * Font Lock
 
+;; The commentary section could elaborate on what "Devicetree" are.
 
 ;;; Code:
 
@@ -44,15 +45,21 @@
 (declare-function treesit-parser-create "treesit.c")
 (declare-function treesit-node-child-by-field-name "treesit.c")
 
+(defgroup devicetree ()
+  "Tree-sitter support for DTS."
+  :prefix "devicetree-ts-"
+  :group 'languages)
+
 (defcustom devicetree-ts-mode-indent-offset 4
   "Number of spaces for each indentation step in `devicetree-ts-mode'."
   :version "29.1"
+  ;; This is not a core package, the version of your package is 0.2,
+  ;; so this doesn't match up
   :type 'natnum
-  :safe 'natnump
-  :group 'devicetree)
+  :safe 'natnump)
 
 ;; Taken from the dts-mode
-(defvar devicetree-ts-mode--syntax-table
+(defvar devicetree-ts-mode-syntax-table
   (let ((table (make-syntax-table)))
 
     (modify-syntax-entry ?<  "(>" table)
@@ -81,6 +88,10 @@
 
 (defvar devicetree-ts-mode--indent-rules
   (let ((offset devicetree-ts-mode-indent-offset))
+    ;; If this is a variable, that is set when the package is loaded,
+    ;; customising the user option `devicetree-ts-mode-indent-offset'
+    ;; will have no effect.  You could turn this into a function
+    ;; instead.
     `((devicetree
        ((node-is ">") parent-bol 0)
        ((node-is "]") parent-bol 0)
@@ -93,13 +104,14 @@
        (no-node parent-bol 0))))
   "Tree-sitter indent rules for `devicetree-ts-mode'.")
 
+;; Could these be defconst?
 (defvar devicetree-ts-mode--treesit-keywords
   '("/delete-node/" "/delete-property/" "#define" "#include"
     "/omit-if-no-ref/" "/dts-v1/"))
 
 (defvar devicetree-ts-mode--treesit-operators
-  '( "!" "~" "-" "+" "*" "/" "%" "||" "&&" "|"
-     "^" "&" "==" "!=" ">" ">=" "<=" ">" "<<" ">>"))
+  '("!" "~" "-" "+" "*" "/" "%" "||" "&&" "|"
+    "^" "&" "==" "!=" ">" ">=" "<=" ">" "<<" ">>"))
 
 (defvar devicetree-ts-mode--font-lock-settings
   (treesit-font-lock-rules
@@ -154,8 +166,8 @@
 
 (defun devicetree-ts-mode--node-addresses (node)
   "List of addresses for NODE."
-  (reverse
-   (seq-reduce
+  (reverse				;Why `reverse'? (or `nreverse'?)
+   (seq-reduce				;Isn't this a `seq-filter'?
     (lambda (acc children)
       (if (string-equal (treesit-node-field-name children)
                         "address")
@@ -175,8 +187,6 @@
 ;;;###autoload
 (define-derived-mode devicetree-ts-mode prog-mode "DTS"
   "Major mode for editing devicetree, powered by tree-sitter."
-  :group 'devicetree
-  :syntax-table devicetree-ts-mode--syntax-table
 
   (when (treesit-ready-p 'devicetree)
     (treesit-parser-create 'devicetree)
@@ -187,7 +197,7 @@
 
     ;; Imenu.
     (setq-local treesit-simple-imenu-settings
-                `((nil "\\`node\\'"
+                `((nil ,(rx bos "node" eos)
                        nil devicetree-ts--mode--name-function)))
     (setq-local which-func-functions nil)
 
@@ -216,9 +226,8 @@
 
     (treesit-major-mode-setup)))
 
-(if (treesit-ready-p 'devicetree)
-    (add-to-list 'auto-mode-alist
-                 '("\\.dtsi?\\'" . devicetree-ts-mode)))
+(when (treesit-ready-p 'devicetree)
+  (add-to-list 'auto-mode-alist '("\\.dtsi?\\'" . devicetree-ts-mode)))
 
 (provide 'devicetree-ts-mode)
 ;;; devicetree-ts-mode.el ends here

[-- Attachment #3: Type: text/plain, Size: 1151 bytes --]


>
> From b69687be8232cfd305893b7c0b6999e6667d6dd8 Mon Sep 17 00:00:00 2001
> Message-ID: <b69687be8232cfd305893b7c0b6999e6667d6dd8.1703075351.git.avityazew@gmail.com>
> From: Aleksandr Vityazev <avityazew@gmail.com>
> Date: Wed, 20 Dec 2023 15:28:35 +0300
> Subject: [PATCH] * elpa-packages (devicetree-ts-mode): New package
>
> ---
>  elpa-packages | 2 ++
>  1 file changed, 2 insertions(+)
>
> diff --git a/elpa-packages b/elpa-packages
> index 612bc676cd..6f86a59cfd 100644
> --- a/elpa-packages
> +++ b/elpa-packages
> @@ -207,6 +207,8 @@
>    :news "CHANGELOG.org"
>    :readme "README.md")
>   (devdocs		:url "https://github.com/astoff/devdocs.el")
> + (devicetree-ts-mode	:url "https://sr.ht/~akagi/devicetree-ts-mode"
> +  :ignored-files ("LICENSE"))

You can track this file in an .elpaignore file, within your repository
(which you appear to have anyway).

>   (dict-tree		:url nil) ;"http://www.dr-qubit.org/git/predictive.git"
>   (diff-hl		:url "https://github.com/dgutov/diff-hl.git")
>   (diffview            	:url "https://github.com/mgalgs/diffview-mode.git")
>
> base-commit: b7bbd439862f2a58151eacacebc1815b7ddf3322
> -- 
> 2.41.0

^ permalink raw reply related	[flat|nested] 12+ messages in thread

* Re: [ELPA] New package devicetree-ts-mode
  2023-12-21 17:47 [ELPA] New package devicetree-ts-mode Aleksandr Vityazev
  2023-12-22  7:36 ` Philip Kaludercic
@ 2023-12-22 16:02 ` Stefan Kangas
  2023-12-22 20:36   ` Aleksandr Vityazev
  1 sibling, 1 reply; 12+ messages in thread
From: Stefan Kangas @ 2023-12-22 16:02 UTC (permalink / raw)
  To: Aleksandr Vityazev, emacs-devel

Aleksandr Vityazev <avityazew@gmail.com> writes:

> I'd like to submit devicetree-ts-mode [1] to GNU ELPA.
> This is tree-sitter major mode for Devicetree [2] files.

Is there any particular reason to add this to GNU ELPA and not to core?
Is it too niche?



^ permalink raw reply	[flat|nested] 12+ messages in thread

* Re: [ELPA] New package devicetree-ts-mode
  2023-12-22  7:36 ` Philip Kaludercic
@ 2023-12-22 20:08   ` Aleksandr Vityazev
  2024-01-07 13:27     ` Aleksandr Vityazev
  0 siblings, 1 reply; 12+ messages in thread
From: Aleksandr Vityazev @ 2023-12-22 20:08 UTC (permalink / raw)
  To: Philip Kaludercic; +Cc: emacs-devel

[-- Attachment #1: Type: text/plain, Size: 6064 bytes --]

Hi,

Thanks for the review, my comments are below.

Commit with changes:
https://git.sr.ht/~akagi/devicetree-ts-mode/commit/bc07c1124545cbf6e5ebe64e92bfaa306e309033

On 2023-12-22 07:36, Philip Kaludercic wrote:

> Aleksandr Vityazev <avityazew@gmail.com> writes:
>
>> Hello,
>>
>> I'd like to submit devicetree-ts-mode [1] to GNU ELPA.
>> This is tree-sitter major mode for Devicetree [2] files.
>>
>> [1] https://git.sr.ht/~akagi/devicetree-ts-mode/
>> [2] https://www.devicetree.org/
>
> Here are a few comments:
>
> diff --git a/devicetree-ts-mode.el b/devicetree-ts-mode.el
> index 1d9f72c..d26937c 100644
> --- a/devicetree-ts-mode.el
> +++ b/devicetree-ts-mode.el
> @@ -33,6 +33,7 @@
>  ;; * IMenu
>  ;; * Font Lock
>  
> +;; The commentary section could elaborate on what "Devicetree" are.

Fixed.

>  ;;; Code:
>  
> @@ -44,15 +45,21 @@
>  (declare-function treesit-parser-create "treesit.c")
>  (declare-function treesit-node-child-by-field-name "treesit.c")
>  
> +(defgroup devicetree ()
> +  "Tree-sitter support for DTS."
> +  :prefix "devicetree-ts-"
> +  :group 'languages)
> +

Applied.

>  (defcustom devicetree-ts-mode-indent-offset 4
>    "Number of spaces for each indentation step in `devicetree-ts-mode'."
>    :version "29.1"
> +  ;; This is not a core package, the version of your package is 0.2,
> +  ;; so this doesn't match up
Fixed.
>    :type 'natnum
> -  :safe 'natnump
> -  :group 'devicetree)
> +  :safe 'natnump)
Applied.  
>  ;; Taken from the dts-mode
> -(defvar devicetree-ts-mode--syntax-table
> +(defvar devicetree-ts-mode-syntax-table
Applied.  
>    (let ((table (make-syntax-table)))
>  
>      (modify-syntax-entry ?<  "(>" table)
> @@ -81,6 +88,10 @@
>  
>  (defvar devicetree-ts-mode--indent-rules
>    (let ((offset devicetree-ts-mode-indent-offset))
> +    ;; If this is a variable, that is set when the package is loaded,
> +    ;; customising the user option `devicetree-ts-mode-indent-offset'
> +    ;; will have no effect.  You could turn this into a function
> +    ;; instead.
>      `((devicetree
>         ((node-is ">") parent-bol 0)
>         ((node-is "]") parent-bol 0)
> @@ -93,13 +104,14 @@
>         (no-node parent-bol 0))))
>    "Tree-sitter indent rules for `devicetree-ts-mode'.")
Replaced by function.

> +;; Could these be defconst?
>  (defvar devicetree-ts-mode--treesit-keywords
>    '("/delete-node/" "/delete-property/" "#define" "#include"
>      "/omit-if-no-ref/" "/dts-v1/"))
>  
>  (defvar devicetree-ts-mode--treesit-operators
> -  '( "!" "~" "-" "+" "*" "/" "%" "||" "&&" "|"
> -     "^" "&" "==" "!=" ">" ">=" "<=" ">" "<<" ">>"))
> +  '("!" "~" "-" "+" "*" "/" "%" "||" "&&" "|"
> +    "^" "&" "==" "!=" ">" ">=" "<=" ">" "<<" ">>"))
>
Replaced defvar with defconst.

>  (defvar devicetree-ts-mode--font-lock-settings
>    (treesit-font-lock-rules
> @@ -154,8 +166,8 @@
>  
>  (defun devicetree-ts-mode--node-addresses (node)
>    "List of addresses for NODE."
> -  (reverse
> -   (seq-reduce
> +  (reverse				;Why `reverse'? (or `nreverse'?)
> +   (seq-reduce				;Isn't this a `seq-filter'?
>      (lambda (acc children)
>        (if (string-equal (treesit-node-field-name children)
>                          "address")

seq-filter returns the element itself, so the code would look like this

#+begin_src elisp 
  (defun devicetree-ts-mode--node-addresses (node)
  "List of addresses for NODE."
  (let ((address-nodes
         (seq-filter
          (lambda (children)
            (when (string-equal (treesit-node-field-name children)
                                "address")
              t))
          (treesit-node-children node))))
    (seq-map (lambda (children)
               (treesit-node-text children t))
             address-nodes)))

#+end_src

I think the code with reverse looks better, so I left it as it was.

> @@ -175,8 +187,6 @@
>  ;;;###autoload
>  (define-derived-mode devicetree-ts-mode prog-mode "DTS"
>    "Major mode for editing devicetree, powered by tree-sitter."
> -  :group 'devicetree
> -  :syntax-table devicetree-ts-mode--syntax-table

Applied.  
>    (when (treesit-ready-p 'devicetree)
>      (treesit-parser-create 'devicetree)
> @@ -187,7 +197,7 @@
>  
>      ;; Imenu.
>      (setq-local treesit-simple-imenu-settings
> -                `((nil "\\`node\\'"
> +                `((nil ,(rx bos "node" eos)
>                         nil devicetree-ts--mode--name-function)))

Applied.  
>      (setq-local which-func-functions nil)
>  
> @@ -216,9 +226,8 @@
>  
>      (treesit-major-mode-setup)))
>  
> -(if (treesit-ready-p 'devicetree)
> -    (add-to-list 'auto-mode-alist
> -                 '("\\.dtsi?\\'" . devicetree-ts-mode)))
> +(when (treesit-ready-p 'devicetree)
> +  (add-to-list 'auto-mode-alist '("\\.dtsi?\\'" . devicetree-ts-mode)))
Applied.

>  (provide 'devicetree-ts-mode)
>  ;;; devicetree-ts-mode.el ends here
>
>
>>
>> From b69687be8232cfd305893b7c0b6999e6667d6dd8 Mon Sep 17 00:00:00 2001
>> Message-ID: <b69687be8232cfd305893b7c0b6999e6667d6dd8.1703075351.git.avityazew@gmail.com>
>> From: Aleksandr Vityazev <avityazew@gmail.com>
>> Date: Wed, 20 Dec 2023 15:28:35 +0300
>> Subject: [PATCH] * elpa-packages (devicetree-ts-mode): New package
>>
>> ---
>>  elpa-packages | 2 ++
>>  1 file changed, 2 insertions(+)
>>
>> diff --git a/elpa-packages b/elpa-packages
>> index 612bc676cd..6f86a59cfd 100644
>> --- a/elpa-packages
>> +++ b/elpa-packages
>> @@ -207,6 +207,8 @@
>>    :news "CHANGELOG.org"
>>    :readme "README.md")
>>   (devdocs		:url "https://github.com/astoff/devdocs.el")
>> + (devicetree-ts-mode	:url "https://sr.ht/~akagi/devicetree-ts-mode"
>> +  :ignored-files ("LICENSE"))
>
> You can track this file in an .elpaignore file, within your repository
> (which you appear to have anyway).

New patch attached.
>
>>   (dict-tree		:url nil) ;"http://www.dr-qubit.org/git/predictive.git"
>>   (diff-hl		:url "https://github.com/dgutov/diff-hl.git")
>>   (diffview            	:url "https://github.com/mgalgs/diffview-mode.git")
>>
>> base-commit: b7bbd439862f2a58151eacacebc1815b7ddf3322
>> -- 
>> 2.41.0
>


[-- Warning: decoded text below may be mangled, UTF-8 assumed --]
[-- Attachment #2: patch --]
[-- Type: text/x-patch, Size: 959 bytes --]

From a0e73d44a933e0a63b340fe1df60e1b5c75a66bf Mon Sep 17 00:00:00 2001
Message-ID: <a0e73d44a933e0a63b340fe1df60e1b5c75a66bf.1703272287.git.avityazew@gmail.com>
From: Aleksandr Vityazev <avityazew@gmail.com>
Date: Fri, 22 Dec 2023 22:10:30 +0300
Subject: [PATCH] * elpa-packages (devicetree-ts-mode): New package.

---
 elpa-packages | 1 +
 1 file changed, 1 insertion(+)

diff --git a/elpa-packages b/elpa-packages
index f2c60cac5d..078d772d00 100644
--- a/elpa-packages
+++ b/elpa-packages
@@ -207,6 +207,7 @@
   :news "CHANGELOG.org"
   :readme "README.md")
  (devdocs		:url "https://github.com/astoff/devdocs.el")
+ (devicetree-ts-mode	:url "https://sr.ht/~akagi/devicetree-ts-mode")
  (dict-tree		:url nil) ;"http://www.dr-qubit.org/git/predictive.git"
  (diff-hl		:url "https://github.com/dgutov/diff-hl.git")
  (diffview            	:url "https://github.com/mgalgs/diffview-mode.git")

base-commit: 080e09a4c6f038818a5a7a9c5e53ff168e0fe00e
-- 
2.41.0


[-- Attachment #3: Type: text/plain, Size: 39 bytes --]



-- 
Best regards,
Aleksandr Vityazev

^ permalink raw reply related	[flat|nested] 12+ messages in thread

* Re: [ELPA] New package devicetree-ts-mode
  2023-12-22 16:02 ` Stefan Kangas
@ 2023-12-22 20:36   ` Aleksandr Vityazev
  2024-02-20  3:04     ` Björn Bidar
  0 siblings, 1 reply; 12+ messages in thread
From: Aleksandr Vityazev @ 2023-12-22 20:36 UTC (permalink / raw)
  To: Stefan Kangas; +Cc: emacs-devel

On 2023-12-22 08:02, Stefan Kangas wrote:

> Aleksandr Vityazev <avityazew@gmail.com> writes:
>
>> I'd like to submit devicetree-ts-mode [1] to GNU ELPA.
>> This is tree-sitter major mode for Devicetree [2] files.
>
> Is there any particular reason to add this to GNU ELPA and not to core?
> Is it too niche?

I can't say whether Devisetree is too niche, but I think 
ELPA is a great place for this package. It will be easier to 
maintain.

-- 
Best regards,
Aleksandr Vityazev



^ permalink raw reply	[flat|nested] 12+ messages in thread

* Re: [ELPA] New package devicetree-ts-mode
  2023-12-22 20:08   ` Aleksandr Vityazev
@ 2024-01-07 13:27     ` Aleksandr Vityazev
  0 siblings, 0 replies; 12+ messages in thread
From: Aleksandr Vityazev @ 2024-01-07 13:27 UTC (permalink / raw)
  To: emacs-devel

Hello,

just a gentle ping -- could anyone review last changes?

I'm looking forward for any feedback.

-- 
Best regards,
Aleksandr Vityazev



^ permalink raw reply	[flat|nested] 12+ messages in thread

* [ELPA] New package devicetree-ts-mode
@ 2024-01-16  0:38 Aleksandr Vityazev
  2024-01-16  6:18 ` Philip Kaludercic
  0 siblings, 1 reply; 12+ messages in thread
From: Aleksandr Vityazev @ 2024-01-16  0:38 UTC (permalink / raw)
  To: emacs-devel; +Cc: Philip Kaludercic, Stefan Kangas

[-- Attachment #1: Type: text/plain, Size: 407 bytes --]

Hello,

Since the previous thread [1] got lost over time and no more replies,
I'm starting a new one.

All of Philip's suggested points have been fixed, is there any reason
why this package should not be included in ELPA? I am willing to listen
and fix if possible.

Link to repository - https://git.sr.ht/~akagi/devicetree-ts-mode/

[1] https://mail.gnu.org/archive/html/emacs-devel/2023-12/msg00629.html


[-- Warning: decoded text below may be mangled, UTF-8 assumed --]
[-- Attachment #2: patch --]
[-- Type: text/x-patch, Size: 959 bytes --]

From a0e73d44a933e0a63b340fe1df60e1b5c75a66bf Mon Sep 17 00:00:00 2001
Message-ID: <a0e73d44a933e0a63b340fe1df60e1b5c75a66bf.1703272287.git.avityazew@gmail.com>
From: Aleksandr Vityazev <avityazew@gmail.com>
Date: Fri, 22 Dec 2023 22:10:30 +0300
Subject: [PATCH] * elpa-packages (devicetree-ts-mode): New package.

---
 elpa-packages | 1 +
 1 file changed, 1 insertion(+)

diff --git a/elpa-packages b/elpa-packages
index f2c60cac5d..078d772d00 100644
--- a/elpa-packages
+++ b/elpa-packages
@@ -207,6 +207,7 @@
   :news "CHANGELOG.org"
   :readme "README.md")
  (devdocs		:url "https://github.com/astoff/devdocs.el")
+ (devicetree-ts-mode	:url "https://sr.ht/~akagi/devicetree-ts-mode")
  (dict-tree		:url nil) ;"http://www.dr-qubit.org/git/predictive.git"
  (diff-hl		:url "https://github.com/dgutov/diff-hl.git")
  (diffview            	:url "https://github.com/mgalgs/diffview-mode.git")

base-commit: 080e09a4c6f038818a5a7a9c5e53ff168e0fe00e
-- 
2.41.0


[-- Attachment #3: Type: text/plain, Size: 39 bytes --]



-- 
Best regards,
Aleksandr Vityazev

^ permalink raw reply related	[flat|nested] 12+ messages in thread

* Re: [ELPA] New package devicetree-ts-mode
  2024-01-16  0:38 Aleksandr Vityazev
@ 2024-01-16  6:18 ` Philip Kaludercic
  2024-01-16  6:42   ` Aleksandr Vityazev
  0 siblings, 1 reply; 12+ messages in thread
From: Philip Kaludercic @ 2024-01-16  6:18 UTC (permalink / raw)
  To: Aleksandr Vityazev; +Cc: emacs-devel, Stefan Kangas

Aleksandr Vityazev <avityazev@disroot.org> writes:

> Hello,
>
> Since the previous thread [1] got lost over time and no more replies,
> I'm starting a new one.

Sorry about that, I hadn't gotten around to answering your last ping.

> All of Philip's suggested points have been fixed, is there any reason
> why this package should not be included in ELPA? I am willing to listen
> and fix if possible.

I think code-wise everything should be fine.  Just to be sure, you have
signed the FSF CA and are familiar with how the ELPA build server works
(at least from a package developer perspective)?

> Link to repository - https://git.sr.ht/~akagi/devicetree-ts-mode/
>
> [1] https://mail.gnu.org/archive/html/emacs-devel/2023-12/msg00629.html



^ permalink raw reply	[flat|nested] 12+ messages in thread

* Re: [ELPA] New package devicetree-ts-mode
  2024-01-16  6:18 ` Philip Kaludercic
@ 2024-01-16  6:42   ` Aleksandr Vityazev
  2024-01-16  7:30     ` Philip Kaludercic
  0 siblings, 1 reply; 12+ messages in thread
From: Aleksandr Vityazev @ 2024-01-16  6:42 UTC (permalink / raw)
  To: Philip Kaludercic; +Cc: emacs-devel, Stefan Kangas

On 2024-01-16 06:18, Philip Kaludercic wrote:

> Aleksandr Vityazev <avityazev@disroot.org> writes:
>
>> Hello,
>>
>> Since the previous thread [1] got lost over time and no more replies,
>> I'm starting a new one.
>
> Sorry about that, I hadn't gotten around to answering your last ping.
>
>> All of Philip's suggested points have been fixed, is there any reason
>> why this package should not be included in ELPA? I am willing to listen
>> and fix if possible.
>
> I think code-wise everything should be fine.  Just to be sure, you have
> signed the FSF CA and are familiar with how the ELPA build server works
> (at least from a package developer perspective)?
>

Yes, I have a signed CA FSF. One of my packages is already in ELPA, so I'm more
or less familiar with the build process.

>> Link to repository - https://git.sr.ht/~akagi/devicetree-ts-mode/
>>
>> [1] https://mail.gnu.org/archive/html/emacs-devel/2023-12/msg00629.html

-- 
Best regards,
Aleksandr Vityazev



^ permalink raw reply	[flat|nested] 12+ messages in thread

* Re: [ELPA] New package devicetree-ts-mode
  2024-01-16  6:42   ` Aleksandr Vityazev
@ 2024-01-16  7:30     ` Philip Kaludercic
  2024-01-16  8:08       ` Aleksandr Vityazev
  0 siblings, 1 reply; 12+ messages in thread
From: Philip Kaludercic @ 2024-01-16  7:30 UTC (permalink / raw)
  To: Aleksandr Vityazev; +Cc: emacs-devel, Stefan Kangas

Aleksandr Vityazev <avityazev@disroot.org> writes:

> On 2024-01-16 06:18, Philip Kaludercic wrote:
>
>> Aleksandr Vityazev <avityazev@disroot.org> writes:
>>
>>> Hello,
>>>
>>> Since the previous thread [1] got lost over time and no more replies,
>>> I'm starting a new one.
>>
>> Sorry about that, I hadn't gotten around to answering your last ping.
>>
>>> All of Philip's suggested points have been fixed, is there any reason
>>> why this package should not be included in ELPA? I am willing to listen
>>> and fix if possible.
>>
>> I think code-wise everything should be fine.  Just to be sure, you have
>> signed the FSF CA and are familiar with how the ELPA build server works
>> (at least from a package developer perspective)?
>>
>
> Yes, I have a signed CA FSF. One of my packages is already in ELPA, so I'm more
> or less familiar with the build process.

OK, great, I have tried building the package and it works.  The package
has been added to the archive and should appear soon enough.

Another tip I can give you is that since you have a mailing list on
Soucehut, I recommend adding that is the maintainer address in the
package header.

Also, is it necessary to distribute both the README.md and README.org in
the package tarball?  I would assume the former is for Sourcehut, right?

>>> Link to repository - https://git.sr.ht/~akagi/devicetree-ts-mode/
>>>
>>> [1] https://mail.gnu.org/archive/html/emacs-devel/2023-12/msg00629.html



^ permalink raw reply	[flat|nested] 12+ messages in thread

* Re: [ELPA] New package devicetree-ts-mode
  2024-01-16  7:30     ` Philip Kaludercic
@ 2024-01-16  8:08       ` Aleksandr Vityazev
  0 siblings, 0 replies; 12+ messages in thread
From: Aleksandr Vityazev @ 2024-01-16  8:08 UTC (permalink / raw)
  To: Philip Kaludercic; +Cc: emacs-devel, Stefan Kangas

On 2024-01-16 07:30, Philip Kaludercic wrote:

> Aleksandr Vityazev <avityazev@disroot.org> writes:
>
>> On 2024-01-16 06:18, Philip Kaludercic wrote:
>>
>>> Aleksandr Vityazev <avityazev@disroot.org> writes:
>>>
>>>> Hello,
>>>>
>>>> Since the previous thread [1] got lost over time and no more replies,
>>>> I'm starting a new one.
>>>
>>> Sorry about that, I hadn't gotten around to answering your last ping.
>>>
>>>> All of Philip's suggested points have been fixed, is there any reason
>>>> why this package should not be included in ELPA? I am willing to listen
>>>> and fix if possible.
>>>
>>> I think code-wise everything should be fine.  Just to be sure, you have
>>> signed the FSF CA and are familiar with how the ELPA build server works
>>> (at least from a package developer perspective)?
>>>
>>
>> Yes, I have a signed CA FSF. One of my packages is already in ELPA, so I'm more
>> or less familiar with the build process.
>
> OK, great, I have tried building the package and it works.  The package
> has been added to the archive and should appear soon enough.
>
> Another tip I can give you is that since you have a mailing list on
> Soucehut, I recommend adding that is the maintainer address in the
> package header.

Thanks, I'll add the mailing list address to the package header.

>
> Also, is it necessary to distribute both the README.md and README.org in
> the package tarball?  I would assume the former is for Sourcehut,
> right?

Yes, README.md for Sourcehut. I prefer to write README in org format. It
is possible to do this by generating html from README.org and adding it
as a custom one. In this case, only README.org can be kept.  Maybe I'll
switch to this method later.

At the moment I can add one of the READMEs to elpaignore if necessary.


>>>> Link to repository - https://git.sr.ht/~akagi/devicetree-ts-mode/
>>>>
>>>> [1] https://mail.gnu.org/archive/html/emacs-devel/2023-12/msg00629.html


-- 
Best regards,
Aleksandr Vityazev



^ permalink raw reply	[flat|nested] 12+ messages in thread

* Re: [ELPA] New package devicetree-ts-mode
  2023-12-22 20:36   ` Aleksandr Vityazev
@ 2024-02-20  3:04     ` Björn Bidar
  0 siblings, 0 replies; 12+ messages in thread
From: Björn Bidar @ 2024-02-20  3:04 UTC (permalink / raw)
  To: Aleksandr Vityazev; +Cc: Stefan Kangas, emacs-devel

Aleksandr Vityazev <avityazew@gmail.com> writes:

> On 2023-12-22 08:02, Stefan Kangas wrote:
>
>> Aleksandr Vityazev <avityazew@gmail.com> writes:
>>
>>> I'd like to submit devicetree-ts-mode [1] to GNU ELPA.
>>> This is tree-sitter major mode for Devicetree [2] files.
>>
>> Is there any particular reason to add this to GNU ELPA and not to core?
>> Is it too niche?
>
> I can't say whether Devisetree is too niche, but I think 
> ELPA is a great place for this package. It will be easier to 
> maintain.

From my point of view DeviceTrees change to slow and the mode is so
simple that a separate package takes more effort that putting into core.
Although I think it would be still beneficial to put it there for older
Emacs versions



^ permalink raw reply	[flat|nested] 12+ messages in thread

end of thread, other threads:[~2024-02-20  3:04 UTC | newest]

Thread overview: 12+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2023-12-21 17:47 [ELPA] New package devicetree-ts-mode Aleksandr Vityazev
2023-12-22  7:36 ` Philip Kaludercic
2023-12-22 20:08   ` Aleksandr Vityazev
2024-01-07 13:27     ` Aleksandr Vityazev
2023-12-22 16:02 ` Stefan Kangas
2023-12-22 20:36   ` Aleksandr Vityazev
2024-02-20  3:04     ` Björn Bidar
  -- strict thread matches above, loose matches on Subject: below --
2024-01-16  0:38 Aleksandr Vityazev
2024-01-16  6:18 ` Philip Kaludercic
2024-01-16  6:42   ` Aleksandr Vityazev
2024-01-16  7:30     ` Philip Kaludercic
2024-01-16  8:08       ` Aleksandr Vityazev

Code repositories for project(s) associated with this public inbox

	https://git.savannah.gnu.org/cgit/emacs.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).