| durham |
| hg-reviewers |
There is no need to make it a separate method. This makes the next change
easier to read.
| Automatic diff as part of commit; lint not applicable. |
| Automatic diff as part of commit; unit tests not applicable. |
| maxfiles = self.configint('blackbox', 'maxfiles', 7) | maxfiles = self.configint('blackbox', 'maxfiles', 7) | ||||
| for i in xrange(maxfiles - 1, 1, -1): | for i in xrange(maxfiles - 1, 1, -1): | ||||
| rotate(oldpath='%s.%d' % (path, i - 1), | rotate(oldpath='%s.%d' % (path, i - 1), | ||||
| newpath='%s.%d' % (path, i)) | newpath='%s.%d' % (path, i)) | ||||
| rotate(oldpath=path, | rotate(oldpath=path, | ||||
| newpath=maxfiles > 0 and path + '.1') | newpath=maxfiles > 0 and path + '.1') | ||||
| return self._bbvfs(name, 'a') | return self._bbvfs(name, 'a') | ||||
| def _bbwrite(self, fmt, *args): | |||||
| self._bbfp.write(fmt % args) | |||||
| self._bbfp.flush() | |||||
| def log(self, event, *msg, **opts): | def log(self, event, *msg, **opts): | ||||
| global lastui | global lastui | ||||
| super(blackboxui, self).log(event, *msg, **opts) | super(blackboxui, self).log(event, *msg, **opts) | ||||
| self._partialinit() | self._partialinit() | ||||
| if not '*' in self.track and not event in self.track: | if not '*' in self.track and not event in self.track: | ||||
| return | return | ||||
| if (ui.configbool('blackbox', 'dirty') and | if (ui.configbool('blackbox', 'dirty') and | ||||
| ctx.dirty(missing=True, merge=False, branch=False)): | ctx.dirty(missing=True, merge=False, branch=False)): | ||||
| changed = '+' | changed = '+' | ||||
| if ui.configbool('blackbox', 'logsource'): | if ui.configbool('blackbox', 'logsource'): | ||||
| src = ' [%s]' % event | src = ' [%s]' % event | ||||
| else: | else: | ||||
| src = '' | src = '' | ||||
| try: | try: | ||||
| ui._bbwrite('%s %s @%s%s (%s)%s> %s', | fp = ui._bbfp | ||||
| date, user, rev, changed, pid, src, formattedmsg) | fmt = '%s %s @%s%s (%s)%s> %s' | ||||
| args = (date, user, rev, changed, pid, src, formattedmsg) | |||||
| fp.write(fmt % args) | |||||
| fp.flush() | |||||
| except IOError as err: | except IOError as err: | ||||
| self.debug('warning: cannot write to blackbox.log: %s\n' % | self.debug('warning: cannot write to blackbox.log: %s\n' % | ||||
| err.strerror) | err.strerror) | ||||
| finally: | finally: | ||||
| ui._bbinlog = False | ui._bbinlog = False | ||||
| def setrepo(self, repo): | def setrepo(self, repo): | ||||
| self._bbfp = None | self._bbfp = None | ||||