JRuby (please use github issues at http://bugs.jruby.org)
  1. JRuby (please use github issues at http://bugs.jruby.org)
  2. JRUBY-6714

require 'java' in verbose mode causes circular dependency warning

    Details

    • Type: Improvement Improvement
    • Status: Resolved Resolved
    • Priority: Major Major
    • Resolution: Fixed
    • Affects Version/s: None
    • Fix Version/s: JRuby 1.7.0.pre2
    • Component/s: None
    • Labels:
      None
    • Testcase included:
      yes
    • Number of attachments :
      0

      Description

      http://www.ruby-forum.com/topic/4340980#new

      $ jruby -ve "require 'java'"
      jruby 1.7.0.preview1 (ruby-1.9.3-p203) (2012-05-17 c550df6) (Java
      HotSpot(TM) Client VM 1.6.0_32) [Windows 7-x86-java]
      c:/jruby/lib/ruby/shared/rubygems/custom_require.rb:36
      warning: loading in progress, circular require considered harmful - java

        Activity

        Hide
        Yoko Harada added a comment -

        Nokogiri test on JRuby master also reports "circular dependency warning," probably, by "require 'java'".

        This is just a warning and doesn't cause error nor failure. However, as Roger said in jruby-user ml, it is scary.

        Show
        Yoko Harada added a comment - Nokogiri test on JRuby master also reports "circular dependency warning," probably, by "require 'java'". This is just a warning and doesn't cause error nor failure. However, as Roger said in jruby-user ml, it is scary.
        Hide
        Thomas E Enebo added a comment -

        fixed on master in commit b93299d. Our java.io.rb needed 'jruby' loaded but jruby needs 'java' loaded....viscious circle. Conditionalized loading of jruby if I can see 'JRuby' constant already exists.

        Show
        Thomas E Enebo added a comment - fixed on master in commit b93299d. Our java.io.rb needed 'jruby' loaded but jruby needs 'java' loaded....viscious circle. Conditionalized loading of jruby if I can see 'JRuby' constant already exists.
        Hide
        Yoko Harada added a comment -

        Thank you! I don't get that annoying warning any more.

        Show
        Yoko Harada added a comment - Thank you! I don't get that annoying warning any more.

          People

          • Assignee:
            Thomas E Enebo
            Reporter:
            Roger Pack
          • Votes:
            0 Vote for this issue
            Watchers:
            3 Start watching this issue

            Dates

            • Created:
              Updated:
              Resolved: