Re: dependencies for generated header files

Поиск
Список
Период
Сортировка
От Tom Lane
Тема Re: dependencies for generated header files
Дата
Msg-id 2619.1250042197@sss.pgh.pa.us
обсуждение исходный текст
Ответ на Re: dependencies for generated header files  (Robert Haas <robertmhaas@gmail.com>)
Ответы Re: dependencies for generated header files  (Robert Haas <robertmhaas@gmail.com>)
Список pgsql-hackers
Robert Haas <robertmhaas@gmail.com> writes:
> Given that the anum.h stuff is gone, "vastly" might be an
> overstatement.  I'm pretty surprised to find out that people don't
> like the idea of having dependencies be correct from anywhere in the
> tree.  Even if I'm the only developer who does partial builds, the
> cost seems to me to be next to nil, so I'm not quite sure what anyone
> gets out of rejecting this patch.

It's not that having the dependencies be 100% up to date wouldn't be
nice; it's that there's a limit to how much we're willing to uglify
the Makefiles to have that.  The makefiles need maintenance too,
you know, and putting things far away from where they should be is
not any better in the makefiles than it is in C code.

As far as I can tell, if you've used --enable-depend then things will
get updated properly before you can ever attempt to run the code
(ie, install a rebuilt postmaster).  The only situation where you'd
actually get an improvement from redoing the dependencies like this
is where lack of an update to a derived file results in a compiler
error/warning.  But there aren't many such cases.  The only one I can
even think of offhand is lack of an fmgroids.h symbol for a newly-added
function ... but we don't use F_XXX symbols enough to make that a
convincing example.  We've intentionally arranged things so that
more-fragile cases like gram.h are not referenced outside their own
directories.
        regards, tom lane


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

Предыдущее
От: Bruce Momjian
Дата:
Сообщение: Re: Re: pgindent timing (was Re: [COMMITTERS] pgsql: Refactor NUM_cache_remove calls in error report path to a PG_TRY)
Следующее
От: Tom Lane
Дата:
Сообщение: Re: WIP: getting rid of the pg_database flat file