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

[Depends] Bump Rust toolchain to 1.69.0 #2872

Merged
merged 10 commits into from
Jul 7, 2023

Conversation

Fuzzbawls
Copy link
Collaborator

Alternative to #2871.

This bumps our Rust Toolchain to v1.69.0, but comes at the cost of ALSO bumping our minimum supported linux glibc to 2.27, which officially drops support for Ubuntu 16.04, Debian 8 (Jessie), Debian 9 (Stretch), and CentOS 7 for binary releases.

Ubuntu 16.04 and CentOS 7 technically don't reach EOL (End Of Life) until 2024, but both are largely unsupported in linux communities already.

The major benefit to this outside of being able to use a more recent Rust toolchain version, is that it allows us to use the official Rust distribution toolchains and no longer maintain our own modified versions, a significant time saver if this ability is carried forward in the future.

fanquake and others added 10 commits June 9, 2023 20:17
The first argument in bin_PROGRAMS (bitcoind) was being silently
dropped and never passed into the check-security.py or check-symbols.py scripts.

This has been the case since the scripts were added to the makefile in
bitcoin@f3d3eaf.

Example of the behavior:

```python
# touch a, touch b, touch c
# python3 args.py < a b c

import sys
if __name__ == '__main__':
    print(sys.argv)
    # ['args.py', 'b', 'c']

    # if you add some lines to "a",
    # you'll see them here..
    for line in sys.stdin:
        print(line)
```
Only for windows and macOS builds currently
@Fuzzbawls Fuzzbawls added High Priority Build System Scripts and Tools Needs Release Notes Placeholder tag for anything needing mention in the "Notable Changes" section of release notes labels Jul 4, 2023
@Fuzzbawls Fuzzbawls added this to the 6.0.0 milestone Jul 4, 2023
@Fuzzbawls Fuzzbawls self-assigned this Jul 4, 2023
Copy link
Member

@Duddino Duddino left a comment

Choose a reason for hiding this comment

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

tACK 1bc92be, works fine with #2870
As for the older linux distributions, I am personally in favor of dropping support, since most people and vps providers tend to use the newer LTS anyways.

Copy link

@panleone panleone left a comment

Choose a reason for hiding this comment

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

tACK 1bc92be
Compiles fine on my pc

@Fuzzbawls Fuzzbawls merged commit 8c4ffa6 into PIVX-Project:master Jul 7, 2023
21 checks passed
@Fuzzbawls Fuzzbawls modified the milestones: 6.0.0, 5.6.0 Feb 6, 2024
@Fuzzbawls Fuzzbawls removed the Needs Release Notes Placeholder tag for anything needing mention in the "Notable Changes" section of release notes label Feb 15, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
Status: No status
Development

Successfully merging this pull request may close these issues.

4 participants