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
cool-retro-term: fix on darwin #51830
Conversation
@@ -7,8 +7,8 @@ stdenv.mkDerivation rec { | |||
src = fetchFromGitHub { | |||
repo = "qmltermwidget"; | |||
owner = "Swordfish90"; | |||
rev = "v${version}"; | |||
sha256 = "0ca500mzcqglkj0i6km0z512y3a025dbm24605xyv18l6y0l2ny3"; | |||
rev = "48274c75660e28d44af7c195e79accdf1bd44963"; |
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.
Was #51572 not enough to fix Darwin build of qmltermwidget
?
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.
Unfortunately not- #51572 fixes the build, but there is another issue once it's running- Swordfish90/cool-retro-term#366
2b6df8c
to
b77bcdb
Compare
wrapProgram $out/bin/cool-retro-term.app/Contents/MacOS/cool-retro-term \ | ||
--set QT_QPA_PLATFORM_PLUGIN_PATH ${qtbase.bin}/lib/qt-*/plugins/platforms \ | ||
--set QT_PLUGIN_PATH ${qtbase.bin}/${qtbase.qtPluginPrefix} \ | ||
--set QML2_IMPORT_PATH ${qmltermwidget}/${qtbase.qtQmlPrefix}:${qtdeclarative.bin}/${qtbase.qtQmlPrefix}:${qtgraphicaleffects}/${qtbase.qtQmlPrefix}:${qtquickcontrols}/${qtbase.qtQmlPrefix} |
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.
Is this also the case if you install retro-term in your nix profile?
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.
Oh, that works! Also works without wrapper using nix run
. I have removed the wrapper, it seems like all that is needed is symlink to binary.
b77bcdb
to
7f82302
Compare
@@ -7,8 +7,8 @@ stdenv.mkDerivation rec { | |||
src = fetchFromGitHub { | |||
repo = "qmltermwidget"; | |||
owner = "Swordfish90"; | |||
rev = "v${version}"; | |||
sha256 = "0ca500mzcqglkj0i6km0z512y3a025dbm24605xyv18l6y0l2ny3"; | |||
rev = "48274c75660e28d44af7c195e79accdf1bd44963"; |
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.
version
should be changed to 2018-11-24
and name to qmltermwidget-unstable-${version}
per https://nixos.org/nixpkgs/manual/#sec-package-naming.
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.
thx, done
Motivation for this change
cool-retro-term had two problems on darwin:
$PATH
due to macOS packagingadd a symlink to the macOS binary on darwin and update qmltermwidget makes it work
Things done
sandbox
innix.conf
on non-NixOS)nix-shell -p nox --run "nox-review wip"
./result/bin/
)nix path-info -S
before and after)