mail imporvements

i have seen that u have added the tests for the speed improvements will be
writing them for the mail improvemnt also. i have added some chages to send
even the reasons but havent tested yet,

used three conditions for sending mails a. if the build has failed for
first time b. only two consecutive builds failed but timeperiod more than
one month c. three builds failed consecutively


On Wed, Aug 13, 2014 at 12:43 AM, Jelmer Vernooij <notifications@github.com>
wrote:

> In buildfarm/build.py:
>
> > @@ -604,13 +603,24 @@ def get_previous_build(self, tree, host, compiler, revision):
> >              raise NoSuchBuildError(tree, host, compiler, revision)
> >          return prev_build
> >
> > -    def get_latest_build(self, tree, host, compiler):
> > -        result = self.store.find(StormBuild,
> > -            StormBuild.tree == tree,
> > -            StormBuild.host == host,
> > -            StormBuild.compiler == compiler)
> > -        result = result.order_by(Desc(StormBuild.id))
> > -        build = result.first()
> > +    def get_latest_build(self, tree, host, compiler, revisions=None):
>
> This function is broken - the argument is named "revisions", but the rest
> of the function uses "revision".
>
> (This is one of the reasons why this code should have unit tests, since
> Python doesn't have a compiler that checks for name errors)
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/jelmer/build-farm/pull/13/files#r16135638>.
>



-- 
krishna teja perannagari