[msbuild] Fix FilterStaticFrameworks task to support custom framework binary names #23045

Copilot · 2025-06-13T18:10:16Z

The FilterStaticFrameworks MSBuild task was hardcoded to assume framework executables are always located at Framework.framework/Framework, but some frameworks (like FFmpeg's libavcodec) use custom executable paths specified in their Info.plist via the CFBundleExecutable property.

This caused build failures with frameworks that use custom binary names:

error MT158: The file '/path/to/libavcodec.framework/libavcodec' does not exist.

The actual executable was at libavcodec.framework/libavcodec.dylib as specified in the framework's Info.plist CFBundleExecutable property.

Solution

Added a GetFrameworkExecutablePath helper method that:

Reads Info.plist: Checks for CFBundleExecutable property in framework's Info.plist
Uses custom path: Returns the path specified by CFBundleExecutable when present
Falls back gracefully: Uses default behavior for standard frameworks or when Info.plist is missing/corrupt
Maintains compatibility: All existing frameworks continue to work unchanged

Example

Before (fails):

libavcodec.framework/
├── Info.plist  # CFBundleExecutable = "libavcodec.dylib"
└── libavcodec.dylib

Task looks for: libavcodec.framework/libavcodec ❌

After (works):
Task reads Info.plist and looks for: libavcodec.framework/libavcodec.dylib ✅

Testing

Added comprehensive tests covering:

✅ Custom CFBundleExecutable scenarios (reproduces the original issue)
✅ Standard frameworks without Info.plist (backward compatibility)
✅ Non-framework paths (edge case handling)
✅ Error handling for malformed/missing Info.plist files

Fixes #23039.

💡 You can make Copilot smarter by setting up custom instructions, customizing its development environment and configuring Model Context Protocol (MCP) servers. Learn more Copilot coding agent tips in the docs.

Co-authored-by: rolfbjarne <[email protected]>

mandel-macaque · 2025-06-13T18:32:46Z

msbuild/Xamarin.MacDev.Tasks/Tasks/FilterStaticFrameworks.cs

+				try {
+					var plist = PDictionary.FromFile (infoPlistPath);
+					var bundleExecutable = plist?.GetCFBundleExecutable ();
+					if (!string.IsNullOrEmpty (bundleExecutable)) {
+						return Path.Combine (frameworkPath, bundleExecutable);
+					}
+				} catch {
+					// If reading the plist fails, fall back to the default behavior
+				}


FromFile does not raise any exception, what exception are you trying to catch? This can me cleaner with a if statement no need for the try/catch.

You're right! I've simplified the code by removing the try-catch and using explicit null checking instead. Since GetFrameworkExecutablePath is called within a larger try-catch block that handles framework validation errors appropriately, any exceptions from PDictionary.FromFile will be caught and logged with proper error messages. The refactored code is cleaner and follows the patterns used elsewhere in the codebase. Fixed in d3f208c.

mandel-macaque · 2025-06-13T18:33:42Z

msbuild/Xamarin.MacDev.Tasks/Tasks/FilterStaticFrameworks.cs

@@ -20,6 +20,29 @@ public class FilterStaticFrameworks : XamarinTask, ITaskCallback {
 		[Output]
 		public ITaskItem []? FrameworkToPublish { get; set; }

+		static string GetFrameworkExecutablePath (string frameworkPath)
+		{
+			if (!frameworkPath.EndsWith (".framework", StringComparison.OrdinalIgnoreCase) || !Directory.Exists (frameworkPath))


Simplify the || to use an and so that it can be short circuit.

Simplified the logical condition to use && with short-circuiting instead of ||. The change is logically equivalent but more efficient - if the path doesn't end with ".framework", we skip the Directory.Exists check entirely. Fixed in 0b203a4.

Co-authored-by: mandel-macaque <[email protected]>

Copilot · 2025-06-13T18:50:43Z

@vs-mobiletools-engineering-service2 Unfortunately I hit an unexpected error while processing your comment. I've automatically reported this to GitHub.

You can ask me to try again later by mentioning me in a new comment.

If you want to contact GitHub about this error, please mention the following identifier so they can better serve you: cb9e1c42-de13-48b7-b6df-6c5a01d20da4

Sorry for the inconvenience!

Copilot · 2025-06-13T18:51:03Z

@vs-mobiletools-engineering-service2 Unfortunately I hit an unexpected error while processing your comment. I've automatically reported this to GitHub.

You can ask me to try again later by mentioning me in a new comment.

If you want to contact GitHub about this error, please mention the following identifier so they can better serve you: 17f50e8e-5c07-48bd-b0a6-bd3ed9a7a89e

Sorry for the inconvenience!

Copilot · 2025-06-13T19:02:02Z

@vs-mobiletools-engineering-service2 Unfortunately I hit an unexpected error while processing your comment. I've automatically reported this to GitHub.

You can ask me to try again later by mentioning me in a new comment.

If you want to contact GitHub about this error, please mention the following identifier so they can better serve you: 4be25d47-1581-484e-82e7-970da6b92d8e

Sorry for the inconvenience!

Copilot · 2025-06-13T19:17:11Z

@vs-mobiletools-engineering-service2 Unfortunately I hit an unexpected error while processing your comment. I've automatically reported this to GitHub.

You can ask me to try again later by mentioning me in a new comment.

If you want to contact GitHub about this error, please mention the following identifier so they can better serve you: dcf50cd4-9439-4ec9-992b-43610425540e

Sorry for the inconvenience!

Copilot · 2025-06-13T19:17:29Z

@vs-mobiletools-engineering-service2 Unfortunately I hit an unexpected error while processing your comment. I've automatically reported this to GitHub.

You can ask me to try again later by mentioning me in a new comment.

If you want to contact GitHub about this error, please mention the following identifier so they can better serve you: 6c8ca4e4-da9c-4123-b939-726c033e967b

Sorry for the inconvenience!

Copilot · 2025-06-13T19:18:50Z

@vs-mobiletools-engineering-service2 Unfortunately I hit an unexpected error while processing your comment. I've automatically reported this to GitHub.

You can ask me to try again later by mentioning me in a new comment.

If you want to contact GitHub about this error, please mention the following identifier so they can better serve you: 4c5bad3c-8c72-4003-b1a0-b90e4818cd77

Sorry for the inconvenience!

vs-mobiletools-engineering-service2 · 2025-07-10T11:31:05Z

✅ [CI Build #84b5ed9] Build passed (Build packages) ✅

Pipeline on Agent
Hash: 84b5ed9f5b0428e978348acc97dd51a94e3d6a2c [PR build]

Copilot · 2025-07-10T11:31:11Z

@vs-mobiletools-engineering-service2 Unfortunately I hit an unexpected error while processing your comment. I've automatically reported this to GitHub.

You can ask me to try again later by mentioning me in a new comment.

If you want to contact GitHub about this error, please mention the following identifier so they can better serve you: 2e123902-209d-4757-89af-a9c89a04eac6

Sorry for the inconvenience!

vs-mobiletools-engineering-service2 · 2025-07-10T11:33:01Z

✅ [PR Build #84b5ed9] Build passed (Detect API changes) ✅

Pipeline on Agent
Hash: 84b5ed9f5b0428e978348acc97dd51a94e3d6a2c [PR build]

Copilot · 2025-07-10T11:33:06Z

@vs-mobiletools-engineering-service2 Unfortunately I hit an unexpected error while processing your comment. I've automatically reported this to GitHub.

You can ask me to try again later by mentioning me in a new comment.

If you want to contact GitHub about this error, please mention the following identifier so they can better serve you: 92a83e84-6838-4e4f-a36d-e4ae0b24a32d

Sorry for the inconvenience!

Copilot · 2025-07-10T11:46:32Z

@vs-mobiletools-engineering-service2 Unfortunately I hit an unexpected error while processing your comment. I've automatically reported this to GitHub.

You can ask me to try again later by mentioning me in a new comment.

If you want to contact GitHub about this error, please mention the following identifier so they can better serve you: acf8377d-c7be-400f-a726-15997b9d3f86

Sorry for the inconvenience!

Copilot · 2025-07-10T11:49:48Z

@vs-mobiletools-engineering-service2 Unfortunately I hit an unexpected error while processing your comment. I've automatically reported this to GitHub.

You can ask me to try again later by mentioning me in a new comment.

If you want to contact GitHub about this error, please mention the following identifier so they can better serve you: 672d9dbb-662f-404d-b6ac-b473274bacaa

Sorry for the inconvenience!

vs-mobiletools-engineering-service2 · 2025-07-10T11:59:56Z

✅ API diff for current PR / commit

.NET ( No breaking changes )

iOS: vsdrops gist ( No breaking changes )
tvOS: vsdrops gist ( No breaking changes )
MacCatalyst: vsdrops gist ( No breaking changes )
macOS: vsdrops gist ( No breaking changes )

✅ API diff vs stable

.NET ( No breaking changes )

iOS: vsdrops gist ( No breaking changes )
tvOS: vsdrops gist ( No breaking changes )
MacCatalyst: vsdrops gist ( No breaking changes )
macOS: vsdrops gist ( No breaking changes )

ℹ️ Generator diff

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

Pipeline on Agent
Hash: 84b5ed9f5b0428e978348acc97dd51a94e3d6a2c [PR build]

Copilot · 2025-07-10T12:00:00Z

@vs-mobiletools-engineering-service2 Unfortunately I hit an unexpected error while processing your comment. I've automatically reported this to GitHub.

You can ask me to try again later by mentioning me in a new comment.

If you want to contact GitHub about this error, please mention the following identifier so they can better serve you: ced72b5a-9464-482b-ac32-6e926b67df85

Sorry for the inconvenience!

Copilot · 2025-07-10T12:00:08Z

@vs-mobiletools-engineering-service2 Unfortunately I hit an unexpected error while processing your comment. I've automatically reported this to GitHub.

You can ask me to try again later by mentioning me in a new comment.

If you want to contact GitHub about this error, please mention the following identifier so they can better serve you: 1f2afd39-310b-442c-9772-ff4835962172

Sorry for the inconvenience!

vs-mobiletools-engineering-service2 · 2025-07-10T12:01:28Z

✅ [CI Build #84b5ed9] Build passed (Build macOS tests) ✅

Pipeline on Agent
Hash: 84b5ed9f5b0428e978348acc97dd51a94e3d6a2c [PR build]

Copilot · 2025-07-10T12:01:33Z

@vs-mobiletools-engineering-service2 Unfortunately I hit an unexpected error while processing your comment. I've automatically reported this to GitHub.

You can ask me to try again later by mentioning me in a new comment.

If you want to contact GitHub about this error, please mention the following identifier so they can better serve you: 5b602204-b3ed-4039-a2c0-0113964dcbd8

Sorry for the inconvenience!

Copilot · 2025-07-10T12:03:04Z

@vs-mobiletools-engineering-service2 Unfortunately I hit an unexpected error while processing your comment. I've automatically reported this to GitHub.

You can ask me to try again later by mentioning me in a new comment.

If you want to contact GitHub about this error, please mention the following identifier so they can better serve you: fc20dbb1-815f-4c60-8817-37f1fa6ee4b8

Sorry for the inconvenience!

vs-mobiletools-engineering-service2 · 2025-07-10T12:04:34Z

💻 [CI Build #4a2623b] Tests on macOS arm64 - Mac Sequoia (15) passed 💻

✅ All tests on macOS arm64 - Mac Sequoia (15) passed.

Pipeline on Agent
Hash: 4a2623b5d6b1e0442202b905ce1329b65f360a3e [PR build]

Copilot · 2025-07-10T12:04:35Z

@vs-mobiletools-engineering-service2 Unfortunately I hit an unexpected error while processing your comment. I've automatically reported this to GitHub.

You can ask me to try again later by mentioning me in a new comment.

If you want to contact GitHub about this error, please mention the following identifier so they can better serve you: c28d6548-725f-4c7c-8f1c-281a2de63997

Sorry for the inconvenience!

Copilot · 2025-07-10T12:04:40Z

@vs-mobiletools-engineering-service2 Unfortunately I hit an unexpected error while processing your comment. I've automatically reported this to GitHub.

You can ask me to try again later by mentioning me in a new comment.

If you want to contact GitHub about this error, please mention the following identifier so they can better serve you: e83eeda3-fa76-461b-931e-a051be840639

Sorry for the inconvenience!

vs-mobiletools-engineering-service2 · 2025-07-10T12:12:55Z

💻 [CI Build #84b5ed9] Tests on macOS X64 - Mac Sonoma (14) passed 💻

✅ All tests on macOS X64 - Mac Sonoma (14) passed.

Pipeline on Agent
Hash: 84b5ed9f5b0428e978348acc97dd51a94e3d6a2c [PR build]

Copilot · 2025-07-10T12:13:02Z

@vs-mobiletools-engineering-service2 Unfortunately I hit an unexpected error while processing your comment. I've automatically reported this to GitHub.

You can ask me to try again later by mentioning me in a new comment.

If you want to contact GitHub about this error, please mention the following identifier so they can better serve you: 4a93a477-e2d1-461e-b996-9cccbca88a1b

Sorry for the inconvenience!

vs-mobiletools-engineering-service2 · 2025-07-10T12:17:00Z

💻 [CI Build #84b5ed9] Tests on macOS M1 - Mac Monterey (12) passed 💻

✅ All tests on macOS M1 - Mac Monterey (12) passed.

Pipeline on Agent
Hash: 84b5ed9f5b0428e978348acc97dd51a94e3d6a2c [PR build]

Copilot · 2025-07-10T12:17:05Z

@vs-mobiletools-engineering-service2 Unfortunately I hit an unexpected error while processing your comment. I've automatically reported this to GitHub.

You can ask me to try again later by mentioning me in a new comment.

If you want to contact GitHub about this error, please mention the following identifier so they can better serve you: c9597572-09f9-455c-b103-98f57c1860ea

Sorry for the inconvenience!

vs-mobiletools-engineering-service2 · 2025-07-10T12:18:26Z

💻 [CI Build #84b5ed9] Tests on macOS M1 - Mac Ventura (13) passed 💻

✅ All tests on macOS M1 - Mac Ventura (13) passed.

Pipeline on Agent
Hash: 84b5ed9f5b0428e978348acc97dd51a94e3d6a2c [PR build]

Copilot · 2025-07-10T12:18:31Z

@vs-mobiletools-engineering-service2 Unfortunately I hit an unexpected error while processing your comment. I've automatically reported this to GitHub.

You can ask me to try again later by mentioning me in a new comment.

If you want to contact GitHub about this error, please mention the following identifier so they can better serve you: 65b4d3ea-c88a-450e-b868-379b18963bc8

Sorry for the inconvenience!

Copilot · 2025-07-10T13:29:16Z

@vs-mobiletools-engineering-service2 Unfortunately I hit an unexpected error while processing your comment. I've automatically reported this to GitHub.

You can ask me to try again later by mentioning me in a new comment.

If you want to contact GitHub about this error, please mention the following identifier so they can better serve you: b6f38803-9ec7-4d99-9877-f56f1a8a1df8

Sorry for the inconvenience!

vs-mobiletools-engineering-service2 · 2025-07-10T13:47:13Z

🔥 [CI Build #84b5ed9] Test results 🔥

Test results

❌ Tests failed on VSTS: test results

0 tests crashed, 1 tests failed, 114 tests passed.

Failures

❌ monotouch tests (iOS)

1 tests failed, 7 tests passed.

monotouch-test/iOS - simulator/Debug: Crashed

Html Report (VSDrops) Download

Successes

✅ 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
✅ fsharp: All 4 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: All 44 tests passed. Html Report (VSDrops) Download
✅ monotouch (MacCatalyst): All 11 tests passed. Html Report (VSDrops) Download
✅ monotouch (macOS): All 9 tests passed. Html Report (VSDrops) Download
✅ monotouch (tvOS): All 8 tests passed. Html Report (VSDrops) Download
✅ msbuild: All 2 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

Pipeline on Agent
Hash: 84b5ed9f5b0428e978348acc97dd51a94e3d6a2c [PR build]

Copilot · 2025-07-10T13:47:20Z

@vs-mobiletools-engineering-service2 Unfortunately I hit an unexpected error while processing your comment. I've automatically reported this to GitHub.

You can ask me to try again later by mentioning me in a new comment.

If you want to contact GitHub about this error, please mention the following identifier so they can better serve you: 8fc6e30c-dcc8-4312-a1ef-2174b581ec11

Sorry for the inconvenience!

Initial plan for issue

e9253cb

Copilot AI assigned Copilot and rolfbjarne Jun 13, 2025

Copilot started work on behalf of rolfbjarne June 13, 2025 18:10 View session

Fix FilterStaticFrameworks to support custom framework binary names

0bd54f1

Co-authored-by: rolfbjarne <[email protected]>

Copilot AI changed the title ~~[WIP] FilterStaticFrameworks task doesn't support custom Framework binary names~~ Fix FilterStaticFrameworks task to support custom framework binary names Jun 13, 2025

Copilot AI requested a review from rolfbjarne June 13, 2025 18:23

Copilot finished work on behalf of rolfbjarne June 13, 2025 18:23

mandel-macaque requested changes Jun 13, 2025

View reviewed changes

Copilot started work on behalf of mandel-macaque June 13, 2025 18:32 View session

mandel-macaque requested changes Jun 13, 2025

View reviewed changes

Clean up PDictionary.FromFile usage by removing unnecessary try-catch

d3f208c

Co-authored-by: mandel-macaque <[email protected]>

Copilot finished work on behalf of mandel-macaque June 13, 2025 18:42

Copilot started work on behalf of mandel-macaque June 13, 2025 18:42 View session

Simplify logical condition to use AND with short-circuiting

0b203a4

Co-authored-by: mandel-macaque <[email protected]>

[msbuild] Fix FilterStaticFrameworks task to support custom framework binary names #23045

Are you sure you want to change the base?

[msbuild] Fix FilterStaticFrameworks task to support custom framework binary names #23045

Uh oh!

Conversation

Copilot AI commented Jun 13, 2025 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Solution

Example

Testing

Uh oh!

mandel-macaque Jun 13, 2025

Choose a reason for hiding this comment

Uh oh!

Copilot AI Jun 13, 2025

Choose a reason for hiding this comment

Uh oh!

mandel-macaque Jun 13, 2025

Choose a reason for hiding this comment

Uh oh!

Copilot AI Jun 13, 2025

Choose a reason for hiding this comment

Uh oh!

This comment has been minimized.

Copilot AI commented Jun 13, 2025

Uh oh!

This comment has been minimized.

Copilot AI commented Jun 13, 2025

Uh oh!

This comment has been minimized.

Copilot AI commented Jun 13, 2025

Uh oh!

This comment has been minimized.

Copilot AI commented Jun 13, 2025

Uh oh!

This comment has been minimized.

Copilot AI commented Jun 13, 2025

Uh oh!

This comment has been minimized.

Copilot AI commented Jun 13, 2025

Uh oh!

This comment has been minimized.

vs-mobiletools-engineering-service2 commented Jul 10, 2025

✅ [CI Build #84b5ed9] Build passed (Build packages) ✅

Uh oh!

Copilot AI commented Jul 10, 2025

Uh oh!

vs-mobiletools-engineering-service2 commented Jul 10, 2025

✅ [PR Build #84b5ed9] Build passed (Detect API changes) ✅

Uh oh!

Copilot AI commented Jul 10, 2025

Uh oh!

This comment has been minimized.

Copilot AI commented Jul 10, 2025

Uh oh!

This comment has been minimized.

Copilot AI commented Jul 10, 2025

Uh oh!

vs-mobiletools-engineering-service2 commented Jul 10, 2025

✅ API diff for current PR / commit

✅ API diff vs stable

ℹ️ Generator diff

Uh oh!

Copilot AI commented Jul 10, 2025

Uh oh!

This comment has been minimized.

Copilot AI commented Jul 10, 2025

Uh oh!

vs-mobiletools-engineering-service2 commented Jul 10, 2025

✅ [CI Build #84b5ed9] Build passed (Build macOS tests) ✅

Uh oh!

Copilot AI commented Jul 10, 2025

Uh oh!

This comment has been minimized.

Copilot AI commented Jul 10, 2025

Uh oh!

This comment has been minimized.

vs-mobiletools-engineering-service2 commented Jul 10, 2025

💻 [CI Build #4a2623b] Tests on macOS arm64 - Mac Sequoia (15) passed 💻

Uh oh!

Copilot AI commented Jun 13, 2025 •

edited

Loading