lmstudio: 0.3.6 -> 0.3.11

[crertel]

Update to 0.3.11.

This also marks lmstudio as broken on mac.

Things done

• 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/)
• 25.05 Release Notes (or backporting 24.11 and 25.05 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.

[deftdawg]

Reviewed this using nix run nixpkgs#nixpkgs-review -- pr 383546 and I'm happy to report it works well on my 6900XT, including 0.3.10's speculative decoding features (accessible from the beaker/lab icon after loading main model; near top right).

A couple of minor things I noticed, not specific to this PR:

1. lms command line interface is present in the AppImage (resources/app/.webpack/lms), but not present in the nix output... that maybe that's an easy add by duplicating:
   https://github.com/NixOS/nixpkgs/blob/f7dafd65c6e75dc30b97550c9bb695cac85e79bc/pkgs/by-name/lm/lmstudio/linux.nix#L30
   and changing it to something like:

   install -D ${appimageContents}/resources/app/.webpack/lms -t $out/bin

   though I'm not certain of that
2. The main binary is actually lm-studio within the AppImage rather than lmstudio

[deftdawg]

Turns out it's slightly more complicated, need to patchelf lms; appending these to extraInstallCommands does it:

      # lms cli tool
      install -m 755 ${appimageContents}/resources/app/.webpack/lms $out/bin/

      ${final.patchelf}/bin/patchelf --set-interpreter "${final.stdenv.cc.bintools.dynamicLinker}" \
      --set-rpath "${final.stdenv.cc.cc.lib}/lib:${final.stdenv.cc.cc.lib}/lib64:$out/lib:${final.lib.makeLibraryPath [final.stdenv.cc.cc.lib]}" $out/bin/lms

Here's the full overlay:

(final: prev: {
  lmstudio = let
    version = "0.3.10";
    rev = "6";
    pname = "lmstudio";
    
    src = final.fetchurl {
      url = "https://installers.lmstudio.ai/linux/x64/${version}-${rev}/LM-Studio-${version}-${rev}-x64.AppImage";
      hash = "sha256-sAGU20gaDaFo5jvuCcN4tkVTlqvuXVCqvrHyl3yg384=";
    };
    appimageContents = final.appimageTools.extractType2 { inherit pname version src; };
  in final.appimageTools.wrapType2 {
    inherit pname version src;

    extraPkgs = pkgs: with pkgs; [
      ocl-icd
    ];

    extraInstallCommands = ''
      mkdir -p $out/share/applications
      cp -r ${appimageContents}/usr/share/icons $out/share
      install -m 444 ${appimageContents}/lm-studio.desktop $out/share/applications/      
      substituteInPlace $out/share/applications/lm-studio.desktop \
        --replace 'Exec=AppRun --no-sandbox %U' 'Exec=lmstudio'

      # lms cli tool
      install -m 755 ${appimageContents}/resources/app/.webpack/lms $out/bin/

      ${final.patchelf}/bin/patchelf --set-interpreter "${final.stdenv.cc.bintools.dynamicLinker}" \
      --set-rpath "${final.stdenv.cc.cc.lib}/lib:${final.stdenv.cc.cc.lib}/lib64:$out/lib:${final.lib.makeLibraryPath [final.stdenv.cc.cc.lib]}" $out/bin/lms
    '';
  };
})

@cig0 can you approve/merge the PR? Then I can try adding lms to another PR (or you can roll it in to this).

[crertel]

@deftdawg that's freakin' awesome, great work! I think it should go in its own PR once this one is merged.

[crertel]

Updated to match the upstream change to overridable urls and hashes.

[deftdawg]

I used this PR on X86_64 and it's good to go.

[deftdawg]

0.3.11 is out now... it kinda sucks we can't get enough reviewers to merge promptly

[crertel]

@deftdawg ...sigh, lemme update again.

[deftdawg]

sha256-T8j0l2ZP53Zc0hgb2EyFeR0mH5YrNlz4UfzN0rO7bKU=

[eeedean]

macOS-Issue persists unfortunately. So no change on darwin side of things.

[deftdawg]

Yep, this PR marks it as broken for Mac, I've pinged LM Studio many times on that issue asking if they'll fix it, they said they would but then went silent.

Anyway need to get this merged... @crertel, not sure what you committed but it doesn't show it having bumped the version to 0.3.11 here (don't forget to edit the issue title too)

[crertel]

@deftdawg should be good now, I think.

[crertel]

@deftdawg I think that tees you up for your PR about pulling lms out (and maybe some of their other stuff)?

[deftdawg]

@deftdawg I think that tees you up for your PR about pulling lms out (and maybe some of their other stuff)?

Done -> #386864 😄