Closed Bug 389114 Opened 17 years ago Closed 17 years ago

Build fails with PR_LOG undefined

Categories

(Core :: Widget: Cocoa, defect)

PowerPC
macOS
defect
Not set
normal

Tracking

()

RESOLVED FIXED

People

(Reporter: crzyhanko, Assigned: jaas)

References

Details

Attachments

(1 file)

User-Agent:       Mozilla/5.0 (Macintosh; U; Intel Mac OS X; en-US; rv:1.9a7pre) Gecko/2007071823 Minefield/3.0a7pre
Build Identifier: 

Since bug Bug 358093 landed, I've been unable to build Firefox with ac_add_options --disable-logging in my mozconfig. Either removing that or commenting out the PR_LOG stuff added by the patch for Bug 358093 will allow Firefox to build.

Reproducible: Always
Can you paste the actual build error you get?
/Developer/moz/mozilla/widget/src/cocoa/nsClipboard.mm
/Developer/moz/mozilla/widget/src/cocoa/nsClipboard.mm: In static member function 'static NSDictionary* nsClipboard::PasteboardDictFromTransferable(nsITransferable*)':
/Developer/moz/mozilla/widget/src/cocoa/nsClipboard.mm:302: error: 'sCocoaLog' was not declared in this scope
/Developer/moz/mozilla/widget/src/cocoa/nsClipboard.mm:302: error: 'PR_LOG_ALWAYS' was not declared in this scope
/Developer/moz/mozilla/widget/src/cocoa/nsClipboard.mm:302: warning: left-hand operand of comma has no effect
/Developer/moz/mozilla/widget/src/cocoa/nsClipboard.mm:302: error: 'PR_LOG' was not declared in this scope
make[6]: *** [nsClipboard.o] Error 1
make[5]: *** [libs] Error 2
make[4]: *** [libs] Error 2
make[3]: *** [libs_tier_gecko] Error 2
make[2]: *** [tier_gecko] Error 2
make[1]: *** [default] Error 2
make: *** [build] Error 2
Blocks: 358093
Attached patch fix v1.0Splinter Review
Attachment #279291 - Flags: review?(stanshebs)
This should be blocking because it is a build failure for a not-uncommon configuration.
Status: UNCONFIRMED → ASSIGNED
Ever confirmed: true
Flags: blocking1.9+
Attachment #279291 - Flags: review?(stanshebs) → review+
Attachment #279291 - Flags: superreview?(roc)
Attachment #279291 - Flags: superreview?(roc) → superreview+
landed on trunk
Status: ASSIGNED → RESOLVED
Closed: 17 years ago
Resolution: --- → FIXED
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Creator:
Created:
Updated:
Size: