Adding FreeBSD build support. by LadySerenaKitty · Pull Request #460 · DiligentGraphics/DiligentCore · GitHub

Making all changes required to build on FreeBSD. Some CMake entries for Linux may have changed, but their functionality remains unchanged.

Is __fastcall a recognized keyword when compiling on FreeBSD?

On this one, the compiler was complaining about fastcall being redefined, as FreeBSD's headers already define fastcall.

Which compiler are you using?

Believe it or not, the entirety of DiligentEngine was built using the compiler provided by the base system: clang 16.0.6.

It is actually strange why it only complains about __fastcall. It also recognized all other attributes
https://clang.llvm.org/docs/AttributeReference.html#target-version

Can you post the error that the compiler produces?

This is what happens without skipping __fastcall:

[ 35%] Building CXX object DiligentCore/Graphics/ShaderTools/CMakeFiles/Diligent-ShaderTools.dir/src/DXCompiler.cpp.o
In file included from /usr/home/jlhawkwell/Projects/DiligentEngine/DiligentCore/Graphics/ShaderTools/src/DXCompiler.cpp:42:
In file included from /usr/home/jlhawkwell/Projects/DiligentEngine/DiligentCore/Graphics/ShaderTools/include/DXCompilerBaseLinux.hpp:31:
In file included from /usr/home/jlhawkwell/Projects/DiligentEngine/DiligentCore/Graphics/ShaderTools/../../ThirdParty/DirectXShaderCompiler/dxc/dxcapi.h:37:
/usr/home/jlhawkwell/Projects/DiligentEngine/DiligentCore/Graphics/ShaderTools/../../ThirdParty/DirectXShaderCompiler/dxc/WinAdapter.h:345:9: error: '__fastcall' macro redefined [-Werror,-Wmacro-redefined]
#define __fastcall
        ^
/usr/include/sys/cdefs.h:365:9: note: previous definition is here
#define __fastcall      __attribute__((__fastcall__))
        ^
1 error generated.
gmake[2]: *** [DiligentCore/Graphics/ShaderTools/CMakeFiles/Diligent-ShaderTools.dir/build.make:122: DiligentCore/Graphics/ShaderTools/CMakeFiles/Diligent-ShaderTools.dir/src/DXCompiler.cpp.o] Error 1
gmake[1]: *** [CMakeFiles/Makefile2:3521: DiligentCore/Graphics/ShaderTools/CMakeFiles/Diligent-ShaderTools.dir/all] Error 2
gmake: *** [Makefile:156: all] Error 2

I updated DXC headers, so this should work now - please try the latest version

A better way to fix this is to check

#ifndef __fastcall

Though I do really not understand why it only complains about __fastcall. All other macros are defined in the same way.

The __fastcall macro is already defined in /usr/include/sys/cdefs.h on line 365.

What I suggested is to use the #ifndef __fastcall guard instead of #ifndef PLATFORM_FREEBSD.
Also note that this file is a third-party header so with the next update, it may break again.

Which compiler are you using?

I updated core module, so that format validation will be disabled on platforms that don't have the script.
Can you update to the latest version and try it?

I updated core module, so that format validation will be disabled on platforms that don't have the script. Can you update to the latest version and try it?

Format validation is successfully skipped (tested after deleting the scripts I added). Would you like a new push with the scripts removed?

Would you like a new push with the scripts removed?

You can force-push to the same branch - the PR will be updated.

I updated DXC headers, so this should work now - please try the latest version

What errors do you get without this include?
This change does not seem right

It is quite correct. Without this, anything that isn't part of base will not be found.

What base?

FreeBSD! "base" is short for "base operating system" and is the stuff that's made by the FreeBSD Project - kernel and world. This includes everything outside of /usr/home and /usr/local, which isn't much.

A mirror of the source repo is available here, so you can see what all is part of FreeBSD's base: https://github.com/freebsd/freebsd-src

I tried to find some info about this problem, and I think that the right way to fix this is to set CMAKE_REQUIRED_INCLUDES at your project's CMake level:

dotnet/runtime#4073

Did you try setting the CMAKE_REQUIRED_INCLUDES in your project CMake?

Sorry for a long delay with response.

A better way to fix this is to check

#ifndef __fastcall

Though I do really not understand why it only complains about __fastcall. All other macros are defined in the same way.

What I suggested is to use the #ifndef __fastcall guard instead of #ifndef PLATFORM_FREEBSD.
Also note that this file is a third-party header so with the next update, it may break again.

Did you try setting the CMAKE_REQUIRED_INCLUDES in your project CMake?

LadySerenaKitty requested a review from TheMostDiligent as a code owner January 31, 2024 07:32

TheMostDiligent reviewed Jan 31, 2024

View reviewed changes

TheMostDiligent reviewed Feb 1, 2024

View reviewed changes

TheMostDiligent requested changes Feb 1, 2024

View reviewed changes

TheMostDiligent reviewed Feb 1, 2024

View reviewed changes

Comment thread BuildTools/CMake/BuildUtils.cmake

LadySerenaKitty force-pushed the master branch 2 times, most recently from d41441e to 73ac8e7 Compare February 2, 2024 17:48

LadySerenaKitty commented Feb 2, 2024

View reviewed changes

Comment thread BuildTools/CMakeLists.txt

TheMostDiligent requested changes Feb 5, 2024

View reviewed changes

LadySerenaKitty force-pushed the master branch from f526296 to 12cdfe3 Compare February 6, 2024 10:05

TheMostDiligent reviewed Feb 19, 2024

View reviewed changes

TheMostDiligent requested changes Feb 19, 2024

View reviewed changes

These things here work on FreeBSD

f72ae4c

TheMostDiligent force-pushed the master branch from 12cdfe3 to f72ae4c Compare February 19, 2024 06:19

TheMostDiligent requested changes Feb 24, 2024

View reviewed changes

               #define __vectorcall
               #define __thiscall
+              #ifndef PLATFORM_FREEBSD
               #define __fastcall

               endif()
+              if(PLATFORM_FREEBSD AND EXISTS /usr/local/include)
+              	target_include_directories(Diligent-BuildSettings SYSTEM AFTER INTERFACE /usr/local/include)

Adding FreeBSD build support. by LadySerenaKitty · Pull Request #460 · DiligentGraphics/DiligentCore · GitHub

Uh oh!

Adding FreeBSD build support. - #460

Adding FreeBSD build support.#460LadySerenaKitty wants to merge 1 commit intoDiligentGraphics:masterDiligentGraphics/DiligentCore:masterfrom LadySerenaKitty:masterLadySerenaKitty/DiligentCore:masterCopy head branch name to clipboard

Conversation

LadySerenaKitty commented Jan 31, 2024

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Choose a reason for hiding this comment

Uh oh!

TheMostDiligent left a comment

Choose a reason for hiding this comment

Uh oh!

LadySerenaKitty commented Feb 1, 2024

Uh oh!

TheMostDiligent commented Feb 1, 2024

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

TheMostDiligent left a comment

Choose a reason for hiding this comment

Uh oh!

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Reviewers

Assignees

Labels

Projects

Milestone

Adding FreeBSD build support.#460
LadySerenaKitty wants to merge 1 commit into
DiligentGraphics:masterfrom
LadySerenaKitty:master