Skip to content

Commit

Permalink
Add pry and pry-byebug to see if I can use it in the CLI
Browse files Browse the repository at this point in the history
  • Loading branch information
jmarrec committed Apr 19, 2024
1 parent 67d3ced commit 7a4055b
Show file tree
Hide file tree
Showing 3 changed files with 5 additions and 2 deletions.
3 changes: 3 additions & 0 deletions Gemfile
Original file line number Diff line number Diff line change
Expand Up @@ -108,5 +108,8 @@ else

end

gem 'pry', '~> 0.14.2'
gem 'pry-byebug', '~> 3.10'

# leave this line in for now as we may try to get nokogiri to compile correctly on windows
# gem 'nokogiri', '= 1.11.0.rc1.20200331222433', :github => 'jmarrec/nokogiri', :ref => 'MSVC_support' # master of 2020-03-31 + gemspec commit
2 changes: 1 addition & 1 deletion build_openstudio_gems.rb
Original file line number Diff line number Diff line change
Expand Up @@ -136,7 +136,7 @@ def make_package(install_dir, tar_exe, expected_ruby_version, bundler_version)
lib_ext = RbConfig::CONFIG['LIBEXT']
libs = Dir.glob("./openstudio-gems/**/*.#{lib_ext}")
lib_names_woext = Set.new(libs.map{|lib| File.basename(lib, File.extname(lib)) })
expected = Set.new(["jaro_winkler_ext", "libll", "liboga", "msgpack"]) # unf_ext: disabled with json_schemer
expected = Set.new(["jaro_winkler_ext", "libll", "liboga", "msgpack", "byebug"]) # unf_ext: disabled with json_schemer
if !is_unix
expected.add("sqlite3_native") # TODO: I don't understand why we don't have it yet...
end
Expand Down
2 changes: 1 addition & 1 deletion lib/rubygems_plugin.rb
Original file line number Diff line number Diff line change
Expand Up @@ -89,7 +89,7 @@ def post_install
elsif extname.to_s == "sqlite3"
extname = "sqlite3_native"
extconf_args = ["--enable-system-libraries", "--with-pkg-config=pkgconf"]
elsif extname.to_s == 'unfext'
elsif ['unfext', 'byebug'].include?(extname.to_s)
# No-op
else
puts "Warning: rubygems_plugin.post_install: no configuration given for extension_dir=#{extension_dir}"
Expand Down

0 comments on commit 7a4055b

Please sign in to comment.