| Commit message (Collapse) | Author | Age | Files | Lines |
| |
|
|
|
|
|
|
| |
And show a proper warning. The other case already shows a message
somewhere else ("Gems for group <bla> were not updated/installed"), so I
just skip any warning in that case.
|
| |
|
|\
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| | |
6329: Stop printing “failed to update” message when changing sources r=segiddins a=segiddins
### What was the end-user problem that led to this PR?
The problem was `bundle update` shows incorrect message when updating git gems
Closes #6325
### What was your diagnosis of the problem?
My diagnosis was we need to check if the source has changed
### What is your fix for the problem, implemented in this PR?
My fix only prints the message when the source has not changed
Note that is does not yet fix the "updated the git sha" case, since the locked spec and the new spec actually share the same source object!
Co-authored-by: Samuel Giddins <segiddins@segiddins.me>
Co-authored-by: David Rodríguez <deivid.rodriguez@riseup.net>
|
| | |
|
| | |
|
|/ |
|
|
|
|
|
|
| |
* Drop bundler 1 stuff from tests.
* Move all feature flags to bundler 3 (like they are in 2-0-stable) and
get them tested.
|
| |
|
| |
|
| |
|
| |
|
| |
|
| |
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
If the lock file has the same gems for different platforms, the
suggestion includes all those gems.
For example, using the Rails's lock file(https://github.com/rails/rails/blob/4a51cbba58435bbba65ca50670bd6ae4887942bd/Gemfile.lock),
it shows like this:
```
$ bundle update mai
Could not find gem 'mai'.
Did you mean ffi, ffi, ffi, ffi, mail, ast, jwt, que or wdm?
```
Since it is unnecessary to include the same gem, removed duplicate gems.
|
| |
|
| |
|
| |
|
| |
|
|
|
|
|
|
|
|
|
|
|
| |
Be more precise when replacing Bundler version in lock file.
Lets say the current Bundler version is 1.16.0. If the test suite is, by a chance, executed in directory which contains such version string, this path is stored in Gemfile.lock file. Later, the test tries to replace
the version of Bundler in the lock file, but instead, it replaces the version in path. Be more careful what should be actually replaced.
This is related to #6185 although it does not resolve the original concern.
(cherry picked from commit 6cef06d19fa04e2cee5ff5faba82e98c08f08d13)
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
r=indirect
Fix NoMethodError during `bundle update --group`
### What was the end-user problem that led to this PR?
#6156: NoMethodError: undefined method `version' for nil:NilClass
### What was your diagnosis of the problem?
The `bundler` gem does not participate in the lockfile, but it can still be included in the list of dependencies that are being updated by `bundle update` if `--group` is specified.
For example, if a Gemfile contains `bundler-audit` (which depends on `bundler`) in the `:development` group, then updating with the option `--group=development` will naturally include `bundler` in the list of gems to evaluate for updating.
The trouble is that since `bundler` is excluded from the lockfile, searching the locked gems for a gemspec for bundler will return `nil`. This caused the following error during `bundle update`:
NoMethodError: undefined method `version' for nil:NilClass
### What is your fix for the problem, implemented in this PR?
This PR solves this bug by skipping over gems (i.e `bundler`) that are not in the lockfile when comparing gem versions at the conclusion of the upgrade command.
Fixes #6156.
### Why did you choose this fix out of the possible options?
I chose this fix because the bug seems to have been introduced by 618c09b59d1318958c23b1b0031c68c93186851a. My fix takes place within the new feature that was added in that commit, so it seems safe and unlikely to have side-effects.
|
| |
|
| |
|
| |
|
| |
|
| |
|
|
|
|
| |
Also ensure the resolver processes specs in the correct order for error messages
|
| |
|
| |
|
| |
|
| |
|
|
|
|
| |
with that name
|
|
|
|
| |
And require it on 2+
|
|\
| |
| |
| |
| |
| |
| |
| |
| | |
Specify `--require spec_helper` in .rspec
Specifying `--require spec_helper` in .rspec will automatically require spec_helper in *_spec.rb.
It isn't necessary to specify `require "spec_helper"` in individual *_spec.rb. I think that it's a [DRY](https://en.wikipedia.org/wiki/Don't_repeat_yourself) way.
Refer: https://github.com/rspec/rspec/wiki#rspec
|
| | |
|
| |
| |
| |
| | |
Run `rubocop -a --only Style/PercentLiteralDelimiters` and `rubocop --auto-gen-config`.
|
|/ |
|
| |
|
| |
|
|
|
|
|
| |
I should have cleaned these up in a prior PR, but no time like the
present.
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
In the discussion on new 1.13 Conservative Bundle Updates (see
https://github.com/bundler/bundler-features/issues/122), some users
would like to have the same Conservative Updating (see
http://bundler.io/v1.12/man/bundle-install.1.html#CONSERVATIVE-UPDATING)
behavior available in `bundle install` when a declared dependency is
altered in the Gemfile, also available in the `bundle update` command.
This adds a new option called `--conservative` to both `bundle update`
and `bundle lock`. The option only applies on `bundle lock` if the
`--update` option is in use.
The internal flag is more descriptive as to what actually takes place:
It locks any shared dependencies from the gem(s) being updated.
This also promotes the previously added patch level options to being
shown in command-line help, anticipating a 1.14 release, to make these
public and documented.
|
|
|
|
| |
This is currently behind the only_update_to_newer_versions setting
|
| |
|
| |
|
| |
|
| |
|
|\
| |
| |
| |
| |
| | |
Re-integrate 1-99-dev into master
@indirect please make sure I didn't miss anything here?
|
| | |
|
| | |
|
| |
| |
| |
| |
| | |
Also some pending specs cleanup. These may be added as new issues and
addressed later.
|