Bug 635439

Summary: FindGTK2.cmake module is broken with recent version of GTK+
Product: [openSUSE] openSUSE 11.4 Reporter: Vincent Untz <vuntz>
Component: DevelopmentAssignee: Petr Uzel <puzel>
Status: RESOLVED FIXED QA Contact: E-mail List <qa-bugs>
Severity: Major    
Priority: P5 - None CC: forgotten_h13THG8RK1
Version: Factory   
Target Milestone: ---   
Hardware: Other   
OS: Other   
Whiteboard: cmake
Found By: --- Services Priority:
Business Priority: Blocker: ---
Marketing QA Status: --- IT Deployment: ---
Attachments: Patch

Description Vincent Untz 2010-08-30 08:56:32 UTC
In GTK+ 2.21, gdk-pixbuf got split from GTK+. The result is that the include path for gdk-pixbuf is different from the include path from GTK+. And this is causing build failures.

Here's a patch I'm adding to yast2-gtk to fix the build there. But we probably want a fix in cmake directly too. This would be the same patch, I guess.

Ricardo: I've submitted this to YaST2:Head (sr#46627), but I guess it should probably be committed to svn?
Comment 1 Vincent Untz 2010-08-30 08:57:04 UTC
Created attachment 386143 [details]
Patch
Comment 2 Forgotten User h13THG8RK1 2010-08-30 14:19:05 UTC
Thanks for that. Coolo already committed it to svn btw.

I kept that FindGTK2 file around because at the time of opensuse 11.2, cmake didn't ship with it. Good it see it has become useful again, but we should definitively get your patch to cmake, so we don't rely on our own FindGTK2.cmake module.
Comment 3 Forgotten User h13THG8RK1 2010-08-30 14:20:52 UTC
Btw, it seems that you guys uploaded the trunk code to YaST:Head. That's fine, but just to tell, we may have to upload the 11_3 branch again in order to issue a patch: see bug 624310.
Comment 4 Forgotten User h13THG8RK1 2010-08-30 17:00:31 UTC
Filled up a bug report at cmake:
http://public.kitware.com/Bug/view.php?id=11186
Comment 5 Forgotten User h13THG8RK1 2010-08-30 17:15:21 UTC
Vincent, I've re-uploaded the 11.3 branch code to YaST:HEAD because we are going to issue a patch (bug 624310).

(Maybe there's a more elegant way to do this -- is there some YaST:11.3 ? -- but I dunno, so let's momentarily leave it broken.)
Comment 6 Forgotten User h13THG8RK1 2010-09-04 11:38:41 UTC
Okay, this issue has been addressed upstream:

Philip Lowman: "Pushed change into next, should be in first RC for 2.8.3"
( http://public.kitware.com/Bug/view.php?id=11186 )

Philip told me via email your patch was useful, Vincent. Thanks!

Petr, should we close the report now?
Comment 7 Petr Uzel 2010-09-14 12:43:04 UTC
Thanks for the patch Vincent. I have patched cmake in Factory until cmake-2.8.3 is out. Closing this bug.
Comment 8 Forgotten User h13THG8RK1 2010-09-15 11:41:23 UTC
Thanks Petr.

I've tried the package from the Factory repository, and that one doesn't have the fix in yet. I guess it takes a little bit of time for a submission to get through.

Can you tell me what version number of the cmake package should I look out for?

http://download.opensuse.org/factory/repo/oss/suse/i586/

Thanks!
Comment 9 Forgotten User h13THG8RK1 2010-09-15 11:50:19 UTC
By the way, with some help from my buddy google, I've just found a way to make the yast2-gtk build compatible with both 11.3 and Factory. I just have to apply this change to the RPM spec file:

- BuildRequires:  gdk-pixbuf-devel
+ %if 0%{?suse_version} > 1130
+ BuildRequires:  gdk-pixbuf-devel
+ %endif
Comment 10 Petr Uzel 2010-09-15 12:17:57 UTC
(In reply to comment #8)
> Thanks Petr.
> 
> I've tried the package from the Factory repository, and that one doesn't have
> the fix in yet. I guess it takes a little bit of time for a submission to get
> through.

Yes, it hasn't been accepted to Factory yet (see request #48122).

> Can you tell me what version number of the cmake package should I look out for?

The release number will be higher than the current one :) Frankly, I don't know how exactly are those release numbers generated by OBS, so I can't tell you.

> http://download.opensuse.org/factory/repo/oss/suse/i586/

You can get the fixed RPM from here:
http://download.opensuse.org/repositories/devel:/tools:/building/openSUSE_Factory/i586/
Comment 11 Forgotten User h13THG8RK1 2010-09-15 23:40:40 UTC
Thx. Will keep an eye on build service:

https://build.opensuse.org/request/show/48122