From: Zac Medico Date: Mon, 12 Jan 2009 08:14:24 +0000 (-0000) Subject: Do not hardlink files unless they are in the same X-Git-Tag: v2.1.6.5~36 X-Git-Url: http://git.tremily.us/?a=commitdiff_plain;h=e9c40b141978327325f3f1e4b16787f937bcfdcd;p=portage.git Do not hardlink files unless they are in the same directory, since otherwise tar may not be able to extract a tarball of the resulting hardlinks due to 'Invalid cross-device link' errors (depends on layout of mount points). Also, don't hardlink zero-byte files since it doesn't save any space. Thanks to Daniel Robbins for reporting these issues which were noticed by funtoo users when attempting to extract stage tarballs. (trunk r12381) svn path=/main/branches/2.1.6/; revision=12451 --- diff --git a/pym/portage/dbapi/vartree.py b/pym/portage/dbapi/vartree.py index 3d8e9d6cb..921ea7de6 100644 --- a/pym/portage/dbapi/vartree.py +++ b/pym/portage/dbapi/vartree.py @@ -2736,12 +2736,21 @@ class dblink(object): # whether config protection or not, we merge the new file the # same way. Unless moveme=0 (blocking directory) if moveme: - hardlink_key = (mymd5, mystat.st_size, + # Do not hardlink files unless they are in the same + # directory, since otherwise tar may not be able to + # extract a tarball of the resulting hardlinks due to + # 'Invalid cross-device link' errors (depends on layout of + # mount points). Also, don't hardlink zero-byte files since + # it doesn't save any space. + parent_dir = os.path.dirname(myrealdest) + print "parent_dir", parent_dir + hardlink_key = (parent_dir, mymd5, mystat.st_size, mystat.st_mode, mystat.st_uid, mystat.st_gid) hardlink_candidates = self._md5_merge_map.get(hardlink_key) if hardlink_candidates is None: hardlink_candidates = [] - self._md5_merge_map[hardlink_key] = hardlink_candidates + if mystat.st_size != 0: + self._md5_merge_map[hardlink_key] = hardlink_candidates mymtime = movefile(mysrc, mydest, newmtime=thismtime, sstat=mystat, mysettings=self.settings, hardlink_candidates=hardlink_candidates)