ruby-core@ruby-lang.org archive (unofficial mirror)
 help / color / mirror / Atom feed
* [ruby-core:114661] [Ruby master Bug#19866] Future of `readline.rb`
@ 2023-09-06 15:20 vo.x (Vit Ondruch) via ruby-core
  2023-09-06 18:31 ` [ruby-core:114666] " rubyFeedback (robert heiler) via ruby-core
                   ` (4 more replies)
  0 siblings, 5 replies; 6+ messages in thread
From: vo.x (Vit Ondruch) via ruby-core @ 2023-09-06 15:20 UTC (permalink / raw
  To: ruby-core; +Cc: vo.x (Vit Ondruch)

Issue #19866 has been reported by vo.x (Vit Ondruch).

----------------------------------------
Bug #19866: Future of `readline.rb`
https://bugs.ruby-lang.org/issues/19866

* Author: vo.x (Vit Ondruch)
* Status: Open
* Priority: Normal
* ruby -v: ruby 3.3.0dev (2023-09-05 master 7c8932365f) [x86_64-linux]
* Backport: 3.0: UNKNOWN, 3.1: UNKNOWN, 3.2: UNKNOWN
----------------------------------------
I wonder what is the future of readline.rb? It still tries to load `readline.so` by default. Is this intended behavior? I'd say that instead, it should rather fire a deprecation warning and load reline. Or maybe the whole readline gem should be completely removed?



-- 
https://bugs.ruby-lang.org/
 ______________________________________________
 ruby-core mailing list -- ruby-core@ml.ruby-lang.org
 To unsubscribe send an email to ruby-core-leave@ml.ruby-lang.org
 ruby-core info -- https://ml.ruby-lang.org/mailman3/postorius/lists/ruby-core.ml.ruby-lang.org/

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

* [ruby-core:114666] [Ruby master Bug#19866] Future of `readline.rb`
  2023-09-06 15:20 [ruby-core:114661] [Ruby master Bug#19866] Future of `readline.rb` vo.x (Vit Ondruch) via ruby-core
@ 2023-09-06 18:31 ` rubyFeedback (robert heiler) via ruby-core
  2023-10-06 14:16 ` [ruby-core:114960] " Eregon (Benoit Daloze) via ruby-core
                   ` (3 subsequent siblings)
  4 siblings, 0 replies; 6+ messages in thread
From: rubyFeedback (robert heiler) via ruby-core @ 2023-09-06 18:31 UTC (permalink / raw
  To: ruby-core; +Cc: rubyFeedback (robert heiler)

Issue #19866 has been updated by rubyFeedback (robert heiler).


I believe hsbt may be able to answer your question. reline is probably the future anyway.

I disagree that:

    require 'readline'

should require reline, though. Users who do say may 100% want to get readline, so a
forceful redirection towards reline may NOT be what ALL users want to have or see.

It's not up to the ruby core team, IMO, to force deprecations onto users like this.

Let people decide on their own if they want to switch towards reline; readline-ruby
is only available externally, anyway - see:

https://github.com/ruby/readline

(I understand that this is not 100% what hsbt wrote, but it is my opinion that when
people do require 'xyz' that ruby should try to load xyz, rather than abc because 
some random developer tries to force people to use that suddenly. Since ruby core
already uses reline, that should it be, and then the proper way to get it used is
via require 'reline', rather than require 'readline'.)

----------------------------------------
Bug #19866: Future of `readline.rb`
https://bugs.ruby-lang.org/issues/19866#change-104495

* Author: vo.x (Vit Ondruch)
* Status: Open
* Priority: Normal
* ruby -v: ruby 3.3.0dev (2023-09-05 master 7c8932365f) [x86_64-linux]
* Backport: 3.0: UNKNOWN, 3.1: UNKNOWN, 3.2: UNKNOWN
----------------------------------------
I wonder what is the future of readline.rb? It still tries to load `readline.so` by default. Is this intended behavior? I'd say that instead, it should rather fire a deprecation warning and load reline. Or maybe the whole readline gem should be completely removed?



-- 
https://bugs.ruby-lang.org/
 ______________________________________________
 ruby-core mailing list -- ruby-core@ml.ruby-lang.org
 To unsubscribe send an email to ruby-core-leave@ml.ruby-lang.org
 ruby-core info -- https://ml.ruby-lang.org/mailman3/postorius/lists/ruby-core.ml.ruby-lang.org/

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

* [ruby-core:114960] [Ruby master Bug#19866] Future of `readline.rb`
  2023-09-06 15:20 [ruby-core:114661] [Ruby master Bug#19866] Future of `readline.rb` vo.x (Vit Ondruch) via ruby-core
  2023-09-06 18:31 ` [ruby-core:114666] " rubyFeedback (robert heiler) via ruby-core
@ 2023-10-06 14:16 ` Eregon (Benoit Daloze) via ruby-core
  2023-10-11  4:05 ` [ruby-core:114995] " hsbt (Hiroshi SHIBATA) via ruby-core
                   ` (2 subsequent siblings)
  4 siblings, 0 replies; 6+ messages in thread
From: Eregon (Benoit Daloze) via ruby-core @ 2023-10-06 14:16 UTC (permalink / raw
  To: ruby-core; +Cc: Eregon (Benoit Daloze)

Issue #19866 has been updated by Eregon (Benoit Daloze).


From https://bugs.ruby-lang.org/issues/19883#change-104825

> Seems odd for a standard library to attempt to load a non-standard library first, and fallback to loading a standard library.

Agreed.
I think either lib/readline.rb should only use reline and not try readline-ext,
or lib/readline.rb should be deprecated/warn, and users then adapt to `require 'readline-ext'`/`require 'reline'`

----------------------------------------
Bug #19866: Future of `readline.rb`
https://bugs.ruby-lang.org/issues/19866#change-104831

* Author: vo.x (Vit Ondruch)
* Status: Open
* Priority: Normal
* ruby -v: ruby 3.3.0dev (2023-09-05 master 7c8932365f) [x86_64-linux]
* Backport: 3.0: UNKNOWN, 3.1: UNKNOWN, 3.2: UNKNOWN
----------------------------------------
I wonder what is the future of readline.rb? It still tries to load `readline.so` by default. Is this intended behavior? I'd say that instead, it should rather fire a deprecation warning and load reline. Or maybe the whole readline gem should be completely removed?



-- 
https://bugs.ruby-lang.org/
 ______________________________________________
 ruby-core mailing list -- ruby-core@ml.ruby-lang.org
 To unsubscribe send an email to ruby-core-leave@ml.ruby-lang.org
 ruby-core info -- https://ml.ruby-lang.org/mailman3/postorius/lists/ruby-core.ml.ruby-lang.org/

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

* [ruby-core:114995] [Ruby master Bug#19866] Future of `readline.rb`
  2023-09-06 15:20 [ruby-core:114661] [Ruby master Bug#19866] Future of `readline.rb` vo.x (Vit Ondruch) via ruby-core
  2023-09-06 18:31 ` [ruby-core:114666] " rubyFeedback (robert heiler) via ruby-core
  2023-10-06 14:16 ` [ruby-core:114960] " Eregon (Benoit Daloze) via ruby-core
@ 2023-10-11  4:05 ` hsbt (Hiroshi SHIBATA) via ruby-core
  2023-10-11  4:07 ` [ruby-core:114996] " hsbt (Hiroshi SHIBATA) via ruby-core
  2023-10-13  3:10 ` [ruby-core:115029] " mame (Yusuke Endoh) via ruby-core
  4 siblings, 0 replies; 6+ messages in thread
From: hsbt (Hiroshi SHIBATA) via ruby-core @ 2023-10-11  4:05 UTC (permalink / raw
  To: ruby-core; +Cc: hsbt (Hiroshi SHIBATA)

Issue #19866 has been updated by hsbt (Hiroshi SHIBATA).


>I wonder what is the future of readline.rb? It still tries to load readline.so by default. Is this intended behavior?

It's intentional. `readline.rb` is same as `yaml.rb` for migrating backend.

`reline` is mostly compatible `libreadline`. But we don't rush to rewrite `readline` to `reline` in Ruby 3.3.

----------------------------------------
Bug #19866: Future of `readline.rb`
https://bugs.ruby-lang.org/issues/19866#change-104864

* Author: vo.x (Vit Ondruch)
* Status: Open
* Priority: Normal
* ruby -v: ruby 3.3.0dev (2023-09-05 master 7c8932365f) [x86_64-linux]
* Backport: 3.0: UNKNOWN, 3.1: UNKNOWN, 3.2: UNKNOWN
----------------------------------------
I wonder what is the future of readline.rb? It still tries to load `readline.so` by default. Is this intended behavior? I'd say that instead, it should rather fire a deprecation warning and load reline. Or maybe the whole readline gem should be completely removed?



-- 
https://bugs.ruby-lang.org/
 ______________________________________________
 ruby-core mailing list -- ruby-core@ml.ruby-lang.org
 To unsubscribe send an email to ruby-core-leave@ml.ruby-lang.org
 ruby-core info -- https://ml.ruby-lang.org/mailman3/postorius/lists/ruby-core.ml.ruby-lang.org/

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

* [ruby-core:114996] [Ruby master Bug#19866] Future of `readline.rb`
  2023-09-06 15:20 [ruby-core:114661] [Ruby master Bug#19866] Future of `readline.rb` vo.x (Vit Ondruch) via ruby-core
                   ` (2 preceding siblings ...)
  2023-10-11  4:05 ` [ruby-core:114995] " hsbt (Hiroshi SHIBATA) via ruby-core
@ 2023-10-11  4:07 ` hsbt (Hiroshi SHIBATA) via ruby-core
  2023-10-13  3:10 ` [ruby-core:115029] " mame (Yusuke Endoh) via ruby-core
  4 siblings, 0 replies; 6+ messages in thread
From: hsbt (Hiroshi SHIBATA) via ruby-core @ 2023-10-11  4:07 UTC (permalink / raw
  To: ruby-core; +Cc: hsbt (Hiroshi SHIBATA)

Issue #19866 has been updated by hsbt (Hiroshi SHIBATA).

Status changed from Open to Assigned
Assignee set to hsbt (Hiroshi SHIBATA)

----------------------------------------
Bug #19866: Future of `readline.rb`
https://bugs.ruby-lang.org/issues/19866#change-104865

* Author: vo.x (Vit Ondruch)
* Status: Assigned
* Priority: Normal
* Assignee: hsbt (Hiroshi SHIBATA)
* ruby -v: ruby 3.3.0dev (2023-09-05 master 7c8932365f) [x86_64-linux]
* Backport: 3.0: UNKNOWN, 3.1: UNKNOWN, 3.2: UNKNOWN
----------------------------------------
I wonder what is the future of readline.rb? It still tries to load `readline.so` by default. Is this intended behavior? I'd say that instead, it should rather fire a deprecation warning and load reline. Or maybe the whole readline gem should be completely removed?



-- 
https://bugs.ruby-lang.org/
 ______________________________________________
 ruby-core mailing list -- ruby-core@ml.ruby-lang.org
 To unsubscribe send an email to ruby-core-leave@ml.ruby-lang.org
 ruby-core info -- https://ml.ruby-lang.org/mailman3/postorius/lists/ruby-core.ml.ruby-lang.org/

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

* [ruby-core:115029] [Ruby master Bug#19866] Future of `readline.rb`
  2023-09-06 15:20 [ruby-core:114661] [Ruby master Bug#19866] Future of `readline.rb` vo.x (Vit Ondruch) via ruby-core
                   ` (3 preceding siblings ...)
  2023-10-11  4:07 ` [ruby-core:114996] " hsbt (Hiroshi SHIBATA) via ruby-core
@ 2023-10-13  3:10 ` mame (Yusuke Endoh) via ruby-core
  4 siblings, 0 replies; 6+ messages in thread
From: mame (Yusuke Endoh) via ruby-core @ 2023-10-13  3:10 UTC (permalink / raw
  To: ruby-core; +Cc: mame (Yusuke Endoh)

Issue #19866 has been updated by mame (Yusuke Endoh).

Status changed from Assigned to Rejected

Discussed at the dev meeting, and agreed that the current status be maintained.

* Dependency on external libraries is not very happy, but not a problem itself. Actually, set.rb had longly depended on rbtree gem.
* As long as it is named `readline.rb`, it should respect `readline` library if it is available.
* We see no benefit in removing it and forcing people to explicitly change to `require "reline"`.


----------------------------------------
Bug #19866: Future of `readline.rb`
https://bugs.ruby-lang.org/issues/19866#change-104900

* Author: vo.x (Vit Ondruch)
* Status: Rejected
* Priority: Normal
* Assignee: hsbt (Hiroshi SHIBATA)
* ruby -v: ruby 3.3.0dev (2023-09-05 master 7c8932365f) [x86_64-linux]
* Backport: 3.0: UNKNOWN, 3.1: UNKNOWN, 3.2: UNKNOWN
----------------------------------------
I wonder what is the future of readline.rb? It still tries to load `readline.so` by default. Is this intended behavior? I'd say that instead, it should rather fire a deprecation warning and load reline. Or maybe the whole readline gem should be completely removed?



-- 
https://bugs.ruby-lang.org/
 ______________________________________________
 ruby-core mailing list -- ruby-core@ml.ruby-lang.org
 To unsubscribe send an email to ruby-core-leave@ml.ruby-lang.org
 ruby-core info -- https://ml.ruby-lang.org/mailman3/postorius/lists/ruby-core.ml.ruby-lang.org/

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

end of thread, other threads:[~2023-10-13  3:10 UTC | newest]

Thread overview: 6+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2023-09-06 15:20 [ruby-core:114661] [Ruby master Bug#19866] Future of `readline.rb` vo.x (Vit Ondruch) via ruby-core
2023-09-06 18:31 ` [ruby-core:114666] " rubyFeedback (robert heiler) via ruby-core
2023-10-06 14:16 ` [ruby-core:114960] " Eregon (Benoit Daloze) via ruby-core
2023-10-11  4:05 ` [ruby-core:114995] " hsbt (Hiroshi SHIBATA) via ruby-core
2023-10-11  4:07 ` [ruby-core:114996] " hsbt (Hiroshi SHIBATA) via ruby-core
2023-10-13  3:10 ` [ruby-core:115029] " mame (Yusuke Endoh) via ruby-core

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).