Opened 9 years ago
Closed 9 years ago
#49412 closed defect (fixed)
oregano: fails to build +quartz, update to version 0.83.2.1
Reported by: | macports@… | Owned by: | dbevans (David B. Evans) |
---|---|---|---|
Priority: | Normal | Milestone: | |
Component: | ports | Version: | |
Keywords: | Cc: | bernhard@… | |
Port: | oregano |
Description (last modified by ryandesign (Ryan Carsten Schmidt))
port deps oregano returns
Full Name: oregano @0.82_1 Build Dependencies: autoconf, automake, intltool, libtool, pkgconfig Library Dependencies: gtk3, gtksourceview3, goocanvas2, rarian, desktop-file-utils, shared-mime-info Runtime Dependencies: yelp, docbook-xml
but some of those dependencies are spurious. Yelp (Gnome’s help viewer) in particular depends on gnome-settings-daemon, which in turn wants to pull in all sorts of stuff for the Gnome desktop environment, including gnome-desktop, which requires gtk3 to be built with +x11 instead of +quartz. (See #32057 )
Configuring from the Github source shows the actual dependencies:
Checking for 'clang' (C compiler) : /usr/bin/clang Checking for program 'glib-genmarshal' : /opt/local/bin/glib-genmarshal Checking for program 'perl' : /opt/local/bin/perl Checking for program 'glib-mkenums' : /opt/local/bin/glib-mkenums Checking for program 'glib-compile-schemas' : /opt/local/bin/glib-compile-schemas Checking for program 'glib-compile-resources' : /opt/local/bin/glib-compile-resources Checking for program 'msgfmt' : /opt/local/bin/msgfmt Checking for program 'intltool-merge' : /opt/local/bin/intltool-merge Checking for header locale.h : yes Checking for library m : yes Checking for program 'pkg-config' : /opt/local/bin/pkg-config Checking for pkg-config version >= '0.26' : yes Checking for 'glib-2.0' : yes Checking for 'gobject-2.0' : yes Checking for 'gtk+-3.0' : yes Checking for 'libxml-2.0' : yes Checking for 'goocanvas-2.0' : yes Checking for 'gtksourceview-3.0' : yes
It builds and runs successfully with just those. There are also optional deps for a debug build:
Checking for program 'clang-format' : not found Checking for program 'gdb' : not found Checking for program 'nemiver' : not found Checking for program 'valgrind' : /opt/local/bin/valgrind
Also, the port is behind the current version 0.83.3.
Change History (8)
comment:1 Changed 9 years ago by ryandesign (Ryan Carsten Schmidt)
Cc: | devans@… added |
---|---|
Description: | modified (diff) |
Summary: | Superfluous dependencies, version update → oregano: Superfluous dependencies, version update |
comment:2 follow-up: 3 Changed 9 years ago by dbevans (David B. Evans)
Resolution: | → invalid |
---|---|
Status: | new → closed |
yelp is declared as a run time dependency because it is required to display the User's Manual when you select Help->User's Manual from the application. If it is not installed this operation will fail. So only superfluous if you think documentation is superfluous.
comment:3 Changed 9 years ago by macports@…
Replying to devans@…:
yelp is declared as a run time dependency because it is required to display the User's Manual when you select Help->User's Manual from the application. If it is not installed this operation will fail. So only superfluous if you think documentation is superfluous.
Okay, maybe not superfluous but certainly not integral to the application’s function. As is, the whole thing fails, not just Help -> User Manual.
Plenty of ports have +docs variants, maybe that could be done here?
comment:4 Changed 9 years ago by dbevans (David B. Evans)
yelp no longer depends on gnome-settings-daemon.
comment:5 Changed 9 years ago by dbevans (David B. Evans)
Resolution: | invalid |
---|---|
Status: | closed → reopened |
Summary: | oregano: Superfluous dependencies, version update → oregano: fails to build +quartz, update to version 0.83.3. |
Concerning +docs, this is used to control installation of documentation that is separate from the application such as man pages. In this case, the Help functionality is an integral part application at run time.
By the way, it would help in reporting problems if you would give more details about failure mode. In particular, if the problem if +quartz related as I perceive it is in this case, you should explicitly state that as part of the summary and description. Further, you should not expect that gtk3 applications will build +quartz if they don't have that variant.
Nonetheless, I will take a look at the +quartz situation and see what I can do.
comment:6 Changed 9 years ago by dbevans (David B. Evans)
Cc: | devans@… removed |
---|---|
Owner: | changed from macports-tickets@… to devans@… |
Status: | reopened → new |
Summary: | oregano: fails to build +quartz, update to version 0.83.3. → oregano: fails to build +quartz, update to version 0.83.2.1 |
Version: | 2.3.4 |
Added +quartz variant and adjusted dependencies to allow build with gtk3 +quartz in r142697. Works for me on Yosemite. Please let me know if you have any further issues/suggestions. In particular, I think the circuit simulation functionality is broken but I don't have a suitable schematic design file to test it with as yet. As a user of this application, if you have such a file or know where one can be found, please let me know.
I'm working on the version update as a separate issue as this involves a change in upstream maintainer and some fairly significant changes including a change of build system to waf. Current upstream version is 0.83.2.1 which the maintainer warns is an unfinished work in progress and should be considered unstable.
Thanks for your input.
comment:7 Changed 9 years ago by dbevans (David B. Evans)
Status: | new → assigned |
---|
comment:8 Changed 9 years ago by dbevans (David B. Evans)
Cc: | bernhard@… added |
---|---|
Resolution: | → fixed |
Status: | assigned → closed |
Oregano updated to version 0.83.2.1 in commit r144007. This update involves a switch of github repo and is based on the Portfile included as part of the upstream distribution, including maintainer (openmaintainer), merged with our existing features. Since this is now a waf build, the waf 1.0 port group is used with a few workarounds as necessary.
So the spurious dependency you are referring to specifically is yelp? You are saying that oregano does not use yelp? The yelp dependency was deliberately added in r108199 by Dave; I'll Cc him for comment.