New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
vscode-extensions: fix aliases #104796
vscode-extensions: fix aliases #104796
Conversation
0f75694
to
ace93fb
Compare
ace93fb
to
348ffdf
Compare
Thanks! I used an overlay with rev 348ffdf. I installed vscode-with-extensions with vscodium, some extensions from nixpkgs (some regular, the ms-vscode.Go alias, ms-vscode.cpptools which were removed by the old alias logic) as well as some extensions from the vscode marketplace. The extensions seem to be ok. I can jump to function definitions in go files, I can jump to variable definitions in c files. |
I can confirm, this closes #104783 for me. |
ms-vscode.Go = golang.Go; | ||
} | ||
); | ||
ms-vscode = lib.recursiveUpdate super.ms-vscode { inherit (super.golang) Go; }; |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
For me this approach feels a little fragile:
- If all ms-vscode extensions are dropped, the aliases need to be changed, because super.ms-vscode does not exist anymore. Right?
- If then a ms-vscode extension is added again, it is not there until the aliases overlay is adjusted again.
Is it possible to use mkMerge or something like this here?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Ah, you did exactly that in the other PR.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
If all ms-vscode extensions are dropped, the aliases need to be changed, because super.ms-vscode does not exist anymore. Right?
If then a ms-vscode extension is added again, it is not there until the aliases overlay is adjusted again.
This shouldn't be too much of an issue, because this is all co-located within the same file. The more elegant implementation is in the other PR, but it might be "controversial" to add a recursiveExtends function.
I was able to rebuild my |
Motivation for this change
closes #104783
I converted it over to an overlay, in case we ever want to also implement an overrides.nix, which would be necessary for doing an auto-update script
cc @buckley310 @priegger
Things done
sandbox
innix.conf
on non-NixOS linux)nix-shell -p nixpkgs-review --run "nixpkgs-review wip"
./result/bin/
)nix path-info -S
before and after)