pld-builder.new: PLD_Builder/rpm_builder.py - copying to cache dir doesn't ...

[glen]

Author: glen                         Date: Mon May 25 12:14:17 2009 GMT
Module: pld-builder.new               Tag: HEAD
---- Log message:
- copying to cache dir doesn't mean that build failed, so ignore result

---- Files affected:
pld-builder.new/PLD_Builder:
   rpm_builder.py (1.76 -> 1.77) 

---- Diffs:

================================================================
Index: pld-builder.new/PLD_Builder/rpm_builder.py
diff -u pld-builder.new/PLD_Builder/rpm_builder.py:1.76 pld-builder.new/PLD_Builder/rpm_builder.py:1.77
--- pld-builder.new/PLD_Builder/rpm_builder.py:1.76	Sat Apr  4 20:30:36 2009
+++ pld-builder.new/PLD_Builder/rpm_builder.py	Mon May 25 14:14:12 2009
@@ -161,9 +161,9 @@
     if b.files != []:
         rpm_cache_dir = config.rpm_cache_dir
         if "test-build" not in r.flags:
-            # XXX missing error check!
+            # NOTE: copying to cache dir doesn't mean that build failed, so ignore result
             b.log_line("copy rpm files to cache_dir=%s" % rpm_cache_dir)
-            res = chroot.run(
+            chroot.run(
                     "cp -f %s %s && poldek --mo=nodiff --mkidxz -s %s/" % \
                         (string.join(b.files), rpm_cache_dir, rpm_cache_dir),
                      logfile = b.logfile, user = "root"
================================================================

---- CVS-web:
    http://cvs.pld-linux.org/cgi-bin/cvsweb.cgi/pld-builder.new/PLD_Builder/rpm_builder.py?r1=1.76&r2=1.77&f=u