Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

wxWidgets debug trace left on #1564

Closed
Germano0 opened this issue Jun 25, 2016 · 5 comments
Closed

wxWidgets debug trace left on #1564

Germano0 opened this issue Jun 25, 2016 · 5 comments

Comments

@Germano0
Copy link
Contributor

Line codes

m_pLog->AddTraceMask(wxT("Function Start/End"));

m_pLog->AddTraceMask(wxT("Function Status"));

left wxWidgets debug trace on, so users get huge debug logs in ~/.BOINC folder.

Downstream bugreport https://bugzilla.redhat.com/show_bug.cgi?id=1347423

@ChristianBeer
Copy link
Member

Those two lines are in the code since 2009. Are you sure you disabled the wxWidgets debugging when building the Manager for Fedora? They changed how debug mode works and you now have to build wxWidgets with -DNDEBUG to turn debug output off.

@Germano0
Copy link
Contributor Author

Mmh a wxWidgets developer just told me that on wxWidgets > 3.x the debugtrace is enabled by default. This explains why in past we did not experience this problem
I solved addind the flag to line
http://pkgs.fedoraproject.org/cgit/rpms/boinc-client.git/tree/boinc-client.spec#n136

@LocutusOfBorg
Copy link
Contributor

I guess the NDEBUG is something that upstream should export by default, I had to do the same patch in Debian

# Disable WXDEBUG assertions (avoiding e.g. https://bugs.debian.org/763113) -
# with -DNDEBUG these are quietly dealt with like they were with wx2.8.
export DEB_CPPFLAGS_MAINT_APPEND := -DNDEBUG

@ChristianBeer
Copy link
Member

Well, this was indeed implemented upstream some time ago:
#1476 commits: 530eb18 and cc67f46
This only works if you use the ./configure file created by ./_autosetup of course. If this doesn't work let me know a place where I should change that upstream.

@LocutusOfBorg
Copy link
Contributor

yes, now that I look at build logs, in Debian they are injected correctly (and then they were injected twice)

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

3 participants