Skip to content

Build.macos.openssl@1.1#4345

Merged
matejk merged 2 commits intopocoproject:develfrom
bas524:build.macos.openssl
Dec 15, 2023
Merged

Build.macos.openssl@1.1#4345
matejk merged 2 commits intopocoproject:develfrom
bas524:build.macos.openssl

Conversation

@bas524
Copy link
Copy Markdown
Contributor

@bas524 bas524 commented Dec 15, 2023

This PR fixes problem whith poco tests on macos when we use makefiles and openssl version 1.1

Problem root couse is "we hope that we build with openssl@1.1 because we installed it, but default openssl in brew is openssl@3"

Problem visalisation is failed test

1: CppUnit::TestCaller<CryptoTest>.testEncryptDecryptGCM
    "Poco::IOException:
I/O error: error:1C800066:Provider routines::cipher operation failed"
    in "<unknown>", line -1

@matejk matejk self-assigned this Dec 15, 2023
@matejk
Copy link
Copy Markdown
Contributor

matejk commented Dec 15, 2023

Changes in the PR are appreciated to correct wronly used paths.

Although it seems that they do not address the real problem with the CGM test, because it also happens in macOS cmake jobs that use openssl 1.1 and 3 properly.

@bas524
Copy link
Copy Markdown
Contributor Author

bas524 commented Dec 15, 2023

Изменения в PR приветствуются для исправления путей, используемых только wronly.

Хотя кажется, что они не решают реальную проблему с тестом CGM, потому что это также происходит в заданиях macOS cmake, которые правильно используют openssl 1.1 и 3.

Let's resolve problems ste by step ))
Now I don't see failed builds with cmake, if they are, of course I'll try to fix them

@matejk
Copy link
Copy Markdown
Contributor

matejk commented Dec 15, 2023

Let's resolve problems ste by step )) Now I don't see failed builds with cmake, if they are, of course I'll try to fix them

I am currently working on the same issue and temporarily disabled the tests. There is a discussion in github that might help resolving the problem permanently.

openssl/openssl#21119

@matejk matejk added this to the Release 1.13.0 milestone Dec 15, 2023
@matejk matejk merged commit a464a4e into pocoproject:devel Dec 15, 2023
@bas524 bas524 deleted the build.macos.openssl branch December 16, 2023 11:21
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.

2 participants