-
Notifications
You must be signed in to change notification settings - Fork 859
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
Update VERSION and NEWS for 5.0.1 #12177
Conversation
7265bbc
to
eee45be
Compare
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.
LGTM
Just a minor note: we should probably remove the opal_common_cuda, and ugni since those don't even exist in v5.0.x
VERSION
Outdated
@@ -41,7 +41,7 @@ flex_min_version=2.5.4 | |||
# requirement is that it must be entirely printable ASCII characters | |||
# and have no white space. | |||
|
|||
greek=rc1 | |||
greek=a1 |
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.
Don't you want to make this rc2
?
We've already made a1
releases -- don't go backwards.
You can make a rc2
release tarball, and if it passes a few sanity tests, make a final tarball (without rc2
). The Jenkins job has a parameter that will set greek
to empty during the tarball build (i.e., you don't have to commit greek=
in VERSION
to make this happen).
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.
Ah you are right. We should only go forward.
Updated. Thanks!
bot:notacherrypick Signed-off-by: Wenduo Wang <[email protected]>
eee45be
to
8899ab0
Compare
bot:notacherrypick