Skip to content

Fix missing SPEED_UNKNOWN definition#2156

Merged
giampaolo merged 1 commit intogiampaolo:masterfrom
arossert:fix_missing_speed_unknown
Oct 20, 2022
Merged

Fix missing SPEED_UNKNOWN definition#2156
giampaolo merged 1 commit intogiampaolo:masterfrom
arossert:fix_missing_speed_unknown

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>
@arossert arossert force-pushed the fix_missing_speed_unknown branch from 3fe5603 to 07be175 Compare October 19, 2022 16:25
@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