-
Notifications
You must be signed in to change notification settings - Fork 3.4k
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
[Ruby] Add explicit the "csv" gem dependency for Ruby 3.4 or later #37259
Comments
@heronshoes @otegami Are you interested in this? |
@kou I want to work on this. Thanks. |
Thanks! |
I understood. Thank you! |
heronshoes
added a commit
to heronshoes/arrow
that referenced
this issue
Sep 1, 2023
heronshoes
added a commit
to heronshoes/arrow
that referenced
this issue
Sep 1, 2023
kou
pushed a commit
that referenced
this issue
Sep 1, 2023
### Rationale for this change The "csv" gem is a default gem for now but it will be a bundled gem in Ruby 3.4. Bundled gem requires an explicit gem dependency. This will cause failure when we do `require "csv"` in Ruby 3.4 (this means in master branch of Ruby = ruby 3.3.0dev for now). See examples in #37259. ### What changes are included in this PR? Added `gem "csv"` in gemspec's runtime dependency of Red Arrow. ### Are these changes tested? Yes. ### Are there any user-facing changes? Users will use 'csv' gem from rubygems, not bundled, for Ruby < 3.4 . * Closes: #37259 Authored-by: Hirokazu SUZUKI <heronshoes877@gmail.com> Signed-off-by: Sutou Kouhei <kou@clear-code.com>
loicalleyne
pushed a commit
to loicalleyne/arrow
that referenced
this issue
Nov 13, 2023
### Rationale for this change The "csv" gem is a default gem for now but it will be a bundled gem in Ruby 3.4. Bundled gem requires an explicit gem dependency. This will cause failure when we do `require "csv"` in Ruby 3.4 (this means in master branch of Ruby = ruby 3.3.0dev for now). See examples in apache#37259. ### What changes are included in this PR? Added `gem "csv"` in gemspec's runtime dependency of Red Arrow. ### Are these changes tested? Yes. ### Are there any user-facing changes? Users will use 'csv' gem from rubygems, not bundled, for Ruby < 3.4 . * Closes: apache#37259 Authored-by: Hirokazu SUZUKI <heronshoes877@gmail.com> Signed-off-by: Sutou Kouhei <kou@clear-code.com>
dgreiss
pushed a commit
to dgreiss/arrow
that referenced
this issue
Feb 19, 2024
### Rationale for this change The "csv" gem is a default gem for now but it will be a bundled gem in Ruby 3.4. Bundled gem requires an explicit gem dependency. This will cause failure when we do `require "csv"` in Ruby 3.4 (this means in master branch of Ruby = ruby 3.3.0dev for now). See examples in apache#37259. ### What changes are included in this PR? Added `gem "csv"` in gemspec's runtime dependency of Red Arrow. ### Are these changes tested? Yes. ### Are there any user-facing changes? Users will use 'csv' gem from rubygems, not bundled, for Ruby < 3.4 . * Closes: apache#37259 Authored-by: Hirokazu SUZUKI <heronshoes877@gmail.com> Signed-off-by: Sutou Kouhei <kou@clear-code.com>
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Describe the enhancement requested
The "csv" gem is a default gem for now but it'll be just a bundled gem in Ruby 3.4.
If red-arrow doesn't have an explicit the "csv" gem dependency,
require "csv"
may be failed with Ruby 3.4 or later.Ruby master reports the following warning when we require red-arrow:
Component(s)
Ruby
The text was updated successfully, but these errors were encountered: