Skip to content
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

Move to GitHub worflow runner macos-12 #13296

Merged
merged 2 commits into from
Jun 4, 2024
Merged

Conversation

daschuer
Copy link
Member

@daschuer daschuer commented May 27, 2024

This fixes: #13248
Deadline is 2024-06-28

@daschuer
Copy link
Member Author

The test fails rather strange.

360/812 Test #360: GlobalTrackCacheTest.concurrentDelete ......................................................***Timeout  45.02 sec
dlopen error : dlopen(libjack.0.dylib, 0x0001): tried: 'libjack.0.dylib' (no such file), '/usr/local/lib/libjack.0.dylib' (no such file), '/usr/lib/libjack.0.dylib' (no such file), '/Users/runner/work/mixxx/mixxx/libjack.0.dylib' (no such file) 
dlopen error : dlopen(/usr/local/lib/libjack.0.dylib, 0x0001): tried: '/usr/local/lib/libjack.0.dylib' (no such file) 
Note: Google Test filter = GlobalTrackCacheTest.concurrentDelete
[==========] Running 1 test from 1 test suite.
[----------] Global test environment set-up.
[----------] 1 test from GlobalTrackCacheTest
[ RUN      ] GlobalTrackCacheTest.concurrentDelete
info [0x6000037f02b0] GlobalTrackCache - Creating instance

For my understanding libjack is linked statically, so there should be no reason for loading the dylib at runtime. It is also interesting why the arm build is not failing.

@daschuer
Copy link
Member Author

jackaudio/jack2#862 might be related.
@fwcd Do you have an idea?

@fwcd
Copy link
Member

fwcd commented May 27, 2024

The linked issue looks unrelated, library validation is different from not finding the library. That shouldn't be needed in the first place though, since we're almost exclusively using static libraries from vcpkg, right?

You could check the configure logs, perhaps CMake decided to pick a system library over vcpkg for JACK, in that case I would have expected link errors at an earlier stage though.

@Swiftb0y
Copy link
Member

Yes, I'm pretty sure the dlopen issue is completely unrelated, I've seen that numerous times on other macOS CI failures which had nothing to do with jack. I'm pretty sure the concurrentDelete test has some threading-related issues which surface now.

@fwcd
Copy link
Member

fwcd commented May 28, 2024

I remember seeing library loading errors with JACK when launching Mixxx normally on macOS too, even though we don't even use JACK on macOS. So probably something to figure out.

But yeah, the test failure is probably completely unrelated.

@JoergAtGithub
Copy link
Member

You could check the configure logs, perhaps CMake decided to pick a system library over vcpkg for JACK, in that case I would have expected link errors at an earlier stage though.

Our vcpkg.json doesn't contain jack2

@daschuer
Copy link
Member Author

The `dlopen error : dlopen(libjack.0.dylib, 0x0001):``is originated from here:
https://github.com/jackaudio/jack2/blob/c46c1b16e0eabbcf55ef69b0ffb96dfe16521cfa/common/JackWeakAPI.c#L55

@daschuer
Copy link
Member Author

We do build jack2 with vcpkg. Int comes in as dependency. I am currently struggling to find out how, because Portaudio does not have this dependency.

2024-05-27T05:28:03.9724760Z -- Found JACK: /Users/runner/buildenv/mixxx-deps-2.4-x64-osx-min1012-release-07b9859/installed/x64-osx-min1012-release/lib/libjack.a (found version "1.9") 

@Swiftb0y
Copy link
Member

Let me emphasize again that the jack dlopen error has nothing to do with the failing test!

@daschuer
Copy link
Member Author

Let me emphasize again that the jack dlopen error has nothing to do with the failing test!

Confirmed. That is not even an error. Portaudio Probes the JACK API like all others. Unfortunately Jack has decided to print this error unconditionally. I think I will file a bug against Jack.

@daschuer
Copy link
Member Author

This is the upstream report: jackaudio/jack2#977

@daschuer
Copy link
Member Author

daschuer commented Jun 4, 2024

This is now working but required a good amount of testing. 24 Days left.

Copy link
Member

@Swiftb0y Swiftb0y left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM. Thank you for the testing.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants