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
notepadqq: use qt5's mkDerivation #66324
Conversation
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.
This is because the executable is a bash script and wrapQtAppsHook
only wraps ELF headers.
You should use wrapQtApp $out/bin/notepadqq
in preFixup
.
4df9831
to
2fb49e7
Compare
Thanks, should have realized it was a bash script.. |
@@ -13,7 +12,7 @@ in stdenv.mkDerivation { | |||
}; | |||
|
|||
nativeBuildInputs = [ | |||
pkgconfig which qttools | |||
pkgconfig which qttools wrapQtAppsHook |
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.
pkgconfig which qttools wrapQtAppsHook | |
pkgconfig which qttools |
is added automatically
2fb49e7
to
ff8987d
Compare
tried a nix-review and while it compiles, it seems stuck on start:
then nothing happens |
I also get that warning on startup, it also complains about some missing icons, but then runs just fine. |
@@ -1,9 +1,8 @@ | |||
{ stdenv, fetchFromGitHub, pkgconfig, which, qtbase, qtsvg, qttools, qtwebkit}: | |||
{ mkDerivation, lib, fetchFromGitHub, pkgconfig, which, qtbase, qtsvg, qttools, qtwebkit, wrapQtAppsHook }: |
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.
{ mkDerivation, lib, fetchFromGitHub, pkgconfig, which, qtbase, qtsvg, qttools, qtwebkit, wrapQtAppsHook }: | |
{ mkDerivation, lib, fetchFromGitHub, pkgconfig, which, qtbase, qtsvg, qttools, qtwebkit }: |
ff8987d
to
6b1d73f
Compare
@teto can you strace the binary? I ran nix-review on this exact pr and had no trouble running the binary. So I guess there is some external state involved? |
I've noticed this in other applications since switching to qt wrappers, perhaps post the error in the tracking issue. |
Here is an strace result |
Motivation for this change
See #65399
Things done
sandbox
innix.conf
on non-NixOS)nix-shell -p nix-review --run "nix-review wip"
./result/bin/
)nix path-info -S
before and after)Notify maintainers
cc @rszibele