TFS 1.X+ Errors when compiling TFS v1.4x (v1.4, v1.4.1, v1.4.2)

[Epsilons]

Dear OTLand community,

I'm reaching out for assistance regarding issues I'm encountering while attempting to compile TFS v1.4.2. I have the exact same errors with TFS v1.4 and v1.4.1. That being said, I compiled TFS v1.2 and TFS v1.5 with no problem whatsoever.

Here's the errors Visual Studio throw:

Error    C2079    '_' uses undefined struct 'fmt::v10::detail::type_is_unformattable_for<T,char>'
        with
        [
            T=AccessList_t
        ]    theforgottenserver    .\vcpkg\installed\x64-windows\include\fmt\core.h    1600    


Error    C2079    '_' uses undefined struct 'fmt::v10::detail::type_is_unformattable_for<T,char>'
        with
        [
            T=MarketAction_t
        ]    theforgottenserver    .\vcpkg\installed\x64-windows\include\fmt\core.h    1600    


Error    C2079    '_' uses undefined struct 'fmt::v10::detail::type_is_unformattable_for<T,char>'
        with
        [
            T=MarketOfferState_t
        ]    theforgottenserver    .\vcpkg\installed\x64-windows\include\fmt\core.h    1600    


Error    C2338    static_assert failed: 'Cannot format an argument. To make type T formattable provide a formatter<T>
specialization: https://fmt.dev/latest/api.html#udt' 
theforgottenserver    .\vcpkg\installed\x64-windows\include\fmt\core.h    1604

Looking forward to any assistance you can provide.

 

[Roddet]

Just apply the iomapserialize.cpp, iomarket.cpp and tools.h changes

Add extra & pedantic warnings and fix the code (#4491) · otland/forgottenserver@61452aa

* Remove discarded qualifiers in return types * Add more warnings and fix underlying enums

github.com

Alternatively, you can cast the enums to int static_cast<int> instead of tfs::to_underlying and ignore the changes on tools.h

 

[Epsilons]

I did that (applied the iomapserialize.cpp, iomarket.cpp and tools.h changes) and it's not working. Throw 295 errors now.

I believe my original issue lie within the fmt lib. I tried to override the more recent version installed with vcpkg back to v9.0.0 (which is the version that was available at the time TFS v1.4.2 was released) with no success.

 

[Roddet]

Yes, the issue is related to fmt library (since v10~)

You can see here that indeed the issue is related to the enums (implicit conversion), iomarket.cpp in this case

Error    C2079    '_' uses undefined struct 'fmt::v10::detail::type_is_unformattable_for<T,char>'
        with
        [
            T=MarketOfferState_t
        ]    theforgottenserver    .\vcpkg\installed\x64-windows\include\fmt\core.h    1600

You can also try the alternative solution.

 

[Epsilons]

Thanks, the alternative solution worked and now TFS v1.4.2 is compiling flawlessly.

I applied the change to iomapserialize.cpp and iomarket.cpp while making sure to cast all the enums to int static_cast<int> (instead of tfs::to_underlying).

I've attached the modified iomapserialize.cpp and iomarket.cpp in a zip archive here to make it simpler for dev who might stumble across this issue in the future.

 

[lordhen98]

Thanks, the alternative solution worked and now TFS v1.4.2 is compiling flawlessly.

I applied the change to iomapserialize.cpp and iomarket.cpp while making sure to cast all the enums to int static_cast<int> (instead of tfs::to_underlying).

I've attached the modified iomapserialize.cpp and iomarket.cpp in a zip archive here to make it simpler for dev who might stumble across this issue in the future.

Thank you so much!

 

[erickroseno]

Thanks, the alternative solution worked and now TFS v1.4.2 is compiling flawlessly.

I applied the change to iomapserialize.cpp and iomarket.cpp while making sure to cast all the enums to int static_cast<int> (instead of tfs::to_underlying).

I've attached the modified iomapserialize.cpp and iomarket.cpp in a zip archive here to make it simpler for dev who might stumble across this issue in the future.

Thank you bro!

 

[alejandro762]

Thanks, the alternative solution worked and now TFS v1.4.2 is compiling flawlessly.

I applied the change to iomapserialize.cpp and iomarket.cpp while making sure to cast all the enums to int static_cast<int> (instead of tfs::to_underlying).

I've attached the modified iomapserialize.cpp and iomarket.cpp in a zip archive here to make it simpler for dev who might stumble across this issue in the future.

Alternative, Check Date of Release of TFS, check on VCPKG > Releases,

TFS 1.4.2 Release The Forgotten Server 1.4.2 · otland/forgottenserver (https://github.com/otland/forgottenserver/releases/tag/v1.4.2) Jun 29 2022
VCPKG Release: Release 2022.06.15 Release · microsoft/vcpkg (https://github.com/microsoft/vcpkg/releases/tag/2022.06.15) Jun 16 2022
or VCPKG Release: Release 2022.05.10 · microsoft/vcpkg (https://github.com/microsoft/vcpkg/releases/tag/2022.05.10) May 11 2022

Compiled with no errors.

For any server, just check release date and take 1-2 month before the release and voila.