Skip to content

Revert "windows: add workaround for unexpected exception (#4747)" #4833

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Merged
merged 1 commit into from
Feb 17, 2025

Conversation

Watson1978
Copy link
Contributor

Which issue(s) this PR fixes:
Fixes #4742

What this PR does / why we need it:
This reverts commit b193dc4.
Because ruby/logger#107 was fixed.

Docs Changes:

Release Note:

This reverts commit b193dc4.

Signed-off-by: Shizuo Fujita <fujita@clear-code.com>
Copy link
Contributor

@daipom daipom left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks!!

@daipom daipom merged commit c19032a into fluent:master Feb 17, 2025
10 checks passed
@Watson1978 Watson1978 deleted the logger branch February 17, 2025 02:33
@daipom daipom added this to the v1.19.0 milestone Apr 23, 2025
@daipom daipom added the backport to LTS We will backport this fix to the LTS branch label Apr 23, 2025
kenhys pushed a commit to kenhys/fluentd that referenced this pull request Apr 23, 2025
…" (fluent#4833)

**Which issue(s) this PR fixes**:
Fixes fluent#4742

**What this PR does / why we need it**:
This reverts commit b193dc4.
Because ruby/logger#107 was fixed.

**Docs Changes**:

**Release Note**:

Signed-off-by: Shizuo Fujita <fujita@clear-code.com>
Signed-off-by: Kentaro Hayashi <hayashi@clear-code.com>
kenhys pushed a commit to kenhys/fluentd that referenced this pull request Apr 23, 2025
…" (fluent#4833)

**Which issue(s) this PR fixes**:
Fixes fluent#4742

**What this PR does / why we need it**:
This reverts commit b193dc4.
Because ruby/logger#107 was fixed.

**Docs Changes**:

**Release Note**:

Signed-off-by: Shizuo Fujita <fujita@clear-code.com>
Signed-off-by: Kentaro Hayashi <hayashi@clear-code.com>
kenhys pushed a commit to kenhys/fluentd that referenced this pull request Apr 23, 2025
…" (fluent#4833)

**Which issue(s) this PR fixes**:
Fixes fluent#4742

**What this PR does / why we need it**:
This reverts commit b193dc4.
Because ruby/logger#107 was fixed.

**Docs Changes**:

**Release Note**:

Signed-off-by: Shizuo Fujita <fujita@clear-code.com>
Signed-off-by: Kentaro Hayashi <hayashi@clear-code.com>
kenhys pushed a commit to kenhys/fluentd that referenced this pull request Apr 23, 2025
…" (fluent#4833)

**Which issue(s) this PR fixes**:
Fixes fluent#4742

**What this PR does / why we need it**:
This reverts commit b193dc4.
Because ruby/logger#107 was fixed.

**Docs Changes**:

**Release Note**:

Signed-off-by: Shizuo Fujita <fujita@clear-code.com>
Signed-off-by: Kentaro Hayashi <hayashi@clear-code.com>
kenhys pushed a commit to kenhys/fluentd that referenced this pull request Apr 23, 2025
…" (fluent#4833)

**Which issue(s) this PR fixes**:
Fixes fluent#4742

**What this PR does / why we need it**:
This reverts commit b193dc4.
Because ruby/logger#107 was fixed.

**Docs Changes**:

**Release Note**:

Signed-off-by: Shizuo Fujita <fujita@clear-code.com>
Signed-off-by: Kentaro Hayashi <hayashi@clear-code.com>
daipom pushed a commit that referenced this pull request Apr 24, 2025
…tion (#4747)" (#4833) (#4923)

**Which issue(s) this PR fixes**: 
Backport #4833
Fixes #4742

**What this PR does / why we need it**: 
This reverts commit b193dc4.
Because ruby/logger#107 was fixed.

**Docs Changes**:

**Release Note**:

Signed-off-by: Shizuo Fujita <fujita@clear-code.com>
Signed-off-by: Kentaro Hayashi <hayashi@clear-code.com>
Co-authored-by: Shizuo Fujita <fujita@clear-code.com>
@daipom daipom added the backported "backport to LTS" is done label Apr 25, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
backport to LTS We will backport this fix to the LTS branch backported "backport to LTS" is done
Projects
None yet
Development

Successfully merging this pull request may close these issues.

CI: Fails "test_reopen(LogTest)" test with "TypeError: no implicit conversion of Pathname into String" since logger v1.6.3 on Windows
2 participants