Skip to content

Commit b436cb1

Browse files
committed
fix platform tag for graalpy
1 parent e597521 commit b436cb1

File tree

1 file changed

+4
-7
lines changed

1 file changed

+4
-7
lines changed

src/python_interpreter/mod.rs

Lines changed: 4 additions & 7 deletions
Original file line numberDiff line numberDiff line change
@@ -520,18 +520,15 @@ impl PythonInterpreter {
520520
)
521521
}
522522
InterpreterKind::GraalPy => {
523-
// GraalPy suffers from pypa/packaging#614, where
524-
// packaging misdetects it as a 32-bit implementation,
525-
// so GraalPy adds the correct platform itself, e.g.
526-
// graalpy 3.10 23.1 => numpy-1.23.5-graalpy310-graalpy231_310_native_x86_64_linux-linux_i686.whl
523+
// GraalPy like PyPy uses its version as part of the ABI
524+
// graalpy 3.10 23.1 => numpy-1.23.5-graalpy310-graalpy231_310_native_manylinux2014_x86_64.whl
527525
format!(
528-
"graalpy{major}{minor}-{abi_tag}_{arch}_{os}-{os}_i686",
526+
"graalpy{major}{minor}-{abi_tag}_{platform}",
529527
major = self.major,
530528
minor = self.minor,
531529
abi_tag = calculate_abi_tag(&self.ext_suffix)
532530
.expect("GraalPy's syconfig didn't define a valid `EXT_SUFFIX` ಠ_ಠ"),
533-
os = target.get_python_os(),
534-
arch = target.get_python_arch(),
531+
platform = platform,
535532
)
536533
}
537534
}

0 commit comments

Comments
 (0)