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

Devendor FindSQLite3.cmake #4569

Closed
wants to merge 1 commit into from
Closed

Conversation

daschuer
Copy link
Member

It is part of CMake since 3.13 and defines SQLite::SQLite3, without 3

https://cmake.org/cmake/help/latest/module/FindSQLite3.html

@github-actions github-actions bot added the build label Dec 16, 2021
@ronso0
Copy link
Member

ronso0 commented Dec 16, 2021

no comment on the code change, but macOS builds fail.

@daschuer
Copy link
Member Author

daschuer commented Dec 16, 2021

Mmm that's strange, I have no idea where 'SQLite3::SQLite3' is comming from on Mac

 CMake Error at /Users/runner/buildenv/mixxx-deps-2.4-x64-osx-058063e/scripts/buildsystems/vcpkg.cmake:538 (_add_executable):
  Target "mixxx-test" links to target "SQLite3::SQLite3" but the target was
  not found.  Perhaps a find_package() call is missing for an IMPORTED
  target, or an ALIAS target is missing?
Call Stack (most recent call first):
  CMakeLists.txt:1571 (add_executable)

@Be-ing
Copy link
Contributor

Be-ing commented Dec 17, 2021

It was your choice to reject a fully working solution. That code hasn't disappeared...

@daschuer
Copy link
Member Author

This PR will succeed, once we have switched to the environment including this: mixxxdj/vcpkg#27

@Be-ing
Copy link
Contributor

Be-ing commented Dec 18, 2021

No, it won't, because you're creating a circular dependency. Again, this was already resolved but you rejected the solution.

mixxxdj/vcpkg#27 was never intended to be used separately from #4432. I don't know why you insist on learning this the hard way.

@Holzhaus
Copy link
Member

Holzhaus commented Jan 1, 2022

I'll mark this as draft until all CI checks are green.

@Holzhaus Holzhaus marked this pull request as draft January 1, 2022 22:54
@daschuer
Copy link
Member Author

daschuer commented Jan 1, 2022

This depends on mixxxdj/vcpkg#38

fwcd added a commit to fwcd/mixxx that referenced this pull request Jan 10, 2022
@github-actions
Copy link

github-actions bot commented Apr 2, 2022

This PR is marked as stale because it has been open 90 days with no activity.

@github-actions github-actions bot added the stale Stale issues that haven't been updated for a long time. label Apr 2, 2022
@daschuer daschuer removed the stale Stale issues that haven't been updated for a long time. label Apr 2, 2022
@daschuer
Copy link
Member Author

daschuer commented Apr 2, 2022

There is a single prerequisite missing, merging from upstream vcpkg: mixxxdj/vcpkg#41
Can one have a look?

It is part of CMake since 3.13 and defines SQLite::SQLite3, without 3
@daschuer
Copy link
Member Author

This needs to become part of #4717 to fix the build failure.

@daschuer daschuer closed this Apr 21, 2022
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