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
gnome-builder: add ctags #46594
gnome-builder: add ctags #46594
Conversation
@GrahamcOfBorg build gnome-builder |
No attempt on x86_64-darwin (full log) The following builds were skipped because they don't evaluate on x86_64-darwin: gnome-builder Partial log (click to expand)
|
Success on x86_64-linux (full log) Attempted: gnome-builder Partial log (click to expand)
|
Success on aarch64-linux (full log) Attempted: gnome-builder Partial log (click to expand)
|
We have decided that Builder should be used inside nix-shell/direnv, see the opening PR: #44217 |
I find this argument about ctags convincing, and it's also possible I was wrong to argue to the contrary for other build-time dependencies. Perhaps the rule should be this: anything that upstream gnome-builder tries to detect while running the meson configure script should be a packaging dependency here. My fear was about trying to track down all gnome-builder's subprocess invocations that assumed that at runtime various tools would be on It's also possible that some of those run-time dependencies should be turned into build-time dependencies. If it really only makes sense to use one version of a tool on a given platform, I'd expect that upstream would accept patches doing so, and then there's no need to maintain the large patches I was worried about here in nixpkgs. In short: @jtojnar, my suggestion would be to go ahead and merge this pull request, and we can think about whether this reasoning applies to other tools later. |
Gnome Builder uses Ctags for autocompletion.
From looking at this adding it as a build input should be sufficient.
cc @jameysharp @jtojnar
Things done
sandbox
innix.conf
on non-NixOS)nix-shell -p nox --run "nox-review wip"
./result/bin/
)nix path-info -S
before and after)