Horton
Menu

Thread

  1. pgsql: Add PGFILEDESC description to Makefiles for all /contrib

    momjian@postgresql.org — 2010-05-12T11:33:10Z

    Log Message:
-----------
Add PGFILEDESC description to Makefiles for all /contrib executables.

Add PGAPPICON to all executable makefiles.

Modified Files:
--------------
    pgsql/contrib/oid2name:
        Makefile (r1.10 -> r1.11)
        (http://anoncvs.postgresql.org/cvsweb.cgi/pgsql/contrib/oid2name/Makefile?r1=1.10&r2=1.11)
    pgsql/contrib/pg_standby:
        Makefile (r1.5 -> r1.6)
        (http://anoncvs.postgresql.org/cvsweb.cgi/pgsql/contrib/pg_standby/Makefile?r1=1.5&r2=1.6)
    pgsql/contrib/pg_upgrade:
        Makefile (r1.1 -> r1.2)
        (http://anoncvs.postgresql.org/cvsweb.cgi/pgsql/contrib/pg_upgrade/Makefile?r1=1.1&r2=1.2)
    pgsql/contrib/pgbench:
        Makefile (r1.17 -> r1.18)
        (http://anoncvs.postgresql.org/cvsweb.cgi/pgsql/contrib/pgbench/Makefile?r1=1.17&r2=1.18)
    pgsql/contrib/vacuumlo:
        Makefile (r1.17 -> r1.18)
        (http://anoncvs.postgresql.org/cvsweb.cgi/pgsql/contrib/vacuumlo/Makefile?r1=1.17&r2=1.18)
    pgsql/src/backend:
        Makefile (r1.141 -> r1.142)
        (http://anoncvs.postgresql.org/cvsweb.cgi/pgsql/src/backend/Makefile?r1=1.141&r2=1.142)
    pgsql/src/bin/initdb:
        Makefile (r1.59 -> r1.60)
        (http://anoncvs.postgresql.org/cvsweb.cgi/pgsql/src/bin/initdb/Makefile?r1=1.59&r2=1.60)
    pgsql/src/bin/pg_config:
        Makefile (r1.23 -> r1.24)
        (http://anoncvs.postgresql.org/cvsweb.cgi/pgsql/src/bin/pg_config/Makefile?r1=1.23&r2=1.24)
    pgsql/src/bin/pg_controldata:
        Makefile (r1.20 -> r1.21)
        (http://anoncvs.postgresql.org/cvsweb.cgi/pgsql/src/bin/pg_controldata/Makefile?r1=1.20&r2=1.21)
    pgsql/src/bin/pg_ctl:
        Makefile (r1.28 -> r1.29)
        (http://anoncvs.postgresql.org/cvsweb.cgi/pgsql/src/bin/pg_ctl/Makefile?r1=1.28&r2=1.29)
    pgsql/src/bin/pg_dump:
        Makefile (r1.69 -> r1.70)
        (http://anoncvs.postgresql.org/cvsweb.cgi/pgsql/src/bin/pg_dump/Makefile?r1=1.69&r2=1.70)
    pgsql/src/bin/pg_resetxlog:
        Makefile (r1.22 -> r1.23)
        (http://anoncvs.postgresql.org/cvsweb.cgi/pgsql/src/bin/pg_resetxlog/Makefile?r1=1.22&r2=1.23)
    pgsql/src/bin/pgevent:
        Makefile (r1.14 -> r1.15)
        (http://anoncvs.postgresql.org/cvsweb.cgi/pgsql/src/bin/pgevent/Makefile?r1=1.14&r2=1.15)
    pgsql/src/bin/psql:
        Makefile (r1.68 -> r1.69)
        (http://anoncvs.postgresql.org/cvsweb.cgi/pgsql/src/bin/psql/Makefile?r1=1.68&r2=1.69)
    pgsql/src/bin/scripts:
        Makefile (r1.44 -> r1.45)
        (http://anoncvs.postgresql.org/cvsweb.cgi/pgsql/src/bin/scripts/Makefile?r1=1.44&r2=1.45)
    pgsql/src/interfaces/ecpg/preproc:
        Makefile (r1.148 -> r1.149)
        (http://anoncvs.postgresql.org/cvsweb.cgi/pgsql/src/interfaces/ecpg/preproc/Makefile?r1=1.148&r2=1.149)

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

    Tom Lane <tgl@sss.pgh.pa.us> — 2010-05-12T15:52:01Z

    momjian@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

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

    Bruce Momjian <bruce@momjian.us> — 2010-05-12T17:38:50Z

    Tom Lane wrote:
> momjian@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.

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

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

    Tom Lane <tgl@sss.pgh.pa.us> — 2010-05-12T17:44:07Z

    Bruce Momjian <bruce@momjian.us> writes:
> Tom Lane wrote:
>> momjian@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

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

    Bruce Momjian <bruce@momjian.us> — 2010-05-12T19:52:26Z

    Tom Lane wrote:
> Bruce Momjian <bruce@momjian.us> writes:
> > Tom Lane wrote:
> >> momjian@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.

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?  I can easily do it.

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

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

    Tom Lane <tgl@sss.pgh.pa.us> — 2010-05-12T19:59:40Z

    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.

			regards, tom lane

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

    Bruce Momjian <bruce@momjian.us> — 2010-05-12T21:42:47Z

    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.

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