← Back to branch summary

~bzr-pqm/bzr/bzr.dev

« back to all changes in this revision

Viewing changes to bzrlib/atomicfile.py

Fix commit message template for non-ascii files, and add test for handling of
non-unicode.

Show diffs side-by-side

added added

removed removed

Lines of Context:
bzrlib/atomicfile.py
1
 
# Copyright (C) 2004, 2005 Canonical Ltd
2
 
#
 
1
# Copyright (C) 2004, 2005 by Canonical Ltd
 
2
 
3
3
# This program is free software; you can redistribute it and/or modify
4
4
# it under the terms of the GNU General Public License as published by
5
5
# the Free Software Foundation; either version 2 of the License, or
6
6
# (at your option) any later version.
7
 
#
 
7
 
8
8
# This program is distributed in the hope that it will be useful,
9
9
# but WITHOUT ANY WARRANTY; without even the implied warranty of
10
10
# MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE.  See the
11
11
# GNU General Public License for more details.
12
 
#
 
12
 
13
13
# You should have received a copy of the GNU General Public License
14
14
# along with this program; if not, write to the Free Software
15
15
# Foundation, Inc., 59 Temple Place, Suite 330, Boston, MA  02111-1307  USA
16
16
 
17
17
 
18
 
import os
19
 
import sys
20
 
 
21
 
from bzrlib.lazy_import import lazy_import
22
 
lazy_import(globals(), """
23
 
import stat
24
 
import socket
25
 
import warnings
26
 
 
27
 
from bzrlib import (
28
 
    errors,
29
 
    osutils,
30
 
    symbol_versioning,
31
 
    )
32
 
""")
33
 
 
34
 
# not forksafe - but we dont fork.
35
 
_pid = os.getpid()
36
 
_hostname = None
37
 
 
 
18
 
 
19
from warnings import warn
 
20
from osutils import rename
 
21
import errno
38
22
 
39
23
class AtomicFile(object):
40
24
    """A file that does an atomic-rename to move into place.
43
27
 
44
28
    Open this as for a regular file, then use commit() to move into
45
29
    place or abort() to cancel.
 
30
 
 
31
    An encoding can be specified; otherwise the default is ascii.
46
32
    """
47
33
 
48
 
    __slots__ = ['tmpfilename', 'realfilename', '_fd']
49
 
 
50
 
    def __init__(self, filename, mode='wb', new_mode=None):
51
 
        global _hostname
52
 
 
53
 
        self._fd = None
54
 
        assert mode in ('wb', 'wt'), \
55
 
            "invalid AtomicFile mode %r" % mode
56
 
 
57
 
        if _hostname is None:
58
 
            _hostname = socket.gethostname()
59
 
 
60
 
        self.tmpfilename = '%s.%d.%s.tmp' % (filename, _pid, _hostname)
61
 
 
 
34
    def __init__(self, filename, mode='wb', encoding=None):
 
35
        if mode != 'wb' and mode != 'wt':
 
36
            raise ValueError("invalid AtomicFile mode %r" % mode)
 
37
 
 
38
        import os, socket
 
39
        self.tmpfilename = '%s.%d.%s.tmp' % (filename, os.getpid(),
 
40
                                             socket.gethostname())
62
41
        self.realfilename = filename
63
42
        
64
 
        flags = os.O_EXCL | os.O_CREAT | os.O_WRONLY
65
 
        if mode == 'wb':
66
 
            flags |= osutils.O_BINARY
67
 
 
68
 
        if new_mode is not None:
69
 
            local_mode = new_mode
 
43
        if encoding:
 
44
            import codecs
 
45
            self.f = codecs.open(self.tmpfilename, mode, encoding)
70
46
        else:
71
 
            local_mode = 0666
72
 
        
73
 
        # Use a low level fd operation to avoid chmodding later.
74
 
        # This may not succeed, but it should help most of the time
75
 
        self._fd = os.open(self.tmpfilename, flags, local_mode)
76
 
 
77
 
        if new_mode is not None:
78
 
            # Because of umask issues, we may need to chmod anyway
79
 
            # the common case is that we won't, though.
80
 
            st = os.fstat(self._fd)
81
 
            if stat.S_IMODE(st.st_mode) != new_mode:
82
 
                os.chmod(self.tmpfilename, new_mode)
83
 
 
84
 
    def _get_closed(self):
85
 
        symbol_versioning.warn('AtomicFile.closed deprecated in bzr 0.10',
86
 
                               DeprecationWarning, stacklevel=2)
87
 
        return self._fd is None
88
 
 
89
 
    closed = property(_get_closed)
 
47
            self.f = open(self.tmpfilename, mode)
 
48
 
 
49
        self.write = self.f.write
 
50
        self.closed = False
 
51
 
90
52
 
91
53
    def __repr__(self):
92
54
        return '%s(%r)' % (self.__class__.__name__,
93
55
                           self.realfilename)
94
 
 
95
 
    def write(self, data):
96
 
        """Write some data to the file. Like file.write()"""
97
 
        os.write(self._fd, data)
98
 
 
99
 
    def _close_tmpfile(self, func_name):
100
 
        """Close the local temp file in preparation for commit or abort"""
101
 
        if self._fd is None:
102
 
            raise errors.AtomicFileAlreadyClosed(path=self.realfilename,
103
 
                                                 function=func_name)
104
 
        fd = self._fd
105
 
        self._fd = None
106
 
        os.close(fd)
 
56
    
107
57
 
108
58
    def commit(self):
109
59
        """Close the file and move to final name."""
110
 
        self._close_tmpfile('commit')
111
 
        osutils.rename(self.tmpfilename, self.realfilename)
 
60
        import sys, os
 
61
        
 
62
        if self.closed:
 
63
            raise Exception('%r is already closed' % self)
 
64
 
 
65
        self.closed = True
 
66
        self.f.close()
 
67
        self.f = None
 
68
        
 
69
        try:
 
70
            stat = os.lstat(self.realfilename)
 
71
            os.chmod(self.tmpfilename, stat.st_mode)
 
72
        except OSError, e:
 
73
            if e.errno != errno.ENOENT:
 
74
                raise
 
75
        rename(self.tmpfilename, self.realfilename)
 
76
 
112
77
 
113
78
    def abort(self):
114
79
        """Discard temporary file without committing changes."""
115
 
        self._close_tmpfile('abort')
 
80
        import os
 
81
 
 
82
        if self.closed:
 
83
            raise Exception('%r is already closed' % self)
 
84
 
 
85
        self.closed = True
 
86
        self.f.close()
 
87
        self.f = None
116
88
        os.remove(self.tmpfilename)
117
89
 
 
90
 
118
91
    def close(self):
119
92
        """Discard the file unless already committed."""
120
 
        if self._fd is not None:
 
93
        if not self.closed:
121
94
            self.abort()
122
95
 
 
96
 
123
97
    def __del__(self):
124
 
        if self._fd is not None:
125
 
            warnings.warn("%r leaked" % self)
 
98
        if hasattr(self, 'closed') and not self.closed:
 
99
            warn("%r leaked" % self)
 
100