After an evaluation, GNOME has moved from Bugzilla to GitLab. Learn more about GitLab.
No new issues can be reported in GNOME Bugzilla anymore.
To report an issue in a GNOME project, go to GNOME GitLab.
Do not go to GNOME Gitlab for: Bluefish, Doxygen, GnuCash, GStreamer, java-gnome, LDTP, NetworkManager, Tomboy.
Bug 593906 - GnuCash crashes at creating account report
GnuCash crashes at creating account report
Status: RESOLVED DUPLICATE of bug 598213
Product: GnuCash
Classification: Other
Component: Reports
2.3.x
Other Linux
: Normal critical
: ---
Assigned To: Andreas Köhler
Andreas Köhler
: 595517 599891 (view as bug list)
Depends on: 596797
Blocks:
 
 
Reported: 2009-09-02 11:39 UTC by noriko
Modified: 2018-06-29 22:28 UTC
See Also:
GNOME target: ---
GNOME version: ---


Attachments
Patch to fix this problem. (574 bytes, patch)
2009-09-17 06:55 UTC, Mike Alexander
needs-work Details | Review
Patch that doesn't use the "@" function (607 bytes, patch)
2009-10-03 19:14 UTC, Mike Alexander
needs-work Details | Review

Description noriko 2009-09-02 11:39:16 UTC
GnuCash 2.3.4 and 2.3.5 crashes when clicking the Account Report menu item. The same happens when clicking Account Transaction Report menu item.

My OS is Linux Debian Lenny.
Comment 1 Phil Longstaff 2009-09-08 19:52:08 UTC
Can you provide a stack trace?  See http://wiki.gnucash.org/wiki/Stack_Trace.

Please attach as an attachment to this bug.
Comment 2 Mike Alexander 2009-09-17 06:54:09 UTC
I'm seeing the same problem.  I tracked it down to the set of changes made in r18253 to automatically load all the reports in the standard-reports directory.  This changed standard-reports.scm to use resolve-module instead of use-modules to load reports.  One difference between these two is that resolve-module doesn't make the interface for the loaded module available to the module containing the call to resolve-module.  This causes a crash when gnc:register-report-create in standard-reports.scm tries to call gnc:register-report-create-internal which is located in register.scm (one of the modules loaded using resolve-module).  I fixed it by explicitly defining that interface using the "@" guile function after the modules are loaded.  You could also probably just add back a call to use-module for that module, but this seems to work and makes it explicit what interface is needed.

I'll attach a patch.
Comment 3 Mike Alexander 2009-09-17 06:55:12 UTC
Created attachment 143321 [details] [review]
Patch to fix this problem.
Comment 4 Christian Stimming 2009-09-21 08:47:06 UTC
*** Bug 595517 has been marked as a duplicate of this bug. ***
Comment 5 Christian Stimming 2009-09-28 17:32:21 UTC
Comment on attachment 143321 [details] [review]
Patch to fix this problem.

r18352, thanks a lot!
Comment 6 Steven N. Severinghaus 2009-09-30 21:10:52 UTC
I just built gnucash from svn head, rev 18359, and I now get a crash on startup with this error:

In /usr/local/share/gnucash/guile-modules/gnucash/report/standard-reports.scm:
 160: 27* (define gnc:register-report-create-internal #)
 160: 28* (@ # gnc:register-report-create-internal)

/usr/local/share/gnucash/guile-modules/gnucash/report/standard-reports.scm:160:45: In expression (@ (gnucash report standard-reports ...) gnc:register-report-create-internal):
/usr/local/share/gnucash/guile-modules/gnucash/report/standard-reports.scm:160:45: Unbound variable: @

It seems related to this bug. Do I have a bad version of guile? I think I'm using guile-1.6-dev, 1.6.8-6.3ubuntu1.
Comment 7 Christian Stimming 2009-10-03 11:35:58 UTC
Patch reverted in r18360:

The bugfix was using the guile function "@" which apparently isn't available
on quite a number of platforms, including the Windows guile-1.6.8 which
we're shipping in our windows binary. So we need to fix the original bug
in a different way without the "@" function.
Comment 8 Mike Alexander 2009-10-03 19:14:49 UTC
Created attachment 144682 [details] [review]
Patch that doesn't use the "@" function

Since the "@" function doesn't exist in some versions of Guile, patch it the way I suggested in comment 2 by calling use-modules for the register report module.
Comment 9 Christian Stimming 2009-10-26 20:24:08 UTC
r18396, thanks a lot!
Comment 10 Akhil Laddha 2009-10-29 04:19:55 UTC
please see bug 599891
Comment 11 Christian Stimming 2009-10-30 19:53:30 UTC
*** Bug 599891 has been marked as a duplicate of this bug. ***
Comment 12 Christian Stimming 2009-10-30 19:56:07 UTC
Unfortunately this change doesn't work on Windows and for this reason I've reverted this in r18400. On Windows with this line added, the splash screen will show the module loading until standard-reports, then every gnucash window disappears. No message in .trace and (because we've disabled console output on windows) no console output.

I didn't have access to a debugger to see where a debugger would end up here. I can only guess this (use-modules ...) implies some sort of library loading order which for whatever reason doesn't work on windows but works fine on Linux...
Comment 13 Sebastian Held 2009-11-07 16:13:49 UTC
The change in r18400 breaks the reports on Linux:

[...]
Unbound variable: gnc:register-report-create-internal

How to fix this on all supported platforms?
Comment 14 Christian Stimming 2009-11-12 14:08:22 UTC
Comment on attachment 144682 [details] [review]
Patch that doesn't use the "@" function

Patch was reverted again because this change still doesn't work on windows.
Comment 15 Phil Longstaff 2009-11-16 01:49:57 UTC

*** This bug has been marked as a duplicate of bug 598213 ***
Comment 16 John Ralls 2018-06-29 22:28:03 UTC
GnuCash bug tracking has moved to a new Bugzilla host. This bug has been copied to https://bugs.gnucash.org/show_bug.cgi?id=593906. Please update any external references or bookmarks.