kdepim/akonadi/agents/nepomuk_contact_feeder/CMakeLists.txt

[Allen Winter]

On Wednesday 16 April 2008 18:42:46 Alexander Neundorf wrote:
> On Thursday 17 April 2008, Allen Winter wrote:
> > On Wednesday 16 April 2008 16:52:00 Alexander Neundorf wrote:
> > > On Wednesday 16 April 2008, Allen Winter wrote:
> > > > Howdy,
> > > >
> > > > Can someone who knows CMake much better than myself please take
> > > > a look at
> > > > kdepim/akonadi/agents/nepomuk_contact_feeder/CMakeLists.txt?
> > > >
> > > > This CMake code has no dependencies and uses execute_process()
> > > > so that all the commands are re-run each time one builds kdepim.
> > > >
> > > > I think it really should use add_custom_command() instead, but that
> > > > doesn't support RESULT_VARIABLE or ERROR_QUIET.. so I'm stumped.
> > > >
> > > > Alex maybe? pretty please?
> > > > -Allen
> > >
> > > Why can that fail, or, why is
> > > kde4_add_executable(akonadi_nepomuk_contact_feeder) inside an
> > > if(NEPOMUK_RESOURCES_GENERATED) ?
> > >
> > > Is this optional ?
> >
> > I have no idea.  I didn't write it.
> > What's irritating is that the files are regenerated each build,
> > and that causes unnecessary recompiles.  We need to add
> > in some dependencies.
>
> Sebastian, I just had a look at revision 773742
> (http://websvn.kde.org/trunk/KDE/kdepim/akonadi/agents/nepomuk_contact_feed
>er/CMakeLists.txt?view=log). In which cases can rcgen fail, once it has been
> found ?
> Is there a reason that this has to run at cmake time or would running it
> dependency-driven at build time also be ok ?
>
Alex,

FYI: I just noticed that all this code is duplicated in
kdelibs/cmake/modules/NepomukMacros.cmake,
presumably for backward compatibility reasons.

So.. whatever changes you make need to be
copied over into both places.