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

Add gcc build dependency #280

Merged
merged 2 commits into from
Sep 19, 2022
Merged

Add gcc build dependency #280

merged 2 commits into from
Sep 19, 2022

Conversation

tjorim
Copy link
Contributor

@tjorim tjorim commented Sep 5, 2022

Proposed Changes

Add gcc build dependency, which seems needed to build the psutil requirement

Related Issues

Needed for #279

needed to build the psutil requirement
@tjorim tjorim added the dependencies Upgrade or downgrade of project dependencies. label Sep 5, 2022
glances/Dockerfile Outdated Show resolved Hide resolved
Co-authored-by: Franck Nijhof <frenck@frenck.nl>
Copy link
Member

@frenck frenck left a comment

Choose a reason for hiding this comment

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

Thanks, @tjorim 👍

@frenck frenck merged commit 647638a into hassio-addons:main Sep 19, 2022
@tjorim tjorim deleted the patch-1 branch September 19, 2022 07:49
@tjorim
Copy link
Contributor Author

tjorim commented Sep 19, 2022

Ugh, why don't they have the prebuilt we need anymore. Looks like it also wants python3-dev now.

@frenck
Copy link
Member

frenck commented Sep 19, 2022

Ugh, why don't they have the prebuilt we need anymore. Looks like it also wants python3-dev now.

Our base images also pick up Home Assistant wheels (https://wheels.home-assistant.io), that is the reason it now needs a bit more (as HA hasn't bumped yet).

@tjorim
Copy link
Contributor Author

tjorim commented Sep 19, 2022

Oh I see, I expected the build to check home assistant wheels in addition to pypi. So if a PR is merged in HA core to bump psutil (used for systemmonitor), the wheels for that will also be build?

@frenck
Copy link
Member

frenck commented Sep 19, 2022

So if a PR is merged in HA core to bump psutil (used for systemmonitor), the wheels for that will also be build?

Yes!

@github-actions github-actions bot locked and limited conversation to collaborators Sep 20, 2022
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
dependencies Upgrade or downgrade of project dependencies.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants