revert previous commit since it had portage_locks stuff mixed in

svn path=/main/branches/2.1.2/; revision=9516

diff --git a/bin/emerge b/bin/emerge
index 2140a02758f9cd6de87cf3cfbada7cac7f36ea86..3623ea2ea8eea07a0250c3e0d5ab2d8b68db489d 100755 (executable)
--- a/bin/emerge
+++ b/bin/emerge
@@ -1635,7 +1635,6 @@ class depgraph:
                                        return 0
                                del e
 
-               noreplace = "--noreplace" in self.myopts
                reinstall_for_flags = None
                merging=1
                if mytype == "installed":
@@ -1649,7 +1648,7 @@ class depgraph:
                        """ If we aren't merging, perform the --newuse check.
                            If the package has new iuse flags or different use flags then if
                            --newuse is specified, we need to merge the package. """
-                       if not noreplace and merging == 0 and \
+                       if merging == 0 and \
                                myroot == self.target_root and \
                                ("--newuse" in self.myopts or
                                "--reinstall" in self.myopts) and \

diff --git a/pym/portage_locks.py b/pym/portage_locks.py
index 340096a7cc5f5427abbe1b6b091f3bd5f7226f01..9b17c60e3c96c2cb16e1a66b57696e8c7c9fffe4 100644 (file)
--- a/pym/portage_locks.py
+++ b/pym/portage_locks.py
@@ -79,7 +79,7 @@ def lockfile(mypath, wantnewlockfile=0, unlinkfile=0, waiting_msg=None):
        except IOError, e:
                if "errno" not in dir(e):
                        raise
-               if e.errno in (errno.EACCES, errno.EAGAIN):
+               if e.errno == errno.EAGAIN:
                        # resource temp unavailable; eg, someone beat us to the lock.
                        if waiting_msg is None:
                                if isinstance(mypath, int):
@@ -111,7 +111,7 @@ def lockfile(mypath, wantnewlockfile=0, unlinkfile=0, waiting_msg=None):
 
                
        if type(lockfilename) == types.StringType and \
-               myfd != HARDLINK_FD and _fstat_nlink(myfd) == 0:
+               myfd != HARDLINK_FD and os.fstat(myfd).st_nlink == 0:
                # The file was deleted on us... Keep trying to make one...
                os.close(myfd)
                portage_util.writemsg("lockfile recurse\n",1)
@@ -122,22 +122,6 @@ def lockfile(mypath, wantnewlockfile=0, unlinkfile=0, waiting_msg=None):
        portage_util.writemsg(str((lockfilename,myfd,unlinkfile))+"\n",1)
        return (lockfilename,myfd,unlinkfile,locking_method)
 
-def _fstat_nlink(fd):
-       """
-       @param fd: an open file descriptor
-       @type fd: Integer
-       @rtype: Integer
-       @return: the current number of hardlinks to the file
-       """
-       try:
-               return os.fstat(fd).st_nlink
-       except EnvironmentError, e:
-               if e.errno == errno.ENOENT:
-                       # Some filesystems such as CIFS return
-                       # ENOENT which means st_nlink == 0.
-                       return 0
-               raise
-
 def unlockfile(mytuple):
        import fcntl
 
@@ -183,7 +167,7 @@ def unlockfile(mytuple):
                        # We won the lock, so there isn't competition for it.
                        # We can safely delete the file.
                        portage_util.writemsg("Got the lockfile...\n",1)
-                       if _fstat_nlink(myfd) == 1:
+                       if os.fstat(myfd).st_nlink == 1:
                                os.unlink(lockfilename)
                                portage_util.writemsg("Unlinked lockfile...\n",1)
                                locking_method(myfd,fcntl.LOCK_UN)