Changeset 65d9457 in rtems-source-builder

Timestamp:

05/14/14 22:49:34 (10 years ago)

Author:

Chris Johns <chrisj@…>

Branches:

4.10, 4.11, 4.9, 5, master

Children:

b09fde0

Parents:

83586f7

Message:

sb: Move error report generation to the build phase.

Generate a separate report of each package being built in a build set.
This creates a better list of faults in the case of nesting build sets
such as */rtems-all.

Location:

source-builder/sb

Files:

• build.py (modified) (5 diffs)
• setbuilder.py (modified) (3 diffs)

source-builder/sb/build.py

@@ -109 +109 @@
         else:
             return name
+
+    def _generate_report_(self, header):
+        ereport.generate('rsb-report-%s.txt' % self.macros['name'], self.opts, header)
 
     def __init__(self, name, create_tar_files, opts, macros = None):
@@ -387 +390 @@
             log.notice('package: nothing to build')
         else:
-            name = package.name()
-            if self.canadian_cross():
-                log.notice('package: (Cxc) %s' % (name))
-            else:
-                log.notice('package: %s' % (name))
-                log.trace('---- macro maps %s' % ('-' * 55))
-                log.trace('%s' % (str(self.config.macros)))
-                log.trace('-' * 70)
-            self.script.reset()
-            self.script.append(self.config.expand('%{___build_template}'))
-            self.script.append('echo "=> ' + name + ':"')
-            self.prep(package)
-            self.build_package(package)
-            if not self.opts.dry_run():
-                self.builddir()
-                sn = path.join(self.config.expand('%{_builddir}'), 'doit')
-                log.output('write script: ' + sn)
-                self.script.write(sn)
+            try:
+                name = package.name()
                 if self.canadian_cross():
-                    log.notice('building: (Cxc) %s' % (name))
+                    log.notice('package: (Cxc) %s' % (name))
                 else:
-                    log.notice('building: %s' % (name))
-                self.run(sn)
+                    log.notice('package: %s' % (name))
+                    log.trace('---- macro maps %s' % ('-' * 55))
+                    log.trace('%s' % (str(self.config.macros)))
+                    log.trace('-' * 70)
+                self.script.reset()
+                self.script.append(self.config.expand('%{___build_template}'))
+                self.script.append('echo "=> ' + name + ':"')
+                self.prep(package)
+                self.build_package(package)
+                if not self.opts.dry_run():
+                    self.builddir()
+                    sn = path.join(self.config.expand('%{_builddir}'), 'doit')
+                    log.output('write script: ' + sn)
+                    self.script.write(sn)
+                    if self.canadian_cross():
+                        log.notice('building: (Cxc) %s' % (name))
+                    else:
+                        log.notice('building: %s' % (name))
+                    self.run(sn)
+            except error.general, gerr:
+                log.notice(str(gerr))
+                log.stderr('Build FAILED')
+                self._generate_report_('Build: %s' % (gerr))
+                raise
+            except error.internal, ierr:
+                log.notice(str(ierr))
+                log.stderr('Internal Build FAILED')
+                self._generate_report_('Build: %s' % (ierr))
+                raise
+            except:
+                raise
+            if self.opts.dry_run():
+                self._generate_report_('Build: dry run')
 
     def name(self):
@@ -455 +473 @@
 def run(args):
     ec = 0
-    opts = None
-    b = None
-    erheader = None
     try:
         optargs = { '--list-configs': 'List available configurations' }
@@ -480 +495 @@
                 b = None
     except error.general, gerr:
-        erheader = 'Build: %s' % (gerr)
-        log.notice(str(gerr))
         log.stderr('Build FAILED')
         ec = 1
     except error.internal, ierr:
-        erheader = 'Build: %s' % (ierr)
-        log.notice(str(ierr))
         log.stderr('Internal Build FAILED')
         ec = 1
@@ -494 +505 @@
         log.notice('abort: user terminated')
         ec = 1
-    if (ec != 0 and erheader and opts and b) or (opts and opts.dry_run()):
-        if opts.dry_run():
-            bname = 'dry-run'
-        else:
-            bname = b.name()
-        ereport.generate('rsb-report-%s.txt' % (bname), opts, erheader)
     sys.exit(ec)
 

source-builder/sb/setbuilder.py

@@ -428 +428 @@
     import sys
     ec = 0
-    opts = None
-    b = None
-    erheader = None
     try:
         optargs = { '--list-configs':  'List available configurations',
@@ -465 +462 @@
                 print 'dep[%d]: %s' % (c, d)
     except error.general, gerr:
-        erheader = 'Build: %s' % (gerr)
-        log.notice(str(gerr))
         log.stderr('Build FAILED')
         ec = 1
     except error.internal, ierr:
-        erheader = 'Build: %s' % (ierr)
-        log.notice(str(ierr))
         log.stderr('Internal Build FAILED')
         ec = 1
@@ -479 +472 @@
         log.notice('abort: user terminated')
         ec = 1
-    if (ec != 0 and erheader and opts and b) or (opts and opts.dry_run()):
-        if opts.dry_run():
-            bset = 'dry-run'
-        else:
-            bset = b.bset
-        ereport.generate('rsb-report-%s.txt' % (bset), opts, erheader)
     sys.exit(ec)