nixpkgs/pkgs/development/interpreters/spidermonkey/common.nix
Alyssa Ross b33fd42c30 pkgsMusl.spidermonkey: fix build on x86_64-linux
rustc 1.73.0 introduced a new x86_64-unikraft-linux-musl target.  This
triple is a bit of a lie — the kernel in this case is unikraft, which
isn't Linux, it just tries to be compatible with it.  But this
User-Agent-like string of a triple now exists, and with it comes some
fallout.

Spidermonkey asks rustc for the list of supported targets, then picks
the one that best matches the GNU triple it's building for, which in
the native case config.guess will say is x86_64-pc-linux-musl.  Prior
to rustc 1.73.0, all was well, because there was only one x86_64 musl
triple, but now there are two: x86_64-unknown-linux-musl, and
x86_64-unikraft-linux-musl, and neither of those are obviously closer
than the other to x86_64-pc-linux-musl to Spidermonkey's build system.
As a result, it can't decide which target to pass to rustc, and bails
out.

We can fix this by telling Spidermonkey what platform it should be
building for, rather than letting it guess, which previously we did
only when cross compiling.  This approach is already used in the
Firefox derivation (which is why it was not affected by the rustc
update), so it's been well tested already.

Closes: https://github.com/NixOS/nixpkgs/pull/266560
Closes: https://github.com/NixOS/nixpkgs/pull/282923
2024-02-01 15:36:20 +01:00

199 lines
6.0 KiB
Nix

{ version, hash }:
{ callPackage
, lib
, stdenv
, fetchurl
, fetchpatch
# build time
, buildPackages
, cargo
, m4
, perl
, pkg-config
, python3
, python39
, rustc
, which
, zip
, autoconf213
, yasm
, xcbuild
# runtime
, icu
, icu67
, nspr
, readline
, zlib
, libobjc
, libiconv
}:
stdenv.mkDerivation (finalAttrs: rec {
pname = "spidermonkey";
inherit version;
outputs = [ "out" "dev" ];
src = fetchurl {
url = "mirror://mozilla/firefox/releases/${version}esr/source/firefox-${version}esr.source.tar.xz";
inherit hash;
};
patches = lib.optionals (lib.versionOlder version "91") [
# Fix build failure on armv7l using Debian patch
# Upstream bug: https://bugzilla.mozilla.org/show_bug.cgi?id=1526653
(fetchpatch {
url = "https://salsa.debian.org/mozilla-team/firefox/commit/fd6847c9416f9eebde636e21d794d25d1be8791d.patch";
hash = "sha512-K8U3Qyo7g4si2r/8kJdXyRoTrDHAY48x/YJ7YL+YBwlpfNQcHxX+EZvhRzW8FHYW+f7kOnJu9QykhE8PhSQ9zQ==";
})
# Remove this when updating to 79 - The patches are already applied upstream
# https://bugzilla.mozilla.org/show_bug.cgi?id=1318905
# Combination of 3 changesets, modified to apply on 78:
# - https://hg.mozilla.org/mozilla-central/rev/06d7e1b6b7e7
# - https://hg.mozilla.org/mozilla-central/rev/ec48f15d085c
# - https://hg.mozilla.org/mozilla-central/rev/6803dda74d33
./add-riscv64-support.patch
] ++ lib.optionals (lib.versionAtLeast version "102") [
# use pkg-config at all systems
./always-check-for-pkg-config.patch
./allow-system-s-nspr-and-icu-on-bootstrapped-sysroot.patch
] ++ lib.optionals (lib.versionAtLeast version "91" && stdenv.hostPlatform.system == "i686-linux") [
# Fixes i686 build, https://bugzilla.mozilla.org/show_bug.cgi?id=1729459
./fix-float-i686.patch
] ++ lib.optionals (lib.versionAtLeast version "91" && lib.versionOlder version "102") [
# Fix 91 compatibility with python311
(fetchpatch {
url = "https://src.fedoraproject.org/rpms/mozjs91/raw/rawhide/f/0001-Python-Build-Use-r-instead-of-rU-file-read-modes.patch";
hash = "sha256-WgDIBidB9XNQ/+HacK7jxWnjOF8PEUt5eB0+Aubtl48=";
})
];
nativeBuildInputs = [
cargo
m4
perl
pkg-config
# 78 requires python up to 3.9
(if lib.versionOlder version "91" then python39 else python3)
rustc
rustc.llvmPackages.llvm # for llvm-objdump
which
zip
] ++ lib.optionals (lib.versionOlder version "91") [
autoconf213
yasm # to buid icu? seems weird
] ++ lib.optionals stdenv.isDarwin [
xcbuild
];
buildInputs = [
(if lib.versionOlder version "91" then icu67 else icu)
nspr
readline
zlib
] ++ lib.optionals stdenv.isDarwin [
libobjc
libiconv
];
depsBuildBuild = [
buildPackages.stdenv.cc
];
setOutputFlags = false; # Configure script only understands --includedir
configureFlags = [
"--with-intl-api"
"--with-system-icu"
"--with-system-nspr"
"--with-system-zlib"
# Fedora and Arch disable optimize, but it doesn't seme to be necessary
# It turns on -O3 which some gcc version had a problem with:
# https://src.fedoraproject.org/rpms/mozjs38/c/761399aba092bcb1299bb4fccfd60f370ab4216e
"--enable-optimize"
"--enable-readline"
"--enable-release"
"--enable-shared-js"
] ++ lib.optionals (lib.versionAtLeast version "91") [
"--disable-debug"
] ++ [
"--disable-jemalloc"
"--disable-strip"
"--disable-tests"
# Spidermonkey seems to use different host/build terminology for cross
# compilation here.
"--host=${stdenv.buildPlatform.config}"
"--target=${stdenv.hostPlatform.config}"
];
# mkDerivation by default appends --build/--host to configureFlags when cross compiling
# These defaults are bogus for Spidermonkey - avoid passing them by providing an empty list
configurePlatforms = [ ];
enableParallelBuilding = true;
# cc-rs insists on using -mabi=lp64 (soft-float) for riscv64,
# while we have a double-float toolchain
env.NIX_CFLAGS_COMPILE = lib.optionalString (with stdenv.hostPlatform; isRiscV && is64bit && lib.versionOlder version "91") "-mabi=lp64d";
postPatch = lib.optionalString (lib.versionOlder version "102") ''
# This patch is a manually applied fix of
# https://bugzilla.mozilla.org/show_bug.cgi?id=1644600
# Once that bug is fixed, this can be removed.
# This is needed in, for example, `zeroad`.
substituteInPlace js/public/StructuredClone.h \
--replace "class SharedArrayRawBufferRefs {" \
"class JS_PUBLIC_API SharedArrayRawBufferRefs {"
'';
preConfigure = lib.optionalString (lib.versionOlder version "91") ''
export CXXFLAGS="-fpermissive"
'' + ''
export LIBXUL_DIST=$out
export PYTHON="${buildPackages.python3.interpreter}"
'' + lib.optionalString (lib.versionAtLeast version "91") ''
export M4=m4
export AWK=awk
export AS=$CC
export AC_MACRODIR=$PWD/build/autoconf/
'' + lib.optionalString (lib.versionAtLeast version "91" && lib.versionOlder version "115") ''
pushd js/src
sh ../../build/autoconf/autoconf.sh --localdir=$PWD configure.in > configure
chmod +x configure
popd
'' + lib.optionalString (lib.versionAtLeast version "115") ''
patchShebangs build/cargo-linker
'' + ''
# We can't build in js/src/, so create a build dir
mkdir obj
cd obj/
configureScript=../js/src/configure
'';
# Remove unnecessary static lib
preFixup = ''
moveToOutput bin/js${lib.versions.major version}-config "$dev"
rm $out/lib/libjs_static.ajs
ln -s $out/bin/js${lib.versions.major version} $out/bin/js
'';
passthru.tests.run = callPackage ./test.nix {
spidermonkey = finalAttrs.finalPackage;
};
meta = with lib; {
description = "Mozilla's JavaScript engine written in C/C++";
homepage = "https://spidermonkey.dev/";
license = licenses.mpl20; # TODO: MPL/GPL/LGPL tri-license for 78.
maintainers = with maintainers; [ abbradar lostnet catap ];
broken = stdenv.isDarwin && versionAtLeast version "115"; # Requires SDK 13.3 (see #242666).
platforms = platforms.unix;
};
})