Skip to content

Fix mac builds#166

Merged
fmarier merged 3 commits intomasterfrom
fix-mac-builds
Feb 4, 2026
Merged

Fix mac builds#166
fmarier merged 3 commits intomasterfrom
fix-mac-builds

Conversation

@fmarier
Copy link
Copy Markdown
Member

@fmarier fmarier commented Jan 29, 2026

Fix for #164 which disabled the zlib tests for the wrong architecture, and disable the tor tests entirely on Mac.

These are broken when cross-compiling for x86_64 on arm64.
The following test currently fails:

16:34:58  perl ./scripts/maint/checkSpace.pl -C \
16:34:58  		./src/lib/*/*.[ch] ./src/core/*/*.[ch] ./src/feature/*/*.[ch] ./src/app/*/*.[ch] ./src/test/*.[ch] ./src/test/*/*.[ch] ./src/tools/*.[ch]
16:34:58  /opt/homebrew/bin/python3 ./scripts/maint/practracker/includes.py .
16:34:58  ./scripts/maint/checkShellScripts.sh
16:34:58  Unusual pattern permitted.h in ./scripts/maint/practracker/testdata
16:34:58  ./scripts/maint/checkShellScripts.sh: Install shellcheck to check shell scripts.
...
16:35:05  FAIL: scripts/maint/practracker/test_practracker.sh

These tests are already disabled on arm64-apple-darwin (#158).
@waybackarchiver
Copy link
Copy Markdown
Contributor

Can this pr solve FAIL: scripts/maint/practracker/test_practracker.sh?

@fmarier
Copy link
Copy Markdown
Member Author

fmarier commented Feb 3, 2026

@waybackarchiver Yes, the tor tests will not be run on Mac anymore.

@fmarier fmarier merged commit e855502 into master Feb 4, 2026
5 of 6 checks passed
@fmarier fmarier deleted the fix-mac-builds branch February 4, 2026 05:12
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.

3 participants