/brz/remove-bazaar

To get this branch, use:
bzr branch http://gegoxaren.bato24.eu/bzr/brz/remove-bazaar

« back to all changes in this revision

Viewing changes to bzrlib/trace.py

renamed copy_ownership to copy_ownership_from_path.
updated .bzr.log to have 0644 permissions.
improved docstring for create_log_file

Show diffs side-by-side

added added

removed removed

Lines of Context:
bzrlib/trace.py
111
111
 
112
112
 
113
113
def note(*args, **kwargs):
114
 
    """Output a note to the user.
115
 
 
116
 
    Takes the same parameters as logging.info.
117
 
 
118
 
    :return: None
119
 
    """
120
114
    # FIXME note always emits utf-8, regardless of the terminal encoding
121
115
    #
122
116
    # FIXME: clearing the ui and then going through the abstract logging
245
239
    """
246
240
    global _bzr_log_filename
247
241
 
248
 
    def _open_or_create_log_file(filename):
249
 
        """Open existing log file, or create with ownership and permissions
 
242
    def create_log_file(filename):
 
243
        """Create the .bzr.log file.
250
244
 
251
245
        It inherits the ownership and permissions (masked by umask) from
252
246
        the containing directory to cope better with being run under sudo
253
247
        with $HOME still set to the user's homedir.
254
248
        """
 
249
        buffering = 0 # unbuffered
255
250
        flags = os.O_WRONLY | os.O_APPEND | osutils.O_TEXT
256
251
        while True:
257
252
            try:
258
253
                fd = os.open(filename, flags)
259
 
                break
 
254
                logfile = os.fdopen(fd, 'at', buffering)
 
255
                return logfile
260
256
            except OSError, e:
261
257
                if e.errno != errno.ENOENT:
262
258
                    raise
263
259
            try:
264
 
                fd = os.open(filename, flags | os.O_CREAT | os.O_EXCL, 0666)
 
260
                flags = flags | os.O_CREAT | os.O_EXCL
 
261
                permissions = 0644
 
262
                fd = os.open(filename, flags, permissions)
 
263
                logfile = os.fdopen(fd, 'at', buffering)
265
264
            except OSError, e:
266
265
                if e.errno != errno.EEXIST:
267
266
                    raise
268
267
            else:
269
268
                osutils.copy_ownership_from_path(filename)
270
 
                break
271
 
        return os.fdopen(fd, 'at', 0) # unbuffered
 
269
                return logfile
272
270
 
273
271
 
274
272
    _bzr_log_filename = _get_bzr_log_filename()
275
273
    _rollover_trace_maybe(_bzr_log_filename)
276
274
    try:
277
 
        bzr_log_file = _open_or_create_log_file(_bzr_log_filename)
 
275
        bzr_log_file = create_log_file(_bzr_log_filename)
278
276
        bzr_log_file.write('\n')
279
277
        if bzr_log_file.tell() <= 2:
280
278
            bzr_log_file.write("this is a debug log for diagnosing/reporting problems in bzr\n")
283
281
 
284
282
        return bzr_log_file
285
283
 
286
 
    except EnvironmentError, e:
 
284
    except IOError, e:
287
285
        # If we are failing to open the log, then most likely logging has not
288
286
        # been set up yet. So we just write to stderr rather than using
289
287
        # 'warning()'. If we using warning(), users get the unhelpful 'no