-
Notifications
You must be signed in to change notification settings - Fork 2.5k
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
FreeBSD support #2508
FreeBSD support #2508
Changes from all commits
d603b18
57e365f
383fb2a
90b43c1
0ff09a2
eb8d908
14a4673
054eabc
23a7751
b5925e3
46ad70b
4a24859
401f621
File filter
Filter by extension
Conversations
Jump to
Diff view
Diff view
There are no files selected for viewing
Original file line number | Diff line number | Diff line change |
---|---|---|
|
@@ -207,6 +207,17 @@ If Doxygen and graphviz are available, the process can also build the documentat | |
|
||
You can also selectively enable/disable other features (e.g., specific plugins you don't care about, or whether or not you want to build the recordings post-processor). Use the --help option when configuring for more info. | ||
|
||
### Building on FreeBSD | ||
* *Note*: rtp_forward of streams only works streaming to IPv6, | ||
because of #2051 and thus the feature is not supported on FreeBSD at the moment. | ||
|
||
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. Small note: missing full stop at the end of the sentence? Also, extra empty line can be removed (I see two). There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. fixed |
||
When building on FreeBSD you can install the depencencies from ports or packages, here only pkg method is used. You also need to use `gmake` instead of `make`, | ||
since it is a GNU makefile. `./configure` can be run without arguments since the default prefix is `/usr/local` which is your default `LOCALBASE`. | ||
Note that the `configure.ac` is coded to use openssl in base. If you wish to use openssl from ports or any other ssl you must change `configure.ac` accordingly. | ||
|
||
pkg install libsrtp2 libusrsctp jansson libnice libmicrohttpd libwebsockets curl opus sofia-sip libogg jansson libnice libconfig \ | ||
libtool gmake autoconf autoconf-wrapper glib gengetopt | ||
|
||
|
||
### Building on MacOS | ||
While most of the above instructions will work when compiling Janus on MacOS as well, there are a few aspects to highlight when doing that. | ||
|
Original file line number | Diff line number | Diff line change |
---|---|---|
|
@@ -14,7 +14,7 @@ | |
#define JANUS_RTP_H | ||
|
||
#include <arpa/inet.h> | ||
#ifdef __MACH__ | ||
#if defined (__MACH__) || defined(__FreeBSD__) | ||
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. We have the same definition in pp-rtp.h, which is in the post-processor code. Have you checked if postprocessing MJR recordings does indeed work as expected on FreeBSD? There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. Sorry for the delay, I could not find the docs to acutally do a test, but I did change pp-rtp.h |
||
#include <machine/endian.h> | ||
#define __BYTE_ORDER BYTE_ORDER | ||
#define __BIG_ENDIAN BIG_ENDIAN | ||
|
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.
Please add some more descriptive description here too, as we do for MacOs. The
pkg
line should be "compacted" a bit too (like what we do above foryum
andapt-get
), as there's too many lines now I believe.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.
Done