After an evaluation, GNOME has moved from Bugzilla to GitLab. Learn more about GitLab.
No new issues can be reported in GNOME Bugzilla anymore.
To report an issue in a GNOME project, go to GNOME GitLab.
Do not go to GNOME Gitlab for: Bluefish, Doxygen, GnuCash, GStreamer, java-gnome, LDTP, NetworkManager, Tomboy.
Bug 663789 - Change IgeMacIntegration to GtkOSXApplication
Change IgeMacIntegration to GtkOSXApplication
Status: RESOLVED FIXED
Product: devhelp
Classification: Applications
Component: General
git master
Other Mac OS
: Normal major
: ---
Assigned To: devhelp-maint
devhelp-maint
Depends on:
Blocks:
 
 
Reported: 2011-11-10 16:57 UTC by Philip Chimento
Modified: 2011-12-17 15:53 UTC
See Also:
GNOME target: ---
GNOME version: ---

Attachments
Patch for gnome-2-32 branch (6.72 KB, patch)
2011-11-10 16:57 UTC, Philip Chimento 		none Details | Review
Patch for master (untested) (6.63 KB, patch)
2011-11-10 16:58 UTC, Philip Chimento 		none Details | Review
Patch for gnome-2-32 branch (6.72 KB, patch)
2011-11-15 17:41 UTC, Philip Chimento 		none Details | Review
Patch for master (untested) (6.63 KB, patch)
2011-11-15 17:42 UTC, Philip Chimento 		none Details | Review

Description Philip Chimento 2011-11-10 16:57:04 UTC 
IgeMacIntegration is deprecated and only works on 32-bit Macs. Here are patches to change IgeMacIntegration to GtkOSXApplication so that Devhelp also builds and runs on 64-bit Macs.

One patch is for the gnome-2-32 branch. This branch still won't build unless you cherry-pick commit [d1ead3ca] from bug #646962.

The other patch is for master. This one I'm not able to test at the moment.
Comment 1 Philip Chimento 2011-11-10 16:57:44 UTC 
Created attachment 201163 [details] [review]
Patch for gnome-2-32 branch
Comment 2 Philip Chimento 2011-11-10 16:58:07 UTC 
Created attachment 201164 [details] [review]
Patch for master (untested)
Comment 3 Philip Chimento 2011-11-15 17:41:49 UTC 
Created attachment 201465 [details] [review]
Patch for gnome-2-32 branch

Correction: pkg-config should test for gtk-mac-integration >= 1.0.0
Comment 4 Philip Chimento 2011-11-15 17:42:22 UTC 
Created attachment 201466 [details] [review]
Patch for master (untested)

Ditto
Comment 5 Frederic Peters 2011-12-17 15:53:13 UTC 
I pushed the patch to master.