-
Notifications
You must be signed in to change notification settings - Fork 14
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
error when building on arm64 MacBook #7
Comments
I'm not keen on having to patch dependencies like this because I'm not familiar with the cpu_features codebase (let alone macs), so I'm not really qualified to say whether the changes in one PR or another is correct (there are multiple PRs now to add M1 support in cpu_features, all delayed for various reasons). |
That's fair, but I propose the issue is kept open until the problem no longer exists. |
@mscdex looks like you've fixed all the build issues for M1 Macs, but I'm not seeing the new commits published to npm. Would you mind getting a new version out? Thanks! |
@mscdex Hello, Are there new plans? |
AFAIK there is still no M1 support upstream. The only Apple-related fixes that I originally floated and were eventually made upstream (which I pulled in awhile back) were for macOS x86. |
@mscdex A new version is now available: https://github.com/google/cpu_features/releases/tag/v0.7.0 |
It took me a minute to realize the dependency is bundled in |
I've landed cpu_features v0.7.0 in the master branch. Everyone feel free to give it a whirl and let me know how it works for you. |
I'm going to assume the aforementioned upgrade fixed things. If not, let me know. |
@mscdex I'm afraid after upgrading to ssh 1.8.0 which brings in cpu-features 0.3 I started getting "dyld missing symbol called" in my jest integration tests and webpack compilation. This is on a Macbook Pro M1 Max with MacOS 12.3 and Node 16.14.2. I'm happy to run any debug commands locally that might help. For now, I've pinned ssh back to 1.7.0. Thanks for your hard work on these libraries 🙇♂️ |
@ckortekaas If you're getting them from |
If anyone here is getting those symbol errors while running the |
FWIW I can't reproduce those errors on Github's infrastructure and I don't have any mac hardware myself to test on. |
I'm not sure if this helps but here is what I get in lldb
|
@alexbosworth What does |
|
Looks like cpu_features' support for M1 macs isn't complete then. 🤷♂️ So I guess if you want to see it fixed, continue to bug them about it/test for them over on their repo. I guess something like this PR will fix things as that should resolve the undefined symbol issue. Whether or not additional changes are needed for reading the M1's CPU features compared to the existing solutions for arm64-based systems (e.g. Linux), I don't know. |
Cool, yeah sounds like the problem derives from lack of support there and it seems like a fix is in progress |
Allegedly fixed in the latest release of cpu_features https://github.com/google/cpu_features/releases/tag/v0.7.0 with google/cpu_features#209 |
@FabianFrank Unfortunately the support is not complete with v0.7.0. |
Seems like this issue went away for me when I updated to MacOS 12.3 |
@alexbosworth The underlying problem still exists (M1 support doesn't actually exist in cpu_features), the only difference is that we now explicitly report the M1 as an unknown CPU without any feature flags instead of crashing at runtime. |
Ah great thanks |
Note that there exists a fix for building on M1:
google/cpu_features#211
Also, this module is broken. The cause of the breakage (upstream or not) doesn't really matter -- unless you're saying "I never plan on fixing this," keeping the issue open helps the community understand what's going on.
Finally, I predict this will become more of a pressing issue now that Apple has stopped selling x64 MacBook Pros entirely, AND this module is a dependency of the electron installer builders.
The text was updated successfully, but these errors were encountered: