Re: mkid reference

From: Magnus Hagander <magnus(at)hagander(dot)net>
To: Tom Lane <tgl(at)sss(dot)pgh(dot)pa(dot)us>
Cc: Daniel Gustafsson <daniel(at)yesql(dot)se>, Bruce Momjian <bruce(at)momjian(dot)us>, PostgreSQL Developers <pgsql-hackers(at)lists(dot)postgresql(dot)org>
Subject: Re: mkid reference
Date: 2021-01-24 13:20:58
Message-ID: CABUevEyCtbB3migDB59nEXFq0v9Rakx77UcSCYMOsp5RmtwYQA@mail.gmail.com
Views: Raw Message | Whole Thread | Download mbox | Resend email
Thread:
Lists: pgsql-hackers

On Fri, Jan 22, 2021 at 7:07 PM Tom Lane <tgl(at)sss(dot)pgh(dot)pa(dot)us> wrote:
>
> Daniel Gustafsson <daniel(at)yesql(dot)se> writes:
> >> On 22 Jan 2021, at 12:56, Magnus Hagander <magnus(at)hagander(dot)net> wrote:
> >> And maybe even more interestnig -- is there a point to this whole
> >> make_diff directory at all in these days of git? Or should we just
> >> remove it rather than try to fix it?
>
> > There's also src/tools/make_mkid which use this mkid tool. +1 for removing.
> > If anything, it seems better replaced by extended documentation on the existing
> > wiki article [0] on how to use "git format-patch".
>
> I found man pages for mkid online --- it's apparently a ctags-like
> code indexing tool, not something for patches. So maybe Bruce still
> uses it, or maybe not. But as long as we've also got make_ctags and
> make_etags in there, I don't have a problem with leaving make_mkid.
>
> make_diff, on the other hand, certainly looks like technology whose
> time has passed. I wonder about pgtest, too.

I'll go kill make_diff then -- quicker than fixing the docs of it.

As for pgtest, that one looks a bit interesting as well -- but it's
been patched on as late as 9.5 and in 2018, so it seems at least Bruce
uses it :)

While at it, what point is "codelines" adding?

--
Magnus Hagander
Me: https://www.hagander.net/
Work: https://www.redpill-linpro.com/

In response to

Responses

Browse pgsql-hackers by date

  From Date Subject
Next Message Alvaro Herrera 2021-01-24 16:01:33 Re: Faulty HEAP_XMAX_LOCK_ONLY & HEAP_KEYS_UPDATED hintbit combination
Previous Message Amit Langote 2021-01-24 12:31:42 Re: POC: postgres_fdw insert batching