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

Develop update build system #1247

Merged

Conversation

hans-schmidt
Copy link
Contributor

The Ravencoin repository tests each new Pull Request submission by attempting to build binaries with the new PR. But for the past year, it has interpreted all PRs as bad because the code for the build system is broken. Specifically, it uses Ubuntu-18.04 which is no longer supported by Github (fails) and it uses Node-16 which is deprecated (generates errors).

This PR updates Ravencoin's build system to use Ubuntu-20.04 and Node-20. It also removes code related to AWS cache storage which Overstock used years ago, and it does some other code clean-up.

This PR is for the develop branch. Another PR has been submitted for the master branch. This PR also updates a URL for the Qt code fetch which the Qt project moved to archive (and which was already updated on the master branch in a previous PR).
No changes are made to the actual Ravencoin source code outside of the build system.

Similar build system changes have already been implemented and tested on the Evrmore blockchain codebase.

Copy link
Contributor

@HyperPeek HyperPeek left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Very clear and straight forward change, approved

@hans-schmidt hans-schmidt merged commit 66c22a6 into RavenProject:develop May 6, 2024
8 checks passed
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

Successfully merging this pull request may close these issues.

2 participants