Bug: Publish optimization uses a hardcoded runtime linux-x64

LambdaDotNetCLIWrapper.cs has the following optimization, which is supposed to make the resulting zip smaller:

                // If you set the runtime linux-x64 it will trim out the Windows and Mac OS specific dependencies but Razor view precompilation
                // will not run. So only do this packaging optimization if there are no Razor views.
                if (Directory.GetFiles(fullProjectLocation, "*.cshtml", SearchOption.AllDirectories).Length == 0)
                {
                    arguments.Append(" -r linux-x64 --self-contained false /p:PreserveCompilationContext=false");
                }

However, if one has a dependency on some native libraries using nuget packages and such a dependency package has a bunch of monikers and not only linux or linux-x64, but e.g. rhel-x64, the resulting output is unusable, since the publish result will contain a wrong library. An example of such a package would be LibGit2Sharp.NativeBinaries:

Temporary workaround:

~Add an empty .cshtml file and a dependency to Microsoft.AspNetCore.Razor so the if in the optimization above is ignored.~

[normj]

Interesting. I’ll take a look to see how we can better handle this.

[normj]

Nothing wrong with the PR. I have the PR staged with the next release. I’m going to close this and use the https://github.com/aws/aws-extensions-for-dotnet-cli/pull/29 to track it.