HI

On Mon, Jul 13, 2015 at 1:59 PM, Sanket Mehta
<sanket.me...@enterprisedb.com> wrote:
> Hi ,
>
> I have attached the patch which is having both functionalities (-v for
> version info and timestamp in log)
>
> Here timestamp is added only for linux platform because on windows pgagent
> logs are viewed in event viewer which provides timestamp on its own.

OK

> Please do review it and let me know in case anything is missing.

Can we not get the version from a preprocessor macro? It's already in
CMakeLists.txt after all. If not, I think we at least need to update
the comment in CMakeLists.txt and other files as appropriate so we
don't forget to update the number anywhere.

Also, please change:

shows version then exit

to

display version info and then exit.

Thanks!

-- 
Dave Page
Blog: http://pgsnake.blogspot.com
Twitter: @pgsnake

EnterpriseDB UK: http://www.enterprisedb.com
The Enterprise PostgreSQL Company


-- 
Sent via pgadmin-support mailing list (pgadmin-support@postgresql.org)
To make changes to your subscription:
http://www.postgresql.org/mailpref/pgadmin-support

Reply via email to