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

Compile osx manager with vcpkg #5926

Draft
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

talregev
Copy link
Contributor

Compile osx manager with vcpkg.

@talregev talregev force-pushed the TalR/osx_vcpkg5 branch 4 times, most recently from 218bab4 to 6687500 Compare November 29, 2024 15:51
@AenBleidd
Copy link
Member

Thank you for your pull request.
Currently we do not plan to support vcpkg in the MacOS build pipeline.
Will keep it in draft for future discussion and review.

Choose a reason for hiding this comment

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

Copilot reviewed 3 out of 13 changed files in this pull request and generated no suggestions.

Files not reviewed (10)
  • 3rdParty/vcpkg_ports/configs/manager/osx/vcpkg.json: Language not supported
  • 3rdParty/vcpkg_ports/patches/boinc_SetItemBitmap.patch: Language not supported
  • 3rdParty/vcpkg_ports/patches/boinc_SetVisibilityHidden.patch: Language not supported
  • 3rdParty/vcpkg_ports/patches/pcre2.patch: Language not supported
  • 3rdParty/vcpkg_ports/patches/wxwidgets.patch: Language not supported
  • 3rdParty/vcpkg_ports/triplets/ci/arm64-osx.cmake: Language not supported
  • 3rdParty/vcpkg_ports/triplets/ci/x64-osx.cmake: Language not supported
  • mac_build/buildMacBOINC-CI.sh: Language not supported
  • osx/build_vcpkg_3rdparty_for_manager.sh: Language not supported
  • osx/update_vcpkg_manager_universal.sh: Language not supported
@CharlieFenton
Copy link
Contributor

The current build system guarantees that each version of BOINC is built for MacOS with a specific versions of all the dependencies. Since vcpkg could result in the MacOS Manager being built with different versions of dependencies, I would no longer be willing to support BOINC because of the added difficulty in debugging bug reports from the field.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
Status: In progress
Development

Successfully merging this pull request may close these issues.

3 participants