376894 - Make DEBUG_PKCS11 work for optimized builds, too

Status: RESOLVED FIXED

(NSS :: Build, defect, P3)

Description

[Kaspar]

Attachment: allow DEBUG_PKCS11 also for optimized builds

NSS tech note 2 ("Using the PKCS #11 Module Logger", see URL) says:

> The logger is available by default in debug builds.  For optimized builds, 
> NSS must be built with the variable DEBUG_PKCS11 set.

Unfortunately, the second statement isn't true right now - for optimized builds, the problem is that when prlog.h gets included by pk11wrap/debug_module.c, the PR_LOG macro will end up with an empty definition. The attached patch fixes this by adding -DFORCE_PR_LOG to the pk11wrap manifest.mn when compiling NSS with DEBUG_PKCS11.

Comment 1

[Nelson Bolyard (seldom reads bugmail)]

Comment on attachment 260990 [details] [diff] [review]
allow DEBUG_PKCS11 also for optimized builds

r=nelson for trunk

Comment 2

[Julien Pierre]

Checked in to NSS_3_11_BRANCH :

Checking in manifest.mn;
/cvsroot/mozilla/security/nss/lib/pk11wrap/manifest.mn,v  <--  manifest.mn
new revision: 1.16.2.1; previous revision: 1.16
done

And to the NSS trunk :

Checking in manifest.mn;
/cvsroot/mozilla/security/nss/lib/pk11wrap/manifest.mn,v  <--  manifest.mn
new revision: 1.17; previous revision: 1.16
done