comparison mercurial/dirstate.py @ 29269:b6f9934cf10b

dirstate: don't use actualfilename to name the backup file

The issue with using actualfilename is that dirstate saved during transaction
with "pending" in filename will be impossible to recover from outside of the
transaction because the recover method will be looking for the name without
"pending".

29268:f200b58497f1

            # ensure that pending file written above is unlinked at
            # failure, even if tr.writepending isn't invoked until the
            # end of this transaction
            tr.registertmp(filename, location='plain')

        self._opener.write(prefix + filename + suffix,
                           self._opener.tryread(filename))

    def restorebackup(self, tr, suffix='', prefix=''):
        '''Restore dirstate by backup file with suffix'''
        # this "invalidate()" prevents "wlock.release()" from writing
        # changes of dirstate out after restoring from backup file
        self.invalidate()
        filename = self._actualfilename(tr)
        self._opener.rename(prefix + filename + suffix, filename)

    def clearbackup(self, tr, suffix='', prefix=''):
        '''Clear backup file with suffix'''
        filename = self._actualfilename(tr)
        self._opener.unlink(prefix + filename + suffix)

29269:b6f9934cf10b

            # ensure that pending file written above is unlinked at
            # failure, even if tr.writepending isn't invoked until the
            # end of this transaction
            tr.registertmp(filename, location='plain')

        self._opener.write(prefix + self._filename + suffix,
                           self._opener.tryread(filename))

    def restorebackup(self, tr, suffix='', prefix=''):
        '''Restore dirstate by backup file with suffix'''
        # this "invalidate()" prevents "wlock.release()" from writing
        # changes of dirstate out after restoring from backup file
        self.invalidate()
        filename = self._actualfilename(tr)
        # using self._filename to avoid having "pending" in the backup filename
        self._opener.rename(prefix + self._filename + suffix, filename)

    def clearbackup(self, tr, suffix='', prefix=''):
        '''Clear backup file with suffix'''
        # using self._filename to avoid having "pending" in the backup filename
        self._opener.unlink(prefix + self._filename + suffix)