On 2011-02-28, Daniel E. Moctezuma <dmoctez...@kmess.org> wrote:
> Yes, this CMake code is to check whether or not the krosspython library is
> installed on the system when  you execute the configure file before build.
> We think it is necessary to check it because it allows the user to know
> which functionalities you will have. As krosspython is used to be able to
> run Python plugins via Kross, the user can decide whether or not to have
> those features.

You know that just because you install krosspython in the build
environment doesn't mean that it will be there at runtime.

People getting your app from a linux distribution will have a completely
different environment than the build environment

>
> Btw, what do you mean by "runtime things that actually changes the result"?

I mean please don't do in your cmake code

(pseudocode)
if(!runtime_component_available) {
  add_definition("-DDISABLE_STUFF")
}

but rather

if(!runtime_component_available) {
  warn("please note that runtime component FOO isn't currently
  available. we can use it at run time. If you want to disable support
  for this, please build with -DDISABLE_STUFF")
}

/Sune

 
>> Visit http://mail.kde.org/mailman/listinfo/kde-devel#unsub to unsubscribe <<

Reply via email to