← Back to branch summary

~bzr-pqm/bzr/bzr.dev

« back to all changes in this revision

Viewing changes to bzrlib/atomicfile.py

  • Committer: v.ladeuil+lp at free
  • Date: 2007-05-15 17:40:32 UTC
  • mto: (2485.8.44 bzr.connection.sharing)
  • mto: This revision was merged to the branch mainline in revision 2646.
  • Revision ID: v.ladeuil+lp@free.fr-20070515174032-qzdkangpv29l9e7g
Add a test that check that init connect only once. It fails.

* __init__.py:
(test_suite): Register the new test class.

* test_init.py: 
(InstrumentedTransport): A transport that can track connections.
(TransportHooks): Transport specific hooks.
(TestInit): Iniit command behavior tests.

* ftp.py:
(FtpTransport.__init__): Mark place that need fixing regarding
transport connection sharing

* builtins.py:
(cmd_init.run): Mark places that need fixing regarding transport
connection sharing.

Show diffs side-by-side

added added

removed removed

Lines of Context:
bzrlib/atomicfile.py
1
 
# Copyright (C) 2005, 2006, 2008, 2009, 2010 Canonical Ltd
 
1
# Copyright (C) 2004, 2005 Canonical Ltd
2
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
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
 
# Foundation, Inc., 51 Franklin Street, Fifth Floor, Boston, MA 02110-1301 USA
 
15
# Foundation, Inc., 59 Temple Place, Suite 330, Boston, MA  02111-1307  USA
16
16
 
17
17
 
18
18
import os
 
19
import sys
19
20
 
20
21
from bzrlib.lazy_import import lazy_import
21
22
lazy_import(globals(), """
22
23
import stat
 
24
import socket
23
25
import warnings
24
26
 
25
27
from bzrlib import (
49
51
        global _hostname
50
52
 
51
53
        self._fd = None
 
54
        assert mode in ('wb', 'wt'), \
 
55
            "invalid AtomicFile mode %r" % mode
52
56
 
53
57
        if _hostname is None:
54
 
            _hostname = osutils.get_host_name()
 
58
            _hostname = socket.gethostname()
55
59
 
56
 
        self.tmpfilename = '%s.%d.%s.%s.tmp' % (filename, _pid, _hostname,
57
 
                                                osutils.rand_chars(10))
 
60
        self.tmpfilename = '%s.%d.%s.tmp' % (filename, _pid, _hostname)
58
61
 
59
62
        self.realfilename = filename
60
 
 
61
 
        flags = os.O_EXCL | os.O_CREAT | os.O_WRONLY | osutils.O_NOINHERIT
 
63
        
 
64
        flags = os.O_EXCL | os.O_CREAT | os.O_WRONLY
62
65
        if mode == 'wb':
63
66
            flags |= osutils.O_BINARY
64
 
        elif mode != 'wt':
65
 
            raise ValueError("invalid AtomicFile mode %r" % mode)
66
67
 
67
68
        if new_mode is not None:
68
69
            local_mode = new_mode
69
70
        else:
70
71
            local_mode = 0666
71
 
 
 
72
        
72
73
        # Use a low level fd operation to avoid chmodding later.
73
74
        # This may not succeed, but it should help most of the time
74
75
        self._fd = os.open(self.tmpfilename, flags, local_mode)
80
81
            if stat.S_IMODE(st.st_mode) != new_mode:
81
82
                os.chmod(self.tmpfilename, new_mode)
82
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)
 
90
 
83
91
    def __repr__(self):
84
92
        return '%s(%r)' % (self.__class__.__name__,
85
93
                           self.realfilename)
111
119
        """Discard the file unless already committed."""
112
120
        if self._fd is not None:
113
121
            self.abort()
 
122
 
 
123
    def __del__(self):
 
124
        if self._fd is not None:
 
125
            warnings.warn("%r leaked" % self)