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

Dockerfile: add PyYaml Python package #57

Merged
merged 1 commit into from
Mar 27, 2024
Merged

Conversation

aabadie
Copy link
Contributor

@aabadie aabadie commented Mar 27, 2024

No description provided.

@maribu maribu merged commit b138e10 into RIOT-OS:master Mar 27, 2024
1 check passed
@maribu
Copy link
Member

maribu commented Mar 27, 2024

Maybe we should revert the PR that added PyYAML via apt to avoid confusion which one is used?

@aabadie aabadie deleted the pyyaml branch March 27, 2024 15:57
@aabadie
Copy link
Contributor Author

aabadie commented Mar 27, 2024

Maybe we should revert the PR that added PyYAML via apt to avoid confusion which one is used?

#56 ? I think so. Would you mind doing it? I'll be AFK in a minute

maribu added a commit to maribu/murdock-scripts that referenced this pull request Mar 27, 2024
In RIOT-OS#57 PyYAML is installed in the correct way. No need to bloat the
container with the same python module installed twice.
aabadie pushed a commit that referenced this pull request Mar 27, 2024
In #57 PyYAML is installed in the correct way. No need to bloat the
container with the same python module installed twice.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants