Re: Splitting up guc.c

Поиск
Список
Период
Сортировка
От Andres Freund
Тема Re: Splitting up guc.c
Дата
Msg-id 20220912205024.6bow7zzl7tin35ul@awork3.anarazel.de
обсуждение исходный текст
Ответ на Re: Splitting up guc.c  (Dagfinn Ilmari Mannsåker <ilmari@ilmari.org>)
Список pgsql-hackers
Hi,

On 2022-09-12 21:12:03 +0100, Dagfinn Ilmari Mannsåker wrote:
> Tom Lane <tgl@sss.pgh.pa.us> writes:
> 
> >> I think this is localized enough that asking people to manually resolve a
> >> conflict around adding a GUC entry wouldn't be asking for that much. And I
> >> think plenty changes might be automatically resolvable, despite the rename.
> >
> > I wonder whether git will be able to figure out that this is mostly a
> > code move.  I would expect so for a straight file rename, but will that
> > work when we're splitting the file 3 ways?
> 
> Git can detect more complicated code movement (see the `--color-moved`
> option to `git diff`), but I'm not sure it's clever enough to realise
> that a change modifying a block of code that was moved in the meanwhile
> should be applied at the new destination.

It sometimes can for large code movements, but not in this case. I think
because guc.c is more self-similar than guc_tables.c.

Greetings,

Andres Freund



В списке pgsql-hackers по дате отправления:

Предыдущее
От: Andres Freund
Дата:
Сообщение: Re: Splitting up guc.c
Следующее
От: "Joel Mariadasan (jomariad)"
Дата:
Сообщение: Permissions denied for the database file system on Windows during restore