Skip to content

HTTPS clone URL

Subversion checkout URL

You can clone with HTTPS or Subversion.

Download ZIP

Loading…

don't let log cleanup prevent engine start #2437

Merged
merged 1 commit into from

1 participant

Min RK
Min RK
Owner

closes #2435

Min RK
Owner

confirmed as fixing the issue in #2435, merging.

Min RK minrk merged commit ca10c42 into from
Gabi Davar mindw referenced this pull request from a commit in mindw/ipython
Min RK minrk Backport PR #2437: don't let log cleanup prevent engine start
closes #2435
e773c53
Min RK minrk deleted the branch
Yaroslav Halchenko yarikoptic referenced this pull request from a commit in yarikoptic/ipython
Yaroslav Halchenko yarikoptic Merge tag 'rel-0.13.1' into debian-01X
* tag 'rel-0.13.1': (27 commits)
  remove rc for 0.13.1 final
  0.13.1.rc3
  qtconsole extras_require to whatsnew
  Backport PR #2492: add missing 'qtconsole' extras_require
  add 2480 to whatsnew
  Backport PR #2480: Add deprecation warnings for sympyprinting
  PR 2384 to whatsnew
  mark 0.13.1.rc2
  Backport PR #2384: Adapt inline backend to changes in matplotlib
  mark 0.13.1.rc1
  update 0.13.1 whatsnew lists
  Backport PR #2479: use new _winapi instead of removed _subprocess
  Backport PR #2437: don't let log cleanup prevent engine start
  Backport PR #2405: clarify TaskScheduler.hwm doc
  Backport PR #2377: Fix installation of man pages in Python 3
  start 0.13.1 whatsnew
  Backport PR #2373: fix missing imports in core.interactiveshell
  Backport PR #2365: fix names of notebooks for download/save
  Backport PR #2345: Remove references to 'version' no longer in argparse. Github issue #2343.
  Backport PR #2347: adjust division error message checking to account for Python 3
  ...
12b344d
Yaroslav Halchenko yarikoptic referenced this pull request from a commit in yarikoptic/ipython
Yaroslav Halchenko yarikoptic Merge tag 'rel-0.13.1' (mtheirs) into releases
* tag 'rel-0.13.1': (51 commits)
  remove rc for 0.13.1 final
  0.13.1.rc3
  qtconsole extras_require to whatsnew
  Backport PR #2492: add missing 'qtconsole' extras_require
  add 2480 to whatsnew
  Backport PR #2480: Add deprecation warnings for sympyprinting
  PR 2384 to whatsnew
  mark 0.13.1.rc2
  Backport PR #2384: Adapt inline backend to changes in matplotlib
  mark 0.13.1.rc1
  update 0.13.1 whatsnew lists
  Backport PR #2479: use new _winapi instead of removed _subprocess
  Backport PR #2437: don't let log cleanup prevent engine start
  Backport PR #2405: clarify TaskScheduler.hwm doc
  Backport PR #2377: Fix installation of man pages in Python 3
  start 0.13.1 whatsnew
  Backport PR #2373: fix missing imports in core.interactiveshell
  Backport PR #2365: fix names of notebooks for download/save
  Backport PR #2345: Remove references to 'version' no longer in argparse. Github issue #2343.
  Backport PR #2347: adjust division error message checking to account for Python 3
  ...
39b9601
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Commits on Sep 27, 2012
  1. Min RK
This page is out of date. Refresh to see the latest.
Showing with 7 additions and 2 deletions.
  1. +7 −2 IPython/parallel/apps/baseapp.py
9 IPython/parallel/apps/baseapp.py
View
@@ -169,8 +169,13 @@ def reinit_logging(self):
log_dir = self.profile_dir.log_dir
if self.clean_logs:
for f in os.listdir(log_dir):
- if re.match(r'%s-\d+\.(log|err|out)'%self.name,f):
- os.remove(os.path.join(log_dir, f))
+ if re.match(r'%s-\d+\.(log|err|out)' % self.name, f):
+ try:
+ os.remove(os.path.join(log_dir, f))
+ except (OSError, IOError):
+ # probably just conflict from sibling process
+ # already removing it
+ pass
if self.log_to_file:
# Start logging to the new log file
log_filename = self.name + u'-' + str(os.getpid()) + u'.log'
Something went wrong with that request. Please try again.