Hi Ryan,
----- Original Message -----
> Updated patch to remove code duplication.
>
> ---
>
> A timeout setting exists in src/libpcp_http/src/http_fetcher.c. This is a
> library that the Apache PMDA uses to read the server status page (the source
> of the apache metrics).
>
> I noticed that this library doesn't honour the timeout value when
> connect()'ing to a HTTP server. The patch implements a timeout for connect()
> via select().
>
> See patch provided, otherwise it is in the branch "ryan" at
> git://github.com/ryandoyle/pcp.git.
I've merged from there, its always easier to do a git pull. For your
next update could you spend some extra time on the commit messages
though?
Your email above would have been a good start (gives further context,
and so the casual reader of a "git log" would know the commit relates
to pmdaapache) - you've got commits in there that are a little on the
terse side, e.g.
commit e713372458f6047e1641eb7e3e098bd706343500
Author: Ryan Doyle <rdoyle@xxxxxxxxxx>
Date: Tue Apr 9 15:38:43 2013 +1000
Syntax
Mhmm. :)
Also, doing an --interactive git-rebase might suit your workflow - ie.
you could squash multiple smaller commits before pushing to github and
sending the pull mail.
Thanks though, that fix looks good to me!
cheers.
--
Nathan
|