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

Remove the PyObjC dependency #62432

Merged

Conversation

s0undt3ch
Copy link
Collaborator

What does this PR do?

See title.

What issues does this PR fix or reference?

This addresses problems with building a one dir build for macOS.
It becomes even more problematic because depending on the macOS version, it pulls different dependencies, and we would either have to build a macos onedir for each macOS supported release, or ship a crippled onedir(because it would be tied to the macOS version where the onedir was built)

@s0undt3ch s0undt3ch requested a review from a team as a code owner August 5, 2022 17:05
@s0undt3ch s0undt3ch requested review from Ch3LL and removed request for a team August 5, 2022 17:05
twangboy
twangboy previously approved these changes Aug 5, 2022
Signed-off-by: Pedro Algarvio <[email protected]>
@s0undt3ch s0undt3ch force-pushed the hotfix/tiamat-ld-library-path branch from e34357d to 887f0ba Compare August 5, 2022 17:20
Copy link
Contributor

@SMillerDev SMillerDev left a comment

Choose a reason for hiding this comment

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

Not a frequent contributor here, but this would improve the packaging of salt in Homebrew a lot.

@s0undt3ch
Copy link
Collaborator Author

Not a frequent contributor here, but this would improve the packaging of salt in Homebrew a lot.

Thank You for the input.
And those that need would still be able to pip install it after salt is installed.

@s0undt3ch
Copy link
Collaborator Author

The required approvals are in and tests have passed.
Merging.

@s0undt3ch s0undt3ch merged commit 27e3745 into saltstack:freeze Aug 8, 2022
@s0undt3ch s0undt3ch deleted the hotfix/tiamat-ld-library-path branch August 8, 2022 04:03
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.

4 participants