Headcrab should build as a library (linux) about headcrab.js HOT 7 OPEN

On 11/6/2012 7:27 PM, James Morgensen wrote:

The C client should be buildable as a library, ideally a static library.

Great work!

Additionally, how should we deal with libwebsockets and jansson?
As-is, they compile to dynamic libraries which must be linked against
and installed.

I'd suggest including them as source.

I'm not sure if having them as static libs first and then linking
would be a good idea, but it'd be nice not to pollute the symbol namespace.

-Chris

from headcrab.js.

Great work!

I believe you misunderstood the original issue statement. I meant that while it SHOULD be able to do that, it does not currently do so, which is why it is filed as an issue.

I'd suggest including them as source.

My concern here is that we then are exposed to their (possibly) complicated build processes. Both already have in-place autotools-based build processes and I don't know how we'll fare with including them as source. But I will give it a shot.

from headcrab.js.

libjansson seems to not have any other library dependencies, which is nice.

Unsure about libwebsockets, it seems to use a fair number of other libraries, but it may have included all of them through source.

So it seems very possible that we can just include them through source and be done with it.

from headcrab.js.

Was poking around with cmake actually, looks promising. Since it creates
makefiles or vs projects.
On Nov 23, 2012 2:27 AM, "Trevor R" [email protected] wrote:

libjansson seems to not have any other library dependencies, which is nice.

Unsure about libwebsockets, it seems to use a fair number of other
libraries, but it may have included all of them through source.

So it seems very possible that we can just include them through source and
be done with it.

—
Reply to this email directly or view it on GitHubhttps://github.com//issues/1#issuecomment-10652774.

from headcrab.js.

I will shoot you in the face if you bring cmake into the picture.

Setup makefiles and vcprojects like a normal human being.

On 11/23/2012 4:16 AM, James Morgensen wrote:

Was poking around with cmake actually, looks promising. Since it creates
makefiles or vs projects.
On Nov 23, 2012 2:27 AM, "Trevor R" [email protected] wrote:

libjansson seems to not have any other library dependencies, which
is nice.

Unsure about libwebsockets, it seems to use a fair number of other
libraries, but it may have included all of them through source.

So it seems very possible that we can just include them through
source and
be done with it.

—
Reply to this email directly or view it on
GitHubhttps://github.com//issues/1#issuecomment-10652774.

—
Reply to this email directly or view it on GitHub
#1 (comment).

from headcrab.js.

With makefiles, I have to embed the commands to manually run the configure and make commands to build libwebsockets, then find and link against the library. No, I won't include their files manually as that breaks compatibility, bypassing their configuration, and as such is a pain to maintain.

@tarice it specifies its dependencies in its configure.ac like any other autotools package; this means that those libraries must be installed on the system. Those libraries are zlib and libdl (no idea why it wants to use libdl, as that is used for dynamic loading of dynamic libraries).

from headcrab.js.

By the way, found a better libwebsockets to use: https://github.com/davidgaleano/libwebsockets
This one is being actively updated.

from headcrab.js.