From: Tom Lane on
momjian(a)postgresql.org (Bruce Momjian) writes:
> Add PGAPPICON to all executable makefiles.

Is it really a good idea to have done that to the server, in particular?
I can't imagine it being a good idea to launch the postmaster from a
GUI, which is what I suppose this is good for.

regards, tom lane

--
Sent via pgsql-hackers mailing list (pgsql-hackers(a)postgresql.org)
To make changes to your subscription:
http://www.postgresql.org/mailpref/pgsql-hackers

From: Tom Lane on
Bruce Momjian <bruce(a)momjian.us> writes:
> Tom Lane wrote:
>> momjian(a)postgresql.org (Bruce Momjian) writes:
>>> Add PGAPPICON to all executable makefiles.
>>
>> Is it really a good idea to have done that to the server, in particular?
>> I can't imagine it being a good idea to launch the postmaster from a
>> GUI, which is what I suppose this is good for.

> Well, I assume this was the "icon", which I assumed should match for all
> the binaries, as compared to config files, for example.

Er... why are you committing undiscussed patches if you just "assume"
they will work, or indeed do anything useful at all? You should not
have touched this if you don't know exactly what it will do and why
changing it is a good thing.

regards, tom lane

--
Sent via pgsql-hackers mailing list (pgsql-hackers(a)postgresql.org)
To make changes to your subscription:
http://www.postgresql.org/mailpref/pgsql-hackers

From: Tom Lane on
Bruce Momjian <bruce(a)momjian.us> writes:
> All other binaries had such a designation, and all /contrib binaries
> were missing them. I assume I was doing cleanup. You want the icon
> removed from the backend makefile?

Yes. I'm prepared to believe that not having the icons set on the
contrib executables was an oversight. I'm much less prepared to assume
that not marking the postgres executable was an oversight. Again,
unless you *know* that this change is needed and appropriate, now is
not the time to be making it, and especially not without discussion.

regards, tom lane

--
Sent via pgsql-hackers mailing list (pgsql-hackers(a)postgresql.org)
To make changes to your subscription:
http://www.postgresql.org/mailpref/pgsql-hackers