Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Add native Arm64 support #763

Merged
merged 3 commits into from
Mar 7, 2024
Merged

Conversation

reflectronic
Copy link
Contributor

The .NET Framework versions of StructuredLogViewer and TaskRunner will run natively on Arm64. This is using the new supportedArchitectures element in the application manifest.

@reflectronic
Copy link
Contributor Author

image

@@ -0,0 +1,30 @@
<?xml version="1.0" encoding="utf-8"?>
<assembly manifestVersion="1.0" xmlns="urn:schemas-microsoft-com:asm.v1" xmlns:asmv3="urn:schemas-microsoft-com:asm.v3">
<assemblyIdentity version="1.0.0.0" name="MyApplication.app"/>
Copy link
Owner

Choose a reason for hiding this comment

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

what should the identity be? Perhaps MSBuildStructuredLogViewer.app?

Copy link
Owner

Choose a reason for hiding this comment

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

if this is intended to be the assembly name, then perhaps StructuredLogViewer.app?

Copy link
Contributor Author

@reflectronic reflectronic Mar 7, 2024

Choose a reason for hiding this comment

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

The article says:

Uniquely names the application or assembly. Use the following format for the name: Organization.Division.Name. For example Microsoft.Windows.mysampleApp. Required.

Maybe we can do something like MSBuildStructuredLog.StructuredLogViewer/MSBuildStructuredLog.TaskRunner? These names aren't used for anything, we don't use the unmanaged assembly binding, it's just for tidiness

@@ -0,0 +1,17 @@
<?xml version="1.0" encoding="utf-8"?>
<assembly manifestVersion="1.0" xmlns="urn:schemas-microsoft-com:asm.v1" xmlns:asmv3="urn:schemas-microsoft-com:asm.v3">
<assemblyIdentity version="1.0.0.0" name="MyApplication.app"/>
Copy link
Owner

Choose a reason for hiding this comment

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

Maybe TaskRunner.app?

@KirillOsenkov
Copy link
Owner

Very cool!

Does the app.manifest file need to be shipped with the app or does it get embedded directly inside the executable?

Let's clarify the assemblyIdentity and then we're good to go!

@reflectronic
Copy link
Contributor Author

The app manifest appears to be embedded into the exe, so, no complications there hopefully

@KirillOsenkov KirillOsenkov merged commit 4387719 into KirillOsenkov:main Mar 7, 2024
1 check was pending
@KirillOsenkov
Copy link
Owner

Thank you!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants