On 2011-Mar-02, 16:48, Michal Varga wrote:
> On Wed, 2011-03-02 at 16:26 +0100, Pietro Cerutti wrote:
> > On 2011-Mar-02, 15:57, Michal Varga wrote:
> > > Out of curiosity - how is it even possible that this update made it as
> > > far as into ports tree, when it - at least as I understand from the PR -
> > > clearly doesn't work on *both* *STABLE* *supported* branches and only
> > > goes well with the very latest CURRENT?
> > 
> > The original update doesn't cause any problems on CURRENT,
> > 7.4-RELEASE, or 8.2-RELEASE. It does cause problems on STABLE branches.
> > 
> 
> Is this guaranteed?
> 
> As here with...
> 
> > uname -a
> FreeBSD 7.4-STABLE FreeBSD 7.4-STABLE #0: Sun Feb 27 22:30:14 CET 2011  
> 
> ...gegl (unpatched) still fails:
> 
> 
> exp_combine.cpp: In function 'gfloat expcombine_get_file_ev(const gchar*)':
> exp_combine.cpp:99: error: 'log2f' was not declared in this scope
> gmake[2]: *** [exp_combine-exp_combine.o] Error 1
> 
> 
> Note that 7.4-RELEASE was back in Feb 21. Or was there a change specific
> to 7.4-RELEASE that for some reason didn't make it into STABLE?

OK, I have nailed down the source of the problem. GEGL has an optional
dependency on graphics/exiv2 which is not tracked by the port, i.e.
exiv2 is not built as a dependency of the port, and GEGL doesn't rely on
it if it doesn't find it. It turns out that the failing C module
(exp_combine.c) is only compiled when EXIV is found.

Thus, on a fresh system (i.e., tinderbox) the error won't appear. It
will, however, on a system where EXIV is installed.
Please check whether you have graphics/exiv2 installed in your system to
confirm this hypothesis.

I have still not decided whether to include exiv2 as a mandatory or
optional dependency, cause configure doesn't handle --without-exiv2
correctly when exiv2 is installed (i.e., GEGL will be built with EXIV
support in that case).

Thanks for the hints, and sorry for the noise.

-- 
Pietro Cerutti
The FreeBSD Project
g...@freebsd.org

PGP Public Key:
http://gahr.ch/pgp

Attachment: pgpM3bdFMOqAB.pgp
Description: PGP signature

Reply via email to