Based on discussion here, I created a PR
https://github.com/rails/rails/pull/12412 with an implicit test. Let me
know, there, if you'd like a more explicit one, or anything else.
def test_tracking_loaded_files
require_dependency 'dependencies/service_one' - require_dependency
'dependencies/service_two' + require_dependency
Pathname('dependencies/service_two')
On Mon, Sep 30, 2013 at 3:12 PM, Xavier Noria <[email protected]> wrote:
> On Mon, Sep 30, 2013 at 10:00 PM, Benjamin Fleischer <
> [email protected]> wrote:
>
> So, is current thought, then
>>
>> ActiveSupport::Dependencies
>>
>> def require_or_load(file_name, const_path = nil)
>>
>>
>>
>>
>>
>> file_name = file_name.to_path if file_name.respond_to?(:to_path)
>>
>>
> The public API is require_dependency, require_or_load is more internal. I
> believe it should go into require_dependency, which is the one doing the
> String check nowadays.
>
>
>
>> Kernel extension #not sure if this is necessary. vanilla ruby appears to
>> be okay with requiring a pathname
>>
>
> Nah, don't think so, we do not have to fix RubyGems own core extensions.
>
> --
> You received this message because you are subscribed to a topic in the
> Google Groups "Ruby on Rails: Core" group.
> To unsubscribe from this topic, visit
> https://groups.google.com/d/topic/rubyonrails-core/ESAONvNYZL4/unsubscribe
> .
> To unsubscribe from this group and all its topics, send an email to
> [email protected].
> To post to this group, send email to [email protected].
> Visit this group at http://groups.google.com/group/rubyonrails-core.
> For more options, visit https://groups.google.com/groups/opt_out.
>
--
You received this message because you are subscribed to the Google Groups "Ruby
on Rails: Core" group.
To unsubscribe from this group and stop receiving emails from it, send an email
to [email protected].
To post to this group, send email to [email protected].
Visit this group at http://groups.google.com/group/rubyonrails-core.
For more options, visit https://groups.google.com/groups/opt_out.
