| Commit message (Collapse) | Author | Age | Files | Lines |
| |
|
|\
| |
| |
| |
| |
| |
| |
| | |
5521: bundler gem: Support test-unit as a testing framework r=hsbt a=kou
Co-authored-by: Kouhei Sutou <kou@clear-code.com>
|
| | |
|
|\ \
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | | |
7545: Enforce Windows CI r=deivid-rodriguez a=deivid-rodriguez
### What was the end-user problem that led to this PR?
The problem was that, although we have a Windows CI, errors are ignored, so whenever we break something on Windows, we don't notice, and having a CI become useless.
### What was your diagnosis of the problem?
My diagnosis was that we should make the Windows CI required.
### What is your fix for the problem, implemented in this PR?
My fix is to actually go ahead and fix some issues, and skip the rest.
Co-authored-by: David Rodríguez <deivid.rodriguez@riseup.net>
|
| | | |
|
| | | |
|
| | | |
|
| | | |
|
| | | |
|
| | |
| | |
| | |
| | |
| | | |
Since it doesn't seem to work, I guess Windows ruby loads it at some
point?
|
| | | |
|
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | | |
Passing paths should work in most cases, but on Windows the driver
letter is interpreted as the scheme and causes some case mismatches
because
```
irb> URI.parse("E:").to_s
=> "e:"
```
We fix this by passing file URI's instead.
|
| | | |
|
|/ /
| |
| |
| | |
Github actions has lowercase drive letters.
|
|\ \
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | | |
7550: Enable parallel specs again r=deivid-rodriguez a=deivid-rodriguez
### What was the end-user problem that led to this PR?
The problem was that specs are slow.
### What is your fix for the problem, implemented in this PR?
My fix is to run them in parallel.
We enabled this a while ago, but run into some issues. I want to try again.
Co-authored-by: David Rodríguez <deivid.rodriguez@riseup.net>
|
| | | |
|
| | | |
|
| | | |
|
| | |
| | |
| | |
| | | |
No need for an extra subprocess.
|
| | |
| | |
| | |
| | | |
It overcomplicates things.
|
| | | |
|
| | |
| | |
| | |
| | |
| | |
| | |
| | | |
We run into a race condition when testing in parallel when all
processors install test gems to their test folder (due to shared gem
cache). So I figure we can instead run this setup sequentially
beforehand.
|
| | | |
|
| | | |
|
| | |
| | |
| | |
| | | |
This is precisely what the `--conservative` flag to `gem install` does.
|
|/ / |
|
|\ \
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | | |
7548: Fix escaping of some subprocesses during specs r=deivid-rodriguez a=deivid-rodriguez
### What was the end-user problem that led to this PR?
The problem was that some specs are failing on Windows due to commands like the following failing:
```
$ C:/hostedtoolcache/windows/Ruby/2.6.3/x64/bin/ruby.exe -Id:/a/bundler/bundler/lib -w -e \
<<EOS
begin
$:.unshift File.expand_path("bundle")
require "bundler/setup"
require "actionpack"
puts ACTIONPACK
require "spec"
rescue LoadError => e
$stderr.puts "ZOMG LOAD ERROR"# if e.message.include?("-- spec")
end
EOS
-e:2: syntax error, unexpected backslash
\$:.unshift File.expand_path("...
^
-e:10: syntax error, unexpected backslash
\$stderr.puts "ZOMG LOAD ERROR...
^
# $? => 1
```
### What was your diagnosis of the problem?
My diagnosis was that if the code being run via `ruby -e` in a subprocess include "$", it was not being properly escaped under Windows.
### What is your fix for the problem, implemented in this PR?
My fix is to use `shellsplit` and `shellescape`.
### Why did you choose this fix out of the possible options?
I chose this fix because after a bit of testing it seemed to do the trick.
Co-authored-by: David Rodríguez <deivid.rodriguez@riseup.net>
|
| | | |
|
| | |
| | |
| | |
| | |
| | | |
So that we don't need to manually escape `$`, that seems to give trouble
on Windows.
|
| | | |
|
| | | |
|
|/ / |
|
|\ \
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | | |
7546: Don't rename folders while being inside r=deivid-rodriguez a=deivid-rodriguez
### What was the end-user problem that led to this PR?
The problem was that some specs were failing under Windows, with error messages like the following:
```
Permission denied @ rb_file_s_rename - (D:/a/1/s/tmp/1/bundled_app, D:/a/1/s/tmp/1/bundled_app2)
Failure/Error: FileUtils.mv(bundled_app, "#{bundled_app}2")
Errno::EACCES:
Permission denied @ rb_file_s_rename - (D:/a/1/s/tmp/1/bundled_app, D:/a/1/s/tmp/1/bundled_app2)
Commands:
$ C:/hostedtoolcache/windows/Ruby/2.6.3/x64/bin/ruby.exe -ID:/a/1/s/lib\;D:/a/1/s/spec -rsupport/hax \
-rsupport/artifice/fail D:/a/1/s/exe/bundle install --path D:/a/1/s/tmp/1/bundled_app/bundle \
--standalone
Fetching d:/a/1/s/tmp/1/libs/devise-1.0
Fetching source index from file:///D:/a/1/s/tmp/1/gems/remote1/
Resolving dependencies...
Fetching rake 12.3.2
Installing rake 12.3.2
Fetching activesupport 2.3.2
Installing activesupport 2.3.2
Fetching actionmailer 2.3.2
Installing actionmailer 2.3.2
Fetching actionpack 2.3.2
Installing actionpack 2.3.2
Fetching activerecord 2.3.2
Installing activerecord 2.3.2
Fetching activeresource 2.3.2
Installing activeresource 2.3.2
Using bundler 2.2.0.dev
Using devise 1.0 from d:/a/1/s/tmp/1/libs/devise-1.0 (at master@1cdf130)
Fetching rails 2.3.2
Installing rails 2.3.2
Bundle complete! 2 Gemfile dependencies, 9 gems now installed.
Bundled gems are installed into `./bundle`
[DEPRECATED] The `--path` flag is deprecated because it relies on being remembered across bundler invocations, which bundler will no longer do in future versions. Instead please use `bundle config set path 'D:/a/1/s/tmp/1/bundled_app/bundle'`, and stop using this flag
# $? => 0
Shared Example Group: "common functionality" called from ./spec/install/gems/standalone_spec.rb:136
Shared Example Group: "bundle install --standalone" called from ./spec/install/gems/standalone_spec.rb:328
./spec/install/gems/standalone_spec.rb:32:in `block (3 levels) in <top (required)>'
./spec/spec_helper.rb:111:in `block (3 levels) in <top (required)>'
./spec/spec_helper.rb:111:in `block (2 levels) in <top (required)>'
./spec/spec_helper.rb:78:in `block (2 levels) in <top (required)>'
./spec/support/rubygems_ext.rb:87:in `load'
./spec/support/rubygems_ext.rb:87:in `gem_load_and_activate'
./spec/support/rubygems_ext.rb:45:in `gem_load'
```
### What was your diagnosis of the problem?
My diagnosis was that Windows doesn't like renaming folders while being inside them.
### What is your fix for the problem, implemented in this PR?
My fix is to move away from the folder to the root of the repo before renaming it.
### Why did you choose this fix out of the possible options?
I chose this fix because it fixes the failures under Windows while still achieving the same thing.
Co-authored-by: David Rodríguez <deivid.rodriguez@riseup.net>
|
| | |
| | |
| | |
| | | |
Since Windows doesn't like that.
|
| | |
| | |
| | |
| | | |
Since their naming leads to code that reads a bit weird without a block.
|
| | | |
|
| | | |
|
| | | |
|
| | | |
|
|/ / |
|
|\ \
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | | |
7529: Remove optional require net-http-pipeline from vendored net-http-persistent. r=deivid-rodriguez a=marciotoshio
### What was the end-user problem that led to this PR?
See issue https://github.com/bundler/bundler/issues/7521.
### What was your diagnosis of the problem?
My diagnosis was If you have net-http-pipeline gem installed bundler loads it and you can't vendor it.
### What is your fix for the problem, implemented in this PR?
My fix is remove the optional require of net-http-pipeline.
### Why did you choose this fix out of the possible options?
I chose this fix because was a recommendation in the issue by @deivid-rodriguez
Fixes #7521.
Co-authored-by: David Rodríguez <deivid.rodriguez@riseup.net>
Co-authored-by: Marcio Toshio <marcio@tosh.io>
|
| | | |
|
| | | |
|
|\ \ \
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | | |
7539: Migrate Windows CI to Github Actions r=deivid-rodriguez a=deivid-rodriguez
### What was the end-user problem that led to this PR?
The problem was that current Windows CI is very slow.
### What is your fix for the problem, implemented in this PR?
My fix is to try migrate it to Github Actions to see if it gets better. It didn't really get much better, but I think Github Actions is better than Azure Pipelines.
Regarding the implementation, Github Actions environment doesn't seem to include an `rb-readline` version, and specs using `readline` just hang. My "solution" is to skip those specs for the time being.
Co-authored-by: David Rodríguez <deivid.rodriguez@riseup.net>
|
| | | | |
|
| | | | |
|
| | | | |
|
| | | | |
|
| | | | |
|
| | | | |
|