Re: garbage variable in GNUmakefile.in

From: Tom Lane <tgl(at)sss(dot)pgh(dot)pa(dot)us>
To: Daniel Gustafsson <daniel(at)yesql(dot)se>
Cc: PostgreSQL-development <pgsql-hackers(at)postgresql(dot)org>, Peter Eisentraut <peter(dot)eisentraut(at)2ndquadrant(dot)com>
Subject: Re: garbage variable in GNUmakefile.in
Date: 2018-08-07 14:06:28
Message-ID: 25928.1533650788@sss.pgh.pa.us
Views: Raw Message | Whole Thread | Download mbox | Resend email
Thread:
Lists: pgsql-hackers

Daniel Gustafsson <daniel(at)yesql(dot)se> writes:
> I happened to notice that there is a variable 'garbage' in GNUmakefile.in,
> which was defined in 32163099d7c43a0244f463eb4e525c711e6e97a3 ~18 years ago,
> but which seems to not be used? Is 'garbage' a special make variable or is a
> leftover which can be removed?

It's not a special make variable, so far as I can find in the gmake manual.

It looks like the idea was to list filename patterns that could be subject
to automatic removal during "make clean", but Peter backed off listing it
in the clean target and then forgot to remove the variable.

I couldn't find any contemporaneous discussion in the archives, but I do
vaguely recall that at some point --- maybe not just then --- we talked
about having such auto removal and decided it was a bad idea. Editor
backup filenames, for instance, vary depending on your weapon of choice;
and core file names vary too, these days. So there's a project policy
that we wouldn't enable any such removal by the makefiles. (These days
people would probably turn to git commands for such cleanup, anyway.)

In short, yeah, I think we could nuke that. But maybe Peter remembers
it differently.

regards, tom lane

In response to

Responses

Browse pgsql-hackers by date

  From Date Subject
Next Message Don Seiler 2018-08-07 14:09:13 Re: [PATCH] Include application_name in "connection authorized" log message
Previous Message Tom Lane 2018-08-07 13:59:09 Re: [PATCH] Include application_name in "connection authorized" log message