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

Allowing direct execution of the bundler if already in the shell #75

Open
wants to merge 4 commits into
base: master
Choose a base branch
from

Conversation

maksar
Copy link

@maksar maksar commented Oct 20, 2020

Might be useful in cases, when custom ruby interpreter is used (not in nixpkgs). So it is not (easily) possible to pass it into subsequent shell.

P.S. It might be a good idea to detect which version of bundler bundix is using, because 2.x does not support --all flag.

@Mic92 Mic92 requested a review from alyssais January 7, 2021 07:25
Ruby 2.7 deprecated using open for URIs, and recommends using URI.open
instead.

    warning: calling URI.open via Kernel#open is deprecated, call URI.open directly or use URI#open

As of Ruby 3.0 the options argument is gone and bundix skips over http
fetches.

Additional context: https://bugs.ruby-lang.org/issues/15893#note-2

Example error:

    Downloading /Users/lorne/.cache/bundix/https_rubygems_org_gems_debase-ruby_core_source-0_10_12_gem from https://rubygems.org/gems/debase-ruby_core_source-0.10.12.gem
    wrong number of arguments (given 4, expected 1..3)
    Skipping debase-ruby_core_source: couldn't fetch hash for debase-ruby_core_source-0.10.12
@shepting
Copy link

@alyssais Does this look ok? I'm trying to use a newer version of bundler and I'm hoping this will help.

@shepting
Copy link

@zimbatm @manveru Does this look ok to you?

@zimbatm
Copy link
Member

zimbatm commented Apr 23, 2021

I haven't touched it for too long to know.

Does anybody want to become a new maintainer for the project?

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

Successfully merging this pull request may close these issues.

4 participants