Opened 11 years ago

Closed 4 years ago

Last modified 4 years ago

#39571 closed defect (duplicate)

yelp 2.31.6 fails to build for +quartz

Reported by: c.herbig@… Owned by: macports-tickets@…
Priority: Normal Milestone:
Component: ports Version: 2.1.3
Keywords: Cc: dbevans (David B. Evans)
Port: yelp

Description

I know that it is not officially supported as a variant for yelp, but I suppose if no one points it out, that guarantees it will never work. In trying to compile yelp +quartz, everything seems to work up to the application.c file, when it generates errors due to specific gdk x11 function calls. I hope that something can be done to work around this. You might be interested to know that if you attempt to install yelp 3.x, the same problem is there.

Here is the relevant (hopefully) part of the log:

:info:build yelp-application.c: In function ‘application_uri_resolved’:
:info:build yelp-application.c:546: warning: implicit declaration of function ‘gdk_x11_window_move_to_current_desktop’
:info:build yelp-application.c:546: warning: nested extern declaration of ‘gdk_x11_window_move_to_current_desktop’
:info:build yelp-application.c:554: warning: implicit declaration of function ‘gdk_x11_get_server_time’
:info:build yelp-application.c:554: warning: nested extern declaration of ‘gdk_x11_get_server_time’
:info:build make[2]: *** [yelp-yelp-application.o] Error 1

P.S. I have never posted a bug ticket before, so I apologize in advance for any problems. I would also like to thank the maintainers for finally updating yelp to a version that will compile on 64 bit x86.

Change History (6)

comment:1 Changed 11 years ago by dbevans (David B. Evans)

As you say, yelp at this revision level is not designed to build +quartz and seeing this is a fairly old version, it's not likely that it will be addressed until the port is upgraded to the GNOME 3 version which has more support for darwin. Will leave it open as a reminder though.

Thanks for your report.

comment:2 in reply to:  description Changed 11 years ago by dbevans (David B. Evans)

P.S. I have never posted a bug ticket before, so I apologize in advance for any problems. I would also like to thank the maintainers for finally updating yelp to a version that will compile on 64 bit x86.

We appreciate your appreciation; the key here was a version based on webkit-gtk instead of xulrunner.

comment:3 in reply to:  1 Changed 11 years ago by c.herbig@…

I don't know much at all about the inner workings of gnome or gtk, but if gnome 3 is newer, has better support for darwin, and since gtk3 is designed to not be dependent on X11, why not just devote the efforts to porting gnome 3?

comment:4 Changed 11 years ago by c.herbig@…

This ticket can be resolved and closed by applying #39756 the patch gets it working well enough.

comment:5 Changed 4 years ago by kencu (Ken)

Resolution: duplicate
Status: newclosed

superceded by 40116

comment:6 Changed 4 years ago by mf2k (Frank Schima)

Keywords: yelp gnome +quartz removed
Note: See TracTickets for help on using tickets.