SVN Trunk crash on price quote

[Mike Evans]

Reverting  app-utils.i changes fixes the issue.  I'll commit it to 2.4
as well as trunk.

Regards
Mike E

On Thu, 24 Nov 2011 14:17:18 +0100
Geert Janssens <janssens-geert at telenet.be> wrote:

> Hmm, most likely candidate is my commit r21588.
> 
> You can try to revert the app-utils.i changes in that file. That will
> re-add a bunch of warnings in most of the scm based tests, but will
> probably fix your price quote problem for now.
> 
> I'm short on time to work on this now. If you want, you can commit
> the fix if it works. I'll check later if I can manage to eliminate
> the test warnings without causing GnuCash to crash.
> 
> By the way, this change was backported to 2.4 also, so we'll need to
> revert the change there as well...
> 
> Geert
> 
> On donderdag 24 november 2011, Mike Evans wrote:
> > Ahem.
> > 
> > Having successfully compiled I now have a crash when updating
> > prices.
> > 
> > Output from;
> > ~/progs/bin/gnucash --logto stdout
> > shown below.
> > 
> > To reproduce:
> > Tools->Price Editor; Get Qoutes.
> > 
> > Crash occurs whether I have funds or not.
> > 
> > Found Finance::Quote version 1.17
> > Backtrace:
> > In unknown file:
> >    ?:  0* [gnc:book-add-quotes # #]
> >    ?:  1* (letrec (# # # # ...) (let* # # #))
> > In /home/mikee/progs/share/gnucash/guile-modules/gnucash/price-quotes.scm:
> >  597:  2  (let* (# # # # ...) (cond # # # ...) (if keep-going? #))
> >  597:  3* [book->commodity->fq-call-data #<swig-pointer QofBook *
> > 99e5510>] 394:  4  (let* ((ct #) (big-list #) (commodity-list
> > 99e5510>#f) ...) (if (and # #)
> > #f ...)) 399:  5* [filter #<procedure #f (a)> ...
> >  401:  6* [call-with-values #<procedure #f ()> #<procedure #f (a
> > b)>] In unknown file:
> >    ?:  7  (@call-with-values (producer consumer))
> >    ?:  8* [#<procedure #f ()>]
> > In /home/mikee/progs/share/gnucash/guile-modules/gnucash/price-quotes.scm:
> >  402:  9* [partition! #<procedure #f (cmd)> ({#})]
> > In unknown file:
> >    ?: 10* [#<procedure #f (cmd)> {#<swig-pointer gnc_commodity *
> > 94f62d0>}]
> > 94f62d0>In /home/mikee/progs/share/gnucash/guile-modules/gnucash/price-quotes.scm:
> >  404: 11* [not ...
> >  404: 12* [string=? ...
> >  404: 13* [car {#<swig-pointer gnc_commodity * 94f62d0>}]
> > 
> > /home/mikee/progs/share/gnucash/guile-modules/gnucash/price-quotes.scm:404:
> > 61: In procedure car in expression (car cmd):
> > /home/mikee/progs/share/gnucash/guile-modules/gnucash/price-quotes.scm:404
> > :61: Wrong type (expecting pair): #<swig-pointer gnc_commodity *
> > 94f62d0>
> > 
> > 
> > /home/mikee/progs/share/gnucash/guile-modules/gnucash/price-quotes.scm:404:
> > 61: In procedure car in expression (car
> > cmd):
> > /home/mikee/progs/share/gnucash/guile-modules/gnucash/price-quotes.scm:404
> > :61: Wrong type (expecting pair): #<swig-pointer gnc_commodity *
> > 94f62d0>
> > 
> > This is recently introduced but not sure when exactly, last week or
> > two I'd guess.  I can't see any obvious commit candidates though.
> > More work for git bisect?
> > 
> > Incidentally, since I suspected it might be due to upgrade issues.
> > 2.4.8 works OK and from the command line: gnc-fq-dump yahoo GOOG
> > Finance::Quote fields Gnucash uses:
> >     symbol: GOOG                 <=== required
> >       date: 11/23/2011           <=== required
> >   currency: USD                  <=== required
> >       last: 570.11               <=\
> >        nav:                      <=== one of these
> >      price: 570.11               <=/
> >   timezone:                      <=== optiona
> > _______________________________________________
> > gnucash-devel mailing list
> > gnucash-devel at gnucash.org
> > https://lists.gnucash.org/mailman/listinfo/gnucash-devel
>