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
toggl: init at 7.4.45 #26510
toggl: init at 7.4.45 #26510
Conversation
It does not compile, I'm not sure what's missing. Maybe somebody with qt exp could take a look?
|
This does not seem qt related, but it seems to ship its own openssl library
|
@Mic92 thanks for checking, LnL took a look and helped me identify some issues yesterday, I fixed some stuff, but still need to fix few. |
{ qtbase, poco, fetchFromGitHub, stdenv, makeQtWrapper, perl, readline, | ||
qtx11extras, qtwebkit, qmakeHook ,uncrustify, libXScrnSaver, scrnsaverproto }: | ||
stdenv.mkDerivation { | ||
name = "toggl"; |
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.
The name should contain a version string.
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.
Thanks, I'll fix this.
but it's still non working package...
sed -i 's@/usr/local/bin/@@' ./third_party/Xcode-formatter/CodeFormatter/scripts/formatAllSources.sh | ||
''; | ||
preBuild = '' | ||
command -v perl |
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.
What does this accomplish?
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 checks if perl is available
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.
But why is this necessary? It is clearly passed as in buildInputs.
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's probably not, I had an issue when build mechanism wanted perl but couldn't find that, so I added this line to make sure it's there. I'll def remove that before merge, after this is "buildable".
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.
Please add the version to the name and delete unnecessary perl check.
@yegortimoshenko thanks, closing. |
Motivation for this change
Things done
(nix.useSandbox on NixOS,
or option
build-use-sandbox
innix.conf
on non-NixOS)
nix-shell -p nox --run "nox-review wip"
./result/bin/
)