-
Notifications
You must be signed in to change notification settings - Fork 423
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
AZP/RELEASE: Fixes for v1.14 release #8823
Conversation
6ba2dee
to
99f1884
Compare
Have you verified test release on your fork with these changes? |
For the most part: |
@Al3xR01 the build failure seems relevant, ucx/buildlib/azure-pipelines-release.yml Line 42 in 88d4bb9
it will trigger release pipeline for this PR so we can see if it works |
56574b2
56574b2
to
34f63d4
Compare
debian/rules.in
Outdated
@@ -16,7 +16,7 @@ CUDA_OPT = $(shell echo ${DEB_BUILD_PROFILES} | grep -qw cuda \ | |||
dh $@ | |||
|
|||
override_dh_auto_configure: | |||
./contrib/configure-release-mt --prefix=/usr \ | |||
../contrib/configure-release-mt --prefix=/usr \ |
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.
some tests fail because of this change
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.
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.
isn't this related?
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.
Resolved by the sed
hack for now.
6e88186
to
94330e2
Compare
94330e2
to
301bed3
Compare
@yosefe, can you please review? |
What
Manually cherry-pick fixes to resolve various CI failures.