FS#69605 - [glib2] geeqie freezes at startup

Attached to Project: Arch Linux
Opened by Evgeny (Rus) - Wednesday, 10 February 2021, 03:43 GMT
Last edited by Jan Alexander Steffens (heftig) - Thursday, 11 February 2021, 14:37 GMT
Task Type Bug Report
Category Packages: Extra
Status Closed
Assigned To Andreas Radke (AndyRTR)
Jan Alexander Steffens (heftig)
Architecture x86_64
Severity Medium
Priority Normal
Reported Version
Due in Version Undecided
Due Date Undecided
Percent Complete 100%
Votes 2
Private No

Details

After the last OS update Geeqie freezes at startup, this problem never happened before
Tried to start with a new config, but it is useless
Tried to start with the terminal to get some output, but with the terminal starts successfully, doesn't freeze, works
This task depends upon

Closed by  Jan Alexander Steffens (heftig)
Thursday, 11 February 2021, 14:37 GMT
Reason for closing:  Fixed
Additional comments about closing:  glib2 2.66.7
Comment by Doug Newgard (Scimmia) - Wednesday, 10 February 2021, 03:59 GMT
With what version of glib2?
Comment by Evgeny (Rus) - Wednesday, 10 February 2021, 04:01 GMT
2.66.6-2
Comment by Andreas Radke (AndyRTR) - Wednesday, 10 February 2021, 06:47 GMT
Please check if geeqie is still running (dead process). What updates have you pulled and please check your system logs. We had a bug in glib2 lately but your version shold be fine.
Comment by Peter Weber (hoschi) - Wednesday, 10 February 2021, 18:20 GMT
I'm sorry. I missed something! But what?
Evgeny is right, this issue is not fixed. In the meantime I followed the instructions[1] from Geeqie upstream, which worked as temporary workaround. Now I've installed the update to glib2 "2.66.2" and went back to an unmodified version of Geeqie from Archlinux. It still doesn't work.


[1] https://github.com/BestImageViewer/geeqie/issues/865#issuecomment-774651175
Comment by Peter Weber (hoschi) - Wednesday, 10 February 2021, 18:45 GMT
Question:
channel->line_term_len = length;
What does this mean? I think
channel->line_term_len = length_unsigned;
makes more sense?

I mean length is there still "-1" and we want instead the value from length_unsigned which should be "19" in the case of "<gq_end_of_command>"?
Comment by Peter Weber (hoschi) - Wednesday, 10 February 2021, 18:53 GMT
Comment by Peter Weber (hoschi) - Wednesday, 10 February 2021, 18:57 GMT
Aha! Simon McVittie recognized that and fixed it already :)

https://gitlab.gnome.org/GNOME/glib/-/commit/e069c50467712e6d607822afd6b6c15c2c343dff
// edit
I changed also this line locally. Now it is works with the unmodified Geeqie.
I don't know if it is better using all current patches from upstream Glib2?
Comment by Andreas Radke (AndyRTR) - Thursday, 11 February 2021, 07:30 GMT
Jan, can you please apply this fix to glib2?

Loading...