Closed Bug 481962 Opened 12 years ago Closed 10 years ago

Force logging on in imagelib


(Core :: ImageLib, defect)

Not set





(Reporter: joe, Assigned: justin.lebar+bug)



(1 file, 2 obsolete files)

We need to have logging forced on, even in release builds. It's impossible to debug problems users have without it.
Attachment #366032 - Flags: review?(vladimir)
Comment on attachment 366032 [details] [diff] [review]
Force logging in build rather than in code

I don't think you even need the ifneq bit.. no reason not to just -DFORCE_PR_LOG unconditionally
Attachment #366032 - Flags: review?(vladimir) → review+
Turns out we need to add the logging bit to just about every Makefile.

Are you sure about -DFORCE_PRLOG everywhere, regardless of whether MOZ_LOGGING is on? Are we going to run into build problems in a build where logging is forced off?
Assignee: nobody → joe
Attachment #366032 - Attachment is obsolete: true
Attachment #366333 - Flags: review?(vladimir)
Comment on attachment 366333 [details] [diff] [review]
Force logging everywhere in build

I don't see how it would, but ask ted?
Attachment #366333 - Flags: review?(vladimir) → review+
The only point of MOZ_LOGGING, it would appear, is to control whether various modules define FORCE_PR_LOG. It looks like everywhere else in the tree just does:
#if defined(MOZ_LOGGING)
#define FORCE_PR_LOG

Do you not have a central header you could do this in? If not, maybe instead you could add MOZ_LOGGING=1 here:

and then AC_SUBST(MOZ_LOGGING) down with the other AC_SUBST bits, add it to, and just use it as a makefile var:
Is this still something we want?  I'd kind of like it for bug 666352 (although there, I really only care about having it on during my custom release builds, not official builds).
Yes, we definitely still want this.
Assignee: joe → justin.lebar+bug
OS: Mac OS X → All
Hardware: x86 → All
Attached patch Patch v1Splinter Review
This forces logging on within ImageLogging.h.  This is similar to how nsHttp.h works.
Attachment #544848 - Flags: review?(joe)
Attachment #366333 - Attachment is obsolete: true
Comment on attachment 544848 [details] [diff] [review]
Patch v1

If'n this passes try, hooray!
Attachment #544848 - Flags: review?(joe) → review+
Closed: 10 years ago
Resolution: --- → FIXED
Whiteboard: [inbound]
Target Milestone: --- → mozilla8
You need to log in before you can comment on or make changes to this bug.