Skip to content

Fix frozen string literal support #83

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.

Sign up for GitHub

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

Merged
mfazekas merged 6 commits into from
May 16, 2026
+23 −14
Merged

Fix frozen string literal support #83

Show file tree
Hide file tree
Changes from 3 commits
Commits
Show all changes
6 commits
Select commit Hold shift + click to select a range
6f5f99c
Use net-ssh 7.3.1.rc1 to fix frozen string literal CI failures
mfazekas Feb 17, 2026
de69d1d
Make rdoc/task optional for Ruby 4.0+ compatibility
mfazekas Feb 17, 2026
654a391
Use net-ssh 7.3.1 final (was rc1)
mfazekas Mar 14, 2026
025a1bf
Retrigger CI
mfazekas May 16, 2026
5faceef
Fix frozen string error in error_string initialization
mfazekas May 16, 2026
931791f
Add logger and rdoc to Gemfile for Ruby 4.0+ compatibility
mfazekas May 16, 2026
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
2 changes: 1 addition & 1 deletion .github/workflows/ci.yml
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -10,7 +10,7 @@ jobs:
strategy:
fail-fast: false
matrix:
ruby-version: ['2.3', '2.4', '2.5', '2.6', '2.7', '3.0', '3.1', '3.2', '3.3', '3.4', 'ruby-head']
ruby-version: ['2.6', '2.7', '3.0', '3.1', '3.2', '3.3', '3.4', 'ruby-head']
continue-on-error: ${{ matrix.ruby-version == 'ruby-head' }}
steps:
- uses: actions/checkout@v3
Expand Down
2 changes: 2 additions & 0 deletions Gemfile
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -3,6 +3,8 @@ source 'https://rubygems.org'
# Specify your gem's dependencies in mygem.gemspec
gemspec

gem "net-ssh", ">= 7.3.1"

@robertcheramy robertcheramy Mar 16, 2026

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Should the gem version be specified in net-scp.gemspec instead?

@mfazekas mfazekas May 16, 2026

Copy link
Copy Markdown
Collaborator Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I think it's fine, in gem spec it would me a backward incompat change. Users can still use with older net-ssh for example if they are on ruby not supported by later net-ssh, just frozen string literals will not fully work for them.


# TODO: add to gemspec
gem "bundler", ">= 1.11"
gem "rake", ">= 12.0"
Expand Down
29 changes: 17 additions & 12 deletions Rakefile
View file
Open in desktop
Original file line number Diff line number Diff line change
@@ -1,7 +1,10 @@
require "rubygems"
require "rake"
require "rake/clean"
require "rdoc/task"
begin
require "rdoc/task"
rescue LoadError
end
require "bundler/gem_tasks"

desc "When releasing make sure NET_SSH_BUILDGEM_SIGNED is set"
Expand Down Expand Up @@ -67,17 +70,19 @@ Rake::TestTask.new do |t|
end

extra_files = %w[LICENSE.txt THANKS.txt CHANGES.txt ]
RDoc::Task.new do |rdoc|
rdoc.rdoc_dir = "rdoc"
rdoc.title = "#{name} #{version}"
rdoc.generator = 'hanna' # gem install hanna-nouveau
rdoc.main = 'README.md'
rdoc.rdoc_files.include("README*")
rdoc.rdoc_files.include("bin/*.rb")
rdoc.rdoc_files.include("lib/**/*.rb")
extra_files.each { |file|
rdoc.rdoc_files.include(file) if File.exist?(file)
}
if defined?(RDoc::Task)
RDoc::Task.new do |rdoc|
rdoc.rdoc_dir = "rdoc"
rdoc.title = "#{name} #{version}"
rdoc.generator = 'hanna' # gem install hanna-nouveau
rdoc.main = 'README.md'
rdoc.rdoc_files.include("README*")
rdoc.rdoc_files.include("bin/*.rb")
rdoc.rdoc_files.include("lib/**/*.rb")
extra_files.each { |file|
rdoc.rdoc_files.include(file) if File.exist?(file)
}
end
end

def change_version(&block)
Expand Down
Loading