Skip to content

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

Open
ctrix wants to merge 1 commit intosaltstack: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 intosaltstack:masterfrom
ctrix:improve_libcrypto_path_discovery

Conversation

@ctrix
Copy link

@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
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
Contributor

twangboy commented Jul 1, 2025

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

@twangboy
Copy link
Contributor

twangboy commented Dec 9, 2025

Please rebase. Needs changelog and a test.

@twangboy twangboy modified the milestones: Sulfur v3006.18, 3006.20 Jan 13, 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