diff options
author | Jesse Keating <jkeating@redhat.com> | 2010-08-17 16:13:46 -0700 |
---|---|---|
committer | Jesse Keating <jkeating@redhat.com> | 2010-08-17 16:13:46 -0700 |
commit | c6c745e8573b8b8df3b9cf673532263608969de1 (patch) | |
tree | 8ad491a3a5bbe02b515c4704d352b46d2ce95317 /src/pyfedpkg | |
parent | 050360dc31c7cee41a47ca0150ef132919408aba (diff) | |
download | fedora-packager-c6c745e8573b8b8df3b9cf673532263608969de1.tar.gz fedora-packager-c6c745e8573b8b8df3b9cf673532263608969de1.tar.xz fedora-packager-c6c745e8573b8b8df3b9cf673532263608969de1.zip |
Fix up chain building
We were not setting up the chains right, and our logging was all funky.
We now handle chains properly and log it sensibly
Diffstat (limited to 'src/pyfedpkg')
-rw-r--r-- | src/pyfedpkg/__init__.py | 8 |
1 files changed, 5 insertions, 3 deletions
diff --git a/src/pyfedpkg/__init__.py b/src/pyfedpkg/__init__.py index 9a5abc9..775306f 100644 --- a/src/pyfedpkg/__init__.py +++ b/src/pyfedpkg/__init__.py @@ -967,10 +967,12 @@ class PackageModule: # Handle the chain build version if chain: log.debug('Adding %s to the chain' % url) - chain[-1].append(url) - cmd.append(url) + chain.append([url]) + # This next list comp is ugly, but it's how we properly get a : + # put in between each build set + cmd.extend(' : '.join([' '.join(sets) for sets in chain]).split()) log.info('Chain building %s + %s for %s' % (self.nvr, - chain, + chain[:-1], self.target)) log.debug('Building chain %s for %s with options %s and a ' \ 'priority of %s' % |