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

use cmake and python to keep version numbers updated #26323

Draft
wants to merge 7 commits into
base: main
Choose a base branch
from

Conversation

arezaii
Copy link
Contributor

@arezaii arezaii commented Nov 26, 2024

This uses a combination of python scripts and cmake dependencies to keep the version number information consistent with the information in CMakeLists.txt. The purpose is to avoid manually having to update the version number across a variety of documents with every release. With this change, the version numbers will be correctly updated based on the status of the CHPL_OFFICIAL_RELEASE flag, and the current, previous, and next version numbers.

This necessitated tracking the next and past version numbers as the versionButton.php file needs the next version number when the CHPL_OFFICIAL_RELEASE flag becomes true, and at the same time the archived-spec.rst file needs to know the previous version number.

Because we can't predict the past or next version number from the current one, it was necessary to add these to the CMakeLists.txt file.

New files that need updating can be handled by creating new instances of the FileUpdater class and implementing the update method.

Signed-off-by: Ahmad Rezaii <ahmad.rezaii@hpe.com>
Signed-off-by: Ahmad Rezaii <ahmad.rezaii@hpe.com>
Signed-off-by: Ahmad Rezaii <ahmad.rezaii@hpe.com>
Signed-off-by: Ahmad Rezaii <ahmad.rezaii@hpe.com>
Signed-off-by: Ahmad Rezaii <ahmad.rezaii@hpe.com>
Signed-off-by: Ahmad Rezaii <ahmad.rezaii@hpe.com>
Signed-off-by: Ahmad Rezaii <ahmad.rezaii@hpe.com>
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.

1 participant