Skip to contentSkip to conversation

libnss: Enable for MIPS64 n32 platforms

PATCH New Vicente Olivert Riera 2 participants 2 messages May 22, 2014 – May 26, 2014

lore
Share & tools
lore
#0 lore raw

PATCH v3 1/1 libnss: Enable for MIPS64 n32 platforms

After talking with upstream we concluded that MIPS64 n32 is treated as a 32-bit architecture by libnss because it has 32-bit pointers, so USE_64=1 should not be set in that case.

Upstream bug:
https://bugzilla.mozilla.org/show_bug.cgi?id=1010730

WRT:
http://git.buildroot.net/buildroot/commit/?id=73e6ae0fe6a69f88183ae33401f2314088b5f799

---
 package/libnss/Config.in |    2 --
 package/libnss/libnss.mk |    4 ++++
 2 files changed, 4 insertions(+), 2 deletions(-)
package/libnss/Config.in +0 -2
index 9a1668c..690f928 100644
@@ -6,7 +6,6 @@ config BR2_PACKAGE_LIBNSS
 	depends on BR2_LARGEFILE
 	depends on BR2_TOOLCHAIN_HAS_THREADS # libnspr
 	depends on BR2_PACKAGE_LIBNSPR_ARCH_SUPPORT # libnspr
-	depends on !BR2_MIPS_NABI32
 	help
 	  Network Security Services (NSS) is a set of libraries designed
 	  to support development of security-enabled client and server
@@ -19,4 +18,3 @@ config BR2_PACKAGE_LIBNSS
 comment "libnss needs a toolchain w/ largefile, threads"
 	depends on BR2_PACKAGE_LIBNSPR_ARCH_SUPPORT
 	depends on !BR2_LARGEFILE || !BR2_TOOLCHAIN_HAS_THREADS
-	depends on !BR2_MIPS_NABI32
package/libnss/libnss.mk +4 -0
index 666df74..dcbf8bb 100644
@@ -30,8 +30,12 @@ LIBNSS_BUILD_VARS = MOZILLA_CLIENT=1 \
 		OS_TEST="$(ARCH)"
 
 ifeq ($(BR2_ARCH_IS_64),y)
+# MIPS64 n32 is treated as a 32-bit architecture by libnss.
+# See: https://bugzilla.mozilla.org/show_bug.cgi?id=1010730
+ifeq ($(BR2_MIPS_NABI32),)
 LIBNSS_BUILD_VARS += USE_64=1
 endif
+endif
 
 
 define LIBNSS_BUILD_CMDS
Signature · 1 line
1.7.1
#1 lore raw

PATCH v3 1/1 libnss: Enable for MIPS64 n32 platforms

Quoted 6 lines
Quoted 1 line
Quoted 1 line
Quoted 1 line
Quoted 1 line

"Vicente" == Vicente Olivert Riera <[email protected]> writes:

After talking with upstream we concluded that MIPS64 n32 is treated as a 32-bit architecture by libnss because it has 32-bit pointers, so USE_64=1 should not be set in that case.

Quoted 7 lines

Upstream bug:

   https://bugzilla.mozilla.org/show_bug.cgi?id=1010730

WRT:

   http://git.buildroot.net/buildroot/commit/?id=73e6ae0fe6a69f88183ae33401f2314088b5f799

Committed, thanks.

Signature · 1 line
Bye, Peter Korsgaard