Report account-piecharts.scm does not work

[Robert Fewell]

Thanks for looking, I am using guile version 1.8.8 and as you say windows
also uses version 1.8.x so that must be the difference to you. I am glad
the changes do not affect version 2.0.0 so will add them to the pull
request. I am a bit surprised that it has not shown up earlier.

Bob

On 8 November 2016 at 13:44, Geert Janssens <geert.gnucash at kobaltwit.be>
wrote:

> On Monday 07 November 2016 18:39:12 Geert Janssens wrote:
>
> > On Monday 07 November 2016 14:07:57 Robert Fewell wrote:
>
> > > All,
>
> > > I have built a new windows gnucash version based on MASTER checked
>
> > > out just before Johns c++ backend stuff as that does not build and
>
> > > I still have the same error with the account-piecharts.scm run from
>
> > > the report "Asset Piechart" or "Expenses PieChart" as they both
>
> > > access the same file.
>
> > >
>
> > > Tried a new file again but the same result.
>
> > >
>
> > > Are you guy's sure you were using a MASTER version ?
>
> >
>
> > Yes, I'm on master.
>
> >
>
> > Can you attach your test file for which your report fails ? If the bug
>
> > is dependent on a certain combination of data, it may behave the same
>
> > on my system. That would help me dig deeper.
>
> >
>
> > Are you using the default report options or did you change anything ?
>
> >
>
> > Regards,
>
> >
>
> > Geert
>
> >
>
> Robert,
>
>
>
> The simple test data file you sent me didn't trigger the guile error you
> got. The report indicated there was no data to display, so I didn't get a
> chart either. However that's normal and from how I read the report it
> should have tripped over the same guile error in both cases.
>
>
>
> Something occurred to me: what version of guile are you building gnucash
> with ? I'm running 2.0.x. Windows is still on 1.8.x, so perhaps that's the
> key to the different behavior.
>
>
>
> I got your altered accounts-piecharts.scm file and see you have mostly
> changed a couple of variable definitions into (lambda) functions. Perhaps
> that's one of the subtle changes between the two guile versions or more
> accurately between the fact that guile 1.8 interprets the file where 2.0
> precompiles it.
>
>
>
> I have tested your altered accounts-piecharts.scm and it works fine on
> guile 2.0. So as far as I'm concerned you can add these changes in a
> separate commit to your PR.
>
>
>
> Regards,
>
>
>
> Geert
>