Enabled deterministic builds.

[azchohfi]

Fixes #3308

PR Type

What kind of change does this PR introduce?

• Build or CI related changes

What is the current behavior?

Builds are not deterministic (https://github.com/clairernovotny/DeterministicBuilds)

What is the new behavior?

Builds are deterministic.

PR Checklist

Please check if your PR fulfills the following requirements:

• Tested code with current supported SDKs
• Pull Request has been submitted to the documentation repository instructions. Link:
• Sample in sample app has been added / updated (for bug fixes / features)
  • Icon has been created (if new sample) following the Thumbnail Style Guide and templates
• Tests for the changes have been added (for bug fixes / features) (if applicable)
• Header has been added to all new source files (run build/UpdateHeaders.bat)
• Contains NO breaking changes

[ghost]

Thanks azchohfi for opening a Pull Request! The reviewers will test the PR and highlight if there is any conflict or changes required. If the PR is approved we will proceed to merge the pull request 🙌

[michael-hawker]

@azchohfi looks like this fails the CI.

[michael-hawker]

@Kyaa-dost want to just double-check you can build and run the Sample App on your machine with this change and we'll merge after?

[azchohfi]

It is still missing some green checks for the Controls, the DataGrid and the GazeInput projects. The Controls and DataGrid are known, since they are missing the pdbs for the design projects, but the GazeInput is warning about a missing pdb for a winmd. @clairernovotny do you know if this is a false positive on NuGet Package Explorer, maybe?

[clairernovotny]

Not sure how it's a false warning if the pdb isn't there? :)

It's possible that there are winmd issues and I'm happy to debug/fix those if they appear.

[azchohfi]

I didn't see the missing pdb for the winmd... I thought it was a false positive since we do have the pdbs for the runtimes:

This PDB is >10MB. Also... we are using the Win32 build of the winmd (it is generated for each arch), and that pdb is there, its just not on the same folder. What are we doing wrong here? https://github.com/windows-toolkit/WindowsCommunityToolkit/blob/26d00f24556dbd274981901b2961f08dc7b1a20e/build/Microsoft.Toolkit.Uwp.Input.GazeInteraction.nuspec#L21
I'm not sure what the right approach is here.

[clairernovotny]

Why is that generating a full pdb? If it's portable it should be a lot smaller...

[clairernovotny]

why is it being built per architecture?

[clairernovotny]

Ah, right, it's C++ :)

I would exclude the full PDB, there's nothing you can do there.

[Kyaa-dost]

Looks good @azchohfi and performing as expected 🚀🚀

CC: @michael-hawker