| Commit message (Collapse) | Author | Age | Files | Lines |
|---|
| | |
|
| | |
|
| |\
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| | |
Add a feature flag for `bundle update —source NAME` not unlocking a gem with that name
### What was the end-user problem that led to this PR?
The problem is that `bundle update --source NAME` will unlock a _gem_ with that name, in addition to the source.
### Was was your diagnosis of the problem?
My diagnosis was that we unlocked based on `spec.name` instead of `spec.source.name`.
### What is your fix for the problem, implemented in this PR?
My fix is to put this backwards-compatibility hack behind a feature flag that will turn off on 2.0.
### Why did you choose this fix out of the possible options?
I chose this fix because it allows the current behavior to continue for those who depend upon it, but the hack will be disabled by default on 2.0, or when `unlock_source_unlocks_spec` is set to true.
|
| |/
|
|
| |
with that name
|
| |\
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| | |
fix a missing space in the group conflict error message
### What was the end-user problem that led to this PR?
There is a typo in the error message that is printed to users when a user specifies a set of conflicting groups when running bundle install. There is a missing space after the first sentence.
```
› bundle install --with foo --without foo
You can't list a group in both, --with and --without.The offending groups are: foo.
```
### Was was your diagnosis of the problem?
execute `bundle install --with foo --without foo`
|
| | | |
|
| |\ \
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | | |
[EnvironmentPreserver] Allow preserving MANPATH
### What was the end-user problem that led to this PR?
The problem was Bundler could throw an exception during setup when a gem contained manpages we were adding to `$MANPATH`.
Fixes #5730.
### Was was your diagnosis of the problem?
My diagnosis was that `$MANPATH` wasn't whitelisted as a key we preserved / were 'allowed' to override via `SharedHelpers.set_env`.
### What is your fix for the problem, implemented in this PR?
My fix is to add `$MANPATH` to that list of keys, as @jules2689 suggested. I also added some test coverage around us setting the proper man path.
### Why did you choose this fix out of the possible options?
I chose this fix because it treats that env var in the same way as others bundler sets.
|
| |/ / |
|
| |\ \
| | |
| | |
| | |
| | |
| | | |
Update bundle-gem.ronn
Fixed minor typo in Docs. Hope this helps!
|
| | | | |
|
| |\ \ \
| | | |
| | | |
| | | |
| | | |
| | | |
| | | | |
Eval Gemfiles one fewer time when running `bundle install`
This is just a straight port of https://github.com/bundler/bundler/pull/4952 to master from `2-0-dev`.
Unfortunately, the Gemfile is still eval'ed twice when plugins are enabled.
|
| | | | | |
|
| |\ \ \ \
| | | | |
| | | | |
| | | | |
| | | | |
| | | | |
| | | | |
| | | | |
| | | | |
| | | | |
| | | | |
| | | | |
| | | | |
| | | | |
| | | | |
| | | | |
| | | | |
| | | | |
| | | | |
| | | | | |
Create Gemfiles with an HTTPS github source defined
### What was the end-user problem that led to this PR?
The problem was that people are creating new Gemfiles that use the built-in `github` git source, which is being removed in 2.0. Additionally, it does _not_ use an encrypted connection to GitHub.
### Was was your diagnosis of the problem?
My diagnosis was that we can't change the default because of backwards compatibility, but we can encourage _new_ Gemfiles to "do the right thing".
### What is your fix for the problem, implemented in this PR?
My fix is to add our new, recommended definition of the shortcut to all bundler-generated gemfiles.
### Why did you choose this fix out of the possible options?
I chose this fix because it will only affect new Gemfiles.
|
| | | | | | |
|
| |\ \ \ \ \
| | | | | |
| | | | | |
| | | | | |
| | | | | |
| | | | | |
| | | | | |
| | | | | |
| | | | | |
| | | | | |
| | | | | |
| | | | | | |
Add --shebang option to binstubs command
Ports https://github.com/bundler/bundler/pull/4070 to master.
bundle install `--binstubs` option is to be removed from Bundler 2.0 but
currently supports setting the shebang using the `--shebang` option.
See #1467 introducing the `--shebang` option, which with this commit
is ported to the binstubs command.
|
| | | | | | |
| | | | | |
| | | | | |
| | | | | |
| | | | | |
| | | | | |
| | | | | |
| | | | | | |
bundle install `--binstubs` option is to be removed from Bundler 2.0 but
currently supports setting the shebang using the `--shebang` option.
See #1467 introducing the `--shebang` option, which with this commit
is ported to the binstubs command.
|
| |\ \ \ \ \ \
| | | | | | |
| | | | | | |
| | | | | | |
| | | | | | |
| | | | | | | |
[2.0] Add a feature flag to prefer gems.rb to Gemfile
With `prefer_gems_rb` set (by default on 2.0), Bundler will prefer a `gems.rb` that is in the same directory as a `Gemfile`
|
| | | | | | | |
| | | | | | |
| | | | | | |
| | | | | | | |
With `prefer_gems_rb` set (by default on 2.0), Bundler will prefer a `gems.rb` that is in the same directory as a `Gemfile`
|
| |\ \ \ \ \ \ \
| |_|_|_|_|/ /
|/| | | | | |
| | | | | | |
| | | | | | |
| | | | | | |
| | | | | | |
| | | | | | | |
Print build metadata when running `bundle version`
Closes #5049.
Will get all the build metadata into `bundle env` once https://github.com/bundler/bundler/pull/5703 lands, since I don't want conflicts and want to use that code for generating the "tables"
|
| | | | | | | | |
|
| | | | | | | | |
|
| | | | | | | | |
|
| | | | | | | | |
|
| | | | | | | | |
|
| | | | | | | | |
|
| | | | | | | | |
|
| | | | | | | |
| | | | | | |
| | | | | | |
| | | | | | | |
Explicitly don’t print it with -v or —version, though
|
| | | | | | | | |
|
| | | | | | | | |
|
| |\ \ \ \ \ \ \
| |_|_|_|/ / /
|/| | | | | |
| | | | | | |
| | | | | | |
| | | | | | |
| | | | | | |
| | | | | | | |
[Installer] Load plugin files from path gems
Closes #5429 .
Because RubyGems doesn't know about path gems before we install, we need to manually load the plugin files for the path gems we're installing. This is basically copying the logic RG uses, but scoped only to those gems that we're entirely responsible for.
|
| | | | | | | | |
|
| | | | | | | | |
|
| | | |_|_|/ /
| |/| | | | |
|
| |\ \ \ \ \ \
| |_|_|/ / /
|/| | | | |
| | | | | |
| | | | | |
| | | | | | |
clearer, more specific wording about sponsorship and contributing
make it clear that everyone's contributions are welcome, and make it clear that Ruby Together funds contributors, but does not maintain the project directly
|
| | | | | | | |
|
| | | | | | | |
|
| | | | | | | |
|
| | | | | | | |
|
| | | | | | | |
|
| |\ \ \ \ \ \
| | | | | | |
| | | | | | |
| | | | | | |
| | | | | | |
| | | | | | |
| | | | | | |
| | | | | | |
| | | | | | |
| | | | | | |
| | | | | | |
| | | | | | |
| | | | | | |
| | | | | | |
| | | | | | |
| | | | | | |
| | | | | | |
| | | | | | |
| | | | | | |
| | | | | | | |
Add `—all` flag to `bundle update`
### What was the end-user problem that led to this PR?
The problem was that some users would think they needed to run `bundle update` instead of `bundle install`, and we want such a (potentially) destructive action to require more explicit user approval.
### Was was your diagnosis of the problem?
My diagnosis was that `bundle update` shouldn't implicitly update _everything_ without an acknowledgement that it will do so.
### What is your fix for the problem, implemented in this PR?
My fix, based on https://github.com/bundler/bundler/pull/2646, it to require `bundle update --all` for the current `bundle update` behavior.
### Why did you choose this fix out of the possible options?
I chose this fix because it was already on 2-0-dev, courtesy of https://github.com/bundler/bundler-features/issues/18.
|
| | | | | | | | |
|
| | | | | | | |
| | | | | | |
| | | | | | |
| | | | | | | |
And require it on 2+
|
| | | | | | | |
| | | | | | |
| | | | | | |
| | | | | | | |
This will avoid spurious deprecation warnings
|
| | | | | | | | |
|
| | | | | | | | |
|
| | | |/ / / /
| |/| | | | |
|
| |\ \ \ \ \ \
| | | | | | |
| | | | | | |
| | | | | | |
| | | | | | |
| | | | | | |
| | | | | | |
| | | | | | |
| | | | | | | |
[DSL] Deprecate the github source for 2.0
Closes #5699.
Closes #5698.
@indirect is this what you wanted?
|
| | |/ / / / / |
|
| |\ \ \ \ \ \
| | | | | | |
| | | | | | |
| | | | | | |
| | | | | | |
| | | | | | |
| | | | | | |
| | | | | | |
| | | | | | |
| | | | | | |
| | | | | | |
| | | | | | |
| | | | | | |
| | | | | | |
| | | | | | |
| | | | | | |
| | | | | | |
| | | | | | |
| | | | | | |
| | | | | | |
| | | | | | |
| | | | | | |
| | | | | | |
| | | | | | |
| | | | | | | |
Fix headers in contributing README
### What was the end-user problem that led to this PR?
Some headers in contributing/README.md aren't displaying properly.
Also, I promised @indirect that I'd start contributing to bundler last night.
### Was was your diagnosis of the problem?
The markdown was missing some spaces.
### What is your fix for the problem, implemented in this PR?
I added some spaces. TADA!
### Why did you choose this fix out of the possible options?
Adding spaces seemed like the most reasonable choice to transition from a state of non-spaces to a spaceful state.
|
| | |/ / / / / |
|
