Issue #16680 has been updated by zw963 (Wei Zheng).


Eregon (Benoit Daloze) wrote in #note-3:
> One issue is that `$LOAD_PATH` entries might not necessarily exist.

Sorry, not really understood what your means about reference comment.

I think maybe it acceptable for only support non-symlink $LOAD_PATH when update to ruby 2.7.
but, it should be mentions on release change log and mention why we change this behavior
before ruby 2.7 release.

----------------------------------------
Bug #16680: [Breaking Change] Ruby 2.7 not support  symlinks folder in $LOAD_PATH to work with autoload.
https://bugs.ruby-lang.org/issues/16680#change-85678

* Author: zw963 (Wei Zheng)
* Status: Open
* Priority: Normal
* ruby -v: 2.7.0
* Backport: 2.5: UNKNOWN, 2.6: UNKNOWN, 2.7: UNKNOWN
----------------------------------------
Following is a full reproducible procees, i will use a gem named 'looksee' to reproduce this.

1. gem install looksee

2. goto gem folder, copy lib content into /tmp/tesst_looksee
so, for now, we have a folder like this:
```sh
 иб $ tree /tmp/test_looksee
/tmp/test_looksee
ижибиб lib
    изибиб looksee
    ив изибиб adapter
    ив ив изибиб base.rb
    ив ив ижибиб rubinius.rb
    ив изибиб adapter.rb
    ив изибиб clean.rb
    ив изибиб columnizer.rb
    ив изибиб core_ext.rb
    ив изибиб editor.rb
    ив изибиб help.rb
    ив изибиб inspector.rb
    ив изибиб lookup_path.rb
    ив изибиб mri.so
    ив ижибиб version.rb
    ижибиб looksee.rb
```

3.  create a new symlinks to this folder.

```sh
 иб $ln -s /tmp/test_looksee /tmp/test_looksee1

  иб $ ls -alhd /tmp/test_looksee*
drwxr-xr-x 3 zw963 zw963 60 2020-03-08 01:39 /tmp/test_looksee/
lrwxrwxrwx 1 zw963 zw963 13 2020-03-08 02:08 /tmp/test_looksee1 -> test_looksee//
```

4. run following command to reproduce this issue.

```sh
 иб $ ruby -I/tmp/test_looksee1/lib/ -rlooksee -e 'puts 100'
Traceback (most recent call last):
        10: from /home/zw963/others/.rvm/rubies/ruby-2.7.0/lib/ruby/2.7.0/rubygems/core_ext/kernel_require.rb:72:in `require'
         9: from /home/zw963/others/.rvm/rubies/ruby-2.7.0/lib/ruby/2.7.0/rubygems/core_ext/kernel_require.rb:72:in `require'
         8: from /tmp/test_looksee1/lib/looksee.rb:1:in `<top (required)>'
         7: from /home/zw963/others/.rvm/rubies/ruby-2.7.0/lib/ruby/2.7.0/rubygems/core_ext/kernel_require.rb:72:in `require'
         6: from /home/zw963/others/.rvm/rubies/ruby-2.7.0/lib/ruby/2.7.0/rubygems/core_ext/kernel_require.rb:72:in `require'
         5: from /tmp/test_looksee1/lib/looksee/clean.rb:4:in `<top (required)>'
         4: from /tmp/test_looksee1/lib/looksee/clean.rb:171:in `<module:Looksee>'
         3: from /home/zw963/others/.rvm/rubies/ruby-2.7.0/lib/ruby/2.7.0/rubygems/core_ext/kernel_require.rb:72:in `require'
         2: from /home/zw963/others/.rvm/rubies/ruby-2.7.0/lib/ruby/2.7.0/rubygems/core_ext/kernel_require.rb:72:in `require'
         1: from /tmp/test_looksee1/lib/looksee/adapter.rb:1:in `<top (required)>'
/tmp/test_looksee1/lib/looksee/adapter.rb:2:in `<module:Looksee>': uninitialized constant Looksee::Adapter (NameError)
```

So, maybe you want to ask why use symlinks folder in $LOAD_PATH, but, i think this should be a quite common case, 
many ruby developer have local gem not manager by `Rubygems` or `Bundler`,  it just use `$LOAD_PATH`
to require/load it, it hard to ensure everyone folder is not a symlink.

I think this is a ** breaking change** , becuase all those code is work quite well before 2.7, because old code always 
following symlinks,  following is a example from ruby 2.6.3, it working.

```sh
 иб $ ruby -I/tmp/test_looksee1/lib/ -rlooksee -e 'puts 100'
Traceback (most recent call last):
        8: from /home/zw963/others/.rvm/rubies/ruby-2.6.3/lib/ruby/2.6.0/rubygems/core_ext/kernel_require.rb:54:in `require'
        7: from /home/zw963/others/.rvm/rubies/ruby-2.6.3/lib/ruby/2.6.0/rubygems/core_ext/kernel_require.rb:54:in `require'
        6: from /tmp/test_looksee/lib/looksee.rb:1:in `<top (required)>'
        5: from /home/zw963/others/.rvm/rubies/ruby-2.6.3/lib/ruby/2.6.0/rubygems/core_ext/kernel_require.rb:54:in `require'
        4: from /home/zw963/others/.rvm/rubies/ruby-2.6.3/lib/ruby/2.6.0/rubygems/core_ext/kernel_require.rb:54:in `require'
        3: from /tmp/test_looksee/lib/looksee/clean.rb:4:in `<top (required)>'
        2: from /tmp/test_looksee/lib/looksee/clean.rb:171:in `<module:Looksee>'
        1: from /home/zw963/others/.rvm/rubies/ruby-2.6.3/lib/ruby/2.6.0/rubygems/core_ext/kernel_require.rb:54:in `require'
/home/zw963/others/.rvm/rubies/ruby-2.6.3/lib/ruby/2.6.0/rubygems/core_ext/kernel_require.rb:54:in `require': incompatible library version - /tmp/test_looksee/lib/looksee/mri.so (LoadError)
```

As we can see, those breaking code come from `autoload :Adapter, 'looksee/adapter'`,  i suspect
maybe nested autoload cause this new issue, but only a guess, please check, thank you.

![](clipboard-202003080224-xglcq.png)




---Files--------------------------------
clipboard-202003080224-xglcq.png (183 KB)


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

Unsubscribe: <mailto:ruby-core-request / ruby-lang.org?subject=unsubscribe>
<http://lists.ruby-lang.org/cgi-bin/mailman/options/ruby-core>