Fixed some bugs in the builder

2 files changed, 21 insertions, 14 deletions

diff --git a/modules/build.py b/modules/build.py
index 60f5104..9955512 100644
--- a/modules/build.py
+++ b/modules/build.py
@@ -40,18 +40,18 @@ class Build(Directory):
         pkg = DirFactory(package)
         with pwd(pkg.dir):
             symlink(pkg.spec_file,
-                    join(self.dir, 'SPECS', pkg.spec_file()))
+                    join(self.dir, 'SPECS', pkg.spec_file))
             sball = pkg.sourceball
             symlink(sball, join(self.dir, 'SOURCES', sball))
             patches = [f for f in listdir(getcwd()) if f.endswith('.patch')]
             for patch in patches:
                 symlink(patch, join(self.dir, 'SOURCES', patch))
 
-    def build_quick_rpm(self):
-        self.rpmbuild('-ba')
+    def build_quick_rpm(self, package):
+        self.rpmbuild('-ba', package)
 
-    def build_source_rpm(self):
-        self.rpmbuild('-bs')
+    def build_source_rpm(self, package):
+        self.rpmbuild('-bs', package)
 
     def rpmbuild(self, param, package, profile=None):
         pkg = DirFactory(package)
@@ -60,16 +60,18 @@ class Build(Directory):
                                    dir_defines(self.dir))
         else:
             defines = dir_defines(self.dir)
+        log.debug('defines are ' + str(defines))
+        log.debug('spec file is ' + pkg.spec_file)
+        cmd = ['rpmbuild'] + defines + ['-v', param, pkg.spec_file]
+        log.debug('cmd is ' + str(cmd))
         with pwd(pkg.dir):
             with log_file('rpmbuild.log') as rpm_out:
                 with pwd(join(self.dir, 'SPECS')):
-                    p = Popen(['rpmbuild',
-                               defines, param,
-                               pkg.spec_file],
-                              stdout=rpm_out, stderr=rpm_out)
+                    p = Popen(cmd, stdout=rpm_out, stderr=rpm_out)
                     log.info('building %s... please wait'
                              % pkg.spec_file)
                     p.wait()
+                    log.debug(p.returncode)
 
     def fetch_rpms(self):
         with pwd(self.dir):
diff --git a/modules/darcs.py b/modules/darcs.py
index 552e9b5..7735e45 100644
--- a/modules/darcs.py
+++ b/modules/darcs.py
@@ -35,11 +35,15 @@ class Darcs(RevisionControl):
     _type = 'darcs'
     def load_dir(self, dir):
         super(RevisionControl, self).load_dir(dir)
-    
+
     @property
     def vc_url(self):
         return self.cfg['vc_url']
 
+    @property
+    def hackage_name(self):
+        return self.cfg['hackage_name']
+
     def source_dir(self, *args):
         return self.cfg['source']
 
@@ -73,6 +77,7 @@ class Darcs(RevisionControl):
 
     def checkout(self, tgt, url, *args):
         self.cfg['vc_url'] = url
+        self.cfg['hackage_name'] = split(tgt)[1]
         self.get(url, tgt, *args)
 
     def set_current_head(self):
@@ -120,12 +125,12 @@ class Darcs(RevisionControl):
         with self.src('--tag', tag):
             yield
 
-    
+
     def setup_sourceball(self):
         log.debug('someone set us up the bomb')
         with pwd(self.dir):
             with pwd(self.source_dir()):
-                name = self.pkg_name
+                name = self.hackage_name
                 ver = self.ver()
                 date = self.date
                 full_name = '%s-%s.%sdarcs' % (name, ver, date)
@@ -134,11 +139,11 @@ class Darcs(RevisionControl):
                     log.debug('we get signal')
                     p = Popen(['darcs', 'dist', '-d', full_name],
                               stdout=darcs_out, stderr=darcs_out)
-                    log.info('generating tarball %s.tar.gz, please wait...' 
+                    log.info('generating tarball %s.tar.gz, please wait...'
                              % full_name)
                     p.wait()
             sourceball = full_name + '.tar.gz'
             move(join(self.source_dir(), sourceball), sourceball)
             self.cfg['sourceball'] = sourceball
-        
+
 __all__ = ['Darcs']