Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Make (or use) .NET Framework ref assemblies as a nuget package #399

Closed
natemcmaster opened this issue Sep 12, 2017 · 21 comments
Closed

Make (or use) .NET Framework ref assemblies as a nuget package #399

natemcmaster opened this issue Sep 12, 2017 · 21 comments
Assignees
Labels
arcade cost: S Will take up to 2 days to complete PRI: 2 - Preferred Preferably should be handled during the milestone.

Comments

@natemcmaster
Copy link
Contributor

Reduce the number of external things that have to be installed in order to compile.
Also, it will reduce the things that are downloaded on Linux builds where .NET Framework isn't actually used.

@davidfowl
Copy link
Member

davidfowl commented Sep 12, 2017

Is this something we'd really want to own? There's an issue for this on corefx.

@natemcmaster
Copy link
Contributor Author

This would be just for us. We've asked netfx to make these a few times, but after a few years of waiting, I think it's safe to say it's not coming.

@natemcmaster
Copy link
Contributor Author

We may be able to punt this: dotnet/sdk#335

@Eilon
Copy link
Member

Eilon commented Sep 14, 2017

Previous related issue: aspnet/KoreBuild#211

@Eilon Eilon added this to the 2.1.0 milestone Sep 14, 2017
@natemcmaster
Copy link
Contributor Author

Note to a potential future implementer: in linux, we acquire netfx assemblies with the "get-netfx.sh" script. If we move to nuget, that piece of the bootstrapper can be deleted.

@dasMulli
Copy link

@natemcmaster
Copy link
Contributor Author

That is one possibility. We're looking into it.

@natemcmaster
Copy link
Contributor Author

natemcmaster commented Dec 6, 2017

@hubuk
Copy link

hubuk commented Dec 6, 2017

@dasMulli
Microsoft.TargetingPack.NETFramework.v* packages lack some important directories described in this issue: https://github.com/dotnet/corefx/issues/25012
With this change they could be a replacement for https://aspnetcore.blob.core.windows.net/buildtools/netfx/4.6.1/netfx.4.6.1.tar.gz used currently.

@natemcmaster natemcmaster changed the title Make .NET Framework ref assemblies a nuget package Make (or use) .NET Framework ref assemblies as a nuget package Dec 6, 2017
@natemcmaster
Copy link
Contributor Author

Good to know. Thanks for the heads up @hubuk .

@natemcmaster
Copy link
Contributor Author

@tmat do you happen to know if roslyn's reference assembly nuget packages are (1) open source and (2) reference-able as a PackageReference?

@tmat
Copy link

tmat commented Feb 7, 2018

@natemcmaster
Copy link
Contributor Author

Thanks @tmat. That package looks almost identical to a package we made too: https://dotnet.myget.org/feed/aspnetcore-tools/package/nuget/NETFrameworkReferenceAssemblies. I don't have any plans to work on this at the moment, but ideally, we'd love to be able to do this.

  <ItemGroup>
    <PackageReference Include="NETFramework.TargetingPack" Version="4.6.1" />
  </ItemGroup>

@tmat
Copy link

tmat commented Feb 7, 2018

I believe there are work items on the dotnet SDK team to make something like that happen. @nguerrera

@nguerrera
Copy link

nguerrera commented Feb 7, 2018

@dsplaisted had a proposal, but I think this falls on corefx. cc @Petermarcu

@natemcmaster
Copy link
Contributor Author

I've been watching dotnet/sdk#335. Is the best one to use to keep track of progress on this?

@nguerrera
Copy link

The corefx team has unofficial targeting pack nupkgs. I think they need to make them official and then we can build support for using them into the SDK. cc @weshaggard.

@weshaggard
Copy link

weshaggard commented Feb 7, 2018

I believe @terrajobst owns writing up a design. However at this point there are no concrete plans around doing this work.

cc @richlander

@natemcmaster
Copy link
Contributor Author

Closing in favor of dotnet/designs#33 and dotnet/sdk#335

@natemcmaster
Copy link
Contributor Author

Changed our minds again. Let's use the ref packs the roslyn team made and put on https://dotnet.myget.org/F/roslyn-tools/api/v3/index.json. cref dotnet/roslyn-tools#190.

@natemcmaster natemcmaster reopened this Aug 9, 2018
@natemcmaster natemcmaster added this to the 2.2.0-preview2 milestone Aug 9, 2018
@natemcmaster natemcmaster added cost: S Will take up to 2 days to complete PRI: 2 - Preferred Preferably should be handled during the milestone. labels Aug 9, 2018
@Eilon Eilon modified the milestones: 2.2.0-preview2, 2.2.0-preview3 Sep 10, 2018
@Eilon Eilon modified the milestones: 2.2.0-preview3, 2.2.0 Oct 18, 2018
@natemcmaster natemcmaster modified the milestones: 2.2.0, 3.0.0-preview3 Jan 25, 2019
@natemcmaster
Copy link
Contributor Author

This was completed in dotnet/aspnetcore#6995

@natemcmaster natemcmaster self-assigned this Apr 30, 2019
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
arcade cost: S Will take up to 2 days to complete PRI: 2 - Preferred Preferably should be handled during the milestone.
Projects
None yet
Development

No branches or pull requests

8 participants