Skip to content
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

1.8.5 build failure on GCC 11 #1899

Closed
xvitaly opened this issue Apr 2, 2021 · 8 comments
Closed

1.8.5 build failure on GCC 11 #1899

xvitaly opened this issue Apr 2, 2021 · 8 comments

Comments

@xvitaly
Copy link
Contributor

xvitaly commented Apr 2, 2021

Failed to build version 1.8.5 on Fedora 35.
GCC version: 11.0.1 (snapshot).

[21/30] /usr/bin/g++ -DFMT_LOCALE -DFMT_SHARED -DSPDLOG_COMPILED_LIB -DSPDLOG_FMT_EXTERNAL -DSPDLOG_SHARED_LIB -I../include -O2 -flto=auto -ffat-lto-objects -fexceptions -g -grecord-gcc-switches -pipe -Wall -Werror=format-security -Wp,-D_FORTIFY_SOURCE=2 -Wp,-D_GLIBCXX_ASSERTIONS -specs=/usr/lib/rpm/redhat/redhat-hardened-cc1 -fstack-protector-strong -specs=/usr/lib/rpm/redhat/redhat-annobin-cc1  -m64  -mtune=generic -fasynchronous-unwind-tables -fstack-clash-protection -fcf-protection -DNDEBUG -std=c++11 -MD -MT tests/CMakeFiles/spdlog-utests.dir/main.cpp.o -MF tests/CMakeFiles/spdlog-utests.dir/main.cpp.o.d -o tests/CMakeFiles/spdlog-utests.dir/main.cpp.o -c ../tests/main.cpp
FAILED: tests/CMakeFiles/spdlog-utests.dir/main.cpp.o 
/usr/bin/g++ -DFMT_LOCALE -DFMT_SHARED -DSPDLOG_COMPILED_LIB -DSPDLOG_FMT_EXTERNAL -DSPDLOG_SHARED_LIB -I../include -O2 -flto=auto -ffat-lto-objects -fexceptions -g -grecord-gcc-switches -pipe -Wall -Werror=format-security -Wp,-D_FORTIFY_SOURCE=2 -Wp,-D_GLIBCXX_ASSERTIONS -specs=/usr/lib/rpm/redhat/redhat-hardened-cc1 -fstack-protector-strong -specs=/usr/lib/rpm/redhat/redhat-annobin-cc1  -m64  -mtune=generic -fasynchronous-unwind-tables -fstack-clash-protection -fcf-protection -DNDEBUG -std=c++11 -MD -MT tests/CMakeFiles/spdlog-utests.dir/main.cpp.o -MF tests/CMakeFiles/spdlog-utests.dir/main.cpp.o.d -o tests/CMakeFiles/spdlog-utests.dir/main.cpp.o -c ../tests/main.cpp
In file included from /usr/include/signal.h:315,
                 from ../tests/catch.hpp:6323,
                 from ../tests/main.cpp:2:
../tests/catch.hpp:9041:58: error: call to non-'constexpr' function 'long int sysconf(int)'
 9041 |     constexpr static std::size_t sigStackSize = 32768 >= MINSIGSTKSZ ? 32768 : MINSIGSTKSZ;
      |                                                          ^~~~~~~~~~~
In file included from /usr/include/bits/sigstksz.h:24,
                 from /usr/include/signal.h:315,
                 from ../tests/catch.hpp:6323,
                 from ../tests/main.cpp:2:
/usr/include/unistd.h:630:17: note: 'long int sysconf(int)' declared here
  630 | extern long int sysconf (int __name) __THROW;
      |                 ^~~~~~~
In file included from ../tests/main.cpp:2:
../tests/catch.hpp:9100:45: error: size of array 'altStackMem' is not an integral constant-expression
 9100 |     char FatalConditionHandler::altStackMem[sigStackSize] = {};
      |                                             ^~~~~~~~~~~~

Full build log: https://kojipkgs.fedoraproject.org//work/tasks/568/65020568/build.log

@xvitaly xvitaly changed the title 1.8.5 build failure 1.8.5 build failure on GCC 11 Apr 2, 2021
@gabime
Copy link
Owner

gabime commented Apr 2, 2021

It is build error of catch2 lib, not spdlog’s.

@xvitaly
Copy link
Contributor Author

xvitaly commented Apr 2, 2021

It is build error of catch2 lib, not spdlog’s.

Yes, bundled into spdlog catch. I think it need to be updated.

@gabime
Copy link
Owner

gabime commented Apr 2, 2021

Could you check if newer version of catch fix this?

@xvitaly
Copy link
Contributor Author

xvitaly commented Apr 2, 2021

Could you check if newer version of catch fix this?

catchorg/Catch2#2178

@xvitaly
Copy link
Contributor Author

xvitaly commented Apr 2, 2021

I will try to use packaged version of Catch.

@xvitaly
Copy link
Contributor Author

xvitaly commented Apr 2, 2021

Packaged version works fine.

@xvitaly xvitaly closed this as completed Apr 2, 2021
@gabime
Copy link
Owner

gabime commented Apr 2, 2021

Packaged version works fine.

You mean spdlog’s bundled version? or latest version(2.3.14) from catch repo?

@xvitaly
Copy link
Contributor Author

xvitaly commented Apr 2, 2021

You mean spdlog’s bundled version? or latest version(2.3.14) from catch repo?

Patched version 2.13.4 from Fedora repositories.

I think you can update bundled version with included patch.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants