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

Python plugin shared libraries missing #676

Merged
merged 1 commit into from Feb 7, 2016

Conversation

halindrome
Copy link
Contributor

When linking against a python with shared libraries, this script ONLY asked for the LD flags, not any additional libraries. This could result in a condition where required libraries (such as libutil on Centos 6.7 against Python 2.7) are not loaded and the load of the plugin thus fails. This change asks the python being linked against which libraries it was linked using, and then ensures those are also linked against by the python plugin for weechat.

When linking against a python with shared libraries, this script ONLY asked for the LD flags, not any additional libraries.  This could result in a condition where required libraries (such as libutil on Centos 6.7 against Python 2.7) are not loaded and the load of the plugin thus fails.  This change asks the python being linked against which libraries it was linked using, and then ensures those are also linked against by the python plugin for weechat.
@flashcode flashcode added the bug Unexpected problem or unintended behavior label Feb 4, 2016
@flashcode flashcode self-assigned this Feb 7, 2016
@flashcode flashcode added this to the 1.5 milestone Feb 7, 2016
@flashcode flashcode merged commit c00ae68 into weechat:master Feb 7, 2016
flashcode added a commit that referenced this pull request Feb 7, 2016
arza-zara pushed a commit to arza-zara/weechat that referenced this pull request Oct 4, 2016
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Unexpected problem or unintended behavior
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants