unofficial mirror of emacs-devel@gnu.org 
 help / color / mirror / code / Atom feed
* Missing entries for Perl in etags-regen-file-extensions -- okay for emacs-30?
@ 2024-09-20  9:15 Sean Whitton
  2024-09-20 10:34 ` Eli Zaretskii
  0 siblings, 1 reply; 4+ messages in thread
From: Sean Whitton @ 2024-09-20  9:15 UTC (permalink / raw)
  To: emacs-devel

Hello,

Can I install this simple change on the release branch, please?

-- >8 --
---
 lisp/progmodes/etags-regen.el | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/lisp/progmodes/etags-regen.el b/lisp/progmodes/etags-regen.el
index 21ea9bfb8b3..e97b0cd7574 100644
--- a/lisp/progmodes/etags-regen.el
+++ b/lisp/progmodes/etags-regen.el
@@ -119,7 +119,7 @@ etags-regen-regexp-alist
 ;; http://lists.gnu.org/archive/html/emacs-devel/2018-01/msg00323.html
 (defcustom etags-regen-file-extensions
   '("rb" "js" "py" "pl" "el" "c" "cpp" "cc" "h" "hh" "hpp"
-    "java" "go" "cl" "lisp" "prolog" "php" "erl" "hrl"
+    "java" "go" "cl" "lisp" "prolog" "php" "erl" "hrl" "pl" "pm"
     "F" "f" "f90" "for" "cs" "a" "asm" "ads" "adb" "ada")
   "Code file extensions for `etags-regen-mode'.
-- 
Sean Whitton



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

* Re: Missing entries for Perl in etags-regen-file-extensions -- okay for emacs-30?
  2024-09-20  9:15 Missing entries for Perl in etags-regen-file-extensions -- okay for emacs-30? Sean Whitton
@ 2024-09-20 10:34 ` Eli Zaretskii
  2024-09-20 13:39   ` Sean Whitton
  0 siblings, 1 reply; 4+ messages in thread
From: Eli Zaretskii @ 2024-09-20 10:34 UTC (permalink / raw)
  To: Sean Whitton, Dmitry Gutov; +Cc: emacs-devel

> From: Sean Whitton <spwhitton@spwhitton.name>
> Date: Fri, 20 Sep 2024 10:15:50 +0100
> 
> Hello,
> 
> Can I install this simple change on the release branch, please?
> 
> -- >8 --
> ---
>  lisp/progmodes/etags-regen.el | 2 +-
>  1 file changed, 1 insertion(+), 1 deletion(-)
> 
> diff --git a/lisp/progmodes/etags-regen.el b/lisp/progmodes/etags-regen.el
> index 21ea9bfb8b3..e97b0cd7574 100644
> --- a/lisp/progmodes/etags-regen.el
> +++ b/lisp/progmodes/etags-regen.el
> @@ -119,7 +119,7 @@ etags-regen-regexp-alist
>  ;; http://lists.gnu.org/archive/html/emacs-devel/2018-01/msg00323.html
>  (defcustom etags-regen-file-extensions
>    '("rb" "js" "py" "pl" "el" "c" "cpp" "cc" "h" "hh" "hpp"
> -    "java" "go" "cl" "lisp" "prolog" "php" "erl" "hrl"
> +    "java" "go" "cl" "lisp" "prolog" "php" "erl" "hrl" "pl" "pm"
>      "F" "f" "f90" "for" "cs" "a" "asm" "ads" "adb" "ada")
>    "Code file extensions for `etags-regen-mode'.

We already have "pl" in the list.  Adding "pm" is okay, but then why
only it? why not add all the extensions that 'etags' supports?

Dmitry, why do we need an explicit list instead of asking 'etags'
which extensions it supports?



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

* Re: Missing entries for Perl in etags-regen-file-extensions -- okay for emacs-30?
  2024-09-20 10:34 ` Eli Zaretskii
@ 2024-09-20 13:39   ` Sean Whitton
  2024-09-20 13:41     ` Eli Zaretskii
  0 siblings, 1 reply; 4+ messages in thread
From: Sean Whitton @ 2024-09-20 13:39 UTC (permalink / raw)
  To: Eli Zaretskii; +Cc: Dmitry Gutov, emacs-devel

Hello,

On Fri 20 Sep 2024 at 01:34pm +03, Eli Zaretskii wrote:

>> From: Sean Whitton <spwhitton@spwhitton.name>
>> Date: Fri, 20 Sep 2024 10:15:50 +0100
>>
>> Hello,
>>
>> Can I install this simple change on the release branch, please?
>>
>> -- >8 --
>> ---
>>  lisp/progmodes/etags-regen.el | 2 +-
>>  1 file changed, 1 insertion(+), 1 deletion(-)
>>
>> diff --git a/lisp/progmodes/etags-regen.el b/lisp/progmodes/etags-regen.el
>> index 21ea9bfb8b3..e97b0cd7574 100644
>> --- a/lisp/progmodes/etags-regen.el
>> +++ b/lisp/progmodes/etags-regen.el
>> @@ -119,7 +119,7 @@ etags-regen-regexp-alist
>>  ;; http://lists.gnu.org/archive/html/emacs-devel/2018-01/msg00323.html
>>  (defcustom etags-regen-file-extensions
>>    '("rb" "js" "py" "pl" "el" "c" "cpp" "cc" "h" "hh" "hpp"
>> -    "java" "go" "cl" "lisp" "prolog" "php" "erl" "hrl"
>> +    "java" "go" "cl" "lisp" "prolog" "php" "erl" "hrl" "pl" "pm"
>>      "F" "f" "f90" "for" "cs" "a" "asm" "ads" "adb" "ada")
>>    "Code file extensions for `etags-regen-mode'.
>
> We already have "pl" in the list.  Adding "pm" is okay, but then why
> only it? why not add all the extensions that 'etags' supports?

Okay thanks, I'll add "pm" so we don't have just one Perl extension, and
hopefully we can figure out a better solution for master.

-- 
Sean Whitton



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

* Re: Missing entries for Perl in etags-regen-file-extensions -- okay for emacs-30?
  2024-09-20 13:39   ` Sean Whitton
@ 2024-09-20 13:41     ` Eli Zaretskii
  0 siblings, 0 replies; 4+ messages in thread
From: Eli Zaretskii @ 2024-09-20 13:41 UTC (permalink / raw)
  To: Sean Whitton; +Cc: dmitry, emacs-devel

> From: Sean Whitton <spwhitton@spwhitton.name>
> Cc: Dmitry Gutov <dmitry@gutov.dev>,  emacs-devel@gnu.org
> Date: Fri, 20 Sep 2024 14:39:11 +0100
> 
> Hello,
> 
> On Fri 20 Sep 2024 at 01:34pm +03, Eli Zaretskii wrote:
> 
> >> From: Sean Whitton <spwhitton@spwhitton.name>
> >> Date: Fri, 20 Sep 2024 10:15:50 +0100
> >>
> >> Hello,
> >>
> >> Can I install this simple change on the release branch, please?
> >>
> >> -- >8 --
> >> ---
> >>  lisp/progmodes/etags-regen.el | 2 +-
> >>  1 file changed, 1 insertion(+), 1 deletion(-)
> >>
> >> diff --git a/lisp/progmodes/etags-regen.el b/lisp/progmodes/etags-regen.el
> >> index 21ea9bfb8b3..e97b0cd7574 100644
> >> --- a/lisp/progmodes/etags-regen.el
> >> +++ b/lisp/progmodes/etags-regen.el
> >> @@ -119,7 +119,7 @@ etags-regen-regexp-alist
> >>  ;; http://lists.gnu.org/archive/html/emacs-devel/2018-01/msg00323.html
> >>  (defcustom etags-regen-file-extensions
> >>    '("rb" "js" "py" "pl" "el" "c" "cpp" "cc" "h" "hh" "hpp"
> >> -    "java" "go" "cl" "lisp" "prolog" "php" "erl" "hrl"
> >> +    "java" "go" "cl" "lisp" "prolog" "php" "erl" "hrl" "pl" "pm"
> >>      "F" "f" "f90" "for" "cs" "a" "asm" "ads" "adb" "ada")
> >>    "Code file extensions for `etags-regen-mode'.
> >
> > We already have "pl" in the list.  Adding "pm" is okay, but then why
> > only it? why not add all the extensions that 'etags' supports?
> 
> Okay thanks, I'll add "pm" so we don't have just one Perl extension, and
> hopefully we can figure out a better solution for master.

My question about adding more extensions was for emacs-30.

Let's wait for Dmitry to chime in, and take it from there.



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

end of thread, other threads:[~2024-09-20 13:41 UTC | newest]

Thread overview: 4+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2024-09-20  9:15 Missing entries for Perl in etags-regen-file-extensions -- okay for emacs-30? Sean Whitton
2024-09-20 10:34 ` Eli Zaretskii
2024-09-20 13:39   ` Sean Whitton
2024-09-20 13:41     ` Eli Zaretskii

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).