[patch] 51_pre12 repoman +x check

51_pre12-repoman-x-bit-check.patch (text/plain), 2.40 KB, created by Brian Harring (RETIRED) on 2004-06-30 18:06:06 UTC

diff -urN portage-2.0.51_pre12-orig/bin/repoman portage-2.0.51_pre12/bin/repoman
--- portage-2.0.51_pre12-orig/bin/repoman	2004-06-21 05:41:41.000000000 -0500
+++ portage-2.0.51_pre12/bin/repoman	2004-06-30 04:29:54.516681464 -0500
@@ -62,6 +62,7 @@
 "changelog.notadded","ebuild.notadded","digest.notadded",
 "ebuild.disjointed","digest.disjointed",
 "digest.unmatch",
+"file.perms", "ebuild.perms",
 "DEPEND.bad","RDEPEND.bad","PDEPEND.bad",
 "DEPEND.badmasked","RDEPEND.badmasked","PDEPEND.badmasked",
 "DEPEND.badindev","RDEPEND.badindev","PDEPEND.badindev",
@@ -89,6 +90,7 @@
 "DEPEND.badindev","RDEPEND.badindev","PDEPEND.badindev",
 "DEPEND.badmaskedindev","RDEPEND.badmaskedindev","PDEPEND.badmaskedindev",
 "IUSE.invalid",
+"file.perms",
 "ebuild.minorsyn",
 "ebuild.badheader",
 "metadata.missing",
@@ -112,6 +114,8 @@
 	"digest.notadded":"Digests that exist but have not been added to cvs",
 	"digest.unmatch":"Digests which are incomplete (please check if your USE/ARCH includes all files)",
 	"ebuild.notadded":"Ebuilds that exist but have not been added to cvs",
+	"file.perms":"Files shouldn't have executable bits set",
+	"ebuild.perms":"Ebuilds cannot have executable bits set",
 	"changelog.notadded":"ChangeLogs that exist but have not been added to cvs",
 	"KEYWORDS.missing":"Ebuilds that have a missing KEYWORDS variable",
 	"LICENSE.missing":"Ebuilds that have a missing LICENSE variable",
@@ -571,6 +575,24 @@
 	if os.path.exists(checkdir+"/files"):
 		filesdirlist=os.listdir(checkdir+"/files")
 		for y in filesdirlist:
+			fstat=os.stat(checkdir+"/files/"+y)[0]
+			if S_ISREG(fstat):
+				if fstat & 0x0248:
+					stats["file.perms"] += 1
+					fails["file.perms"].append(x+"/files/"+y)
+			elif S_ISDIR(fstat):
+				stack=[checkdir+"/files/"+y]
+				while stack:
+					e=stack.pop()
+					fstat=os.stat(e)[0]
+					if S_ISREG(fstat):
+						if fstat & 0x0248:
+							stats["file.perms"] += 1
+							fails["file.perms"].append(e)
+					if S_ISDIR(fstat) and e[-3:] != "CVS":
+						for x in os.listdir(e):
+							stack.append(e+"/"+x)
+
 			if y[:7]=="digest-":
 				if y[7:] not in dadded:
 					#digest not added to cvs
@@ -624,6 +646,10 @@
 
 	arch_caches={}
 	for y in ebuildlist:
+		if os.stat(checkdir+"/"+y+".ebuild")[0] & 0x0248:
+			stats["ebuild.perms"] += 1
+			fails["ebuild.perms"].append(x+"/"+y+".ebuild")
+
 		if y not in eadded:
 			#ebuild not added to cvs
 			stats["ebuild.notadded"]=stats["ebuild.notadded"]+1