[#35027] [Ruby 1.9-Bug#4352][Open] [patch] Fix eval(s, b) backtrace; make eval(s, b) consistent with eval(s) — "James M. Lawrence" <redmine@...>

Bug #4352: [patch] Fix eval(s, b) backtrace; make eval(s, b) consistent with eval(s)

16 messages 2011/02/01

[#35114] [Ruby 1.9-Bug#4373][Open] http.rb:677: [BUG] Segmentation fault — Christian Fazzini <redmine@...>

Bug #4373: http.rb:677: [BUG] Segmentation fault

59 messages 2011/02/06

[#35171] [Ruby 1.9-Bug#4386][Open] encoding: directive does not affect regex expressions — mathew murphy <redmine@...>

Bug #4386: encoding: directive does not affect regex expressions

9 messages 2011/02/09

[#35237] [Ruby 1.9-Bug#4400][Open] nested at_exit hooks run in strange order — Suraj Kurapati <redmine@...>

Bug #4400: nested at_exit hooks run in strange order

12 messages 2011/02/15

[ruby-core:35264] Re: [Ruby 1.9-Bug#4403][Open] require command loads file twice when required symlink or file under directory which is symlink (symbolic link)

From: Kirk Haines <wyhaines@...>
Date: 2011-02-16 18:24:33 UTC
List: ruby-core #35264
On Wed, Feb 16, 2011 at 7:48 AM, Robert Pankowecki
<redmine@ruby-lang.org> wrote:
> Bug #4403: require command loads file twice when required symlink or file under directory which is symlink (symbolic link)
> http://redmine.ruby-lang.org/issues/show/4403
>
> Author: Robert Pankowecki
> Status: Open, Priority: Normal
> ruby -v: ruby 1.9.2p136 (2010-12-25 revision 30365) [i686-linux]
>
> Consider this file structure:
>
> ~/test/symbolic  tree
> .
> |-- a
> | `-- a.rb
> |-- b -> a
> |-- c.rb
> `-- d.rb -> c.rb
>
>
> Current behavior:
>
> ruby-1.9.2-p136 :001 > require './c'
> > true
> ruby-1.9.2-p136 :002 > require './d'
> ./d.rb:1: warning: already initialized constant CON2
> > true
> ruby-1.9.2-p136 :003 > require './a/a'
> > true
> ruby-1.9.2-p136 :004 > require './b/a'
> ./b/a.rb:1: warning: already initialized constant CON
> > true
>
> Expected behavior:
>
> ruby-1.9.2-p136 :001 > require './c'
> > true
> ruby-1.9.2-p136 :002 > require './d'
> > false
> ruby-1.9.2-p136 :003 > require './a/a'
> > true
> ruby-1.9.2-p136 :004 > require './b/a'
> > false
>
> This bug can be reproduced at Ruby 1.8.7, too.

The behavior that you are seeing is what is expected. Ruby is using
the path of the required file to differentiate between files which
have been required before, and those which have not been.

This has been well documented in the code and elsewhere for a long time.

Take a look in eval.c, searching for "rb_f_require", in a 1.8.x series
source code, and in the 1.9.x code, look in load.c. The
comment//documentation is identical except for the last sentence,
which details a single important difference:


Ruby 1.8.6:

[This first chunk is identical]

 *  Ruby tries to load the library named _string_, returning
 *  +true+ if successful. If the filename does not resolve to
 *  an absolute path, it will be searched for in the directories listed
 *  in <code>$:</code>. If the file has the extension ``.rb'', it is
 *  loaded as a source file; if the extension is ``.so'', ``.o'', or
 *  ``.dll'', or whatever the default shared library extension is on
 *  the current platform, Ruby loads the shared library as a Ruby
 *  extension. Otherwise, Ruby tries adding ``.rb'', ``.so'', and so on
 *  to the name. The name of the loaded feature is added to the array in
 *  <code>$"</code>. A feature will not be loaded if it's name already

[This last sentence is what is different]

 *  appears in <code>$"</code>. However, the file name is not converted
 *  to an absolute path, so that ``<code>require 'a';require
 *  './a'</code>'' will load <code>a.rb</code> twice.

Ruby 1.9.3 trunk, as of today:

[Just the part that is different.]

 * .... The file name is converted to an absolute
 *  path, so ``<code>require 'a'; require './a'</code>'' will not load
 *  <code>a.rb</code> twice.

In both cases, though, a link, which creates two different _absolute_
paths to the the same file, will allow it to get required twice. This
is expected.


Kirk Haines

In This Thread

Prev Next