[fedora-packager/el5/master] add patch for no tty

Dennis Gilmore ausil at fedoraproject.org
Fri Aug 13 14:06:57 UTC 2010


commit c1227166564206e349deb6c8da638001a712e30f
Author: Dennis Gilmore <dennis at ausil.us>
Date:   Fri Aug 13 09:05:51 2010 -0500

    add patch for no tty

 ...-missing-process-call-for-non-pipe-no-tty.patch |   28 ++++++++++++++++++++
 fedora-packager.spec                               |    8 ++++-
 2 files changed, 34 insertions(+), 2 deletions(-)
---
diff --git a/0001-Add-missing-process-call-for-non-pipe-no-tty.patch b/0001-Add-missing-process-call-for-non-pipe-no-tty.patch
new file mode 100644
index 0000000..e72e8f4
--- /dev/null
+++ b/0001-Add-missing-process-call-for-non-pipe-no-tty.patch
@@ -0,0 +1,28 @@
+From 050360dc31c7cee41a47ca0150ef132919408aba Mon Sep 17 00:00:00 2001
+From: Jesse Keating <jkeating at redhat.com>
+Date: Thu, 12 Aug 2010 22:03:31 -0700
+Subject: [PATCH] Add missing process call for non-pipe no tty.
+
+---
+ src/pyfedpkg/__init__.py |    5 +++++
+ 1 files changed, 5 insertions(+), 0 deletions(-)
+
+diff --git a/src/pyfedpkg/__init__.py b/src/pyfedpkg/__init__.py
+index 9acdf82..9a5abc9 100644
+--- a/src/pyfedpkg/__init__.py
++++ b/src/pyfedpkg/__init__.py
+@@ -164,6 +164,11 @@ def _run_command(cmd, shell=False, env=None, pipe=[]):
+                                          stdout=subprocess.PIPE,
+                                          stderr=subprocess.PIPE, shell=shell)
+                 output, error = proc2.communicate()
++            else:
++                proc = subprocess.Popen(command, env=environ,
++                                        stdout=subprocess.PIPE,
++                                        stderr=subprocess.PIPE, shell=shell)
++                output, error = proc.communicate()
+         except OSError, e:
+             raise FedpkgError(e)
+         log.info(output)
+-- 
+1.7.2.1
+
diff --git a/fedora-packager.spec b/fedora-packager.spec
index 9781c35..ecfff2c 100644
--- a/fedora-packager.spec
+++ b/fedora-packager.spec
@@ -4,7 +4,7 @@
 
 Name:           fedora-packager
 Version:        0.5.1.2
-Release:        1%{?dist}
+Release:        2%{?dist}
 Summary:        Tools for setting up a fedora maintainer environment
 
 Group:          Applications/Productivity
@@ -12,6 +12,7 @@ License:        GPLv2+
 URL:            https://fedorahosted.org/fedora-packager
 Source0:        https://fedorahosted.org/releases/f/e/fedora-packager/fedora-packager-%{version}.tar.bz2
 BuildRoot:      %{_tmppath}/%{name}-%{version}-%{release}-root-%(%{__id_u} -n)
+Patch0:         0001-Add-missing-process-call-for-non-pipe-no-tty.patch
 
 BuildRequires:  python-devel
 Requires:       koji bodhi-client 
@@ -41,7 +42,7 @@ Provides the fedpkg command for working with dist-git
 
 %prep
 %setup -q
-
+%patch0 -p1 
 
 %build
 %configure
@@ -70,6 +71,9 @@ rm -rf $RPM_BUILD_ROOT
 %{python_sitelib}/pyfedpkg
 
 %changelog
+* Fri Aug 13 2010 Dennis Gilmore <dennis at ausil.us> - 0.5.1.2-2
+- add patch to deal with no real tty
+
 * Thu Aug 12 2010 Dennis Gilmore <dennis at asuil.us> - 0.5.1.2-1
 - fix rh bz 619733 619879 619935 620254 620465 620595 620648
 - 620653 620750 621148 621808 622291 622716


More information about the scm-commits mailing list