Skip to content
GitLab
Projects Groups Snippets
  • /
  • Help
    • Help
    • Support
    • Community forum
    • Submit feedback
    • Contribute to GitLab
  • Sign in / Register
  • P PyAV
  • Project information
    • Project information
    • Activity
    • Labels
    • Members
  • Repository
    • Repository
    • Files
    • Commits
    • Branches
    • Tags
    • Contributors
    • Graph
    • Compare
  • Issues 37
    • Issues 37
    • List
    • Boards
    • Service Desk
    • Milestones
  • Merge requests 26
    • Merge requests 26
  • CI/CD
    • CI/CD
    • Pipelines
    • Jobs
    • Schedules
  • Deployments
    • Deployments
    • Environments
    • Releases
  • Packages and registries
    • Packages and registries
    • Package Registry
    • Infrastructure Registry
  • Monitor
    • Monitor
    • Incidents
  • Analytics
    • Analytics
    • Value stream
    • CI/CD
    • Repository
  • Wiki
    • Wiki
  • Snippets
    • Snippets
  • Activity
  • Graph
  • Create a new issue
  • Jobs
  • Commits
  • Issue Boards
Collapse sidebar
  • PyAV
  • PyAV
  • Merge requests
  • !528

Detect static library and pthread awareness in setup.py

  • Review changes

  • Download
  • Email patches
  • Plain diff
Closed Administrator requested to merge github/fork/stepheny/detect_static into develop May 20, 2019
  • Overview 0
  • Commits 3
  • Pipelines 0
  • Changes 1

Created by: stepheny

It is great to stop trying to build against static ffmpeg library early. But only showing "unknown flags" -pthread is very confusing. This patch trys to check 'library_dirs' returned by parse_cflags. If system ffmpeg was used, this should be empty. If a custom built ffmpeg was used, probably in /usr/local/lib, this would be filled by pkg-config. And then check if target .so file exists there. If it does not exist, the library is probably static. In case this test could go wrong, an env var was added to bypass the test. I think this should work on linux. I've no idea how this would affect windows or macos users. I've no idea whether the env var name and error message are proper.

Assignee
Assign to
Reviewers
Request review from
Time tracking
Source branch: github/fork/stepheny/detect_static