Skip to content

Conversation

@blacksam07
Copy link
Contributor

Board:


Description:

  • Add missing bundle require
Replacement completed successfully.
== Installing dependencies ==
bin/setup:16:in 'block in <main>': undefined local variable or method 'bundler_version' for main (NameError)

  system! "bin/web 'gem install bundler -v '#{bundler_version}''"

Notes:


Tasks:

  • Add each element in this format

Risk:


Preview:

@sonarqubecloud
Copy link

@@ -1,5 +1,7 @@
#!/usr/bin/env ruby
require 'fileutils'
require_relative 'bundler'
Copy link
Member

@santib santib Nov 19, 2025

Choose a reason for hiding this comment

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

I don't think this is needed. Notice that this file comes from Rails https://github.com/rails/rails/blob/cf4e3c508adf930eb70e4b3e6e16d2ca83b7e9b3/railties/lib/rails/generators/rails/app/templates/bin/setup.tt#L4

what is the exact command that you are running?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

This error comes when run the bootstrap.sh script, and I saw the file history and previous to the Rails 8 update have this require

7474d9e#diff-83a07cb3b17898a530fb9aadd9b41c7fc9217cb6d0cdbcd6fbb0577cba79780bL3

Copy link
Member

@santib santib left a comment

Choose a reason for hiding this comment

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

Good catch, not sure why this is needed but it's true that it was removed by mistake 👍

@santib santib merged commit dfaf8e5 into main Nov 19, 2025
5 checks passed
@santib santib deleted the fix-setup-script branch November 19, 2025 18:48
@blacksam07
Copy link
Contributor Author

@santib We are using a different was to install bundler, and we are setting an specify bundler version that way required bundler in this file, maybe with the base setup is not need it

base file

  puts "== Installing dependencies =="
  system! "gem install bundler --conservative"
  system("bundle check") || system!("bundle install")

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants