Xamarin-macios: [msbuild] warning MSB9004: ManifestResourceWithNoCulture item type is deprecated.

[ajax16384]

Steps to Reproduce

1. Create new project "Bindings Library (iOS)"
2. Add Native Reference to some ios framework (e.g. "Test.framework")
3. Try build

Expected Behavior

Build with no warns

Actual Behavior

Build with warning:
[skip]Microsoft.Common.CurrentVersion.targets(3214,5): warning MSB9004: ManifestResourceWithNoCulture item type is deprecated. Emit EmbeddedResource items instead, with metadata WithCulture='false', Type='Resx', and optional LogicalName.

Possible reason

msbuild file Xamarin.iOS.ObjCBinding.CSharp.targets ( https://github.com/xamarin/xamarin-macios/blob/master/msbuild/Xamarin.iOS.Tasks.Core/Xamarin.iOS.ObjCBinding.CSharp.targets ) contains rule for PrepareNativeReferences task which use deprecated syntax

Environment

=== Visual Studio Community 2017 for Mac ===

Version 7.4.2 (build 12)
Installation UUID: e6bf2be1-b435-4d47-b39c-e45bc71bf6c3
Runtime:
    Mono 5.8.1.0 (2017-10/6bf3922f3fd) (64-bit)
    GTK+ 2.24.23 (Raleigh theme)

    Package version: 508010000

=== NuGet ===

Version: 4.3.1.4445

=== .NET Core ===

Runtime: /usr/local/share/dotnet/dotnet
Runtime Version: 2.0.5
SDK: /usr/local/share/dotnet/sdk/2.1.4/Sdks
SDK Version: 2.1.4
MSBuild SDKs: /Library/Frameworks/Mono.framework/Versions/5.8.1/lib/mono/msbuild/15.0/bin/Sdks

=== Xamarin.Profiler ===

Version: 1.6.1
Location: /Applications/Xamarin Profiler.app/Contents/MacOS/Xamarin Profiler

=== Xamarin.Android ===

Not Installed

=== Apple Developer Tools ===

Xcode 9.3 (14154)
Build 9E145

=== Xamarin Inspector ===

Version: 1.4.0
Hash: b3f92f9
Branch: master
Build date: Fri, 19 Jan 2018 22:00:34 GMT
Client compatibility: 1

=== Xamarin.Mac ===

Version: 4.2.1.28 (Visual Studio Community)

=== Xamarin.iOS ===

Version: 11.9.1.24 (Visual Studio Community)
Hash: f62de472
Branch: xcode9.3
Build date: 2018-03-29 19:30:53-0400

=== Build Information ===

Release ID: 704020012
Git revision: 0d8e3f0a4d683771f17959739956fa09c7ba21e3
Build date: 2018-03-30 10:45:17-04
Xamarin addins: 958839ea56ab1e331caf7c92b6ad50fb9e6ee9d2
Build lane: monodevelop-lion-d15-6

=== Operating System ===

Mac OS X 10.13.4
Darwin 17.5.0 Darwin Kernel Version 17.5.0
    Mon Mar  5 22:24:32 PST 2018
    root:xnu-4570.51.1~1/RELEASE_X86_64 x86_64

[VincentDondain]

Bonjour,

I could confirm the warning with the following environment (all stable): https://gist.github.com/VincentDondain/57b9b3f47019b174ee6d6f5955761097

Same warning happened with XI 11.6.1.4

Here's a simple test case: https://www.dropbox.com/s/t4dmc4kcj0f0ri4/BDLib.zip?dl=0

[ajax16384]

to clarify: there is not build error (framework successfully links with project), just want to get warning free build

[spouliot]

@ajax16384 noted, it's why it was marked as an enhancement (and not a bug)

[chamons]

This also occurs with Xamarin.Mac binding projects.

[chamons]

I believe something like this patch should work: https://git.io/fxvuG
With a test like this: https://git.io/fxvuW

_However_ the fix does not work with xbuild, which we currently use with our tests, so if I fix this warning, I break our tests. So it's blocked until https://github.com/xamarin/xamarin-macios/pull/4111 ever lands.

[viralanomaly]

It appears that #4111 is in (unless I'm misreading something). Is there any chance of this ticket proceeding?

[chamons]

Yes, and PRs are welcome if you'd like to contribute one. :)

One reason I haven't worked on this yet is that I'm doing https://github.com/xamarin/xamarin-macios/pull/5167 which adds and option that changes how this is done completely.

[awattar]

Any update?

[chamons]

Given the fact that this issue is still open - no.

[stonedreamforest]

+1

[chamons]

So for future me or anyone else looking at this:

We set these in here.

https://github.com/xamarin/xamarin-macios/blob/master/msbuild/Xamarin.iOS.Tasks.Core/Xamarin.iOS.ObjCBinding.CSharp.targets#L171

The error message suggests we could change them to Embedded directly:

C:\Program Files (x86)\Microsoft Visual Studio\2019\Community\MSBuild\Current\Bin\Microsoft.Common.CurrentVersion.targets(3280,5): warning MSB9004: ManifestResourceWithNoCulture item type is deprecated. Emit EmbeddedResource items instead, with metadata WithCulture='false', Type='Resx', and optional LogicalName. [C:\Users\chhamo\source\repos\iOS-Binding\iOS-Binding\iOS-Binding.csproj]

However, that did not work (we stopped embedding the lib). Any change here will require more understanding that a brief once over (and making sure we don't regress bindings.

[GrigoriiVakhmistrov]

+1

[SanduEmr]

+1

[ibocon]

+1

[ricardocolombo]

+1