Skip to content

Commit 2f4e622

Browse files
Vipul-Cariappavgvassilev
authored andcommitted
brew remove unxip on OSX-ARM exclusively
fix for the osx dependency issue
1 parent 3540eff commit 2f4e622

File tree

2 files changed

+5
-2
lines changed

2 files changed

+5
-2
lines changed

.github/workflows/ci.yml

Lines changed: 4 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -282,7 +282,10 @@ jobs:
282282
run: |
283283
brew update
284284
brew remove [email protected]
285-
brew remove unxip
285+
export ARCHITECHURE=$(uname -m)
286+
if [[ "$ARCHITECHURE" != "x86_64" ]]; then
287+
brew remove unxip
288+
fi
286289
# workaround for https://github.com/actions/setup-python/issues/577
287290
for pkg in $(brew list | grep '^python@'); do
288291
brew unlink "$pkg"

test/Makefile

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -18,7 +18,7 @@ all : $(dicts)
1818

1919
#cppflags=$(shell llvm-config --cxxflags) -O3 -fPIC -I$(shell python -c 'import sysconfig as sc; print(sc.get_config_var("INCLUDEPY"))') -Wno-register
2020
# FIXME: stltypes.cxx does not compile with clang!
21-
cppflags= -std=c++17 -O0 -g -fPIC -I$(shell python -c 'import sysconfig as sc; print(sc.get_config_var("INCLUDEPY"))') -Wno-register
21+
cppflags= -std=c++17 -O3 -fPIC -I$(shell python -c 'import sysconfig as sc; print(sc.get_config_var("INCLUDEPY"))') -Wno-register
2222

2323
PLATFORM := $(shell uname -s)
2424
ifeq ($(PLATFORM),Darwin)

0 commit comments

Comments
 (0)