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

fix(docker-build): use platform flag only when available #190

Merged
merged 1 commit into from
Feb 28, 2024
Merged

fix(docker-build): use platform flag only when available #190

Show file tree
Hide file tree
Changes from all commits
Commits
Show all changes
1 commit
Select commit
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
5 changes: 5 additions & 0 deletions .changeset/brown-clouds-knock.md
View file
Open in desktop
Original file line number Diff line number Diff line change
@@ -0,0 +1,5 @@
---
'arui-scripts': patch
---

Исправлен запуск команды docker-build при использовании докера версии 17-20
7 changes: 4 additions & 3 deletions packages/arui-scripts/src/commands/util/docker-build.ts
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -114,10 +114,11 @@ type DockerBuildCommandParams = {

export function getDockerBuildCommand({ tempDirName, imageFullName }: DockerBuildCommandParams) {
// если пытаться собрать проект на маках с m1, докер будет пытаться вытянуть базовый образ под свою платформу и
// упадет с ошибкой. Чтобы этого избежать - достаточно использовать флаг --platform. Но он поддерживается только начиная
// с docker 17.12, а на многих серверах, используемых для сборки до сих пор живет докер 1.13.1
// упадет с ошибкой. Чтобы этого избежать - достаточно использовать флаг --platform. Но он поддерживается без экспериментальных
// флагов только начиная с docker 20.10.21, а на многих серверах, используемых для сборки до сих пор живет докер 1.13.1
// Соответственно они будут падать при наличии этого флага.
const canUsePlatformFlag = dockerVersionSatisfies('>=17.12');
// https://docs.docker.com/engine/release-notes/20.10/
const canUsePlatformFlag = dockerVersionSatisfies('>=20.10.21');

return `docker build ${
canUsePlatformFlag ? '--platform linux/x86_64' : ''
Expand Down
Loading