-
-
Notifications
You must be signed in to change notification settings - Fork 7
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
MySQL 8.4.0 is released #1051
MySQL 8.4.0 is released #1051
Conversation
WalkthroughThis update primarily introduces support for MySQL version 8.4.0, involving modifications to build configurations and patch management. The changes streamline the building process and adjust the handling of specific files during installation, particularly for Windows environments. These adjustments are crucial for accommodating the new version in both development workflows and automated testing. Changes
TipsChatThere are 3 ways to chat with CodeRabbit:
Note: Be mindful of the bot's finite context window. It's strongly recommended to break down tasks such as reading entire modules into smaller chunks. For a focused discussion, use review comments to chat about specific files and their changes, instead of using the PR comments. CodeRabbit Commands (invoked as PR comments)
Additionally, you can add CodeRabbit Configration File (
|
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Actionable comments posted: 1
cno_huffman_generator | ||
cno_huffman_generator.cc | ||
SKIP_INSTALL | ||
- EXCLUDE_FROM_ALL |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Consider the implications of removing EXCLUDE_FROM_ALL
.
Removing EXCLUDE_FROM_ALL
from MYSQL_ADD_EXECUTABLE
makes cno_huffman_generator
part of the default build targets. This change could potentially increase build times or include unnecessary targets in some builds. It's important to verify whether this target is essential for all default builds or if it should be optional.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Actionable comments posted: 1
Review Details
Configuration used: CodeRabbit UI
Review profile: CHILL
Files selected for processing (1)
- patches/mysql/8.4.0/windows/fix-building-libcno.patch (1 hunks)
Additional comments not posted (1)
patches/mysql/8.4.0/windows/fix-building-libcno.patch (1)
10-10
: Removal ofEXCLUDE_FROM_ALL
directive.The removal of the
EXCLUDE_FROM_ALL
directive aligns with the PR objectives to streamline the build process for MySQL 8.4.0. This change should help in making thecno_huffman_generator
part of the default build targets, which is intended per the PR summary.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Actionable comments posted: 0
Review Details
Configuration used: CodeRabbit UI
Review profile: CHILL
Files selected for processing (1)
- versions/mysql.json (1 hunks)
Files skipped from review as they are similar to previous changes (1)
- versions/mysql.json
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Actionable comments posted: 0
Review Details
Configuration used: CodeRabbit UI
Review profile: CHILL
Files selected for processing (1)
- .github/workflows/test.yml (2 hunks)
Files skipped from review as they are similar to previous changes (1)
- .github/workflows/test.yml
Summary by CodeRabbit