Skip to content

fix(macOS): macdeployqt now finds libvpx#9497

Merged
i2h3 merged 1 commit intomasterfrom
i2h3/fix/libvpx-not-found
Feb 24, 2026
Merged

fix(macOS): macdeployqt now finds libvpx#9497
i2h3 merged 1 commit intomasterfrom
i2h3/fix/libvpx-not-found

Conversation

@i2h3
Copy link
Collaborator

@i2h3 i2h3 commented Feb 24, 2026

Finally get rid of that not critical build error on macOS:

ERROR: no file at "/usr/lib/libvpx.11.dylib"

@i2h3 i2h3 added this to the 33.0.0 milestone Feb 24, 2026
@i2h3 i2h3 self-assigned this Feb 24, 2026
@i2h3 i2h3 requested a review from Aiiaiiio as a code owner February 24, 2026 09:43
Copilot AI review requested due to automatic review settings February 24, 2026 09:43
@i2h3 i2h3 added bug os: 🍎 macOS Apple macOS, formerly also known as OS X labels Feb 24, 2026
@github-project-automation github-project-automation bot moved this to 🧭 Planning evaluation (don't pick) in 💻 Desktop Clients team Feb 24, 2026
@i2h3 i2h3 moved this from 🧭 Planning evaluation (don't pick) to 🏗️ In progress in 💻 Desktop Clients team Feb 24, 2026
@i2h3 i2h3 force-pushed the i2h3/fix/libvpx-not-found branch from cc4218f to 1c2a117 Compare February 24, 2026 09:43
@i2h3 i2h3 changed the title macdeployqt now finds libvpx fix(macOS): macdeployqt now finds libvpx Feb 24, 2026
Copy link
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull request overview

This PR updates the macOS bundle deployment step so macdeployqt is given an explicit Qt library search path, improving its ability to locate and bundle dependencies like libvpx.

Changes:

  • Derive the macdeployqt installation directory and an associated ../lib directory.
  • Pass the derived library directory to macdeployqt via -libpath=... during the post-build bundle step.

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

@i2h3
Copy link
Collaborator Author

i2h3 commented Feb 24, 2026

/backport to stable-33.0

@github-actions
Copy link

Artifact containing the AppImage: nextcloud-appimage-pr-9497.zip

Digest: sha256:77d60dc721236188b4b7725518ef8e4f56967bc7f569050d2576652490af3327

To test this change/fix you can download the above artifact file, unzip it, and run it.

Please make sure to quit your existing Nextcloud app and backup your data.

Added two lines after find_program(MACDEPLOYQT_EXECUTABLE ...) to compute the lib directory adjacent to the macdeployqt binary (i.e., ../lib relative to its bin/ directory), then passed it as -libpath to the macdeployqt invocation.

Root cause of the libvpx error: The craft-built libvpx.11.dylib has a bare install name (libvpx.11.dylib, no @rpath/ prefix). When macdeployqt encounters it as a dependency of ffmpeg/libavcodec and tries to locate the file, it falls back to searching /usr/lib/ — where it doesn't exist. The -libpath option adds the craft lib/ directory to macdeployqt's search path, so it finds the library at .mac-crafter/macos-clang-arm64/lib/libvpx.11.dylib.

Signed-off-by: Iva Horn <iva.horn@nextcloud.com>
@i2h3 i2h3 force-pushed the i2h3/fix/libvpx-not-found branch from 1c2a117 to b5914af Compare February 24, 2026 11:24
@i2h3 i2h3 merged commit 5d1ce2f into master Feb 24, 2026
17 of 18 checks passed
@i2h3 i2h3 deleted the i2h3/fix/libvpx-not-found branch February 24, 2026 11:29
@github-project-automation github-project-automation bot moved this from 🏗️ In progress to ☑️ Done in 💻 Desktop Clients team Feb 24, 2026
@sonarqubecloud
Copy link

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

Labels

bug os: 🍎 macOS Apple macOS, formerly also known as OS X

Projects

Status: ☑️ Done

Development

Successfully merging this pull request may close these issues.

2 participants