Skip to content

🔨 Detect package manager install paths#27655

Merged
thinkyhead merged 5 commits intoMarlinFirmware:bugfix-2.1.xfrom
thisiskeithb:pr/detect_mac_architecture
Feb 15, 2025
Merged

🔨 Detect package manager install paths#27655
thinkyhead merged 5 commits intoMarlinFirmware:bugfix-2.1.xfrom
thisiskeithb:pr/detect_mac_architecture

Conversation

@thisiskeithb
Copy link
Contributor

@thisiskeithb thisiskeithb commented Jan 26, 2025

Description

Update mac_gcc script to detect MacPorts & Homebrew install paths.

Benefits

Script will be more universal (and work on Intel Macs)

Related Issues

@thisiskeithb thisiskeithb added the T: Development Makefiles, PlatformIO, Python scripts, etc. label Jan 26, 2025
@thisiskeithb thisiskeithb changed the title 🔨 mac_gcc: Detect architecture for Homebrew 🔨 Improve Homebrew path detection Feb 15, 2025
@thisiskeithb thisiskeithb force-pushed the pr/detect_mac_architecture branch from a40f3b4 to d60fdc0 Compare February 15, 2025 05:48
@thisiskeithb thisiskeithb changed the title 🔨 Improve Homebrew path detection 🔨 Add Homebrew path detection Feb 15, 2025
@thisiskeithb thisiskeithb changed the title 🔨 Add Homebrew path detection 🔨 Detect package manager install paths Feb 15, 2025
@thisiskeithb
Copy link
Contributor Author

thisiskeithb commented Feb 15, 2025

The script now detects MacPorts & Homebrew install paths instead of assuming defaults (and works on Intel).

@thisiskeithb thisiskeithb force-pushed the pr/detect_mac_architecture branch from 5d31d30 to 0ce9aac Compare February 15, 2025 19:29
@thinkyhead thinkyhead merged commit 12b3f18 into MarlinFirmware:bugfix-2.1.x Feb 15, 2025
64 checks passed
@thisiskeithb thisiskeithb deleted the pr/detect_mac_architecture branch February 16, 2025 01:44
EvilGremlin pushed a commit to EvilGremlin/Marlin that referenced this pull request May 15, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

C: Build / Toolchain T: Development Makefiles, PlatformIO, Python scripts, etc.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants