GenerateAssemblyInfo is not ordered before BeforeCompile

[tmat]

The common targets convention is to generate all source inputs to the compiler before BeforeCompile target.

Targets that fail to do so break Source Link (see dotnet/sourcelink#572).

[bording]

@tmat If the fix for this was merged into master, what SDK version does that mean it would arrive in? Does it need to be ported to any other branches?

[tmat]

@nguerrera

[nguerrera]

master is 5.0. If this is needed sooner than that, then this would need to be ported to 3.1.2xx to coincide with 16.5. That will require approval at this point. The 16.6 / 3.1.3xx would next opportunity, but branch does not exist yet.

I imagine we could get this approved if you want to put up a port to release/3.1.2xx

[nguerrera]

Reopening and targeting 3.1.2xx milestone for now.

[tmat]

Yeah, I can port it to release/3.1.2xx.

[nguerrera]

FYI, @wli3 Note that there is a possibility it will not be approved for 3.1.2xx, but then we can leave the PR open and retarget to 3.1.3xx when the branch opens. That is still much sooner than 5.0.1xx that master is targeting.

[tmat]

PR for 3.1.2: #10616

[rainersigwald]

#10616 was merged to 3.1.3xx.