Re: [SyncEvolution] SyncEvolution in Fremantle

-----Original Message----- From: Zhu, Yongsheng Sent: Wednesday, January 06, 2010 2:05 PM To: Chen, Congwu; Chen, Congwu; Ove Kaaven Cc: SyncEvolution Subject: RE: [SyncEvolution] SyncEvolution in Fremantle Ok for me. please go forward. Cheers, Yongsheng > -----Original Message----- > From: syncevolution-bounces(a)syncevolution.org > [mailto:syncevolution-bounces@syncevolution.org] 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 > SyncEvolution(a)syncevolution.org > http://lists.syncevolution.org/listinfo/syncevolution