Bug #16738: RbConfig::CONFIG["SDKROOT"] not frozen on macOS

• Status: Closed
• Assignee: -
• Target version: -
• ruby -v: 2.6.5, 2.7.0
• Backport: 2.5: DONE, 2.6: DONE, 2.7: DONE

[bjfish (Brandon Fish)]

The RbConfig::CONFIG["SDKROOT"] is not frozen on macOS when the SDKROOT environment variable is set which causes the kernel spec (https://github.com/ruby/ruby/blob/1b3339528c0804aa0e673bd3f15be8b087d17bd8/spec/ruby/library/rbconfig/rbconfig_spec.rb#L27) to fail.

Spec Failure

1)
RbConfig::CONFIG contains no frozen strings even with --enable-frozen-string-literal FAILED
Expected "SDKROOT Failure\n" + "Done\n" == "Done\n"
to be truthy but was false
/Users/bfish/Documents/projects/spec/library/rbconfig/rbconfig_spec.rb:28:in `block (2 levels) in <top (required)>'
/Users/bfish/Documents/projects/spec/library/rbconfig/rbconfig_spec.rb:4:in `<top (required)>'

Example

% unset SDKROOT
% ruby -e 'p RbConfig::CONFIG["SDKROOT"].frozen?'
false
% export SDKROOT="test"
% ruby -e 'p RbConfig::CONFIG["SDKROOT"].frozen?'
true

Environment
macOS

Versions Tested
2.6.5
2.7.0

[Eregon (Benoit Daloze)]

The reason RbConfig entries cannot be frozen currently is because RbConfig.expand which calls String#replace.
So I think this is a bug, and we should ensure all RbConfig entries are mutable, hence the spec.

Alternatively, one could try to rewrite RbConfig.expand, but not sure that's possible in a compatible way
(I don't understand that code in details).

[nobu (Nobuyoshi Nakada)]

• Status changed from Open to Closed

Applied in changeset git|4adb2d655dcd54b28366b4d17e86b21e2b622cd1.

---

Make RbConfig::CONFIG values mutable [Bug #16738]

As RbConfig.expand modifies the argument and involved CONFIG
values, its values should be mutable.

[nobu (Nobuyoshi Nakada)]

• Backport changed from 2.5: UNKNOWN, 2.6: UNKNOWN, 2.7: UNKNOWN to 2.5: REQUIRED, 2.6: REQUIRED, 2.7: REQUIRED

[naruse (Yui NARUSE)]

• Backport changed from 2.5: REQUIRED, 2.6: REQUIRED, 2.7: REQUIRED to 2.5: REQUIRED, 2.6: REQUIRED, 2.7: DONE

ruby_2_7 5ccf57f23eb7a35ac699edc2fc598c47fa1844a0.

[nagachika (Tomoyuki Chikanaga)]

• Backport changed from 2.5: REQUIRED, 2.6: REQUIRED, 2.7: DONE to 2.5: REQUIRED, 2.6: DONE, 2.7: DONE

ruby_2_6 r67859 merged revision(s) 4adb2d655dcd54b28366b4d17e86b21e2b622cd1.

[usa (Usaku NAKAMURA)]

• Backport changed from 2.5: REQUIRED, 2.6: DONE, 2.7: DONE to 2.5: DONE, 2.6: DONE, 2.7: DONE

[[e-mail] (منصة شليله)]

• File network-2402637_640.jpg added

[nobu (Nobuyoshi Nakada)]

• File deleted (network-2402637_640.jpg)