emacs-devel
[Top][All Lists]
Advanced

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

Re: [PATCH] Re: etags name collision.


From: Ergus
Subject: Re: [PATCH] Re: etags name collision.
Date: Mon, 11 Apr 2022 21:19:33 +0200

On Mon, Apr 11, 2022 at 07:40:29PM +0300, Eli Zaretskii wrote:
Date: Mon, 11 Apr 2022 18:19:42 +0200
From: Ergus <spacibba@aol.com>
Cc: emacs-devel@gnu.org

On Mon, Apr 11, 2022 at 06:51:58PM +0300, Eli Zaretskii wrote:

>The code should test whether another version of 'ctags' is already
>installed, and if so, that it isn't our 'ctags'.  _Then_ we could
>default to not installing our 'ctags'.  Your proposal doesn't make
>that test, so please add it.
>
>Thanks.
>
I do this tests like with mailutils and movemail:

(ctags --version) >/dev/null 2>&1 || with_ctags=no

Isn't this enough?

No, because AFAIU this test will succeed also if the installed ctags
is (an older version) of the program that came with (an older version)
of Emacs.  You need to make sure the text emitted by --version does
NOT include "GNU Emacs".

Now?

Attachment: ctags.patch
Description: Text document


reply via email to

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