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
Update: easyjson, quicktemplate, statik, qtchan, vlang #67236
Conversation
@GrahamcOfBorg build easyjson quicktemplate statik qtchan vlang |
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.
pname (I believe) is preferred over name, also, "unstable" shouldn't be part of the package name.
I'm seeing a lot of these have |
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.
qtchan
is packaged improperly for a qt application, and needs updates for wrapQtAppsHook
.
See the updated manual
Migration looks like
- call the package with
libsForQt5.callPackage
- don't reference the
qt
package set in the package
See Tracking issue for wrapQtAppsHook #65399 (comment) - use
mkDerivation
instead ofstdenv.mkDerivation
- drop custom wrapper in
preFixup
Note that using qt5's mkDerivation
automatically includes wrapQtAppsHook
.
go2nix already checks if there are updates needed for deps.nix, there were none needed in this case. |
That should take care of the issues. |
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.
LGTM, executed all binaries. Checked qtchan
is wrapped.
Motivation for this change
General updates for my packages.
meguca/hydron will come later, pending a possible go2nix issue.
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 @