Bug 314688

Summary:	Crash in GIMP
Product:	[Plasma] Oxygen	Reporter:	nucleo <nucleo>
Component:	gtk2-engine	Assignee:	Hugo Pereira Da Costa <hugo.pereira.da.costa>
Status:	RESOLVED FIXED
Severity:	normal	CC:	b7.10110111, hugo.pereira.da.costa, web
Priority:	NOR
Version:	unspecified
Target Milestone:	---
Platform:	Other
OS:	Linux
Latest Commit:		Version Fixed In:

Description nucleo 2013-02-08 16:12:36 UTC

Gimp crases with oxygen-gtk2-1.3.2 but with 1.3.1 don't.

Reproducible: Always

Actual Results:  
See more details here https://bugzilla.redhat.com/show_bug.cgi?id=909019

Comment 1 Ruslan Kabatsayev 2013-02-08 16:18:41 UTC

Most likely duplicate of bug 314545

Comment 2 Hugo Pereira Da Costa 2013-02-11 11:13:55 UTC

Hello
Can't reproduce the crash, but that does not prove anything
Nucleo: can you reproduce it with latest source from git ? (either master or 1.3 branch ?)

Thanks in advance, 

@Ruslan: if indeed this crash is fixed for current 1.3, I think I'll make a new release today. 
Could not fix all these crashes under gnome shell (for the same reason as you have them now with other DEs, because initialization cannot be done in theme_init, and crashes for some apps in instance_init, but if bugs arise, I'll mark these as "upstream" as this really is a gnome3 issue, I guess. GtkDisplay should be defined in theme_init, and what I tried to fix is just a workaround, that does not work all the time ...).

Objections ? 

Hugo

Comment 3 Ruslan Kabatsayev 2013-02-11 11:40:03 UTC

(In reply to comment #2)

> @Ruslan: if indeed this crash is fixed for current 1.3, I think I'll make a
> new release today. 
> Could not fix all these crashes under gnome shell (for the same reason as
> you have them now with other DEs, because initialization cannot be done in
> theme_init, and crashes for some apps in instance_init, but if bugs arise,
> I'll mark these as "upstream" as this really is a gnome3 issue, I guess.
> GtkDisplay should be defined in theme_init, and what I tried to fix is just
> a workaround, that does not work all the time ...).
I agree. It's quite weird for GdkDisplay to be undefined at theme loading stage anyway.

Comment 4 nucleo 2013-02-11 14:31:07 UTC

I asked bug reporter to confirm that fix from bug 314545 also fixes his crash, but still no answer.
https://bugzilla.redhat.com/show_bug.cgi?id=909019#c21

Comment 5 Hugo Pereira Da Costa 2013-02-11 15:52:44 UTC

@Nucleo
ok, thanks for the info. I'll make another release tomorrow (3.2.1-1, I guess) and we'll see.

Comment 6 Hugo Pereira Da Costa 2013-02-11 15:53:24 UTC

... meant 1.3.2-1

Comment 7 nucleo 2013-02-11 19:20:46 UTC

1.3.2.1 will be better, if possible, because "-" can't be used in rpm Version tag.

Comment 8 Hugo Pereira Da Costa 2013-02-12 08:21:07 UTC

@Nucleo
ok. Will do so.

Comment 9 nucleo 2013-02-12 17:26:35 UTC

Thanks.
Finally confirmed that fix from bug 314545 also fixes GIMP crash
https://bugzilla.redhat.com/show_bug.cgi?id=909019#c23

Comment 10 Hugo Pereira Da Costa 2013-02-13 13:00:16 UTC

ok. New release available at 
download.kde.org/stable/oxygen-gtk2/1.3.2.1/src/oxygen-gtk2-1.3.2.1.tar.bz2

So closing.