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
gtkspell3: add valaSupport #43129
gtkspell3: add valaSupport #43129
Conversation
@@ -1,4 +1,4 @@ | |||
{stdenv, fetchurl, gtk3, aspell, pkgconfig, enchant, isocodes, intltool, gobjectIntrospection}: | |||
{stdenv, fetchurl, gtk3, aspell, pkgconfig, enchant, isocodes, intltool, gobjectIntrospection, valaSupport ? true, vala_0_40}: |
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.
Let's just build it always with vala support, it should be pretty small and go to dev output anyway.
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.
@jtojnar Did the thing
configureFlags = [ "--enable-introspection" ]; | ||
configureFlags = [ | ||
"--enable-introspection" | ||
"--enable-vala=yes" |
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 can get rid of the yes here too.
@@ -11,11 +11,14 @@ stdenv.mkDerivation rec { | |||
sha256 = "09jdicmpipmj4v84gnkqwbmj4lh8v0i6pn967rb9jx4zg2ia9x54"; | |||
}; | |||
|
|||
nativeBuildInputs = [ pkgconfig intltool gobjectIntrospection ]; | |||
nativeBuildInputs = [ pkgconfig intltool gobjectIntrospection vala_0_40 ]; |
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 would just use vala
here. It is backwards compatible most of the time and updating every file when new version is released is annoying.
@GrahamcOfBorg build gtkspell3 |
Success on x86_64-linux (full log) Attempted: gtkspell3 Partial log (click to expand)
|
Success on x86_64-darwin (full log) Attempted: gtkspell3 Partial log (click to expand)
|
Success on aarch64-linux (full log) Attempted: gtkspell3 Partial log (click to expand)
|
Motivation for this change
I was packaging elementary-code and it cried about gtkspell3 not having vala support.
cc @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)