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

[openssl] Fix building in Release only mode #24004

Conversation

AenBleidd
Copy link
Contributor

Signed-off-by: Vitalii Koshura lestat.de.lionkur@gmail.com

Copy link

@github-actions github-actions bot left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This is a new experimental fast check for PR issues. Please let us know if this bot is helpful!

PRs must add only one version and must not modify any published versions

When making any changes to a library, the version or port-version in vcpkg.json or CONTROL must be modified.

Error: Local changes detected for openssl but no changes to version or port version.
-- Version: 3.0.2
-- Old SHA: 2ba2e59ee4f32c11e30aeccc0ecabc09b69c5d22
-- New SHA: 9c3f6ad15d765bc9e5c8754dea85793bd33d4183
-- Did you remember to update the version or port version?
-- Pass `--overwrite-version` to bypass this check.
***No files were updated.***

@AenBleidd
Copy link
Contributor Author

This is a fix for #22878 (comment)
@strega-nil-ms, @BillyONeal, may I have your review?

Signed-off-by: Vitalii Koshura <lestat.de.lionkur@gmail.com>
@AenBleidd AenBleidd force-pushed the vko_fix_openssl_for_release_only_builds branch from 64f6d24 to 4493e92 Compare April 6, 2022 21:42
AenBleidd and others added 2 commits April 6, 2022 23:50
Co-authored-by: Alexander Neumann <30894796+Neumann-A@users.noreply.github.com>
Copy link

@github-actions github-actions bot left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This is a new experimental fast check for PR issues. Please let us know if this bot is helpful!

After committing all other changes, the version database must be updated
git add -u && git commit
git checkout 59facb4cf71bff961a7a7fe18ffa07539c19183f -- versions
./vcpkg x-add-version --all
Diff
diff --git a/versions/o-/openssl.json b/versions/o-/openssl.json
index d64cbab..7734cda 100644
--- a/versions/o-/openssl.json
+++ b/versions/o-/openssl.json
@@ -1,7 +1,7 @@
 {
   "versions": [
     {
-      "git-tree": "6abb68591e9636cdf310ce1d1c3a04e3ebce9cab",
+      "git-tree": "7f45a98732d599c67e9cf0069783308f71f8666a",
       "version": "3.0.2",
       "port-version": 1
     },

…AenBleidd/vcpkg into vko_fix_openssl_for_release_only_builds

Signed-off-by: Vitalii Koshura <lestat.de.lionkur@gmail.com>
@strega-nil-ms strega-nil-ms added the info:reviewed Pull Request changes follow basic guidelines label Apr 6, 2022
Signed-off-by: Vitalii Koshura <lestat.de.lionkur@gmail.com>
@AenBleidd
Copy link
Contributor Author

Merged #23932 as @strega-nil-ms asked

@JackBoosY JackBoosY added the category:port-bug The issue is with a library, which is something the port should already support label Apr 7, 2022
@winterz
Copy link

winterz commented Apr 7, 2022

anxiously waiting for this fix to land
#24003

@strega-nil-ms
Copy link
Contributor

Thanks!

@strega-nil-ms strega-nil-ms merged commit 1b3a783 into microsoft:master Apr 7, 2022
@AenBleidd AenBleidd deleted the vko_fix_openssl_for_release_only_builds branch April 7, 2022 14:55
@winterz
Copy link

winterz commented Apr 7, 2022

still doesn't work. same error as reported in #24003
how can I roll back to version 1.1.1n#1? is there a way to pass OPENSSL_VERSION to vcpkg install openssl ?

@AenBleidd
Copy link
Contributor Author

@winterz, I'll take a look at this issue tonight

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
category:port-bug The issue is with a library, which is something the port should already support info:reviewed Pull Request changes follow basic guidelines
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants