-
Notifications
You must be signed in to change notification settings - Fork 2.2k
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
Load dynamic libraries on darwin #3901
base: master
Are you sure you want to change the base?
Load dynamic libraries on darwin #3901
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Do TestListImages, TestPluginStepping and TestPluginVariables pass on macOS with this? The WithPlugins function that disables them wasn't changed.
Yes, at least as far as I can tell. I'm running it on M1 Air with MacOS 15.2. edit: however. i cannot confirm these fail on master either. |
Did you comment out the line at pkg/proc/test/support.go:359 or exclude darwin? |
thanks for the pointer. Seems like I wasn't getting the feedback the test was skipped. Looks like TestListImages fails after enabling plugin tests on darwin. |
Do you mean in CI or also on your computer? |
I mean on my machine (see attachment). Other tests run fine. CI seems to fail only for Go 1.24 if I understand correctly? |
This could be just a problem with the test:
plugin1 is there but the test doesn't find it, probably because of the symlink stuff that macos does. |
Followup on #3669
Resolves #1628