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

fix(aarch64): correct protoc for linux_aarch64 #2839

Closed
wants to merge 1 commit into from

Conversation

s22chan
Copy link
Contributor

@s22chan s22chan commented Nov 7, 2024

protoc was always being download as x86-64 when under linux.

Fixes #2838

Pull Request Checklist

  • Description which clearly states what problems the PR solves.
  • Description contains a link to the Github Issue, and any relevent
    Discussions, this PR applies to.
  • Include new tests that fail without this PR but passes with it.
  • Include any relevent Documentation changes related to this change.
  • Verify all commits have been signed in accordance with the DCO policy.
  • Reviewed PR commit history to remove unnecessary changes.
  • Make sure your PR passes build, test and lint steps completely.

@texodus
Copy link
Member

texodus commented Nov 27, 2024

@s22chan Thanks for the PR!

This does not compile and is beginning to bitrot, so closing for now. While I'm in favor of adding broader architecture support, I would like test coverage for these platforms (or surrogate diligence in understanding why GitHub Actions CI does not allow this), or new releases are only going to work incidentally.

@texodus texodus closed this Nov 27, 2024
@s22chan
Copy link
Contributor Author

s22chan commented Nov 27, 2024

It did not compile due to an unrelated issue and I wasn't sure how to retrigger a build.

For the second point, that's fair, I'll just make my own builds for now. Thanks for your project btw, its' very useful

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.

protoc cannot be found when building perspective-python linux_aarch64
2 participants