clang static analysis updated with tinderbox

[Dirk Wallenstein]

On Fri, Apr 29, 2011 at 11:38:38AM -0700, Jeremy Huddleston wrote:
> 
> On Apr 29, 2011, at 1:35 AM, Dirk Wallenstein wrote:
> 
> >> --- a/jhbuild/modtypes/autotools.py
> >> +++ b/jhbuild/modtypes/autotools.py
> >> @@ -121,13 +121,14 @@ class AutogenModule(Package, DownloadableModule):
> >>         if self.autogen_template:
> >>             template = self.autogen_template
> > If this is configured, %(scan_build) won't be added. Would it make sense
> > to add it independently, after this branching?
> 
> I would expect that %(scan_build) would be provided in self.autogen_template in the modules xml file if it was desired.  Is that not custom?  I'm not really sure what should take precedence here.

(Ups, new-mail-notification must have failed when visiting my old OS)
I don't know clang but it looks like the command can be prefixed to any
autogen.sh command, so that there is no need to edit existing
modulesets. It seems that it would be possible to simply prefix it to
the command without going through the template.

    cmd = self.scan_build_template(buildscript) + cmd

-- 
Cheers,
  Dirk