Add a workaround for #176 by mattleibow · Pull Request #177 · novotnyllc/MSBuildSdkExtras (original) (raw)

@mattleibow

@mattleibow

The @(NativeReference) is used for other things in .NET, so make sure to remove the incorrect file.

novotnyllc#176

@clairernovotny

Happy to merge, but any chance you can add a test case/repro to the test projects?

mattleibow

Choose a reason for hiding this comment

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

I updated this to exclude the binding projects test because @(NativeReference) is also valid in non-binding projects.

@clairernovotny

mattleibow

Choose a reason for hiding this comment

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

The actual contents of the file is unimportant because it is only used at app link-time. Also, in order for this test to pass/fail, you need to pack. The issue does not arise in build because iOS never references the "generated" file. But, the pack target does when it tries to pack it.

@mattleibow

I think we are good to go!

This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.Learn more about bidirectional Unicode characters

[ Show hidden characters]({{ revealButtonHref }})