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

monero: Use latest builder #12009

Draft
wants to merge 2 commits into
base: master
Choose a base branch
from
Draft

monero: Use latest builder #12009

wants to merge 2 commits into from

Conversation

maflcko
Copy link
Contributor

@maflcko maflcko commented May 28, 2024

No description provided.

@maflcko maflcko marked this pull request as draft May 28, 2024 16:36
Copy link

maflcko has previously contributed to projects/monero. The previous PR was #11714

@maflcko
Copy link
Contributor Author

maflcko commented May 28, 2024

cc @binaryFate Can you please take a look here? This requires bumping the boost version, I presume.

@maflcko
Copy link
Contributor Author

maflcko commented Jun 17, 2024

cc @luigi1111 @moneromooo-monero or maybe one of you can take a look at the boost upgrade?

@selsta
Copy link

selsta commented Jun 18, 2024

@maflcko I'll look into it

@selsta
Copy link

selsta commented Jun 25, 2024

Just updating boost doesn't work unfortunately, I might have to try different versions or ask someone else who is more experienced.

@maflcko
Copy link
Contributor Author

maflcko commented Jun 25, 2024

Yes, the fix would be to bump boost. Currently boost is downloaded from && curl -s -L -o boost_${BOOST_VERSION}.tar.bz2 https://downloads.getmonero.org/libs/boost_${BOOST_VERSION}.tar.bz2 \. So to fix it, one would have to upload a different boost version there, or change the download source. (Among possibly other changes)

@selsta
Copy link

selsta commented Jun 25, 2024

I did update it to 1.80 and it fails

#8 30.41 /usr/local/bin/../include/c++/v1/__config:13:10: fatal error: '__config_site' file not found
#8 30.41    13 | #include <__config_site>
#8 30.41       |          ^~~~~~~~~~~~~~~
#8 30.41 1 error generated.

https://github.com/selsta/testrepo/actions/runs/9566788394/job/26372907450#step:3:843

I can also try newer or older versions.

@maflcko
Copy link
Contributor Author

maflcko commented Jun 25, 2024

Yeah, you could try 1.79 for now. It may be needed to set -DBOOST_NO_CXX98_FUNCTION_BASE in the pre-processor.

For example via ... && ./b2 --build-type=minimal link=static runtime-link=static --with-chrono --with-date_time --with-filesystem --with-program_options --with-regex --with-serialization --with-system --with-thread --with-locale threading=multi threadapi=pthread cflags="$CFLAGS -DBOOST_NO_CXX98_FUNCTION_BASE" cxxflags="$CXXFLAGS -DBOOST_NO_CXX98_FUNCTION_BASE" stage

@maflcko
Copy link
Contributor Author

maflcko commented Jun 25, 2024

Not sure why the last push wasn't fixing the build. Somehow -std=c++11 wasn't propagated properly?

@maflcko
Copy link
Contributor Author

maflcko commented Jun 28, 2024

Looks like boost 1.84 (or 1.81) is enough to avoid the deprecated c++11 code and the need to specify -DBOOST_NO_CXX98_FUNCTION_BASE.

To work around the missing __config_site. You can manually copy it for now as a temporary workaround. Just add this before downloading boost:

RUN cp /usr/local/include/x86_64-unknown-linux-gnu/c++/v1/__config_site /usr/local/include/c++/v1/

Also ./b2 ... stage would have to be replaced by ./b2 ... install.

If it works for you as well, and you agree with the changes, you can go ahead and create a new pull request against the master branch here.

@selsta
Copy link

selsta commented Jun 28, 2024

It did build the docker image correctly, it failed to run it but that appears to be unrelated. Will open a PR with the changes.

@selsta
Copy link

selsta commented Jun 28, 2024

#12138

@selsta
Copy link

selsta commented Jun 28, 2024

I don't have a Google account unfortunately to sign the CLA. Can you add the commit to this PR, or should I ask someone else from the monero project to open the PR?

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

Successfully merging this pull request may close these issues.

None yet

2 participants