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
Geany: 1.30 -> 1.30.1 #24067
Geany: 1.30 -> 1.30.1 #24067
Conversation
@frlan, thanks for your PR! By analyzing the history of the files in this pull request, we identified @domenkozar, @bbenoist and @vcunat to be potential reviewers. |
Cosmetic change
buildInputs = [ gtk2 which pkgconfig intltool file ]; | ||
|
||
nativeBuildInputs = [ pkgconfig ]; | ||
buildInputs = [ gtk2 which intltool file ]; |
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.
intltool also belongs to nativeBuildInputs.
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.
@Mic92 was thinking it may be, will fix it :) running nox-review right now before I merge
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.
Seems to be missing deps
Motivation for this change
This PR updates Geany to 1.30 as there was a bug found in 1.30 causing e.g. calltips popping up on wrong display when using Geany in a multi-display environment and not running on left top monitor.
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/
)