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.

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: improve "npx playwright install-deps" in frontend/dockerfile #130

Merged
merged 2 commits into from
Jun 5, 2024

Conversation

Marc-AntoineA
Copy link
Collaborator

What

make build was broken on my Linux Mint 21  because of this line npm playwright install-deps chromium.

The solution is to replace this line by an exhaustive apt-get install.

This command is broken on Linux Mint 21 (Vanessa)
Replaced by an exhaustive apt-get install obtained by executing
"npm playwright install-deps --dry-run"

The dependencies list shouldn't change regularly.
@Marc-AntoineA
Copy link
Collaborator Author

This change breaks build docker container on Debian.
Error :

Package 'xfonts-cyrillic' has no installation candidate

We should find another solution ^^

frontend/Dockerfile Outdated Show resolved Hide resolved
@alexgarel
Copy link
Member

alexgarel commented Jun 5, 2024

This change breaks build docker container on Debian. Error :

Package 'xfonts-cyrillic' has no installation candidate

We should find another solution ^^

EDIT: I'm wrong here ! It does not exists in bookworm !

Strange because it should exist: https://packages.debian.org/bookworm/xfonts-cyrillic

and node:lts is based upon bookworm (see https://hub.docker.com/_/node)

* Keep mount=type=cache
* use node:lts-bullseye
Copy link
Member

@alexgarel alexgarel left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks !

@alexgarel alexgarel merged commit b50e422 into main Jun 5, 2024
6 checks passed
@alexgarel alexgarel deleted the fix-docker-mint branch June 5, 2024 16:38
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
Status: Done
Development

Successfully merging this pull request may close these issues.

2 participants