Merged, I also backported it to syncevolution-0-9-branch
From: Zhu, Yongsheng
Sent: Wednesday, January 06, 2010 2:05 PM
To: Chen, Congwu; Chen, Congwu; Ove Kaaven
Subject: RE: [SyncEvolution] SyncEvolution in Fremantle
Ok for me. please go forward.
> -----Original Message-----
> From: syncevolution-bounces(a)syncevolution.org
> [mailto:firstname.lastname@example.org] On Behalf Of Chen,
> Sent: Wednesday, January 06, 2010 1:25 PM
> To: Chen, Congwu; Ove Kaaven
> Cc: SyncEvolution
> Subject: Re: [SyncEvolution] SyncEvolution in Fremantle
> I fixed this issue in 9aad6cee430d198d7376bdf45075f7a18ff9d25e congwu
> Yongsheng, please review and I will merge it later.
> Chen, Congwu wrote:
> >On Sat, 26 Dec 2009, Ove Kaaven wrote:
> >My comments for your reference.
> >> > On Thu, 2009-12-24 at 08:03 +0000, Ove Kaaven wrote:
> >> >> It can't yet be built with a buildbot, primarily because not
all of its
> >> >> build-dependencies exist in maemo-devel, cppunit in particular.
> >> >
> >> > This shouldn't be a hard dependency. Without cppunit,
> >> > can't be built, but that is not necessary during normal usage. If
> >> > ran into a compile problem without cppunit, then please file a bug
> >> > report.
> >> src/backends/evolution/Makefile.am and src/backends/file/Makefile.am
> >> both enforce -DENABLE_INTEGRATION_TESTS flag regardless of how the
> >> source is configured. This doesn't seem to cause a run-time dependency,
> >> but it definitely causes a compile-time dependency when --enable-shared
> >> is used (not with --disable-shared, since only the *Register.cpp files
> >> actually includes test.h). So I should go to bugzilla with that?
> >That indeed looks like a bug. I will look at this further, thanks for your
> Best Regards,
> SyncEvolution mailing list