From: Zac Medico <zmedico@gentoo.org>
Date: Wed, 11 Mar 2009 05:48:37 +0000 (-0000)
Subject: Use a regular expression for the file.name check. This replaces some odd
X-Git-Tag: v2.1.6.8~167
X-Git-Url: http://git.tremily.us/?a=commitdiff_plain;h=01ce00cee05ceea56eb37de9c0b6585fcbb2725a;p=portage.git

Use a regular expression for the file.name check. This replaces some odd
map() usage that 2to3 warns about. (trunk r12638)

svn path=/main/branches/2.1.6/; revision=12910
---

diff --git a/bin/repoman b/bin/repoman
index bc0997dc4..70a0587a6 100755
--- a/bin/repoman
+++ b/bin/repoman
@@ -73,11 +73,7 @@ util.initialize_logger()
 # 14 is the length of DESCRIPTION=""
 max_desc_len = 100
 allowed_filename_chars="a-zA-Z0-9._-+:"
-allowed_filename_chars_set = {}
-map(allowed_filename_chars_set.setdefault, map(chr, range(ord('a'), ord('z')+1)))
-map(allowed_filename_chars_set.setdefault, map(chr, range(ord('A'), ord('Z')+1)))
-map(allowed_filename_chars_set.setdefault, map(chr, range(ord('0'), ord('9')+1)))
-map(allowed_filename_chars_set.setdefault, map(chr, map(ord, [".", "-", "_", "+", ":"])))
+disallowed_filename_chars_re = re.compile(r'[^a-zA-Z0-9._\-+:]')
 bad = create_color_func("BAD")
 
 # A sane umask is needed for files that portage creates.
@@ -864,11 +860,11 @@ for x in scanlist:
 		continue
 
 	for y in checkdirlist:
-		for c in y.strip(os.path.sep):
-			if c not in allowed_filename_chars_set:
-				stats["file.name"] += 1
-				fails["file.name"].append("%s/%s: char '%s'" % (checkdir, y, c))
-				break
+		m = disallowed_filename_chars_re.search(y.strip(os.sep))
+		if m is not None:
+			stats["file.name"] += 1
+			fails["file.name"].append("%s/%s: char '%s'" % \
+				(checkdir, y, m.group(0)))
 
 		if not (y in ("ChangeLog", "metadata.xml") or y.endswith(".ebuild")):
 			continue
@@ -1003,11 +999,12 @@ for x in scanlist:
 				stats["file.size"] += 1
 				fails["file.size"].append("("+ str(mystat.st_size/1024) + "K) "+x+"/files/"+y)
 
-			for c in os.path.basename(y.rstrip(os.path.sep)):
-				if c not in allowed_filename_chars_set:
-					stats["file.name"] += 1
-					fails["file.name"].append("%s/files/%s: char '%s'" % (checkdir, y, c))
-					break
+			m = disallowed_filename_chars_re.search(
+				os.path.basename(y.rstrip(os.sep)))
+			if m is not None:
+				stats["file.name"] += 1
+				fails["file.name"].append("%s/files/%s: char '%s'" % \
+					(checkdir, y, m.group(0)))
 
 			if desktop_file_validate and desktop_pattern.match(y):
 				status, cmd_output = commands.getstatusoutput(