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
hexchat: 2.12.4 -> 2.14.2 #67714
hexchat: 2.12.4 -> 2.14.2 #67714
Conversation
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.
tested with nix-review, worked fine.
I don't know enough about meson to comment the patchPhase though
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.
Tested by connecting to my ZNC with a very large number of channels, I did not see any crashes or regressions
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, I just noticed that 2.14.2 always dumps core when I close it:
# /run/current-system/sw/bin/hexchat -d $HOME/.config/hexchats/full
Fatal Python error: PyInterpreterState_Delete: remaining subinterpreters
zsh: abort (core dumped) /run/current-system/sw/bin/hexchat -d $HOME/.config/hexchats/full
Upstream bug is hexchat/hexchat#2237 but maybe this is just something users have to live with until it's fixed upstream?
noticed the same bug, I believe it's fine merging. |
85964d6
to
97d185a
Compare
I've added the fedora 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.
Tested again, no core dump on exit any more; LGTM
@GrahamcOfBorg build hexchat |
Motivation for this change
Things done
sandbox
innix.conf
on non-NixOS)nix-shell -p nix-review --run "nix-review wip"
./result/bin/
)nix path-info -S
before and after)Notify maintainers
cc @romildo