chore(macOS): Updated Xcode Project Settings#9499
Conversation
github-project-automation
bot
moved this to 🧭 Planning evaluation (don't pick)
in 💻 Desktop Clients team
i2h3
moved this from 🧭 Planning evaluation (don't pick)
to 🏗️ In progress
in 💻 Desktop Clients team
Based on the suggestions of the latest Xcode, the project settings have been updated. - Build phase "Embed App Extensions" was renamed to "Embed Foundation Extensions" - ALWAYS_EMBED_SWIFT_STANDARD_LIBRARIES has been removed - DEAD_CODE_STRIPPING was enabled in general - ENABLE_MODULE_VERIFIER has been turned on - ONLY_ACTIVE_ARCH for debug configurations Signed-off-by: Iva Horn <iva.horn@nextcloud.com>
i2h3
force-pushed
the
i2h3/chore/updated-xcode-project
branch
from
94bd199 to
198448f
Compare
There was a problem hiding this comment.
Pull request overview
This PR updates the macOS Xcode project settings for the Nextcloud desktop client's shell integration. The changes reflect an upgrade to a newer version of Xcode (from 12.4.0 to a much higher version), applying modern Xcode build settings and best practices. The update includes standardizing bundle identifier handling, enabling new compiler features, and renaming build phases to match current Xcode conventions.
Changes:
- Standardized bundle identifier handling by replacing hardcoded values with
$(PRODUCT_BUNDLE_IDENTIFIER)variable - Updated Xcode project to version 26.2.0 with modern build settings including dead code stripping, module verification, and localization analysis
- Renamed "Embed App Extensions" to "Embed Foundation Extensions" following newer Xcode conventions
Reviewed changes
Copilot reviewed 3 out of 3 changed files in this pull request and generated no comments.
| File | Description |
|---|---|
| shell_integration/MacOSX/NextcloudIntegration/desktopclient/Info.plist | Changed CFBundleIdentifier from hardcoded value to use $(PRODUCT_BUNDLE_IDENTIFIER) variable |
| shell_integration/MacOSX/NextcloudIntegration/NextcloudIntegration.xcodeproj/project.pbxproj | Updated Xcode project version, added modern build settings (DEAD_CODE_STRIPPING, ENABLE_MODULE_VERIFIER, etc.), renamed embed phase, removed deprecated ALWAYS_EMBED_SWIFT_STANDARD_LIBRARIES setting, and added PRODUCT_BUNDLE_IDENTIFIER to build configurations |
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
|
Artifact containing the AppImage: nextcloud-appimage-pr-9499.zip Digest: 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. |
|
|
/backport to stable-33.0 |
github-project-automation
bot
moved this from 🏗️ In progress
to ☑️ Done
in 💻 Desktop Clients team
Every now and then new Xcode releases suggest updates and optimizations to projects and its settings. These apparently have been overdue since a long time now.