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

salt: migrate to python@3.9 #62864

Closed
wants to merge 1 commit into from
Closed

Conversation

fxcoudert
Copy link
Member

Splitting from #62560
Updating cffi

@BrewTestBot BrewTestBot added the python Python use is a significant feature of the PR or issue label Oct 14, 2020
@fxcoudert
Copy link
Member Author

This is weird, maybe we're not building the right version of cffi?

    In file included from c/_cffi_backend.c:7552:
    In file included from c/cffi1_module.c:20:
    c/call_python.c:26:30: error: incomplete definition of type 'struct _is'
        builtins = tstate->interp->builtins;
                   ~~~~~~~~~~~~~~^

@fxcoudert
Copy link
Member Author

Even if we provide a newer cffi in the resources, it downloads and uses cffi 1.12.2
It actually downloads a lot of other packages, it seems…

@bayandin
Copy link
Member

It actually downloads a lot of other packages, it seems…

It is. Salt installs of its dependencies + we use resources for installing additional ones (the switch was done in #52835)

If we want to install a custom version of cffi, we need to use inreplace it in https://github.com/saltstack/salt/blob/v3001.1/pkg/osx/req.txt#L4 (and it looks like we'll need to rework it for the next version of salt because of saltstack/salt#56904)

@fxcoudert
Copy link
Member Author

Upstream report: saltstack/salt#58809

@fxcoudert fxcoudert added the upstream issue An upstream issue report is needed label Oct 28, 2020
@fxcoudert
Copy link
Member Author

Closing, will reopen once upstream has fixed this.

@fxcoudert fxcoudert closed this Oct 28, 2020
@fxcoudert fxcoudert deleted the salt branch October 28, 2020 11:35
@fxcoudert
Copy link
Member Author

Weird:

==> brew audit salt --online --git --skip-style
==> FAILED
Error: 1 problem in 1 formula detected
Missing libraries:
  unexpected (/System/Library/Frameworks/AuthenticationServices.framework/Versions/A/AuthenticationServices)
  unexpected (/System/Library/Frameworks/OSLog.framework/Versions/A/OSLog)
  unexpected (/System/Library/Frameworks/PushKit.framework/Versions/A/PushKit)
  unexpected (/System/Library/Frameworks/Speech.framework/Versions/A/Speech)
  unexpected (/System/Library/Frameworks/SystemExtensions.framework/Versions/A/SystemExtensions)
Unexpected missing library linkage detected
salt:
  * salt has broken dynamic library links:

@fxcoudert
Copy link
Member Author

I think those might be a brew bug: Homebrew/brew#9015

@MikeMcQuaid
Copy link
Member

Given the failure is only on 10.14 and 10.13: have you checked those files actually exist in the macOS workers on those versions?

@MikeMcQuaid
Copy link
Member

Yes, looking at the code: I'm almost certain these frameworks don't exist on these macOS versions.

I've opened Homebrew/brew#9031 which should fix this failure when it's merged and CI is rerun. It may be that brew test fails after that point, though, in which case the error message should be adjusted in brew linkage --test and the PR perhaps reverted.

@MikeMcQuaid
Copy link
Member

Rerunning CI now Homebrew/brew#9031 is merged.

@MikeMcQuaid
Copy link
Member

Looks good now. Seems those broken system framework referenced didn't matter 🤷🏻.

@fxcoudert should be ✅ to merge.

@BrewTestBot
Copy link
Member

🤖 A scheduled task has triggered a merge.

@fxcoudert fxcoudert deleted the salt branch November 2, 2020 16:53
@BrewTestBot BrewTestBot added the outdated PR was locked due to age label Dec 3, 2020
@Homebrew Homebrew locked as resolved and limited conversation to collaborators Dec 3, 2020
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
outdated PR was locked due to age python Python use is a significant feature of the PR or issue python-3.9-migration upstream issue An upstream issue report is needed
Projects
None yet
Development

Successfully merging this pull request may close these issues.

6 participants