From: | Yugo NAGATA <nagata(at)sraoss(dot)co(dot)jp> |
---|---|
To: | Tatsuo Ishii <ishii(at)sraoss(dot)co(dot)jp> |
Cc: | masao(dot)fujii(at)oss(dot)nttdata(dot)com, alvherre(at)alvh(dot)no-ip(dot)org, pgsql-hackers(at)postgresql(dot)org |
Subject: | Re: make_ctags: use -I option to ignore pg_node_attr macro |
Date: | 2023-02-07 13:34:41 |
Message-ID: | 20230207223441.27126c53e002009fe4871276@sraoss.co.jp |
Views: | Raw Message | Whole Thread | Download mbox | Resend email |
Thread: | |
Lists: | pgsql-hackers |
On Tue, 07 Feb 2023 21:29:04 +0900 (JST)
Tatsuo Ishii <ishii(at)sraoss(dot)co(dot)jp> wrote:
> > Does is make sense to change make_etags as the attached patch does?
> > This allows make_etags to use etags if Exuberant-type ctags is not
> > available. This allows users to use make_etags if hey has either
> > Exuberant-type ctags or etags.
>
> The patch drops support for "-n" option :-<
>
> Attached is the patch by fixing make_ctags (make_etags is not
> touched).
>
> If Exuberant-type ctags is available, use it (not changed).
> If Exuberant-type ctags is not available, try old ctags (not changed).
> If the old ctags does not support "-e" option, try etags (new).
I am not sure if this is good way to check if ctags supports "-e" or not.
+ then ctags --version 2>&1 | grep -- -e >/dev/null
Perhaps, "--help" might be intended rather than "--version" to check
supported options? Even so, ctags would have other option whose name contains
"-e" than Emacs support, so this check could end in a wrong result. Therefore,
it seems to me that it is better to check immediately if etags is available
in case that we don't have Exuberant-type ctags.
Regards,
Yugo Nagata
> If etags is not available, give up (new).
>
> Best reagards,
> --
> Tatsuo Ishii
> SRA OSS LLC
> English: http://www.sraoss.co.jp/index_en/
> Japanese:http://www.sraoss.co.jp
--
Yugo NAGATA <nagata(at)sraoss(dot)co(dot)jp>
From | Date | Subject | |
---|---|---|---|
Next Message | Aleksander Alekseev | 2023-02-07 13:39:45 | Re: [PATCH] Compression dictionaries for JSONB |
Previous Message | Andrew Dunstan | 2023-02-07 13:33:33 | Re: run pgindent on a regular basis / scripted manner |