[gentoo-portage-dev] PATCH: Make lchown a no-op rather than aliasing to chown on Darwin (#98827)

[Jason Stubbs <jstubbs@gentoo.org>]

[-- Attachment #1.1: Type: text/plain, Size: 595 bytes --]

http://bugs.gentoo.org/show_bug.cgi?id=98827

Author: Jason Stubbs

Portage presently alters permissions of the target of a symlink rather 
than the symlink itself on Darwin. As well as the possibility that 
the target's permissions are modified incorrectly, there is also the 
case that the symlink points somewhere on the live filesystem or 
doesn't point anywhere at all. The former case will is definately 
incorrect and the latter case will cause portage to fail with a 
traceback. Making lchown a no-op should fix all these cases without 
any regressions.

-- 
Jason Stubbs

[-- Attachment #1.2: 98827_darwin_noop_lchown.patch --]
[-- Type: text/x-diff, Size: 550 bytes --]

diff -uNr portage-stable/pym/portage_data.py portage-fixed/pym/portage_data.py
--- portage-stable/pym/portage_data.py	2005-05-15 19:20:58.000000000 +0900
+++ portage-fixed/pym/portage_data.py	2005-07-18 11:27:09.096800056 +0900
@@ -16,8 +16,9 @@
 	os.environ["XARGS"]="xargs -r"
 elif ostype == "Darwin":
 	userland="Darwin"
-	os.environ["XARGS"]="xargs"	
-	lchown=os.chown
+	os.environ["XARGS"]="xargs"
+	def lchown(*pos_args, **key_args):
+		pass
 elif ostype in ["FreeBSD","OpenBSD"]:
 	userland="BSD"
 	os.environ["XARGS"]="xargs"

[-- Attachment #2: Type: application/pgp-signature, Size: 189 bytes --]