Bromvlieg / gm_bromsock

Networking module for Garrysmod
MIT License
62 stars 11 forks source link

Build Release Request #43

Open The-Lord-of-Owls opened 3 months ago

The-Lord-of-Owls commented 3 months ago

I'd like to get a build release of the linux64 for the V2 branch if possible. I'm having trouble getting it to properly build with Travis CI and with github actions

Bromvlieg commented 3 months ago

Im afraid I cannot provide you a build, but there's a makefile inside of gm_bromsock/Source/gm_bromsock, so if you cd into that directory and run make it should compile. But haven't tried so in a lot of years, so might have issues 😄

The-Lord-of-Owls commented 3 months ago

I think I might have to try reworking the files to use premake instead, I can't seem to get it working with cmake

EDIT: For referance this is where I'm stuck at

C:\Users\administrator\Downloads\gm_bromsock-2\gm_bromsock-2>cmake . -- Selecting Windows SDK version 10.0.19041.0 to target Windows 10.0.19045. -- Conan: Using cmake-multi generator -- Conan: Automatic detection of conan settings from cmake -- Conan: Detected VS runtime: MD -- Conan: Settings= -s;arch=x86_64;-s;build_type=Release;-s;compiler=Visual Studio;-s;compiler.version=17;-s;compiler.runtime=MD;-s;os=Windows -- Conan: checking conan executable -- Conan: Found program C:/Program Files/Conan/conan/conan.exe -- Conan: Version found 2.5.0 -- Conan executing: C:/Program Files/Conan/conan/conan.exe install C:/Users/administrator/Downloads/gm_bromsock-2/gm_bromsock-2/conanfile.txt -s arch=x86_64 -s build_type=Release -s compiler=Visual Studio -s compiler.version=17 -s compiler.runtime=MD -s os=Windows -g=cmake_multi --build=missing ERROR: Invalid setting 'Visual Studio' is not a valid 'settings.compiler' value. Possible values are ['sun-cc', 'gcc', 'msvc', 'clang', 'apple-clang', 'intel-cc', 'qcc', 'mcst-lcc'] Read "http://docs.conan.io/2/knowledge/faq.html#error-invalid-setting" CMake Error at conan.cmake:566 (message): Conan install failed='1' Call Stack (most recent call first): conan.cmake:912 (old_conan_cmake_install) CMakeLists.txt:31 (conan_cmake_run)

-- Configuring incomplete, errors occurred!

Bromvlieg commented 3 months ago

I'd strongly advise against using Conan as a package manager. Seems that's misconfiguring it. Alternatively you need to change the profile for Conan. It's been a pain to use it with github actions for me atleast. I don't think there's a component in bromsock that requires external packages other than build-in ones that come with CMake (such as openssl)

There's also some unused packages in the conan.txt, seems I never finished that setup in the V2 branch.