-
Notifications
You must be signed in to change notification settings - Fork 3.5k
reinstate standalone flag and remove .bundle in vendor task #2184
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
Conversation
|
💚 Test passed. |
|
💚 Test passed. |
|
Rule #1 of using Bundler: Always, always, always delete TBH, I'd be happy monkeypatching Bundler to purge its ability to write .bundle/config. |
|
I wonder if we should file a feature request on bundler/bundler to ignore .bundle/config if we have any flags on the cli? IMO |
|
LGTM. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Let's move this to a separate method, maybe, that we invoke after any bundler invocation?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
since there are 2 or 3 calls to bundler we should have a DRYer version of doing it indeed..but they differ quite a lot and, on a first glance, the number of arguments to this new function is pretty big (jruby+bundler path, environment tweaks, bundler arguments) so I have to come up with a proper solution..
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
+1 on improving how we invoke bundler later.
|
Merged sucessfully into master 1.5! |
the "standalone" flag during rake bootstrap is needed to generate the bundler setup.rb flag.
However, that generates a "tools/.bundle/config" file that sets the BUNDLE_PATH and will later mess with
rake plugin:install-defaults, forcing all plugins to be installed in "vendor/bundle" instead of "vendor/plugins".Setting "--path" in
rake plugin:install-defaultsisn't an option because then gems from "vendor/bundle" will not be reused, generating many duplication and some installation errors.The only option I could find was to remove the ".bundle" dir after running
rake vendor's "bundle install"Closes #2183