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

update libdjinterop to 0.20.2 #12846

Closed
wants to merge 1 commit into from
Closed

Conversation

Be-ing
Copy link
Contributor

@Be-ing Be-ing commented Feb 19, 2024

to fix Clang build
xsco/libdjinterop#112

@github-actions github-actions bot added the build label Feb 19, 2024
@Be-ing
Copy link
Contributor Author

Be-ing commented Feb 20, 2024

Linking libdjinterop fails on Windows when built via ExternalProject_Add?? PR to update vcpkg is here: microsoft/vcpkg#36869

@daschuer daschuer self-requested a review February 20, 2024 08:15
Copy link
Member

@daschuer daschuer left a comment

Choose a reason for hiding this comment

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

This does unfortunately not link on windows:
fatal error LNK1120: 8 unresolved externals

I think It would be reasonable to stick with the VCPKG provided version for now and just add your change as patch. PATCH_COMMAND

Other option would be to required 0.20.1 but download 0.20.2
We need also make sure that the djinterop PPA has the newest version in this case.

@Be-ing
Copy link
Contributor Author

Be-ing commented Feb 21, 2024

The update of libdjinterop in vcpkg has been merged.

@daschuer
Copy link
Member

Do you have an idea why we have a linker error on Windows? Will the same happen with the vcpkg build?

@Be-ing
Copy link
Contributor Author

Be-ing commented Feb 21, 2024

It may have to do with this https://github.com/xsco/libdjinterop/blob/main/CMakeLists.txt#L156

set_target_properties(DjInterop PROPERTIES C_VISIBILITY_PRESET hidden)
set_target_properties(DjInterop PROPERTIES CXX_VISIBILITY_PRESET hidden)

which makes me wonder how it works with vcpkg 🤔 Anyway, considering that it worked with vcpkg before somehow, I think it will work again when updating vcpkg.

@Be-ing
Copy link
Contributor Author

Be-ing commented Feb 22, 2024

Let's update vcpkg then try rebasing this.

@daschuer
Copy link
Member

@mr-smidge Can you please you Launchpad PPA to 0.20.2? Than we can advance with Mixxx to that version. Thank you.

@mr-smidge
Copy link
Contributor

@mr-smidge Can you please you Launchpad PPA to 0.20.2? Than we can advance with Mixxx to that version. Thank you.

I've pushed 0.20.2, and bionic is building now... will copy for other distros tomorrow once it's done.

@Be-ing Be-ing force-pushed the libdjinterop_0.20.2 branch from f863412 to 0ee7a04 Compare February 28, 2024 02:11
@daschuer
Copy link
Member

Thank you

@mr-smidge
Copy link
Contributor

0.20.2 now available for all Ubuntu series: https://launchpad.net/~xsco/+archive/ubuntu/djinterop/+packages

@daschuer
Copy link
Member

daschuer commented Mar 4, 2024

Merged via #12898
Thank you.

@daschuer daschuer closed this Mar 4, 2024
@Be-ing
Copy link
Contributor Author

Be-ing commented Mar 4, 2024

That was to the main branch. This needs to go to the 2.4 branch to fix the build failure.

@Be-ing Be-ing reopened this Mar 4, 2024
@Be-ing Be-ing added this to the 2.4.1 milestone Mar 9, 2024
@daschuer daschuer mentioned this pull request Apr 22, 2024
@JoergAtGithub
Copy link
Member

Merged in #13154

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

Successfully merging this pull request may close these issues.

4 participants