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
qjournalctl: init at 0.6.2 #78815
qjournalctl: init at 0.6.2 #78815
Conversation
Submitted as is. feel free to alprove/reject. |
|
||
meta = with stdenv.lib; { | ||
description = "Qt-based Graphical User Interface for systemd's journalctl command"; | ||
homepage = https://github.com/pentix/qjournalctl; |
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.
Quote URL please.
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.
Looks good, just few notes.
''; | ||
|
||
nativeBuildInputs = [ qmake pkgconfig libssh ]; | ||
buildInputs = [ qtbase 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.
wrapQtAppsHook
belongs to nativeBuildInputs
.
substituteInPlace qjournalctl.pro --replace /usr/ $out/ | ||
''; | ||
|
||
nativeBuildInputs = [ qmake pkgconfig libssh ]; |
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.
In new expressions, pkg-config
should be used instead of the pkgconfig
alias.
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.
libssh
belongs to buildInputs
.
}; | ||
|
||
postPatch = '' | ||
substituteInPlace qjournalctl.pro --replace /usr/ $out/ |
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.
It might be nice to have this configurable through makeFlags
like here: jamulussoftware/jamulus#247
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'm trying to pick up this PR. How could this be achieved? By providing a patch for the .pro file?
qmake, | ||
pkgconfig, | ||
libssh, | ||
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.
wrapQtAppsHook | |
wrapQtAppsHook |
@@ -0,0 +1,36 @@ | |||
{ |
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 marked this as stale due to inactivity. → More info |
Closing in favour of #129246 |
Motivation for this change
bring back https://github.com/NixOS/nixpkgs/pull/55762/commits from dead
Things done
Copy pasted aanderse's code plus some qt specific changes plus a version bump
sandbox
innix.conf
on non-NixOS linux)nix-shell -p nixpkgs-review --run "nixpkgs-review wip"
./result/bin/
)nix path-info -S
before and after)