Re: [PATCH] Issue 443 : post-commit hook (error) output lost: Step 6 V2

[Madan U Sreenivasan <ma...@collab.net>]

On Wed, 2005-11-02 at 05:10, Garrett Rooney wrote:
> On 10/31/05, Madan U Sreenivasan <ma...@collab.net> wrote:
> 
> In the change to subversion/libsvn_repos/commit.c:
[snip]
> +      if (err->child->message)
> +        post_commit_err = apr_pstrdup (pool, err->child->message) ;
> +
> 
> Is there any chance that err->child could be NULL?
If the post-commit hook errors out, but doesnt have any info in
stderr...
> Also, this file doesn't appear in the log message.
oh, yes. I just noticed. I must have it in the earlier copies of the
log(yes, I have multiple copies of the log as I go ;) )...will fix this.
Thanks.

Regards,
Madan.


---------------------------------------------------------------------
To unsubscribe, e-mail: dev-unsubscribe@subversion.tigris.org
For additional commands, e-mail: dev-help@subversion.tigris.org

Re: [PATCH] Issue 443 : post-commit hook (error) output lost: Step 6 V2

[Madan U Sreenivasan <ma...@collab.net>]

On Thu, 2005-11-03 at 03:01, Madan U Sreenivasan wrote:
> On Wed, 2005-11-02 at 05:10, Garrett Rooney wrote:
> > On 10/31/05, Madan U Sreenivasan <ma...@collab.net> wrote:
> > 
> > In the change to subversion/libsvn_repos/commit.c:
> [snip]
> > +      if (err->child->message)
> > +        post_commit_err = apr_pstrdup (pool, err->child->message) ;
> > +
> > 
> > Is there any chance that err->child could be NULL?
> If the post-commit hook errors out, but doesnt have any info in
> stderr...
> > Also, this file doesn't appear in the log message.
> oh, yes. I just noticed. I must have it in the earlier copies of the
> log(yes, I have multiple copies of the log as I go ;) )...will fix this.
> Thanks.
here they come.
> 
> Regards,
> Madan.
> 
> 
> ---------------------------------------------------------------------
> To unsubscribe, e-mail: dev-unsubscribe@subversion.tigris.org
> For additional commands, e-mail: dev-help@subversion.tigris.org