summaryrefslogtreecommitdiff
Commit message (Collapse)AuthorAgeFilesLines
* [MatchPlatform] Allow matching two platforms as a functionseg-rg-1.3.6Samuel Giddins2016-12-272-9/+13
|
* [RubyGemsIntegration] Restore 1.3.6 compatibility when installingSamuel Giddins2016-12-271-1/+1
|
* Auto merge of #5279 - bundler:seg-parallel-extensions, r=indirectThe Bundler Bot2016-12-273-2/+11
|\ | | | | | | | | | | | | | | [GemInstaller] Allow installing extensions in parallel I believe this closes #4684 \c @lynnco
| * [GemInstaller] Allow installing extensions in parallelseg-parallel-extensionsSamuel Giddins2016-12-253-2/+11
| |
* | Auto merge of #5277 - bundler:Paxa-master, r=segiddinsThe Bundler Bot2016-12-274-11/+21
|\ \ | | | | | | | | | | | | | | | Don't include dev deps in Index#dependency_names See #5256
| * | Remove unecessary hack in the specsPaxa-masterSamuel Giddins2016-12-251-7/+0
| | |
| * | [Index] Avoid uncessary array allocations for dependency_namesSamuel Giddins2016-12-251-5/+8
| | |
| * | Add a spec for Index#dependency_names not include dev depsSamuel Giddins2016-12-242-0/+12
| | |
| * | Do not pull development dependenciesPavel2016-12-241-1/+3
| | |
* | | Auto merge of #5267 - bundler:seg-parallel-installer-missing-specs-warning, ↵The Bundler Bot2016-12-273-24/+83
|\ \ \ | | | | | | | | | | | | | | | | | | | | | | | | | | | | r=indirect [ParallelInstaller] Allow installing with corrupted lockfiles - [x] Specs
| * | | Add specs for falling back to 1 thread when the lockfile is corruptseg-parallel-installer-missing-specs-warningSamuel Giddins2016-12-263-15/+49
| | | |
| * | | [ParallelInstaller] Only print 1 warning for missing dependenciesSamuel Giddins2016-12-261-9/+24
| | | |
| * | | [ParallelInstaller] Allow installing with corrupted lockfilesSamuel Giddins2016-12-261-9/+19
|/ / /
* | | Auto merge of #5282 - bundler:seg-travis-2.4.0, r=segiddinsThe Bundler Bot2016-12-251-5/+7
|\ \ \ | |_|/ |/| | | | | | | | | | | [Travis] Test against Ruby 2.4 Yay larger build matrix!
| * | [Travis] Specify ruby version patch levelsseg-travis-2.4.0Samuel Giddins2016-12-251-6/+6
| | |
| * | [Travis] Test against Ruby 2.4Samuel Giddins2016-12-251-0/+2
|/ /
* | Merge tag 'v1.13.7'Samuel Giddins2016-12-253-2/+8
|\ \ | | | | | | | | | Version 1.13.7
| * | Version 1.13.7 with changelogv1.13.71-13-stableSamuel Giddins2016-12-252-1/+7
| | |
| * | Auto merge of #5175 - biow0lf:biow0lf-fix-typo, r=segiddinsHomu2016-12-251-1/+1
| | | | | | | | | | | | | | | | | | Fix x64_mingw_24. Typo in PR #5170 (cherry picked from commit 6e0dcf48ee70314e42a65dcfd1f81c1494646d91)
| * | Auto merge of #5170 - amatsuda:mri_24, r=segiddinsHomu2016-12-253-3/+17
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Support :mri_24 platform Current ruby trunk is versioned as "2.4.0". % ruby -ve 'p RUBY_VERSION' ruby 2.4.0dev (2016-11-15 trunk 56792) [x86_64-darwin15] "2.4.0" We'd like to run tests on some gems against ruby-trunk or 2.4 pre-releases, so here's a support for MRI 2.4. (This is Ruby 2.4 version of #3865) (cherry picked from commit aa9414e78f527ee2066d21783abb7d52a55f68a9)
* | | Auto merge of #5023 - renuo:4854-gemnotfound-error, r=segiddinsThe Bundler Bot2016-12-244-4/+24
|\ \ \ | |_|/ |/| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | 4854 gemnotfound error Fixes broken test Wanting to help out, I was on the outlook for something easy to fix in bundler. I stumbled upon this issue #4854, which was already tackled by @b-ggs but somehow not merged yet. I went on to check whether I could do something. Here's what I've come up with: @b-ggs' PR included everything to solve the issue, but introduced the use of `Bundler.app_cache` in the resolver. This works out just fine until you run `bundle plugin install`, which does not seem to require a Gemfile. `Bundler.app_cache` however calls `root`, which then again tries to find a Gemfile. This obviously fails and raises a `GemfileNotFound` exception. The error message "Could not locate Gemfile or .bundle/ directory" starts very similar to the one expected in the failing test ("Could not find"). This however, is a completely different error. The solution is now is to rescue the GemfileNotFound error when it comes up to here. All credit for solving the issue goes to @b-ggs and his PR here: https://github.com/bundler/bundler/pull/4865
| * | Remove debug codeDiego Steiner2016-09-271-1/+0
| | |
| * | Fixes broken testDiego Steiner2016-09-271-1/+6
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Wanting to help out, I was on the outlook for something easy to fix in bundler. I stumbled upon this issue 4854, which was already tackled by b-ggs but somehow not merged yet. I went on to check whether I could do something. Here's what I've come up with: b-ggs' PR included everything to solve the issue, but introduced the use of ```Bundler.app_cache``` in the resolver. This works out just fine until you run ```bundle plugin install```, which does not seem to require a Gemfile. ```Bundler.app_cache``` however calls ```root```, which then again tries to find a Gemfile. This obviously fails and raises a ```GemfileNotFound``` exception. The error message "Could not locate Gemfile or .bundle/ directory" starts very similar to the one expected in the failing test ("Could not find"). This however, is a completely different error. The solution is now is to rescue the GemfileNotFound error when it comes up to here. All credit for solving the issue goes to b-ggs and his PR here: bundler/bundler pull:4865 Fix rubocop offense
| * | Merge remote-tracking branch 'origin/master' into 4854-gemnotfound-errorDiego Steiner2016-09-27101-333/+1394
| |\ \
| * | | Update failing specBoggs2016-08-171-1/+1
| | | |
| * | | Update specsBoggs2016-08-172-2/+17
| | | |
| * | | Modify error message for GemNotFound error messageBoggs2016-08-171-1/+2
| | | |
* | | | Auto merge of #5270 - pixeltrix:fix-hook-name, r=segiddinsThe Bundler Bot2016-12-241-1/+1
|\ \ \ \ | | | | | | | | | | | | | | | | | | | | | | | | | Use correct hook name in comment I checked `Bundler::Source::Path::Installer#post_install` - it's definitely `pre_install` and not `post_install` twice though I wasn't sure given the method name 😄
| * | | | Use correct hook name in commentAndrew White2016-12-221-1/+1
| | | | |
* | | | | Auto merge of #5273 - chrismo:new-pr-4734, r=segiddinsThe Bundler Bot2016-12-231-0/+27
|\ \ \ \ \ | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Specs for multiple platform bug fix in outdated. This supersedes PR #4734, which fixed a bug but didn't have proper spec coverage. Since that PR was submitted, a different commit (291a807c in PR #5176) fixed a similar bug at the same time. That caused a merge conflict in #4734. It was unknown if the same fix fixed both bugs, since there were no successfully implemented specs. These specs show that the bug was properly fixed. These specs were mostly already implemented by @b-ggs, just needed a little additional guidance.
| * | | | | Specs for multiple platform bug fix in outdated.chrismo2016-12-231-0/+27
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | This supersedes PR #4734, which fixed a bug but didn't have proper spec coverage. Since that PR was submitted, a different commit (291a807c in PR #5176) fixed a similar bug at the same time. That caused a merge conflict in #4734. It was unknown if the same fix fixed both bugs, since there were no successfully implemented specs. These specs show that the bug was properly fixed. These specs were mostly already implemented by @b-ggs, just needed a little additional guidance.
* | | | | | Auto merge of #5260 - bundler:seg-require-speed, r=indirectThe Bundler Bot2016-12-224-14/+16
|\ \ \ \ \ \ | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Improve -rbundler performance Along with https://github.com/rubygems/rubygems/pull/1801, speeds up `-rbundler` by ~5x Profiled via ```ruby # frozen_string_literal: true REQUIRE_TIMES = {} # rubocop:disable Style/MutableConstant module Kernel require "benchmark" alias_method :require_before_req, :require def require(path) ret = nil Benchmark.realtime do ret = require_before_req(path) end.tap do |t| if ret from = begin caller[2] rescue nil end ::REQUIRE_TIMES["#{path} : #{from}"] ||= t end end ret end end at_exit { puts REQUIRE_TIMES.to_a.sort_by(&:last).reverse.map {|k, v| "#{k} =>\n\t#{v}" } } ```
| * | | | | | [ParallelInstaller] Pre-require gem_remote_fetcher on 1.8.7seg-require-speedSamuel Giddins2016-12-211-0/+4
| | | | | | |
| * | | | | | [SharedHelpers] Speed up #clean_load_pathSamuel Giddins2016-12-211-10/+8
| | | | | | |
| * | | | | | [SharedHelpers] Document why we’re overriding #requirementSamuel Giddins2016-12-211-0/+1
| | | | | | |
| * | | | | | [SharedHelpers] Remove unused attributeSamuel Giddins2016-12-211-2/+0
| | | | | | |
| * | | | | | Improve -rbundler performance by using autoloadsSamuel Giddins2016-12-212-2/+3
| | | | | | |
* | | | | | | Auto merge of #5209 - bundler:seg-spec-improvements, r=indirectThe Bundler Bot2016-12-224-26/+41
|\ \ \ \ \ \ \ | |_|_|/ / / / |/| | | | | | | | | | | | | | | | | | | | | | | | | | | Random spec improvements Called "sam has a flight and forgot to run `rake spec:deps`"
| * | | | | | Define the artifice module in fail.rbseg-spec-improvementsSamuel Giddins2016-12-221-0/+2
| | | | | | | | | | | | | | | | | | | | | | | | | | | | This is so persistent will use our override class
| * | | | | | Also load net/https before stubbing Net::HTTPSamuel Giddins2016-12-211-1/+11
| | | | | | |
| * | | | | | Avoid using the Fail endpoint when we want to request against localhostSamuel Giddins2016-12-211-2/+2
| | | | | | |
| * | | | | | Don’t use Artifice for the fail endpointSamuel Giddins2016-12-211-10/+17
| | | | | | | | | | | | | | | | | | | | | | | | | | | | This avoids loading rack, which conflicts with some specs
| * | | | | | Verify no un-mocked requests are being made in the specsSamuel Giddins2016-12-211-1/+3
| | | | | | |
| * | | | | | [Source] Add an inspect methodSamuel Giddins2016-12-211-0/+4
| | | | | | | | | | | | | | | | | | | | | | | | | | | | This will avoid printing out every single cached spec
| * | | | | | Use the bundle helper when calling bundle_rubySamuel Giddins2016-12-211-12/+2
| | | | | | |
| * | | | | | Use a regexp instead of a convoluted start_with? method callSamuel Giddins2016-12-211-1/+1
| |/ / / / /
* | | | | | Auto merge of #5261 - bundler:seg-specs-2.4-fix, r=segiddinsThe Bundler Bot2016-12-212-4/+6
|\ \ \ \ \ \ | |/ / / / / |/| | | | | | | | | | | Fix the specs on Ruby 2.4
| * | | | | Fix the specs on Ruby 2.4seg-specs-2.4-fixSamuel Giddins2016-12-201-0/+2
| | | | | |
| * | | | | Remove rubygems.org from gemfiles in the specsSamuel Giddins2016-12-182-4/+4
| |/ / / /
* | | | | Auto merge of #5247 - colby-swandale:missing-new-line-on-fetch, r=segiddinsThe Bundler Bot2016-12-193-3/+22
|\ \ \ \ \ | |/ / / / |/| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Add a new line for before priring BundlerError errors This PR is fixing a problem where the warnings and errors on a `bundle install` using an invalid source are not being put on a newline correctly. ``` › bundle install Fetching source index from https://www.google.com/ Retrying fetcher due to error (2/4): Bundler::HTTPError Could not fetch specs from https://www.google.com/ Retrying fetcher due to error (3/4): Bundler::HTTPError Could not fetch specs from https://www.google.com/ Retrying fetcher due to error (4/4): Bundler::HTTPError Could not fetch specs from https://www.google.com/Could not fetch specs from https://www.google.com/ ``` This is happening because the new lines for the warnings are being printed before the error itself. This behaviour is happening due to another UI problem, see https://github.com/bundler/bundler/commit/2ac26bc88ba30b28aba6e11c9b2d04cdb0276a25. Maybe a better way of fixing this problem is to add a new line to the exception message itself rather then what i have submitted. Its a bit of a weird one. Let me know.