| From: | Andrew Dunstan <andrew(at)dunslane(dot)net> |
|---|---|
| To: | Dagfinn Ilmari Mannsåker <ilmari(at)ilmari(dot)org> |
| Cc: | Michael Paquier <michael(at)paquier(dot)xyz>, Alvaro Herrera <alvherre(at)alvh(dot)no-ip(dot)org>, Peter Eisentraut <peter(dot)eisentraut(at)enterprisedb(dot)com>, PostgreSQL-development <pgsql-hackers(at)postgresql(dot)org> |
| Subject: | Re: multi-install PostgresNode |
| Date: | 2021-03-24 12:11:41 |
| Message-ID: | b6fc7a5a-cd6d-5260-94ba-c117549e4d73@dunslane.net |
| Views: | Raw Message | Whole Thread | Download mbox | Resend email |
| Thread: | |
| Lists: | pgsql-hackers |
On 3/24/21 7:54 AM, Dagfinn Ilmari Mannsåker wrote:
> Andrew Dunstan <andrew(at)dunslane(dot)net> writes:
>
>> And here it is. No subclass, no eval, no magic :-) Some of my colleagues
>> are a lot happier ;-)
>>
>> The downside is that we need to litter PostgresNode with a bunch of
>> lines like:
>>
>> local %ENV = %ENV;
>> _set_install_env($self);
> I think it would be even neater having a method that returns the desired
> environment and then have the other methods do:
>
> local %ENV = $self->_get_install_env();
Yeah, that's nice. I'll do that. Thanks.
cheers
andrew
--
Andrew Dunstan
EDB: https://www.enterprisedb.com
| From | Date | Subject | |
|---|---|---|---|
| Next Message | Bruce Momjian | 2021-03-24 12:13:40 | Re: Feature improvement: can we add queryId for pg_catalog.pg_stat_activity view? |
| Previous Message | Dagfinn Ilmari Mannsåker | 2021-03-24 11:54:03 | Re: multi-install PostgresNode |