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

Fix/macos m2 detection bug #19369

Merged
merged 9 commits into from
Jan 26, 2024

Conversation

cwall-dev
Copy link
Contributor

Task name: UseDotNetV2

Description: Fixed os architecture detection for Apple M1/2

Documentation changes required: N

Added unit tests: N

Attached related issue: #19174

Checklist:

  • Task version was bumped - please check instruction how to do it
  • Checked that applied changes work as expected

@cwall-dev
Copy link
Contributor Author

cwall-dev commented Dec 12, 2023 via email

@merlynomsft
Copy link
Contributor

/azp run

Copy link

Azure Pipelines successfully started running 3 pipeline(s).

@merlynomsft
Copy link
Contributor

/azp run

Copy link

Azure Pipelines successfully started running 3 pipeline(s).

@merlynomsft
Copy link
Contributor

@cwall-dev Thank you for the contribution. I added some generated output needed by our integration system.

Could you share test results for x64 / ARM64? (Ideally, we'd have an integration test for this, but currently we target a x64 macos agent)

@cwall-dev
Copy link
Contributor Author

@merlynomsft I successfully ran the existing unit tests on a Macbook Air M2 2022. I also copied the build output to the current version (2.232.0) on my local agent on the same Macbook and then ran a build successfully.
The result looks like this:
image

@cwall-dev cwall-dev closed this Jan 25, 2024
@cwall-dev cwall-dev reopened this Jan 25, 2024
@merlynomsft
Copy link
Contributor

/azp run

Copy link

Azure Pipelines successfully started running 3 pipeline(s).

@merlynomsft merlynomsft merged commit 44b9b29 into microsoft:master Jan 26, 2024
8 checks passed
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.

None yet

2 participants