Build issues, Clang, C++11 or C++17, FreeBSD about nymphcast HOT 3 CLOSED

Hello, and thank you for the comments :)

The issues you mentioned should be addressed in the current master. I changed the use of pthread_t, and the entire codebase compiles cleanly with MSVC 2017.

from nymphcast.

Remaining patches to build on FreeBSD are in that PR.

There's two things that I still change in packaging (e.g. relevant for #61 and not-really here):

• moving version to version.mk (and corresponding change in the Makefile). This is very much an implementation detail, but Clang has a <version> header (so does gcc, for that matter) and because of -I . it ends up being picked up by system headers. Compilation errors look like:

  In file included from /usr/include/c++/v1/exception:85:
  ./version:1:3: error: invalid preprocessing directive
  # Versioning file for the NymphCast server.
  

  It's easiest just to rename the file and get it out of the way.
• Fixing up the Makefile:
  • replacing make by $(MAKE) to ensure the same flavor of make is run each time. (This could probably be upstreamed, though)
  • using c++ instead of g++, cc instead of gcc.
  • dropping unneeded libraries like -latomic and -lstdc++fs, which don't exist and are not needed on the FreeBSD / Clang side of things.

from nymphcast.

One last one, (which would apply to the upstream ES-Core that is vendored into NymphCast, too):

--- gui/core/utils/TimeUtil.h.orig      2022-02-16 14:27:26 UTC
+++ gui/core/utils/TimeUtil.h
@@ -2,6 +2,7 @@
 #ifndef ES_CORE_UTILS_TIME_UTIL_H
 #define ES_CORE_UTILS_TIME_UTIL_H
 
+#include <ctime>
 #include <string>
 
 namespace Utils

This is one of those implicit-includes-that-annoys-people things. In FreeBSD recent releases do not need this anymore, as there was an effort to make the system headers less pedantically-annoying. But older, still supported versions, do. Not PR'ing this because it's not particularly necessary upstream, it's in packaging downstream anyway, and the problem is going away with the EoL of FreeBSD 12.

(Closing the issue, too: NymphCast builds on FreeBSDs 12, 13 and 14 and is available from packaging).

from nymphcast.