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
mkvtoolnix: use wrapQtApp #66684
mkvtoolnix: use wrapQtApp #66684
Conversation
@@ -4,6 +4,7 @@ | |||
, withGUI ? true | |||
, qtbase ? null | |||
, qtmultimedia ? null | |||
, wrapQtAppsHook ? null |
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.
even though it's supposed to be denote it's optional, weird to see extra whitespace.
Personal preference. Don't feel strongly about it.
, wrapQtAppsHook ? null | |
, wrapQtAppsHook ? null |
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’ve just followed the style of the two lines above.
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.
either way, I don't have much of a preference. Just a slight preference :)
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.
nix-review
passes on NixOS
diff LGTM
binaries seem to all work
leaf package
@GrahamcOfBorg build mkvtoolnix |
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've reproduce the issue on master, upon building the package I ensured the correct binary was wrapped.
Executes without an error at runtime ✔️
922377e
to
b43c368
Compare
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)