#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)
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 |
---|
comment:2 Changed 16 years ago by gwhitney
Port: | gnucash added |
---|---|
Priority: | Normal → Low |
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: | new → closed |
Cc Me!