-
Notifications
You must be signed in to change notification settings - Fork 45
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 pkgconfig support in the setup.py #120
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I see some stylistic issues that should be fixed by running make format
, and a few other small issues as well:
Ah, the Coverage workflow failed because pystack/.github/workflows/coverage.yml Line 50 in 8473602
pkgconfig .
|
Added support for pkgconfig Signed-off-by: Apurva Khatri <apurvakhatri2011@gmail.com>
Installing pkgconfig Signed-off-by: Apurva Khatri <apurvakhatri2011@gmail.com>
Instructions for pkg-config added Signed-off-by: Apurva Khatri <apurvakhatri2011@gmail.com>
3a5806d
to
0e33282
Compare
I have rebased and manually applied @godlygeek suggestions to move this forward. Thanks a lot for the contribution @apurvakhatri 🚀 |
Signed-off-by: Pablo Galindo <pablogsal@gmail.com>
Issue number of the reported bug or feature request: #54
Describe your changes
I have added support for pkgconfig in setup.py. Python's pkgconfig depends on system package pkg-config. The instructions on how to install pkg-config has also been added to README.md. The changes uses pkgconfig to look for clfags & ldflags to use for libelf and libdw. If pkgconfig is unable to determine the flags then the default pre-configured behavior is used.
Testing performed
Testing has been performed on Ubuntu 22.04.2 LTS
Additional context
Closes: #54