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
brewtarget: 2.3.1 -> 3.0.6 #217648
base: master
Are you sure you want to change the base?
brewtarget: 2.3.1 -> 3.0.6 #217648
Conversation
43d4f84
to
f53114f
Compare
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.
Actually, please make sure that the package name in commit and PR title match the pname
exactly, including the case
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.
@avnik Could you please also replace Qt's mkDerivation
with stdenv.mkDerivation
to fix
#180841
For that to work wrapQtAppsHook
will be needed.
If you can, please also update it to 3.0.7 which has already been released in the meantime:
https://github.com/Brewtarget/brewtarget/releases/tag/v3.0.7
Also this PR would fix the issue with qtwebkit for brewtarget (for cross-reference): |
Drop insecure qtwebkit
@totoroot I fixed all suggestions (except bumping to 3.0.7 -- newer brewtarget depends on boost_stacktrace_backtrace which not yet packaged), also rebased PR on newer master |
nativeBuildInputs = [ cmake ]; | ||
buildInputs = [ qtbase qttools qtmultimedia qtwebkit qtsvg ]; | ||
nativeBuildInputs = [ cmake qt5.wrapQtAppsHook ]; | ||
buildInputs = [ boost xalanc xercesc ] ++ |
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.
buildInputs = [ boost xalanc xercesc ] ++ | |
buildInputs = [ boost curl xalanc xercesc ] ++ |
, fetchFromGitHub | ||
, bash | ||
, boost |
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.
, boost | |
, boost | |
, curl |
@avnik Sorry for the late response. The last I checked, it did not build on darwin, which can be seen in ofborg's tests as well. |
@@ -2958,7 +2958,7 @@ with pkgs; | |||
|
|||
brakeman = callPackage ../development/tools/analysis/brakeman { }; | |||
|
|||
brewtarget = libsForQt5.callPackage ../applications/misc/brewtarget { } ; |
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.
Why?
Upgrade to 3.0.6
Also drop "insecure" qtwebkit
Things done
sandbox = true
set innix.conf
? (See Nix manual)nix-shell -p nixpkgs-review --run "nixpkgs-review rev HEAD"
. Note: all changes have to be committed, also see nixpkgs-review usage./result/bin/
)