rpms/planet/devel planet-1.0-no-shebang.patch, 1.1, 1.2 planet.spec, 1.4, 1.5

[Richard Dawe (richdawe)]

Author: richdawe

Update of /cvs/extras/rpms/planet/devel
In directory cvs-int.fedora.redhat.com:/tmp/cvs-serv24832

Modified Files:
	planet-1.0-no-shebang.patch planet.spec 
Log Message:
Reinstate the shebang patch, to pass QA checks; add explanation to patch

planet-1.0-no-shebang.patch:

Index: planet-1.0-no-shebang.patch
===================================================================
RCS file: /cvs/extras/rpms/planet/devel/planet-1.0-no-shebang.patch,v
retrieving revision 1.1
retrieving revision 1.2
diff -u -r1.1 -r1.2
--- planet-1.0-no-shebang.patch	2 Mar 2006 05:52:59 -0000	1.1
+++ planet-1.0-no-shebang.patch	14 Sep 2006 21:35:58 -0000	1.2
@@ -1,3 +1,9 @@
+Fedora has QA checks on the rpms, such that files that have shebang lines
+should be executable. I had two choices: make the .py files executable
+in the packaging; or remove the shebang lines. Since these modules
+don't seem to be intended to be run on the command-line, I decided to remove
+the shebang lines. 
+
 diff -pur planet-1.0~pre1.orig/planet/cache.py planet-1.0~pre1/planet/cache.py
 --- planet-1.0~pre1.orig/planet/cache.py	2005-11-15 09:35:07.000000000 +0000
 +++ planet-1.0~pre1/planet/cache.py	2005-11-16 18:50:41.000000000 +0000


Index: planet.spec
===================================================================
RCS file: /cvs/extras/rpms/planet/devel/planet.spec,v
retrieving revision 1.4
retrieving revision 1.5
diff -u -r1.4 -r1.5
--- planet.spec	14 Sep 2006 21:29:03 -0000	1.4
+++ planet.spec	14 Sep 2006 21:35:58 -0000	1.5
@@ -9,6 +9,7 @@
 License: Python Software Foundation License
 URL: http://www.planetplanet.org/
 Source0: http://www.planetplanet.org/download/%{name}-%{version}.tar.bz2
+Patch1: planet-1.0-no-shebang.patch
 BuildRoot: %{_tmppath}/%{name}-%{version}-%{release}-root-%(%{__id_u} -n)
 
 BuildArch: noarch
@@ -26,6 +27,7 @@
 
 %prep
 %setup -q
+%patch1 -p1 -b .no-shebang
 
 # Fix line endings.
 %{__sed} -i 's/\r//' LICENCE