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 missing SPEED_UNKNOWN definition #2156

Merged
merged 1 commit into from
Oct 20, 2022

Conversation

arossert
Copy link
Contributor

Signed-off-by: Amir Rossert amir.rossert@gmail.com

Summary

  • OS: Linux
  • Bug fix: yes
  • Type: core
  • Fixes: fix build on old compiler where SPEED_UNKNOWN is not defined

Description

Old compiler fails on

      psutil/_psutil_linux.c: In function 'psutil_net_if_duplex_speed':
      psutil/_psutil_linux.c:451: error: 'SPEED_UNKNOWN' undeclared (first use in this function)
      psutil/_psutil_linux.c:451: error: (Each undeclared identifier is reported only once
      psutil/_psutil_linux.c:451: error: for each function it appears in.)
      error: command '/usr/bin/gcc' failed with exit code 1

Signed-off-by: Amir Rossert <amir.rossert@gmail.com>

Signed-off-by: Amir Rossert <amir.rossert@safebreach.com>
@giampaolo giampaolo merged commit 8e58bce into giampaolo:master Oct 20, 2022
@arossert
Copy link
Contributor Author

@giampaolo Thanks for merging this one, any estimation on the next release?

ddelange added a commit to ddelange/psutil that referenced this pull request Oct 28, 2022
* 'master' of https://github.com/giampaolo/psutil:
  add windows test for free physical mem giampaolo#2074
  fix OSX tests broken by accident
  update HISTORY + give CREDITS for @arossert, @smoofra, @mayeut for giampaolo#2102, giampaolo#2156, giampaolo#2010
  build fix for Mac OS, Apple Silicon (giampaolo#2010)
  Linux: fix missing SPEED_UNKNOWN definition (giampaolo#2156)
  Use system-level values for Windows virtual memory (giampaolo#2077)
  feature: use ABI3 for cp36+ (giampaolo#2102)
  fix py2 compatibility
  improve API speed benchmark script giampaolo#2102
  fix: linter issues from giampaolo#2146 (giampaolo#2155)
  chore: skip test_cpu_freq on macOS arm64 (giampaolo#2146)
  pre-release + give CREDITS to @mayeut (PR giampaolo#2040) and @eallrich (new supporter)
  Fix a typo (giampaolo#2047)
  move isort and coverage config into pyproject.toml
  fix giampaolo#2021, fix giampaolo#1954, provide OSX arm64 bins + add pyproject.toml (giampaolo#2040)
  refactor git_log.py
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.

3 participants