Skip to content

[master] This addresses the discovery of libcrypto on the latest OSX #66206

Open
ctrix wants to merge 1 commit into
saltstack:masterfrom
ctrix:improve_libcrypto_path_discovery
Open

[master] This addresses the discovery of libcrypto on the latest OSX #66206
ctrix wants to merge 1 commit into
saltstack:masterfrom
ctrix:improve_libcrypto_path_discovery

Conversation

@ctrix
Copy link
Copy Markdown

@ctrix ctrix commented Mar 12, 2024

…atest brew.

What does this PR do?

On the latest darwin with homebrew salt-ssh doesn't work because (among all the issues) it doesn't find libcrypto.

What issues does this PR fix or reference?

Fixes: this allows proper discovery of libcrypto

Merge requirements satisfied?

Probably not.

Commits signed with GPG?

No

@ctrix ctrix requested a review from a team as a code owner March 12, 2024 08:44
@ctrix ctrix requested review from dwoz and removed request for a team March 12, 2024 08:44
@salt-project-bot-prod-environment salt-project-bot-prod-environment Bot changed the title This addresses the discovery of libcrypto on the latest OSX [master] This addresses the discovery of libcrypto on the latest OSX Mar 12, 2024
@dwoz
Copy link
Copy Markdown
Contributor

dwoz commented May 1, 2024

@ctrix Is there any logic to prefer the onedir package's library over homebrew's?

@dwoz dwoz requested a review from a team as a code owner March 16, 2025 22:09
@twangboy twangboy added the test:full Run the full test suite label Jul 1, 2025
@twangboy twangboy added this to the Sulfur v3006.14 milestone Jul 1, 2025
@twangboy
Copy link
Copy Markdown
Contributor

twangboy commented Jul 1, 2025

We should fix this on 3006.x branch instead of master

@twangboy twangboy force-pushed the improve_libcrypto_path_discovery branch from 192b08f to 4db0c7d Compare July 9, 2025 20:50
@twangboy twangboy force-pushed the improve_libcrypto_path_discovery branch from 4db0c7d to 25881e7 Compare July 30, 2025 16:10
@twangboy
Copy link
Copy Markdown
Contributor

twangboy commented Dec 9, 2025

Please rebase. Needs changelog and a test.

@dwoz
Copy link
Copy Markdown
Contributor

dwoz commented Mar 30, 2026

@ctrix Is this still an issue? I thought I recalled merging another PR that addressed this recently.

@twangboy twangboy removed this from the Sulpher v3006.24 milestone May 4, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

test:full Run the full test suite

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants