Skip to content

fix(macOS): Defined PRODUCT_BUNDLE_IDENTIFIER for FinderSync#9498

Merged
i2h3 merged 1 commit intomasterfrom
i2h3/fix/missing-product-bundle-identifier
Feb 24, 2026
Merged

fix(macOS): Defined PRODUCT_BUNDLE_IDENTIFIER for FinderSync#9498
i2h3 merged 1 commit intomasterfrom
i2h3/fix/missing-product-bundle-identifier

Conversation

@i2h3
Copy link
Collaborator

@i2h3 i2h3 commented Feb 24, 2026

The FinderSync extension target was missing the PRODUCT_BUNDLE_IDENTIFIER build setting which resulted in compiler warnings in Xcode. That build setting is defined like for the other extensions.

The FinderSync extension target was missing the PRODUCT_BUNDLE_IDENTIFIER build setting which resulted in compiler warnings in Xcode. That build setting is defined like for the other extensions.

Signed-off-by: Iva Horn <iva.horn@nextcloud.com>
@i2h3 i2h3 added this to the 33.0.0 milestone Feb 24, 2026
@i2h3 i2h3 self-assigned this Feb 24, 2026
Copilot AI review requested due to automatic review settings February 24, 2026 09:50
@i2h3 i2h3 added the bug label Feb 24, 2026
@i2h3 i2h3 requested a review from Aiiaiiio as a code owner February 24, 2026 09:50
@i2h3 i2h3 added the os: 🍎 macOS Apple macOS, formerly also known as OS X label 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
Copy link
Collaborator Author

i2h3 commented Feb 24, 2026

/backport to stable-33.0

@i2h3 i2h3 moved this from 🧭 Planning evaluation (don't pick) to 🏗️ In progress in 💻 Desktop Clients team 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 fixes a missing PRODUCT_BUNDLE_IDENTIFIER build setting for the FinderSync extension target in the Xcode project, which was causing compiler warnings. The fix adds the setting to both Debug and Release build configurations using the same pattern found in the extension's Info.plist file.

Changes:

  • Added PRODUCT_BUNDLE_IDENTIFIER build setting to FinderSyncExt Debug configuration
  • Added PRODUCT_BUNDLE_IDENTIFIER build setting to FinderSyncExt Release configuration

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

@github-actions
Copy link

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

Digest: sha256:44e72932a86986150f3e8e2bfafb94e29c548c057512576bd43f6c99c14c14ef

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.

@sonarqubecloud
Copy link

@i2h3 i2h3 merged commit 58223bf into master Feb 24, 2026
27 of 28 checks passed
@i2h3 i2h3 deleted the i2h3/fix/missing-product-bundle-identifier branch February 24, 2026 11:27
@github-project-automation github-project-automation bot moved this from 🏗️ In progress to ☑️ Done in 💻 Desktop Clients team Feb 24, 2026
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