Skip to content

[msbuild] Only run _CreateInstaller target for desktop platforms. Fixes #19790#25825

Draft
rolfbjarne wants to merge 1 commit into
mainfrom
rolfbjarne-fix-createpackage-ios-pkg
Draft

[msbuild] Only run _CreateInstaller target for desktop platforms. Fixes #19790#25825
rolfbjarne wants to merge 1 commit into
mainfrom
rolfbjarne-fix-createpackage-ios-pkg

Conversation

@rolfbjarne

Copy link
Copy Markdown
Member

The _CreateInstaller target creates a .pkg installer package, which is only
meaningful for desktop platforms (macOS and Mac Catalyst). Previously, setting
CreatePackage=true for an iOS or tvOS project would attempt to create a .pkg,
which is useless and fails if /usr/bin/productbuild is not available.

Add a $(SdkIsDesktop) == 'true' condition to the _CreateInstaller target so
it only executes for macOS and Mac Catalyst projects.

Fixes #19790

🤖 Pull request created by Copilot

The _CreateInstaller target creates a .pkg installer, which is only useful for desktop platforms (macOS and Mac Catalyst). Previously, setting CreatePackage=true for an iOS or tvOS project would attempt to create a .pkg, which is useless and would fail if /usr/bin/productbuild is not available.

Add a 'SdkIsDesktop' == 'true' condition to the target so it only runs for macOS and Mac Catalyst projects.

Fixes #19790

Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
@vs-mobiletools-engineering-service2

This comment has been minimized.

@vs-mobiletools-engineering-service2

This comment has been minimized.

@vs-mobiletools-engineering-service2

This comment has been minimized.

@vs-mobiletools-engineering-service2

Copy link
Copy Markdown
Collaborator

✅ [CI Build #8281934] Build passed (Build packages) ✅

Pipeline on Agent
Hash: 828193401623af0e0ad200f7b7dd655fa7082bc2 [PR build]

@vs-mobiletools-engineering-service2

This comment has been minimized.

@vs-mobiletools-engineering-service2

Copy link
Copy Markdown
Collaborator

✅ [CI Build #8281934] Build passed (Detect API changes) ✅

Pipeline on Agent
Hash: 828193401623af0e0ad200f7b7dd655fa7082bc2 [PR build]

@vs-mobiletools-engineering-service2

This comment has been minimized.

@vs-mobiletools-engineering-service2

Copy link
Copy Markdown
Collaborator

✅ API diff for current PR / commit

NET (empty diffs)

✅ API diff vs stable

NET (empty diffs)

ℹ️ Generator diff

Generator Diff: vsdrops (html) vsdrops (raw diff) gist (raw diff) - Please review changes)

Pipeline on Agent
Hash: 828193401623af0e0ad200f7b7dd655fa7082bc2 [PR build]

@vs-mobiletools-engineering-service2

Copy link
Copy Markdown
Collaborator

✅ [CI Build #8281934] Build passed (Build macOS tests) ✅

Pipeline on Agent
Hash: 828193401623af0e0ad200f7b7dd655fa7082bc2 [PR build]

@vs-mobiletools-engineering-service2

Copy link
Copy Markdown
Collaborator

🔥 [CI Build #8281934] Test results 🔥

Test results

❌ Tests failed on VSTS: test results

0 tests crashed, 2 tests failed, 205 tests passed.

Failures

❌ fsharp tests

2 tests failed, 2 tests passed.

Failed tests

  • fsharp/iOS - simulator/Debug: LaunchTimedOut
  • fsharp/tvOS - simulator/Debug: LaunchTimedOut

Html Report (VSDrops) Download

Successes

✅ assembly-processing: All 1 tests passed. Html Report (VSDrops) Download
✅ cecil: All 1 tests passed. Html Report (VSDrops) Download
✅ dotnettests (iOS): All 1 tests passed. Html Report (VSDrops) Download
✅ dotnettests (MacCatalyst): All 1 tests passed. Html Report (VSDrops) Download
✅ dotnettests (macOS): All 1 tests passed. Html Report (VSDrops) Download
✅ dotnettests (Multiple platforms): All 1 tests passed. Html Report (VSDrops) Download
✅ dotnettests (tvOS): All 1 tests passed. Html Report (VSDrops) Download
✅ framework: All 2 tests passed. Html Report (VSDrops) Download
✅ generator: All 5 tests passed. Html Report (VSDrops) Download
✅ interdependent-binding-projects: All 4 tests passed. Html Report (VSDrops) Download
✅ introspection: All 4 tests passed. Html Report (VSDrops) Download
✅ linker (iOS): All 15 tests passed. Html Report (VSDrops) Download
✅ linker (MacCatalyst): All 15 tests passed. Html Report (VSDrops) Download
✅ linker (macOS): All 21 tests passed. Html Report (VSDrops) Download
✅ linker (tvOS): All 15 tests passed. Html Report (VSDrops) Download
✅ monotouch (iOS): All 20 tests passed. Html Report (VSDrops) Download
✅ monotouch (MacCatalyst): All 19 tests passed. Html Report (VSDrops) Download
✅ monotouch (macOS): All 20 tests passed. Html Report (VSDrops) Download
✅ monotouch (tvOS): All 20 tests passed. Html Report (VSDrops) Download
✅ msbuild: All 2 tests passed. Html Report (VSDrops) Download
✅ sharpie: All 1 tests passed. Html Report (VSDrops) Download
✅ windows: All 3 tests passed. Html Report (VSDrops) Download
✅ xcframework: All 4 tests passed. Html Report (VSDrops) Download
✅ xtro: All 1 tests passed. Html Report (VSDrops) Download

macOS tests

✅ Tests on macOS Monterey (12): All 5 tests passed. Html Report (VSDrops) Download
✅ Tests on macOS Ventura (13): All 5 tests passed. Html Report (VSDrops) Download
✅ Tests on macOS Sonoma (14): All 5 tests passed. Html Report (VSDrops) Download
✅ Tests on macOS Sequoia (15): All 5 tests passed. Html Report (VSDrops) Download
✅ Tests on macOS Tahoe (26): All 5 tests passed. Html Report (VSDrops) Download

Linux Build Verification

Linux build succeeded

Pipeline on Agent
Hash: 828193401623af0e0ad200f7b7dd655fa7082bc2 [PR build]

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.

Setting 'CreatePackage=true' for an iOS build creates a pkg

2 participants