Add ASP.NET Core to https://source.dot.net

[davidfowl]

So that we can join the family of APIs on https://source.dot.net/, we should add this repository and Extensions and EF to the list.

The source is here:

• https://github.com/dotnet/source-indexer
• We need to add to this list https://github.com/dotnet/source-indexer/blob/master/src/index/repositories.props

cc @KirillOsenkov @stephentoub @natemcmaster

[stephentoub]

Thanks for following up on this.

cc: @alexperovich, @danmosemsft

[natemcmaster]

Duplicate of dotnet/source-indexer#5?

[Eilon]

@natemcmaster only a dup if you want to support project.json

[davidfowl]

FWIW it looks simple and I gave it a try but I couldn't figure out how to build just the managed projects in our stack. It doesn't seem to work anymore but I haven't dug into why.

[alexperovich]

If you don't have any weird post-compile processes that change type forwards in the assemblies then the indexer build should not actually build any of the ASP.NET projects. It only needs to do a complete package restore so the compiler can get the correct ReferencePaths to analyze the sources.

[davidfowl]

Then maybe a build.cmd /t:Restore is enough to make it work. Let me try that.

[natemcmaster]

Tangent - does source-indexer have the ability to take symbols as inputs and use sourcelink metadata to find the code?

[alexperovich]

I don't believe it does. Its all based on Roslyn analysis and hasn't changed significantly in a while. @KirillOsenkov wrote https://github.com/KirillOsenkov/SourceBrowser which powers https://source.dot.net/. He can give a more complete answer.

[davidfowl]

I think I got this working. I'll send a PR.

[KirillOsenkov]

@natemcmaster no, the way it works it needs both the full built source laid out next to obj, bin and packages on the same machine, so that Roslyn MSBuildWorkspace can open the .sln and populate projects/compilations. We don't need the source to be built strictly speaking, but we need all references (NuGet) to be in place, as well as all generated files in obj, otherwise compilation will have errors and hyperlinks won't work.

[davidfowl]

Hmm, so it does need to build?

[KirillOsenkov]

No, mostly to restore packages. It'll be good enough. Don't worry about full accuracy now.

[davidfowl]

P2Ps are currently broken. I need to figure out why.

[natemcmaster]

Does MSBuildWorkspace run the ResolveReferences target?