From: | Fabien COELHO <coelho(at)cri(dot)ensmp(dot)fr> |
---|---|
To: | Alvaro Herrera <alvherre(at)dcc(dot)uchile(dot)cl> |
Cc: | Tom Lane <tgl(at)sss(dot)pgh(dot)pa(dot)us>, Peter Eisentraut <peter_e(at)gmx(dot)net>, PostgreSQL Patches <pgsql-patches(at)postgresql(dot)org>, PostgreSQL Developers <pgsql-hackers(at)postgresql(dot)org> |
Subject: | Re: pgxs default installation + various fixes |
Date: | 2004-09-03 15:28:36 |
Message-ID: | Pine.LNX.4.58.0409031707200.28337@sablons.cri.ensmp.fr |
Views: | Raw Message | Whole Thread | Download mbox | Resend email |
Thread: | |
Lists: | pgsql-hackers pgsql-patches |
Dear Alvaro,
> > As I'm into these files, I can say that one of the reason for that is that
> > the shell scripts in the makefile looks inefficient, with nested for-loops
> > and one-at-a-time config/install-sh forked-script copies for 350 header
> > files, on the 971 files of a standard installation.
>
> Also the install-sh script apparently is way more complex than it needs
> to be.
Maybe. I guess the options are there because they might be useful
sometimes?
> There's probably a lot of that complexity (and subsequent slowness) that
> install-all-headers doesn't need.
Maybe. I don't have a clear view about portability issues that I guess
justify this script.
> A lot of time goes into processing the script itself rather than doing
> useful work.
Yes.
> Is there an objection to trying to convert it to a simpler, faster
> alternative? Maybe even one that receives multiple files as arguments,
> which would reduce the number of times it is called by an order of
> magnitude.
Yes, handling several files at a time could indeed improve the stuff.
But this means changing the syntax somehow, and fixing makefiles...
Also, most unix box have an "install" program which might be more
efficient and which handles several files. I do not know whether it
has all the required facilities.
For instance, apache looks for a "bsd" install program at configuration
time... and a slow but compatible shell substitute is used instead if none
is available. Maybe this can be reused quite simply by postgresql, with
their kind permission. As apache is quite portable, it might be good
enough for pg.
This seems a reasonnable todo objective, but I'm not sure it should be
done for 8.0 as it changes the installation procedure significantly? Well,
maybe it could be done quite quiclky. It does not look too difficult to
implement with apache example at hand.
--
Fabien Coelho - coelho(at)cri(dot)ensmp(dot)fr
From | Date | Subject | |
---|---|---|---|
Next Message | Alvaro Herrera | 2004-09-03 16:06:07 | Bug in numeric modulo operation remains |
Previous Message | Alvaro Herrera | 2004-09-03 15:05:38 | Re: pgxs default installation + various fixes |
From | Date | Subject | |
---|---|---|---|
Next Message | Tom Lane | 2004-09-03 16:07:53 | Re: pgxs default installation + various fixes |
Previous Message | Alvaro Herrera | 2004-09-03 15:05:38 | Re: pgxs default installation + various fixes |