spoom icon indicating copy to clipboard operation
spoom copied to clipboard

Problem with `Gemfile.lock`

Open andyw8 opened this issue 1 year ago • 3 comments

Running bundle locally on main is causing changes to Gemfile.lock:

spoom main % g diff
diff --git a/Gemfile.lock b/Gemfile.lock
index d0a6850..2cf1a68 100644
--- a/Gemfile.lock
+++ b/Gemfile.lock
@@ -40,8 +40,8 @@ GEM
     racc (1.7.3)
     rainbow (3.1.1)
     rake (13.1.0)
-    rbi (0.1.6)
-      prism (>= 0.18.0, < 0.20)
+    rbi (0.1.9)
+      prism (>= 0.18.0, < 0.25)
       sorbet-runtime (>= 0.5.9204)
     rdoc (6.6.2)
       psych (>= 4.0.0)
@@ -71,7 +71,6 @@ GEM
       sorbet-static (= 0.5.11287)
     sorbet-runtime (0.5.11287)
     sorbet-static (0.5.11287-universal-darwin)
-    sorbet-static (0.5.11287-x86_64-linux)
     sorbet-static-and-runtime (0.5.11287)
       sorbet (= 0.5.11287)
       sorbet-runtime (= 0.5.11287)
@@ -93,10 +92,7 @@ GEM
       yard (>= 0.9)
 
 PLATFORMS
-  arm64-darwin-21
   arm64-darwin-23
-  universal-darwin-22
-  x86_64-linux
 
 DEPENDENCIES
   bundler (>= 2.2.10)

andyw8 avatar Mar 20 '24 14:03 andyw8

If we check out the second-newest commit on main, 614084b54bc7b4bbaef8743b5620055d1fa9c3a5, bundle runs fine without changing to Gemfile.lock. So f529cdcab2a8c4e585ffe1a4e24fe850b260db4c is to blame.

andyw8 avatar Mar 21 '24 14:03 andyw8

~The problem seems to begin with rbi 0.1.8, it's fine with 0.1.7.~

andyw8 avatar Mar 21 '24 14:03 andyw8

There must be a bug in Bundler, try this on main:

  • Add gem "rbi" to Gemfile (probably any gem would work)
  • Run bundle
  • Run git diff and observe that the PLATFORMS haven't changed
  • Run bundle again
  • Run git diff and observe this time the PLATFORMS have changed

andyw8 avatar Mar 21 '24 18:03 andyw8

Opened an issue on bundler: https://github.com/rubygems/rubygems/issues/7536

egiurleo avatar Mar 25 '24 17:03 egiurleo

https://github.com/Shopify/spoom/pull/528 got auto-closed because https://github.com/Shopify/spoom/pull/534 was opened, and it seems to no longer have the PLATFORMS problem. 🤔

andyw8 avatar Apr 02 '24 00:04 andyw8

I'll close this since it seems to have resolved itself.

andyw8 avatar Apr 02 '24 20:04 andyw8

Should I close the bundler issue since this is no longer reproducible?

egiurleo avatar Apr 03 '24 14:04 egiurleo

Yeah, let's close and we can re-open if it happens again.

andyw8 avatar Apr 03 '24 14:04 andyw8