cpptools needs an override

[fzakaria]

cpptools downloads a binary and it's not executable or able to run.

❯ readlink -f /home/fmzakari/.vscode/extensions/ms-vscode.cpptools/debugAdapters/bin/OpenDebugAD7

/nix/store/cx9451yl9icy2zyh9shggq5vq8j0gy1c-vscode-extension-ms-vscode-cpptools-1.21.1/share/vscode/extensions/ms-vscode.cpptools/debugAdapters/bin/OpenDebugAD7

Nixpkgs has special handling for it (https://github.com/NixOS/nixpkgs/blob/nixos-24.05/pkgs/applications/editors/vscode/extensions/ms-vscode.cpptools/default.nix)

[deemp]

@fzakaria, there's quite a lot of code, and I don't want to duplicate it. I suggest to use the version from nixpkgs. Alternatively, make a fork of nixpkgs, update the extension there, and use that version.

[fzakaria]

okay

[fzakaria]

Should the extension be removed at least so people don't install it?

…

On Fri, Aug 23, 2024, 9:38 AM Danila Danko ***@***.***> wrote: Closed #69 <#69> as completed. — Reply to this email directly, view it on GitHub <#69 (comment)>, or unsubscribe <https://github.com/notifications/unsubscribe-auth/AAETXDRNIWUMJJAL44ZAIOLZS5QPJAVCNFSM6AAAAABLBK6FBSVHI2DSMVQWIX3LMV45UABCJFZXG5LFIV3GK3TUJZXXI2LGNFRWC5DJN5XDWMJTHE4TKNJRGU3TCMY> . You are receiving this because you were mentioned.Message ID: <nix-community/nix-vscode-extensions/issue/69/issue_event/13995515713@ github.com>

[deemp]

@fzakaria, nice idea! I'll provide a mechanism for removing particular extensions and will remove this extension.

[Naxdy]

Good call on removing the extension. However, it would be nice if upon trying to install it, nix would throw an error message during eval indicating what's wrong. error: attribute 'cpptools' missing isn't super descriptive as it stands.

[0x50F1A]

Good call on removing the extension. However, it would be nice if upon trying to install it, nix would throw an error message during eval indicating what's wrong. error: attribute 'cpptools' missing isn't super descriptive as it stands.

+1 just ran into this and wouldn't have figured it out without searching cpptools in this repo

[stereomato]

+2, just got hit with this

[deemp]

@Naxdy, @0x50F1A, @stereomato, is the solution in #87 OK?