net-voip/telepathy-salut-0.5.0 fails to configure if it cannot find xmldiff: # emerge telepathy-salut [...] checking to see if compiler understands -Werror... yes checking to see if compiler understands -Wno-missing-field-initializers... yes checking to see if compiler understands -Wno-error=missing-field-initializers... yes checking to see if compiler understands -Wno-unused-parameter... yes checking to see if compiler understands -Wno-error=unused-parameter... yes checking for xsltproc... xsltproc checking for xmldiff... no configure: error: xmldiff is required for the gibber tests !!! Please attach the following file when seeking support: !!! /var/tmp/portage/net-voip/telepathy-salut-0.5.0/work/telepathy-salut-0.5.0/config.log * ERROR: net-voip/telepathy-salut-0.5.0 failed (configure phase): * econf failed With app-text/xmldiff-0.6.10 installed, telepathy-salut-0.5.0 emerges correctly. So app-text/xmldiff needs to be added to DEPEND of the telepathy-salut-0.5.0 ebuild.
I thought this was only for tests. I'll fix it asap.
(In reply to comment #1) > I thought this was only for tests. I'll fix it asap. Will you fix it simply adding the xmldiff dep unconditionally? Looks like it should only be needed for tests :-/ (or configure output is wrong)
My "test" flag is not set, and configure fails... So an unconditional dep is needed...
Created attachment 273245 [details] ebuild that skips xmldiff configure check when FEATURES=-test (In reply to comment #2) > (In reply to comment #1) > > I thought this was only for tests. I'll fix it asap. > > Will you fix it simply adding the xmldiff dep unconditionally? Looks like it > should only be needed for tests :-/ (or configure output is wrong) You are correct, it's only needed for tests, but configure is checking for it unconditionally due to upstream silliness (see NEWS). Fortunately, autoconf provides an easy way to bypass the AC_CHECK_PROGS check: simply have ac_cv_prog_XMLDIFF point to some working executable. The attached ebuild sets ac_cv_prog_XMLDIFF=/bin/true in src_configure() when FEATURES=-test. I have verified that it builds correctly without xmldiff installed.
I had the same issue, configure fails without xmldiff, compiles fine after its added.
Well, in case someone happens to commit the fix, there's also following QA issue: * configure: WARNING: unrecognized options: --disable-plugins
The QA warning can't be fixed as it comes from another autotools configure triggered by the toplevel one for a lib that might be split in the future.
Created attachment 274943 [details, diff] xmldiff-warn.patch Downgrade xmldiff error to a warning. I prefer this patch as it seems less magical.
07 Jun 2011; Olivier Crête <tester@gentoo.org> telepathy-salut-0.5.0.ebuild: xmldiff is alwasy required for the configure to pass
