Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

mini_portile2 will not run on Ruby 3.1.0.dev (master) where net-ftp is a bundled gem #101

Closed
2 tasks done
flavorjones opened this issue May 28, 2021 · 5 comments
Closed
2 tasks done

Comments

@flavorjones
Copy link
Owner

flavorjones commented May 28, 2021

When using mini_portile2 on Ruby 3.1.0.dev, we see this error:

/home/runner/work/nokogiri/nokogiri/vendor/bundle/ruby/3.1.0/gems/mini_portile2-2.5.1/lib/mini_portile2/mini_portile.rb:4:in `require': cannot load such file -- net/ftp (LoadError)
	from /home/runner/work/nokogiri/nokogiri/vendor/bundle/ruby/3.1.0/gems/mini_portile2-2.5.1/lib/mini_portile2/mini_portile.rb:4:in `<top (required)>'
	from /home/runner/work/nokogiri/nokogiri/vendor/bundle/ruby/3.1.0/gems/mini_portile2-2.5.1/lib/mini_portile2.rb:2:in `require'
	from /home/runner/work/nokogiri/nokogiri/vendor/bundle/ruby/3.1.0/gems/mini_portile2-2.5.1/lib/mini_portile2.rb:2:in `<top (required)>'
	from ../../../../ext/nokogiri/extconf.rb:402:in `require'
	from ../../../../ext/nokogiri/extconf.rb:402:in `process_recipe'
	from ../../../../ext/nokogiri/extconf.rb:871:in `<main>'

Recent commits on Ruby master (e.g., ruby/ruby@e49c998) moved the net-* set of libraries from the standard library to be bundled gems.

I'm creating this issue to:

  • add ruby-head to the set of rubies being tested in CI
  • address the above error (and any other related errors)
flavorjones added a commit that referenced this issue May 28, 2021
@flavorjones flavorjones changed the title mini_portile2 will not run on Ruby 3.1.0.dev (master) where net-ftp/net-http are bundled gems mini_portile2 will not run on Ruby 3.1.0.dev (master) where net-ftp is a bundled gem May 28, 2021
@flavorjones
Copy link
Owner Author

Fixed in v2.5.2.

@KartikSoneji
Copy link

Hi, apologies if v2.4.0 is not supported but I couldn't find any info on supported versions.

While working on another package (asciidoctor), I noticed installing nokogiri@1.10.10 breaks on ruby 3.1 since it depends on mini_portile2@2.4.0, which has this issue.

I'm not too familiar with ruby, but it seems this can be fixed:

  • by nokogiri:
    • release nokogiri@1.10.11 with net-ftp as a dependency
    • release nokogiri@1.10.11 using mini_portile2~>2.5.0
  • by mini_portile:
    • release mini_portile2@2.4.1 with net-ftp as a dependency

I'm not sure which fix would be best, but I'll be happy to open a PR in whichever repo it should go to.
Unless both nokogiri@1.10.10 and mini_portile2@2.4.0 are no longer supported, in which case I'll open an issue in asciidoctor.
Thanks.

@flavorjones
Copy link
Owner Author

Hi! Nokogiri v1.10 is not a supported version, since v1.11 was released in January 2021 (the current supported branch is v1.13.x). Please ask the asciidoctor maintainers to update the gemspec!

@flavorjones
Copy link
Owner Author

I actually did this while I had context swapped in: asciidoctor/asciidoctor#4339

@KartikSoneji
Copy link

KartikSoneji commented Aug 23, 2022

Hi @flavorjones

Nokogiri v1.10 is not a supported version, since v1.11 was released in January 2021

Got it.

I actually did this while I had context swapped in: asciidoctor/asciidoctor#4339

Thanks!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants