Skip to content

Revert unintended toolchain changes from PR #920#921

Merged
cavearr merged 1 commit into
FPGAwars:developfrom
ProbabilityEngineer:revert-pr920-toolchain
May 28, 2026
Merged

Revert unintended toolchain changes from PR #920#921
cavearr merged 1 commit into
FPGAwars:developfrom
ProbabilityEngineer:revert-pr920-toolchain

Conversation

@ProbabilityEngineer

@ProbabilityEngineer ProbabilityEngineer commented May 27, 2026

Copy link
Copy Markdown
Contributor

Summary

I should have made a new branch to continue working after making PR #920. I didn't, I'm sorry. I didn't realize PRs aren't snapshots.

PR #920 was intended to fix the macOS ARM build, but its source branch continued to move before merge and unintentionally included partial Apio 1.x/toolchain installer changes.

This PR restores the toolchain-related files to the state from the intended macOS build-fix commit while keeping the macOS build fixes that PR #920 was meant to introduce.

Reverted from PR #920

  • Apio version pin changes
  • Python 3.11 requirement changes
  • partial Apio 1.x installer command changes
  • macOS Homebrew/driver behavior changes
  • accidental local .tickets file
  • macOS Homebrew notes doc added during local investigation

Validation

npm run jshint

passes locally.

PR FPGAwars#920 was intended to only fix the macOS ARM build, but the source branch advanced before merge and included partial Apio 1.x/toolchain installer work. Restore the toolchain-related files to the macOS-build-fix state while keeping the build fixes from PR FPGAwars#920.

Also remove the accidentally tracked local tk ticket database and ignore .tickets going forward.
@cavearr

cavearr commented May 28, 2026

Copy link
Copy Markdown
Member

let's go! don't worry for this, the important is that you learn and contribute to the project.

@cavearr cavearr merged commit cfa40bd into FPGAwars:develop May 28, 2026
1 check passed
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.

2 participants