cake-build / website

:earth_americas: The Cake website: https://cakebuild.net
https://cakebuild.net
MIT License
42 stars 230 forks source link

Multiple documents output to api/Cake.Talend.Models/Executiontime/index.html (this probably wasn't intended) #465

Open devlead opened 6 years ago

devlead commented 6 years ago

@daveaglick while monitoring build logs during, I noticed this warning, something to worry about? It doesn't fail the build, so everything seems to work, but maybe not as intended? 😉

Executing pipeline "RenderApi" (18/26) with 1 child module(s)
                Multiple documents output to api/Cake.Talend.Models/Executiontime/index.html (this probably wasn't intended):
  Cake.Talend.Models.TalendApiResponse.Executiontime
  Cake.Talend.Models.Executiontime
        Executed pipeline "RenderApi" (18/26) in 134888 ms resulting in 13474 output document(s)

Example: https://ci.appveyor.com/project/cakebuild/website/build/1.0.1395#L461

daveaglick commented 6 years ago

Interesting - what this really means is that more than one symbol was found in the compilation for an object named Cake.Talend.Models.Executiontime since that's what drives the file output. I decompiled the assembly in the package and it looks fine to me. There's only one definition and use of that object (source here: https://github.com/RadioSystems/Cake.Talend/blob/97c23ae1cc7cd6bfd6caa605fe51b92ade70b9eb/src/Cake.Talend/Models/TalendApiResponse.cs).

I'll have to dig further to see what's really going on. Guesses include Cake.Talend.Models.Executiontime being used in another package/addin or Roslyn creating duplicate symbols (which can happen with generics, though that doesn't look like the case here). Anyway, it's not a high priority and the build should be fine - I'll circle back around when I've got the time.

devlead commented 6 years ago

@daveaglick excellent 👍

pascalberger commented 4 years ago

Another addin with a similar warning:

Multiple documents output to api/Cake.Warp.dll/index.html (this probably wasn't intended):