diff options
| author | Jesse Keating <jkeating@redhat.com> | 2010-08-11 17:58:04 -0700 |
|---|---|---|
| committer | Jesse Keating <jkeating@redhat.com> | 2010-08-11 22:09:06 -0700 |
| commit | 411e3abeddb3a3d232b07baf9dfa499269f7002e (patch) | |
| tree | 8646cea012a629b4a870aebfb40d84fcace5207f /src | |
| parent | b73b60991cd6eeddf5bf4144e3e5741003c6c6a4 (diff) | |
| download | fedora-packager-411e3abeddb3a3d232b07baf9dfa499269f7002e.tar.gz fedora-packager-411e3abeddb3a3d232b07baf9dfa499269f7002e.tar.xz fedora-packager-411e3abeddb3a3d232b07baf9dfa499269f7002e.zip | |
Make clean use the _run_command
Diffstat (limited to 'src')
| -rw-r--r-- | src/pyfedpkg/__init__.py | 15 |
1 files changed, 3 insertions, 12 deletions
diff --git a/src/pyfedpkg/__init__.py b/src/pyfedpkg/__init__.py index a95ab92..a811707 100644 --- a/src/pyfedpkg/__init__.py +++ b/src/pyfedpkg/__init__.py @@ -282,7 +282,7 @@ def clean(dry=False, useignore=True): Can optionally not use the ignore rules - Logs output and returns the returncode + Logs output and returns nothing """ @@ -293,17 +293,8 @@ def clean(dry=False, useignore=True): if not useignore: cmd.append('-x') # Run it! - log.debug('Running: %s' % subprocess.list2cmdline(cmd)) - try: - proc = subprocess.Popen(cmd, stderr=subprocess.PIPE, - stdout=subprocess.PIPE) - output, error = proc.communicate() - except OSError, e: - raise FedpkgError(e) - log.info(output) - if error: - log.error(error) - return proc.returncode + _run_command(cmd) + return def clone(module, user, path=os.getcwd(), branch=None, bare_dir=None): """Clone a repo, optionally check out a specific branch. |