Re: [kmymoney/4.8] /: Another cmake fix.

2016-09-30 Thread Thomas Baumgart
Hi all, On Friday 30 September 2016 10:57:07 Jack wrote: > On 2016.09.30 05:29, Christian Dávid wrote: > > Hi Ralf, > > > > I think it should be > > > > if (${LibAlkimia_FOUND} AND DEFINED LibAlkimia_VERSION) > > > > without ${}. This explains why you had an error if LibAlkimia_VERSION > > is

Re: [kmymoney/4.8] /: Another cmake fix.

2016-09-30 Thread Jack
On 2016.09.30 05:29, Christian Dávid wrote: Hi Ralf, I think it should be if (${LibAlkimia_FOUND} AND DEFINED LibAlkimia_VERSION) without ${}. This explains why you had an error if LibAlkimia_VERSION is not defined — because the expresion evaluates to "if (${LibAlkimia_FOUND} AND

Re: Review Request 112847: Display equity accounts when enabled in payees page and unknown payee import dialog

2016-09-30 Thread Ralf Habacker
> On Sept. 21, 2013, 4:24 nachm., Alvaro Soliverez wrote: > > I fail to see why you would want equity accounts displayed for imports. Can > > you explain the use case? > > > > Keep in mind that equity accounts are not stock or share accounts of some > > kind. > > Ralf Habacker wrote: >

Re: [kmymoney/4.8] /: Another cmake fix.

2016-09-30 Thread Christian Dávid
Hi Ralf, I think it should be if (${LibAlkimia_FOUND} AND DEFINED LibAlkimia_VERSION) without ${}. This explains why you had an error if LibAlkimia_VERSION is not defined — because the expresion evaluates to "if (${LibAlkimia_FOUND} AND DEFINED)". Also I want to highlight that Alkimia 6 uses