-
Notifications
You must be signed in to change notification settings - Fork 54
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
Broken on MacOS #223
Comments
P.S. the old version works fine,
|
It is still open and occurring since the fix was not part of a release. The fix went in Aug 2019, the last release was in May. |
Still an issue on macOS:
|
It seems like this fails to build any source version, and the bug is that wheels aren't getting uploaded. Are there some build options one needs to set? |
I'm having a very similar problem on Ubuntu 20 /usr/bin/ld: build/temp.linux-x86_64-3.8/src/vmprof_common.o:/tmp/pip-install-mvuv1_s9/vmprof/src/vmprof_common.h:92: multiple definition of |
This worked for me as well on Ubuntu. |
I'm also getting this duplicate symbol error with mac 10.15.7. Can we just upload the wheels for py3.7 and py3.8 on macOS as part of the release? |
Same here on MacOs Catalina 10.15.7 with python 3.8 Install fails:
|
Moving to cibuildwheel #241 would make releasing a new version less painful. Help welcome. |
We now have wheels for macos x86_64. We do not have wheels for arm64. Can you use the wheels? |
@mattip awesome. It has been forever and a day since I tried to use vmprof due to this issue, so, can you remind me how to do a quick test case? :) |
I think this is supposed to work
then There is a also a way to get the output into a flamegraph, @cfbolz might know better. I guess we should add that to the FAQ... |
And, it works! Thanks so much for following up on this bug. Generating the flamegraph is a bit annoying, since it involves cloning two repos (one python, one perl) but it did work without me having to install anything else. |
What was the exact workflow? We should document it and maybe we can streamline it. |
@glyph a student of mine has been working on an importer to the Firefox profiling web app: https://github.com/Cskorpion/vmprof-firefox-converter |
The text was updated successfully, but these errors were encountered: