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
mapmap: Fix startup issues and unbreak the derivation. #105793
Conversation
Thank you very much for fixing mapmap. Additionally to @SuperSandro2000's comment I'd prefer it if you could squash the three commits into one. |
Result of 1 package built:
|
Done. |
Result of 1 package built:
|
Result of |
patches = [ | ||
(fetchpatch { | ||
name = "message-handler-segfault.patch"; | ||
url = "https://github.com/mapmapteam/mapmap/pull/519.patch"; |
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.
Just one last request: Could you use https://github.com/mapmapteam/mapmap/pull/519/commits/22eeee59ba7de6de7b73ecec3b0ea93bdc7f04e8.patch
here? This way it's obvious which commit is meant to be fetched and we won't have the problem of the PR changing later.
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.
Good catch. Done.
* Apply patch from not yet merged upstream pull request to fix segfault * Add wrapQtAppsHook to make it start again * Mark as unbroken
Motivation for this change
mapmap has been broken as far as I can see for two reasons:
wrapQtAppsHook
and therefor failed on startup with the usual error message.I'm not familiar with mapmap right now, so I so far can only tell that it starts up again.
Things done
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)