[ruby-trunk - Bug #9046][Open] [PATCH] Inter-process locking for log rotation is required

Ruby Development
1

Issue #9046 has been reported by sonots (Naotoshi Seo).

Bug #9046: [PATCH] Inter-process locking for log rotation is required

Author: sonots (Naotoshi Seo)
Status: Open
Priority: Normal
Assignee:
Category: lib
Target version: current: 2.1.0
ruby -v: ruby 2.1.0dev (2013-10-20 trunk 43373) [x86_64-darwin12.2.1]
Backport: 1.9.3: UNKNOWN, 2.0.0: UNKNOWN

I sent a pull request on Inter-process locking for log rotation by sonots · Pull Request #428 · ruby/ruby · GitHub. Please
see details on it.

2

Issue #9046 has been updated by shyouhei (Shyouhei U.).

Description updated

Bug #9046: [PATCH] Inter-process locking for log rotation is required

Author: sonots (Naotoshi Seo)
Status: Open
Priority: Normal
Assignee:
Category: lib
Target version: current: 2.1.0
ruby -v: ruby 2.1.0dev (2013-10-20 trunk 43373) [x86_64-darwin12.2.1]
Backport: 1.9.3: UNKNOWN, 2.0.0: UNKNOWN

I sent a pull request on Inter-process locking for log rotation by sonots · Pull Request #428 · ruby/ruby · GitHub . Please
see details on it.

3

Issue #9046 has been updated by naruse (Yui NARUSE).

The point “current implementation is wrong on multi-process environment”
sounds reasonable and need to fix.

I commented some implementation points to the pull request.

Bug #9046: [PATCH] Inter-process locking for log rotation is required

Author: sonots (Naotoshi Seo)
Status: Open
Priority: Normal
Assignee:
Category: lib
Target version: current: 2.1.0
ruby -v: ruby 2.1.0dev (2013-10-20 trunk 43373) [x86_64-darwin12.2.1]
Backport: 1.9.3: UNKNOWN, 2.0.0: UNKNOWN

I sent a pull request on Inter-process locking for log rotation by sonots · Pull Request #428 · ruby/ruby · GitHub . Please
see details on it.

4

Issue #9046 has been updated by nobu (Nobuyoshi N.).

Status changed from Closed to Open
Priority changed from Normal to High
% Done changed from 100 to 0

It broke on Windows.

  • don’t use fork.
  • don’t assume that open file can be removed or renamed.

I’ll try to fix or revert it.

Bug #9046: [PATCH] Inter-process locking for log rotation is required

Author: sonots (Naotoshi Seo)
Status: Open
Priority: High
Assignee:
Category: lib
Target version: current: 2.1.0
ruby -v: ruby 2.1.0dev (2013-10-20 trunk 43373) [x86_64-darwin12.2.1]
Backport: 1.9.3: UNKNOWN, 2.0.0: UNKNOWN

I sent a pull request on Inter-process locking for log rotation by sonots · Pull Request #428 · ruby/ruby · GitHub . Please
see details on it.