-
-
Notifications
You must be signed in to change notification settings - Fork 9.8k
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
Add a workflow to test building from a read-only source tree [3.1 & 3.0] #22934
Conversation
I have verified that this cherry-picks cleanly to openssl-3.0 |
Wheres the 'deal with the consequences part' :) |
That's the second commit, which modifies |
Is the test failure not related? |
PING! #22908 has been approved... any chance this can as well? |
What about the comment from @slontis above? |
Whoopsie, of course there should be no |
Tests still failing. |
Why is this being approved with test failures? |
I think that failure is irrelevant for this PR. I can rebase it, though, and see if fares better |
This also adds the configuration options 'enable-quic'. Fixes openssl#22907
…urce tree Modify 'out-of-source-and-install' to work with a read-only source tree Fixes openssl#22907
dd04ea3
to
a0db2a4
Compare
And that exact same error is in the current openssl-3.1 branch: https://github.com/openssl/openssl/actions/runs/7117073364/job/19376972990 So, I'll count the approvals as valid. |
24 hours has passed since 'approval: done' was set, but as this PR has been updated in that time the label 'approval: ready to merge' is not being automatically set. Please review the updates and set the label manually. |
Merged 3.1: 3.0: |
Fixes #22907 Reviewed-by: Tomas Mraz <tomas@openssl.org> Reviewed-by: Neil Horman <nhorman@openssl.org> (Merged from openssl/openssl#22934) (cherry picked from commit d48db01f06afbc0557ad04548d07cb16eecaa3e4) Signed-off-by: fly2x <fly2x@hitls.org>
Fixes #22907 Reviewed-by: Tomas Mraz <tomas@openssl.org> Reviewed-by: Neil Horman <nhorman@openssl.org> (Merged from openssl/openssl#22934) (cherry picked from commit 487628b62efc5703e44b2f88800e7a340ce17822) Signed-off-by: fly2x <fly2x@hitls.org>
Fixes #22907 Reviewed-by: Tomas Mraz <tomas@openssl.org> Reviewed-by: Neil Horman <nhorman@openssl.org> (Merged from openssl/openssl#22934) Signed-off-by: fly2x <fly2x@hitls.org>
Fixes #22907 Reviewed-by: Tomas Mraz <tomas@openssl.org> Reviewed-by: Neil Horman <nhorman@openssl.org> (Merged from openssl/openssl#22934) Signed-off-by: fly2x <fly2x@hitls.org>
... and deal with the consequences.
Note: this is a backport of #22908 for openssl-3.1 and openssl-3.0