From: | Mike Fowler <mike(at)mlfowler(dot)com> |
---|---|
To: | Andrew Dunstan <andrew(at)dunslane(dot)net> |
Cc: | Alvaro Herrera <alvherre(at)commandprompt(dot)com>, pgsql-jdbc(at)postgresql(dot)org |
Subject: | Re: JDBC specific Buildfarm member |
Date: | 2011-05-05 05:49:24 |
Message-ID: | 4DC23A64.9050908@mlfowler.com |
Views: | Raw Message | Whole Thread | Download mbox | Resend email |
Thread: | |
Lists: | pgsql-jdbc |
On 04/05/11 23:17, Andrew Dunstan wrote:
>
>
> This looks pretty good.
>
> You should probably fill in $steps_completed. You'd need to change its
> declaration so it was visible - change it so the declarations is :
>
> use vars qw($steps_completed);
>
> As a matter of style, I stopped using globs for file handles long ago,
> in favor of local variables. It would be best to keep to that style for
> consistency (although you've done a pretty good job on that front, I
> must say.)
>
> Go ahead and enable it and let's see the results on the server.
>
> Maybe I should factor out the core tests in a similar fashion.
>
> Nice work.
>
> cheers
>
> andrew
>
>
>
Thanks, I've switched it on and 8.4 has been building for the last
little while so we should see some results in a hour or so. I'll try and
make your suggested changes and send a revised patch over the weekend.
On the subject of refactoring, at some point it might be in idea to
change the SCM module so that it's project agnostic. Once that's done I
can drop the CVS handling in the JDBC module.
Regards,
--
Mike Fowler
Registered Linux user: 379787
From | Date | Subject | |
---|---|---|---|
Next Message | Andrew Dunstan | 2011-05-05 15:26:25 | Re: JDBC specific Buildfarm member |
Previous Message | Johnny Luong | 2011-05-05 02:31:55 | jdbc driver support for prepared statement executeBatch() |