Re: [COMMITTERS] pgsql: Add PGFILEDESC description to Makefiles for all /contrib

Поиск
Список
Период
Сортировка
От Bruce Momjian
Тема Re: [COMMITTERS] pgsql: Add PGFILEDESC description to Makefiles for all /contrib
Дата
Msg-id 201005122347.o4CNl3Q10985@momjian.us
обсуждение исходный текст
Ответы Re: Re: [COMMITTERS] pgsql: Add PGFILEDESC description to Makefiles for all /contrib  (Magnus Hagander <magnus@hagander.net>)
Список pgsql-hackers
bruce wrote:
> Tom Lane wrote:
> > Bruce Momjian <bruce@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.
>
> OK, done with attached patch, and I added a comment about why it is not
> labeled.

I did some research on PGFILEDESC and it does what I thought it does ---
in embeds the 'ico' file into the executable in
/pg/tools/msvc/Project.pm, and the image looks like the attached JPEG.
The image is of a blue elephant head.

So, currently, every binary uses that icon, except for the postmaster.
Is that what we want?  You could make the argument that a daemon, like
the postmaster, shouldn't have one, which I think is Tom's point.

--
  Bruce Momjian  <bruce@momjian.us>        http://momjian.us
  EnterpriseDB                             http://enterprisedb.com

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

Предыдущее
От: "David E. Wheeler"
Дата:
Сообщение: Re: pg_upgrade versus MSVC build scripts
Следующее
От: Bruce Momjian
Дата:
Сообщение: Re: pg_upgrade versus MSVC build scripts