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

Switch to use pkg-config to detect libevent and openssl #27

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

tpetazzoni
Copy link

Switching to pkg-config fixes a number of problems when detecting the
libraries. For example the detection of libpthread was failing,
because libevent_threads was added to LIBS before libevent itself,
causing the libpthread test to fail due to missing symbols. pkg-config
is anyway nowadays the preferred way for detecting libraries. It also
has the benefit of working properly in static library situations.

Signed-off-by: Thomas Petazzoni [email protected]

Switching to pkg-config fixes a number of problems when detecting the
libraries. For example the detection of libpthread was failing,
because libevent_threads was added to LIBS before libevent itself,
causing the libpthread test to fail due to missing symbols. pkg-config
is anyway nowadays the preferred way for detecting libraries. It also
has the benefit of working properly in static library situations.

Signed-off-by: Thomas Petazzoni <[email protected]>
@JoakimSoderberg
Copy link

Hello, I googled for an email address / homepage to contact payden about my pull request #28. However it seems like he has died recently http://www.meaningfulfunerals.net/fh/obituaries/obituary.cfm?o_id=2723507&fh_id=10680 :(

At least the name and place of residence is the same, and it mentions him being a software engineer.

Just thought I'd give you a heads up.

@tpetazzoni
Copy link
Author

@JoakimSoderberg Thanks for the heads up. Apparently, there is a fork at https://github.com/JonnyWhatshisface/libwebsock, whose description explicitly indicate that the fork was done following the death of Payden.

@JoakimSoderberg
Copy link

Yes I noticed this as well after writing to you from reading #26. As I mentioned there, it would have been better to simply fork this repo instead of restarting an entirely new repo. Especially since there's loads more people favoriting and having forked this repo. It just creates more hazzle, instead of simply changing the pull request target.

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.

2 participants