[patch] final repoman commit manifest patch

repoman.patch (text/plain), 2.76 KB, created by Fabian Groffen on 2008-07-31 13:50:14 UTC

Index: bin/repoman
===================================================================
--- bin/repoman	(revision 11210)
+++ bin/repoman	(revision 11225)
@@ -1737,19 +1744,23 @@
 		mynew=portage.cvstree.findnew(mycvstree,recursive=1,basedir="./")
 		myremoved=portage.cvstree.findremoved(mycvstree,recursive=1,basedir="./")
 		bin_blob_pattern = re.compile("^-kb$")
-		bin_blobs = set(portage.cvstree.findoption(mycvstree, bin_blob_pattern,
+		expansion = set(portage.cvstree.findoption(mycvstree, bin_blob_pattern,
 			recursive=1, basedir="./"))
 
 
 	if vcs == "svn":
 		svnstatus = os.popen("svn status").readlines()
 		mychanged = [ elem.rstrip()[7:] for elem in svnstatus if elem.startswith("M") ]
-		for manifest in [ file for file in mychanged if '/Manifest' in file ]:
-			mychanged.remove(manifest)
-		mynew = [ elem.rstrip()[7:] for elem in svnstatus if elem.startswith("A") ]
+		mynew     = [ elem.rstrip()[7:] for elem in svnstatus if elem.startswith("A") ]
 		myremoved = [ elem.rstrip()[7:] for elem in svnstatus if elem.startswith("D") ]
-		# no idea how to detect binaries in SVN
-		bin_blobs = []
+		# in contrast to CVS, SVN expands nothing by default.
+		# bin_blobs historically
+		# were just there to see what files need to be checked for
+		# keyword expansion, which is exactly what we do here, so
+		# slightly change the semantic meaning of "bin_blob"...  In the
+		# future we could store which keyword is expanded.
+		props = os.popen("svn propget -R svn:keywords").readlines()
+		expansion = [ prop.split("-")[0].strip() for prop in props ]
 
 	if vcs:
 		if not (mychanged or mynew or myremoved):
@@ -1769,8 +1780,12 @@
 	headerstring = "'\$(Header|Id)"
 	headerstring += ".*\$'"
 	for myfile in myupdates:
-		if myfile in bin_blobs:
+		# for CVS, expansion contains files that are excluded from expansion
+		if vcs == "cvs" and myfile in expansion:
 			continue
+		# for SVN, expansion contains files that are included in expansion
+		if vcs == "svn" and myfile not in expansion:
+			continue
 		myout = commands.getstatusoutput("egrep -q "+headerstring+" "+myfile)
 		if myout[0] == 0:
 			myheaders.append(myfile)
@@ -1825,6 +1840,8 @@
 
 	if myupdates or myremoved:
 		myfiles = myupdates + myremoved
+		if not myheaders and "sign" not in repoman_settings.features:
+			myfiles += mymanifests
 		fd, commitmessagefile = tempfile.mkstemp(".repoman.msg")
 		mymsg = os.fdopen(fd, "w")
 		mymsg.write(commitmessage)
@@ -1911,8 +1928,8 @@
 			write_atomic(x, "".join(mylines))
 
 	manifest_commit_required = True
-	if myheaders or myupdates or myremoved or mynew:
-		myfiles=myheaders+myupdates+myremoved+mynew
+	if myupdates or myremoved or mynew:
+		myfiles=myupdates+myremoved+mynew
 		for x in range(len(myfiles)-1, -1, -1):
 			if myfiles[x].count("/") < 4-repolevel:
 				del myfiles[x]