Re: PgAgent Patch

From: Ashesh Vashi <ashesh(dot)vashi(at)enterprisedb(dot)com>
To: Dave Page <dpage(at)pgadmin(dot)org>
Cc: Mehmet Emin KARAKAŞ <emin100(at)gmail(dot)com>, pgadmin-hackers <pgadmin-hackers(at)postgresql(dot)org>
Subject: Re: PgAgent Patch
Date: 2015-03-04 05:11:36
Message-ID: CAG7mmoy-OL7oW8sCJP2OF6qxfNKzyfnuzRPwm8DhtMJtgLr54Q@mail.gmail.com
Views: Raw Message | Whole Thread | Download mbox | Resend email
Thread:
Lists: pgadmin-hackers

Sure

--

Thanks & Regards,

Ashesh Vashi
EnterpriseDB INDIA: Enterprise PostgreSQL Company
<http://www.enterprisedb.com>

*http://www.linkedin.com/in/asheshvashi*
<http://www.linkedin.com/in/asheshvashi>

On Tue, Mar 3, 2015 at 2:01 PM, Dave Page <dpage(at)pgadmin(dot)org> wrote:

> Ashesh; can you please get this reviewed and applied if it looks OK?
>
> Thanks.
>
> On Tue, Mar 3, 2015 at 12:00 PM, Mehmet Emin KARAKAŞ <emin100(at)gmail(dot)com>
> wrote:
> > Hi,
> > If batch cron is failed, pgagent doesn't receive error messages. We fix
> it.
> > We canalized the stderr to stdout. This patch batch cron receive error
> > massages from nix machine. Git Diff Message and job.cpp file is attached.
> >
> > Thanks.
> >
> > --
> > MEHMET EMİN KARAKAŞ
> > Postgresql Global Username: emin100
> >
> >
> >
> > diff --git a/job.cpp b/job.cpp
> > index 00334ab..694be22 100644
> > --- a/job.cpp
> > +++ b/job.cpp
> > @@ -273,8 +273,11 @@ int Job::Execute()
> > // The *nix way.
> > FILE *fp_script;
> > char buf[4098];
> > +
> > + wxString wxSt = wxT("");
> > + wxSt.Printf( _("%s 2>&1"), filename.c_str());
> >
> > - fp_script =
> > popen(filename.mb_str(wxConvUTF8), "r");
> > + fp_script =
> popen(wxSt.mb_str(wxConvUTF8) ,
> > "r");
> > if (!fp_script)
> > {
> > output.Printf(_("Couldn't execute
> > script: %s, errno = %d"), filename.c_str(), errno);
> >
> >
> >
> >
> >
> >
> >
> > --
> > Sent via pgadmin-hackers mailing list (pgadmin-hackers(at)postgresql(dot)org)
> > To make changes to your subscription:
> > http://www.postgresql.org/mailpref/pgadmin-hackers
> >
>
>
>
> --
> Dave Page
> Blog: http://pgsnake.blogspot.com
> Twitter: @pgsnake
>
> EnterpriseDB UK: http://www.enterprisedb.com
> The Enterprise PostgreSQL Company
>
>
> --
> Sent via pgadmin-hackers mailing list (pgadmin-hackers(at)postgresql(dot)org)
> To make changes to your subscription:
> http://www.postgresql.org/mailpref/pgadmin-hackers
>

In response to

Responses

Browse pgadmin-hackers by date

  From Date Subject
Next Message Ashesh Vashi 2015-03-04 05:12:01 Re: assert "!IsAttached()" failed in SetInvokingWindow()
Previous Message Ashesh Vashi 2015-03-04 05:05:53 Re: Patch : PGPASSFILE fix