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 ARM machine types #407

Merged
merged 1 commit into from
Jul 17, 2023
Merged

Add ARM machine types #407

merged 1 commit into from
Jul 17, 2023

Conversation

mdanish-kh
Copy link
Contributor

@mdanish-kh mdanish-kh commented Jul 11, 2023

Reference: https://learn.microsoft.com/en-us/windows/win32/debug/pe-format#machine-types

In cases where we don't get the architecture from URL, winget-create parses the portable exe for the architecture. Need to add these extra values so that we can correctly parse out ARM packages.


Microsoft Reviewers: codeflow:open?pullrequest=https://github.com/microsoft/winget-create/pull/407&drop=dogfoodAlpha

@mdanish-kh mdanish-kh requested a review from a team as a code owner July 11, 2023 19:32
@mdanish-kh mdanish-kh requested review from yao-msft and ryfu-msft and removed request for a team July 11, 2023 19:32
@ryfu-msft
Copy link
Contributor

/azp run

@azure-pipelines
Copy link

No commit pushedDate could be found for PR 407 in repo microsoft/winget-create

@mdanish-kh
Copy link
Contributor Author

mdanish-kh commented Jul 17, 2023

@ryfu-msft Similar issue we're having over at winget-pkgs with the pipelines. I found re-running the command helps but sometimes it takes 3-4 retries

@ryfu-msft
Copy link
Contributor

/azp run

@azure-pipelines
Copy link

Azure Pipelines successfully started running 1 pipeline(s).

@ryfu-msft ryfu-msft merged commit 9724c54 into microsoft:main Jul 17, 2023
4 checks passed
@mdanish-kh mdanish-kh deleted the machineTypes branch July 17, 2023 23:16
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Architecture isn't parsed correctly for portable installers in interactive update
2 participants