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

build fails on fedora 37 #1204

Closed
grubles opened this issue Jan 30, 2023 · 4 comments
Closed

build fails on fedora 37 #1204

grubles opened this issue Jan 30, 2023 · 4 comments

Comments

@grubles
Copy link

grubles commented Jan 30, 2023

at commit 0a20d6e
arch: ppc64le

  CXX      wallet/test/test_test_bitcoin-init_test_fixture.o
wallet/test/init_test_fixture.cpp: In constructor ‘InitWalletDirTestingSetup::InitWalletDirTestingSetup(const std::string&)’:
wallet/test/init_test_fixture.cpp:35:52: error: ‘std::map<std::__cxx11::basic_string<char>, boost::filesystem::path>::mapped_type’ {aka ‘class boost::filesystem::path’} has no member named ‘BOOST_FILESYSTEM_C_STR’
   35 |     std::ofstream f(m_walletdir_path_cases["file"].BOOST_FILESYSTEM_C_STR);
      |                                                    ^~~~~~~~~~~~~~~~~~~~~~
make[2]: *** [Makefile:17049: wallet/test/test_test_bitcoin-init_test_fixture.o] Error 1
make[2]: Leaving directory '/home/user/builds/elements/src'
make[1]: *** [Makefile:18581: all-recursive] Error 1
make[1]: Leaving directory '/home/user/builds/elements/src'
make: *** [Makefile:833: all-recursive] Error 1

@delta1
Copy link
Member

delta1 commented Jan 31, 2023

Do you have the same issue when building bitcoin @grubles ?

@psgreco
Copy link
Contributor

psgreco commented Jan 31, 2023

@grubles look at this MR #1142, it should include the commits for master to build. The reason it's not merged yet is because those commits will come from the "normal" merge process from Core. You can either use 22.1 (which includes almost everything in master) or cherry-pick those commits. Hopefully we'll merge the 24 codebase soon so we won't have this problem anymore

@delta1
Copy link
Member

delta1 commented Jan 31, 2023

I did manage to build this commit for that arch with the guix build, if you just need the binary

https://github.com/delta1/elements/releases/tag/v21.99-0a20d6e3275d

@grubles
Copy link
Author

grubles commented Jan 31, 2023

Just tried building bitcoind from master and it worked. elements-22.1 tag also worked. Appreciate the explanation @psgreco @delta1.

@grubles grubles closed this as completed Jan 31, 2023
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

3 participants