Ok for me. please go forward.

Cheers,
Yongsheng

> -----Original Message-----
> From: [email protected]
> [mailto:[email protected]] On Behalf Of Chen, Congwu
> 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 branch.
> 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, "client-test"
> >> > can't be built, but that is not necessary during normal usage. If you
> >> > 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
> >report!
> 
> Best Regards,
> Congwu
> _______________________________________________
> SyncEvolution mailing list
> [email protected]
> http://lists.syncevolution.org/listinfo/syncevolution
_______________________________________________
SyncEvolution mailing list
[email protected]
http://lists.syncevolution.org/listinfo/syncevolution

Reply via email to