buildRubyGem: Include platform in suffix for non-Ruby platforms

[amarshall]

Description of changes

This adheres to how Bundler handles such gems. We must also adjust the generated URL to use this full suffix.

Things done

Result of nixpkgs-review run on x86_64-linux 1 (nothing built).

I have tested this with this by applying this patch and nix-build -A rubyPackages.sorbet-static. Before this change, building non-ruby-platform gems was not trivially possible.

diff --git a/pkgs/top-level/ruby-packages.nix b/pkgs/top-level/ruby-packages.nix
index 232c48c739b1..4ffe6db90665 100644
--- a/pkgs/top-level/ruby-packages.nix
+++ b/pkgs/top-level/ruby-packages.nix
@@ -10,6 +10,18 @@
     };
     version = "7.1.3.2";
   };
+  sorbet-static = {
+    dependencies = [ ];
+    groups = [ "default" ];
+    platform = "x86_64-linux";
+    platforms = [ ];
+    source = {
+      remotes = ["https://rubygems.org"];
+      type = "gem";
+      sha256 = "5f6aaa02f28ceed4bf43dd40fdbe9a47c8d3589ccb9917bb1f9da90ee546ee8d";
+    };
+    version = "0.5.11294";
+  };
   actionmailbox = {
     dependencies = ["actionpack" "activejob" "activerecord" "activestorage" "activesupport" "mail" "net-imap" "net-pop" "net-smtp"];
     groups = ["default"];

• Built on platform(s)
  • x86_64-linux
  • aarch64-linux
  • x86_64-darwin
  • aarch64-darwin
• For non-Linux: Is sandboxing enabled in nix.conf? (See Nix manual)
  • sandbox = relaxed
  • sandbox = true
• Tested, as applicable:
  • NixOS test(s) (look inside nixos/tests)
  • and/or package tests
  • or, for functions and "core" functionality, tests in lib/tests or pkgs/test
  • made sure NixOS tests are linked to the relevant packages
• Tested compilation of all packages that depend on this change using nix-shell -p nixpkgs-review --run "nixpkgs-review rev HEAD". Note: all changes have to be committed, also see nixpkgs-review usage
• Tested basic functionality of all binary files (usually in ./result/bin/)
• 24.05 Release Notes (or backporting 23.05 and 23.11 Release notes)
  • (Package updates) Added a release notes entry if the change is major or breaking
  • (Module updates) Added a release notes entry if the change is significant
  • (Module addition) Added a release notes entry if adding a new NixOS module
• Fits CONTRIBUTING.md.

---

Add a 👍 reaction to pull requests you find important.