ruby-core@ruby-lang.org archive (unofficial mirror)
 help / color / mirror / Atom feed
* [ruby-core:106854] [Ruby master Bug#18443] IO read/write/wait hook bug fixes.
@ 2021-12-27 22:29 ioquatix (Samuel Williams)
  2022-01-30  6:42 ` [ruby-core:107353] " nagachika (Tomoyuki Chikanaga)
  2022-02-01 12:56 ` [ruby-core:107412] " naruse (Yui NARUSE)
  0 siblings, 2 replies; 3+ messages in thread
From: ioquatix (Samuel Williams) @ 2021-12-27 22:29 UTC (permalink / raw
  To: ruby-core

Issue #18443 has been reported by ioquatix (Samuel Williams).

----------------------------------------
Bug #18443: IO read/write/wait hook bug fixes.
https://bugs.ruby-lang.org/issues/18443

* Author: ioquatix (Samuel Williams)
* Status: Open
* Priority: Normal
* Backport: 2.6: UNKNOWN, 2.7: UNKNOWN, 3.0: UNKNOWN, 3.1: UNKNOWN
----------------------------------------
After testing Ruby 3.1.0 extensively, I found some obscure bugs which should be fixed. This should be back ported to 3.1.1 ASAP.

- `console.c` incorrect `rb_io_wait` argument.
- `rb_read_internal` incorrect logic when handling partial non-blocking reads in `io.c`.
- Prefer `wait_readable` rather than `IO.select` in irb implementation.

Please see https://github.com/ruby/ruby/pull/5353 for details.



-- 
https://bugs.ruby-lang.org/

^ permalink raw reply	[flat|nested] 3+ messages in thread

* [ruby-core:107353] [Ruby master Bug#18443] IO read/write/wait hook bug fixes.
  2021-12-27 22:29 [ruby-core:106854] [Ruby master Bug#18443] IO read/write/wait hook bug fixes ioquatix (Samuel Williams)
@ 2022-01-30  6:42 ` nagachika (Tomoyuki Chikanaga)
  2022-02-01 12:56 ` [ruby-core:107412] " naruse (Yui NARUSE)
  1 sibling, 0 replies; 3+ messages in thread
From: nagachika (Tomoyuki Chikanaga) @ 2022-01-30  6:42 UTC (permalink / raw
  To: ruby-core

Issue #18443 has been updated by nagachika (Tomoyuki Chikanaga).


I have confirmed the PR https://github.com/ruby/ruby/pull/5473 contains the two whole backports and a partial backport from master branch.

I think e4cd319f3d6560b76dbc1c7e89aa4d790a2d1e43 should be backported. @naruse can I backport it?

The others are changesets for default gems stdlibs (irb and io-console).
I prefer to sync released version from upstream for default gems.

io-console-0.5.11.gem was released with the corresponding commits. I think these changesets are okey to backport. f27eb8148f5a72bbacfebfecc7de9305471bb5c9, d5836db1b398a7936b0461b3011db66f6cc8c490, d75f7078c831d45ab5ba2fae4fbb308d01a21e46.

irb was not released yet with the changeset https://github.com/ruby/irb/commit/1c03bd33737e9c69ffa34a86618d69a543795e75. @aycabta, do you have any plan to release new version of irb?

----------------------------------------
Bug #18443: IO read/write/wait hook bug fixes.
https://bugs.ruby-lang.org/issues/18443#change-96252

* Author: ioquatix (Samuel Williams)
* Status: Closed
* Priority: Normal
* Backport: 2.6: DONTNEED, 2.7: DONTNEED, 3.0: DONTNEED, 3.1: REQUIRED
----------------------------------------
After testing Ruby 3.1.0 extensively, I found some obscure bugs which should be fixed. This should be back ported to 3.1.1 ASAP.

- `console.c` incorrect `rb_io_wait` argument.
- `rb_read_internal` incorrect logic when handling partial non-blocking reads in `io.c`.
- Prefer `wait_readable` rather than `IO.select` in irb implementation.

Please see https://github.com/ruby/ruby/pull/5353 for details.



-- 
https://bugs.ruby-lang.org/

^ permalink raw reply	[flat|nested] 3+ messages in thread

* [ruby-core:107412] [Ruby master Bug#18443] IO read/write/wait hook bug fixes.
  2021-12-27 22:29 [ruby-core:106854] [Ruby master Bug#18443] IO read/write/wait hook bug fixes ioquatix (Samuel Williams)
  2022-01-30  6:42 ` [ruby-core:107353] " nagachika (Tomoyuki Chikanaga)
@ 2022-02-01 12:56 ` naruse (Yui NARUSE)
  1 sibling, 0 replies; 3+ messages in thread
From: naruse (Yui NARUSE) @ 2022-02-01 12:56 UTC (permalink / raw
  To: ruby-core

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

Backport changed from 2.6: DONTNEED, 2.7: DONTNEED, 3.0: DONTNEED, 3.1: REQUIRED to 2.6: DONTNEED, 2.7: DONTNEED, 3.0: DONTNEED, 3.1: DONE

ruby_3_1 cd4aae430da8303dac9f1be2d1e1b7ce1bc8a458 merged revision(s) f27eb8148f5a72bbacfebfecc7de9305471bb5c9,d75f7078c831d45ab5ba2fae4fbb308d01a21e46,d5836db1b398a7936b0461b3011db66f6cc8c490.

----------------------------------------
Bug #18443: IO read/write/wait hook bug fixes.
https://bugs.ruby-lang.org/issues/18443#change-96311

* Author: ioquatix (Samuel Williams)
* Status: Closed
* Priority: Normal
* Backport: 2.6: DONTNEED, 2.7: DONTNEED, 3.0: DONTNEED, 3.1: DONE
----------------------------------------
After testing Ruby 3.1.0 extensively, I found some obscure bugs which should be fixed. This should be back ported to 3.1.1 ASAP.

- `console.c` incorrect `rb_io_wait` argument.
- `rb_read_internal` incorrect logic when handling partial non-blocking reads in `io.c`.
- Prefer `wait_readable` rather than `IO.select` in irb implementation.

Please see https://github.com/ruby/ruby/pull/5353 for details.



-- 
https://bugs.ruby-lang.org/

^ permalink raw reply	[flat|nested] 3+ messages in thread

end of thread, other threads:[~2022-02-01 12:56 UTC | newest]

Thread overview: 3+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2021-12-27 22:29 [ruby-core:106854] [Ruby master Bug#18443] IO read/write/wait hook bug fixes ioquatix (Samuel Williams)
2022-01-30  6:42 ` [ruby-core:107353] " nagachika (Tomoyuki Chikanaga)
2022-02-01 12:56 ` [ruby-core:107412] " naruse (Yui NARUSE)

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for read-only IMAP folder(s) and NNTP newsgroup(s).