Opened 17 years ago

Closed 16 years ago

Last modified 16 years ago

#15327 closed defect (fixed)

gnucash doesn't depend on evince if built from gtk2+quartz+no_x11

Reported by: mp@… Owned by: macports-tickets@…
Priority: Low Milestone:
Component: ports Version: 1.6.0
Keywords: Cc: mf2k (Frank Schima)
Port: gnucash

Description

Gnucash's evince runtime dependency was added for printing support, but this is only needed with gtk2(x11).

When built from gtk2+quartz+no_x11, evince is not required. In fact, as far as I can tell, evince and it's dependencies (eg, gnome-desktop) will not build from a quartz version of gtk2, and so a 'port install gnucash' will fail in the dependency chain.

I have attached a patch to the gnucash Portfile which detects whether gtk2 was installed with the quartz variant, and if so removes the evince dependency. I chose not to add a quartz variant to gnucash because it's not really a gnucash 'feature'. This way the gnucash Portfile is smart enough to build properly based on the variants of its dependencies.

Attachments (1)

gnucash-Portfile.diff (487 bytes) - added by mp@… 17 years ago.

Download all attachments as: .zip

Change History (5)

Changed 17 years ago by mp@…

Attachment: gnucash-Portfile.diff added

comment:1 Changed 16 years ago by mf2k (Frank Schima)

Cc: macsforever2000@… added

Cc Me!

comment:2 Changed 16 years ago by gwhitney

Port: gnucash added
Priority: NormalLow

I have just committed r44456 which updates gnucash to 2.2.8. Compiling this under Mac OS X 10.5 with MacPorts variants.conf containing +no_x11 +quartz on a machine with Apple's X11 installed, I could not get the final gnucash application to link unless I disabled the check for Xlib.h in gnucash's configure script. So to get it to work, I added a +no_x11 variant at the gnucash level. Since I was adding such a variant for independent reasons, it seemed a no-brainer to eliminate the evince dependency in that variant. I figure this pretty much takes care of this ticket, but I wasn't sure if everyone would agree with this course, or exactly how I should mark the resolution (fixed, wontfix, etc.) So anyone involved with this ticket should feel free to close it as far as I'm concerned.

comment:3 Changed 16 years ago by mf2k (Frank Schima)

Resolution: fixed
Status: newclosed

comment:4 Changed 16 years ago by (none)

Milestone: Port Bugs

Milestone Port Bugs deleted

Note: See TracTickets for help on using tickets.