Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

Add architecture input check for PyPy for Windows platform #520

Merged
merged 17 commits into from Oct 18, 2022
Merged

Add architecture input check for PyPy for Windows platform #520

Show file tree
Hide file tree
Changes from 13 commits
Commits
Show all changes
17 commits
Select commit Hold shift + click to select a range
7e403af
Revert cache index.js
Oct 12, 2022
de93cfa
build cache index file
Oct 12, 2022
5d1b76c
Refactor
Oct 12, 2022
29adb1a
Debug
Oct 13, 2022
7bb5a4b
Debug
Oct 13, 2022
69f5a56
Debug
Oct 13, 2022
ae01777
Debug
Oct 13, 2022
c139ac5
Debug
Oct 13, 2022
ffca1d4
Debug
Oct 13, 2022
5041c3b
Debug
Oct 13, 2022
82e58e0
Debug
Oct 13, 2022
97960c6
Format code
Oct 13, 2022
0a1e867
Rebuild dist
Oct 14, 2022
71e2dec
Minor refactor
Oct 14, 2022
957735f
Format code
Oct 14, 2022
d1ffee0
Minor fixes
Oct 17, 2022
9854107
Check platform firstly
Oct 17, 2022
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Jump to
Jump to file
Failed to load files.
Diff view
Diff view
27 changes: 19 additions & 8 deletions dist/setup/index.js
View file
Open in desktop
Expand Up @@ -66577,7 +66577,7 @@ function findRelease(releases, pythonVersion, pypyVersion, architecture) {
semver.satisfies(pypyVersionToSemantic(item.pypy_version), pypyVersion);
const isArchPresent = item.files &&
(utils_1.IS_WINDOWS
? isArchPresentForWindows(item)
? isArchPresentForWindows(item, architecture)
: isArchPresentForMacOrLinux(item, architecture, process.platform));
return isPythonVersionSatisfied && isPyPyVersionSatisfied && isArchPresent;
});
Expand All @@ -66590,7 +66590,7 @@ function findRelease(releases, pythonVersion, pypyVersion, architecture) {
});
const foundRelease = sortedReleases[0];
const foundAsset = utils_1.IS_WINDOWS
? findAssetForWindows(foundRelease)
? findAssetForWindows(foundRelease, architecture)
: findAssetForMacOrLinux(foundRelease, architecture, process.platform);
return {
foundAsset,
Expand All @@ -66613,24 +66613,35 @@ function pypyVersionToSemantic(versionSpec) {
return versionSpec.replace(prereleaseVersion, '$1-$2.$3');
}
exports.pypyVersionToSemantic = pypyVersionToSemantic;
function isArchPresentForWindows(item) {
return item.files.some((file) => utils_1.WINDOWS_ARCHS.includes(file.arch) &&
utils_1.WINDOWS_PLATFORMS.includes(file.platform));
function isArchPresentForWindows(item, architecture) {
architecture = ReplaceX32toX86(architecture);
return item.files.some((file) => {
return (file.arch === architecture && utils_1.WINDOWS_PLATFORMS.includes(file.platform));
});
}
exports.isArchPresentForWindows = isArchPresentForWindows;
function isArchPresentForMacOrLinux(item, architecture, platform) {
return item.files.some((file) => file.arch === architecture && file.platform === platform);
}
exports.isArchPresentForMacOrLinux = isArchPresentForMacOrLinux;
function findAssetForWindows(releases) {
return releases.files.find((item) => utils_1.WINDOWS_ARCHS.includes(item.arch) &&
utils_1.WINDOWS_PLATFORMS.includes(item.platform));
function findAssetForWindows(releases, architecture) {
architecture = ReplaceX32toX86(architecture);
return releases.files.find((item) => {
return (item.arch === architecture && utils_1.WINDOWS_PLATFORMS.includes(item.platform));
});
}
exports.findAssetForWindows = findAssetForWindows;
function findAssetForMacOrLinux(releases, architecture, platform) {
return releases.files.find((item) => item.arch === architecture && item.platform === platform);
}
exports.findAssetForMacOrLinux = findAssetForMacOrLinux;
function ReplaceX32toX86(architecture) {
// convert x32 to x86 because os.arch() return x32 for 32-bit systems but PyPy releases json has x86 arch value.
if (architecture === 'x32') {
architecture = 'x86';
}
return architecture;
}


/***/ }),
Expand Down
38 changes: 24 additions & 14 deletions src/install-pypy.ts
View file
Open in desktop
Expand Up @@ -157,7 +157,7 @@ export function findRelease(
const isArchPresent =
item.files &&
(IS_WINDOWS
? isArchPresentForWindows(item)
? isArchPresentForWindows(item, architecture)
: isArchPresentForMacOrLinux(item, architecture, process.platform));
return isPythonVersionSatisfied && isPyPyVersionSatisfied && isArchPresent;
});
Expand All @@ -181,7 +181,7 @@ export function findRelease(

const foundRelease = sortedReleases[0];
const foundAsset = IS_WINDOWS
? findAssetForWindows(foundRelease)
? findAssetForWindows(foundRelease, architecture)
: findAssetForMacOrLinux(foundRelease, architecture, process.platform);

return {
Expand All @@ -205,12 +205,13 @@ export function pypyVersionToSemantic(versionSpec: string) {
return versionSpec.replace(prereleaseVersion, '$1-$2.$3');
}

export function isArchPresentForWindows(item: any) {
return item.files.some(
(file: any) =>
WINDOWS_ARCHS.includes(file.arch) &&
vsafonkin marked this conversation as resolved.
Show resolved Hide resolved
WINDOWS_PLATFORMS.includes(file.platform)
);
export function isArchPresentForWindows(item: any, architecture: string) {
architecture = ReplaceX32toX86(architecture);
return item.files.some((file: any) => {
return (
file.arch === architecture && WINDOWS_PLATFORMS.includes(file.platform)
vsafonkin marked this conversation as resolved.
Show resolved Hide resolved
);
});
vsafonkin marked this conversation as resolved.
Show resolved Hide resolved
}

export function isArchPresentForMacOrLinux(
Expand All @@ -223,12 +224,13 @@ export function isArchPresentForMacOrLinux(
);
}

export function findAssetForWindows(releases: any) {
return releases.files.find(
(item: any) =>
WINDOWS_ARCHS.includes(item.arch) &&
WINDOWS_PLATFORMS.includes(item.platform)
);
export function findAssetForWindows(releases: any, architecture: string) {
architecture = ReplaceX32toX86(architecture);
return releases.files.find((item: any) => {
return (
vsafonkin marked this conversation as resolved.
Show resolved Hide resolved
item.arch === architecture && WINDOWS_PLATFORMS.includes(item.platform)
);
});
}

export function findAssetForMacOrLinux(
Expand All @@ -240,3 +242,11 @@ export function findAssetForMacOrLinux(
(item: any) => item.arch === architecture && item.platform === platform
);
}

function ReplaceX32toX86(architecture: string): string {
// convert x32 to x86 because os.arch() return x32 for 32-bit systems but PyPy releases json has x86 arch value.
vsafonkin marked this conversation as resolved.
Show resolved Hide resolved
if (architecture === 'x32') {
architecture = 'x86';
}
return architecture;
}