From: "Daniel Martín" <mardani29@yahoo.es>
To: Ergus <spacibba@aol.com>
Cc: emacs-devel@gnu.org
Subject: Re: Yet another global/gtags package into elpa..
Date: Sat, 02 Apr 2022 02:47:44 +0200 [thread overview]
Message-ID: <m1mth4nwen.fsf@yahoo.es> (raw)
In-Reply-To: <20220328232754.feeavepjtqauvnv5@Ergus> (Ergus's message of "Tue, 29 Mar 2022 01:27:54 +0200")
Ergus <spacibba@aol.com> writes:
> Hi:
>
> After many years dealing with issues (specially with tramp) with all the
> global/gtags packages around (agtags, gxref, counsel-gtags, ggtags and
> global-tags) I definitively implemented my own simpler one only with
> emacs internal infrastructure, and I am wondering if it may be fine to
> add it to elpa. The package could be even added to vanilla due to it's
> simplicity and because fully integrates into the emacs ecosystem.
>
Thanks, I think it's a good idea that Emacs has first class support for
GNU Global. I have some suggestions for your package:
;; Package-Requires: ((emacs "28"))
Is there a reason the package requires such a recent version of Emacs?
If it's only because you use string-lines, that logic can be implemented
in terms of split-string, which is available since Emacs 20.1, at least.
;;; Commentary:
;; GNU Global integration with xref, project and imenu.
It'd be nice to add a ;;; Usage section that describes how to make the
package work. You can reuse some of the information you have in the
Readme.md file. For example, you should advertise the gtags-mode-create
command, which generates the tags files for the project.
(defcustom gtags-mode-lighter "Gtags"
"Gtags executable."
:type 'string
:risky t)
To make things look good in the modeline it's better to add a space
before the lighter, so it should be " Gtags". Also, the docstring seems
incorrect.
(add-hook 'after-save-hook #'gtags-mode--after-save-hook nil t)
Could this be optional/configurable? I think some people would prefer to
update the GTAGS database manually via a command and not each time the
file is saved.
(when (called-interactively-p 'all)
(message "Couldn't enable gtags-mode. Not root found."))
^^^^^^^
Should be (error ...
This error message is also a bit confusing. I think it should say
something like "No tags file found" or similar.
Thanks.
prev parent reply other threads:[~2022-04-02 0:47 UTC|newest]
Thread overview: 9+ messages / expand[flat|nested] mbox.gz Atom feed top
[not found] <20220328232754.feeavepjtqauvnv5.ref@Ergus>
2022-03-28 23:27 ` Yet another global/gtags package into elpa Ergus
2022-03-29 8:21 ` Michael Albinus
2022-03-29 9:56 ` Ergus
2022-03-29 10:08 ` Michael Albinus
2022-03-29 12:16 ` Dmitry Gutov
2022-03-29 16:07 ` Ergus
2022-03-29 16:16 ` Dmitry Gutov
2022-03-29 17:15 ` Ergus
2022-04-02 0:47 ` Daniel Martín [this message]
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=m1mth4nwen.fsf@yahoo.es \
--to=mardani29@yahoo.es \
--cc=emacs-devel@gnu.org \
--cc=spacibba@aol.com \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
Code repositories for project(s) associated with this external index
https://git.savannah.gnu.org/cgit/emacs.git
https://git.savannah.gnu.org/cgit/emacs/org-mode.git
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.