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.0] Update chainbase to tip (initialization / move operations update) #561

Merged
merged 1 commit into from
Aug 15, 2024

Conversation

greg7mdp
Copy link
Contributor

Update chainbase to tip, which includes some fixes for missing member initialization, as well as move operator fixes.
These were found when investigating boost 1.85 ci/cd failures with asserton builds.

@greg7mdp greg7mdp requested review from heifner and linh2931 August 15, 2024 19:14
@heifner heifner changed the title Update chainbase to tip (initialization / move operations update) [1.0] Update chainbase to tip (initialization / move operations update) Aug 15, 2024
@greg7mdp greg7mdp requested a review from spoonincode August 15, 2024 19:47
@spoonincode
Copy link
Member

This really needs to go to release branch? Does it fix a defect?

@greg7mdp
Copy link
Contributor Author

greg7mdp commented Aug 15, 2024

This really needs to go to release branch? Does it fix a defect?

I think it fixes a potential defect (possible `_segment_manager overwrite in move assignment operator), but not a reported issue.

PRs #558 and #550 did not fix defects. I think this may be useful for updating boost in 1.0, and Kevin also thought it should go to release/1.0.

@arhag which branch would you like me to target this PR to?

@greg7mdp greg7mdp merged commit 034a586 into release/1.0 Aug 15, 2024
36 checks passed
@greg7mdp greg7mdp deleted the chainbase_tip_update branch August 15, 2024 21:41
@arhag arhag added this to the Spring v1.0.0-rc2 milestone Aug 15, 2024
@ericpassmore
Copy link
Contributor

Note:start
group: STABILITY
category: CHORE
summary: Upgrade chainbase, fixes some issues found in CI/CD tests.
Note:end

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.

5 participants