Skip to content

Conversation

@ericstj
Copy link
Member

@ericstj ericstj commented Oct 6, 2021

Customer Impact

Reliably detect machine architecture

Testing

Manually built and consumed changes, built installer in runtime and tested to validate behavior.

Risk

Low - we don't expect this to break anything, but it requires coordination across repositories and adds a new component to the installers including a custom action DLL which we didn't previously use.

dotnet#8000)

* Use WIX_NATIVE_MACHINE to detect native architecture of target machine

* Reference WIX_NATIVE_MACHINE property
@ericstj ericstj self-assigned this Oct 6, 2021
@ericstj ericstj added the servicing-approved Approved for servicing label Oct 7, 2021
@ericstj
Copy link
Member Author

ericstj commented Oct 7, 2021

This was approved over email. Could I have a review and merge?

@ericstj ericstj requested a review from mmitche October 7, 2021 01:40
@joeloff
Copy link
Member

joeloff commented Oct 7, 2021

Are you updating the workload task's templates separately, or is that already completed?

@ericstj
Copy link
Member Author

ericstj commented Oct 7, 2021

Can you clarify what you mean by that? The source file is shared between installers and workloads package. I spot checked the places we consume that file and made sure they pass the extension. Is there something I missed? It's quite possible since those only build in official builds of runtime and we did miss something there before, but I tried to be diligent about that.

@joeloff
Copy link
Member

joeloff commented Oct 7, 2021

Can you clarify what you mean by that? The source file is shared between installers and workloads package. I spot checked the places we consume that file and made sure they pass the extension. Is there something I missed? It's quite possible since those only build in official builds of runtime and we did miss something there before, but I tried to be diligent about that.

Sorry we're good, I forgot you linked that file into the workload task's .csproj

@ericstj
Copy link
Member Author

ericstj commented Oct 7, 2021

@mmitche @markwilkie can I get a merge on this? It was approved over email yesterday.

@mmitche mmitche merged commit e58cb82 into dotnet:release/6.0 Oct 7, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

servicing-approved Approved for servicing

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants