Created attachment 305278 [details] [diff] [review] Proposed patch The default makefile target should be 'all' (which is equivalent to 'export libs') so that when we say gmake it will build the 'all' makefile target. This requires that 'all' (from coreconf/rules.mk) be the first makefile target in the makefile. This is not the case in lib/softoken and lib/softoken/legacydb because it has a makefile target defined in the local config.mk, which is included before coreconf/rules.mk. The attached patch fixes this problem.
I checked in the patch on the NSS trunk for NSS 3.12. Checking in Makefile; /cvsroot/mozilla/security/nss/lib/softoken/Makefile,v <-- Makefile new revision: 1.6; previous revision: 1.5 done Checking in config.mk; /cvsroot/mozilla/security/nss/lib/softoken/config.mk,v <-- config.mk new revision: 1.24; previous revision: 1.23 done Checking in legacydb/Makefile; /cvsroot/mozilla/security/nss/lib/softoken/legacydb/Makefile,v <-- Makefile new revision: 1.3; previous revision: 1.2 done Checking in legacydb/config.mk; /cvsroot/mozilla/security/nss/lib/softoken/legacydb/config.mk,v <-- config.mknew revision: 1.6; previous revision: 1.5 done
Created attachment 307117 [details] [diff] [review] Supplemental patch lib/ssl needs the same change because the SSL bypass code also uses freebl.
Comment on attachment 307117 [details] [diff] [review] Supplemental patch I checked in this patch (for lib/ssl) on the NSS trunk for NSS 3.12. Checking in Makefile; /cvsroot/mozilla/security/nss/lib/ssl/Makefile,v <-- Makefile new revision: 1.7; previous revision: 1.6 done Checking in config.mk; /cvsroot/mozilla/security/nss/lib/ssl/config.mk,v <-- config.mk new revision: 1.24; previous revision: 1.23 done