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
irony-server: init at 'same-version-as-irony-of-chosen-elpa` #21815
Conversation
56a045e
to
2912f08
Compare
2912f08
to
bcc708f
Compare
meta = { | ||
description = "The server part of irony."; | ||
homepage = "https://melpa.org/#/irony"; | ||
license = lib.licenses.free; |
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.
missing maintainers
and platforms
@@ -5094,6 +5094,8 @@ in | |||
|
|||
intercal = callPackage ../development/compilers/intercal { }; | |||
|
|||
irony-server = callPackage ../development/tools/irony-server/default.nix { }; |
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.
I think it's better to pass in irony
as an argument here instead of resolving it in the expression, eg.
irony-server = callPackage ../development/tools/irony-server/default.nix {
inherit (emacsPackagesNg.melpaStablePackages) irony;
};
in | ||
stdenv.mkDerivation rec { | ||
name = "irony-server-${version}"; | ||
version = irony.version; |
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.
You could write this as inherit (irony) version;
63165f3
to
267908e
Compare
@LnL7, adjusted to accord with your suggestions. |
267908e
to
7ceca3d
Compare
@LnL7, thank you for your review! |
Motivation for this change
Add the
irony-server
package, piggybacking on the autogeneratedirony
package from emacsPackagesNg.Things done
(nix.useSandbox on NixOS,
or option
build-use-sandbox
innix.conf
on non-NixOS)
nix-shell -p nox --run "nox-review wip"
./result/bin/
)