Bug #9276

"RUBY_FREE_MIN is obsolete. Use RUBY_GC_HEAP_FREE_SLOTS instead" warning should not be issued when both ENV vars are set.

Added by Myron Marston over 1 year ago. Updated about 1 year ago.

[ruby-core:59255]
Status:Closed
Priority:Normal
Assignee:Yui NARUSE
ruby -v:ruby 2.1.0dev (2013-12-20 trunk 44301) [x86_64-darwin12.0] Backport:1.9.3: DONTNEED, 2.0.0: DONTNEED, 2.1: DONE

Description

Trying out ruby 2.1.0-rc1, I keep getting this warning. I understand why, and I've updated my ~/.zshrc to set RUBY_GC_HEAP_FREE_SLOTS for ruby 2.1, but I need to keep RUBY_GC_HEAP_FREE_SLOTS set for projects running on earlier versions. Ideally, this warning would only be emitted when RUBY_FREE_MIN is set and RUBY_GC_HEAP_FREE_SLOTS is not set. If RUBY_GC_HEAP_FREE_SLOTS is set, the programmer clearly knows that's the ENV var to use for 2.1, and likely needs to keep RUBY_FREE_MIN in place for running code against older rubies. After all, having RUBY_FREE_MIN set on 2.1 doesn't cause any problems, right?

Associated revisions

Revision 44327
Added by Koichi Sasada over 1 year ago

  • gc.c (ruby_gc_set_params): don't show obsolete warnings for RUBY_FREE_MIN/RUBY_HEAP_MIN_SLOTS if RUBY_GC_HEAP_FREE_SLOTS/RUBY_GC_HEAP_INIT_SLOTS are given. [Bug #9276]

Revision 44327
Added by Koichi Sasada over 1 year ago

  • gc.c (ruby_gc_set_params): don't show obsolete warnings for RUBY_FREE_MIN/RUBY_HEAP_MIN_SLOTS if RUBY_GC_HEAP_FREE_SLOTS/RUBY_GC_HEAP_INIT_SLOTS are given. [Bug #9276]

History

#1 Updated by Koichi Sasada over 1 year ago

  • Status changed from Open to Closed
  • % Done changed from 0 to 100

This issue was solved with changeset r44327.
Myron, thank you for reporting this issue.
Your contribution to Ruby is greatly appreciated.
May Ruby be with you.


  • gc.c (ruby_gc_set_params): don't show obsolete warnings for RUBY_FREE_MIN/RUBY_HEAP_MIN_SLOTS if RUBY_GC_HEAP_FREE_SLOTS/RUBY_GC_HEAP_INIT_SLOTS are given. [Bug #9276]

#2 Updated by Koichi Sasada over 1 year ago

  • Assignee set to Yui NARUSE
  • Category set to core
  • Status changed from Closed to Assigned

I think this request is reasonable.

myronmarston:
could you check the behavior of trunk?

Naruse-san:
could you decide to introduce this patch or not?

#3 Updated by Myron Marston over 1 year ago

could you check the behavior of trunk?

Is there an easy way to build trunk using chruby? That's what I use these days to install my rubies. I haven't played around with setting up an MRI dev environment at all.

#4 Updated by Benoit Daloze over 1 year ago

myronmarston (Myron Marston) wrote:

Is there an easy way to build trunk using chruby? That's what I use these days to install my rubies. I haven't played around with setting up an MRI dev environment at all.

It does not seems to be. Although it might be worth to add an issue to ruby-install about that.
It should be as simple as:

$ wget https://ftp.ruby-lang.org/pub/ruby/snapshot.tar.gz
$ tar xf snapshot.tar.gz
$ cd ruby-*
$ mkdir build
$ ./configure --prefix=pwd/build
$ make install
$ ./build/bin/ruby -v

#5 Updated by Yui NARUSE over 1 year ago

ko1 (Koichi Sasada) wrote:

Naruse-san:
could you decide to introduce this patch or not?

It sounds reasonable but will be 2.1.1.

myronmarston (Myron Marston) wrote:

could you check the behavior of trunk?

Is there an easy way to build trunk using chruby? That's what I use these days to install my rubies. I haven't played around with setting up an MRI dev environment at all.

What you need sound rbenv.
http://rbenv.org/

#6 Updated by Hal Brodigan over 1 year ago

It appears MRI builds the ruby executable in the root of trunk, not in the bin/ directory where chruby expects it to be. Until this is fixed, you will have to install trunk somewhere (~/.rubies/ or append the installation dir to RUBIES).

#7 Updated by Hiroshi SHIBATA about 1 year ago

  • Target version changed from 2.1.0 to current: 2.2.0

#8 Updated by Aman Gupta about 1 year ago

  • Backport changed from 1.9.3: UNKNOWN, 2.0.0: UNKNOWN, 2.1: UNKNOWN to 1.9.3: DONTNEED, 2.0.0: DONTNEED, 2.1: REQUIRED

#9 Updated by Yui NARUSE about 1 year ago

  • Status changed from Assigned to Closed

#10 Updated by Yui NARUSE about 1 year ago

  • Backport changed from 1.9.3: DONTNEED, 2.0.0: DONTNEED, 2.1: REQUIRED to 1.9.3: DONTNEED, 2.0.0: DONTNEED, 2.1: DONE

ruby_2_1 r45054 merged revision(s) 44327.

Also available in: Atom PDF