Skip to content

Commit ed533be

Browse files
committed
work
1 parent bf95f5a commit ed533be

File tree

1 file changed

+3
-2
lines changed

1 file changed

+3
-2
lines changed

designs/NuGet-CPPCLI-Support.md

Lines changed: 3 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -72,9 +72,9 @@ To understand the proposed changes, here's a mapping of all `C++` project types
7272
In the background, it is said that C++/CLI projects are supposed to support installing both managed and native packages. This would require some amendments to NuGet's framework model.
7373

7474
`native` framework is not compatible with anything but `native` and that will remain.
75-
NuGet also has a fallback one.
75+
NuGet has a fallback compatibility mode as well.
7676
For example: .NET Core projects have a fallback to .NET Framework with a warning. This is also callled `Asset Target Fallback`.
77-
If a package supports .NET Core or .NET Standard, it's fully compatible with a .NET Core project. Otherwise if a package has .NET Framework assets, the package is installed, but with a warning.
77+
If a package supports .NET Core or .NET Standard, it's fully compatible with a .NET Core project. Otherwise if a package has .NET Framework assets, the package is installed, but with a *warning*.
7878

7979
The `Asset Target Fallback` implementation currently suffers from a bug where the dependencies are not pulled in correctly, see [5957](https://github.com/NuGet/Home/issues/5957).
8080

@@ -83,6 +83,7 @@ Whatever the effective frameworks turn out to be, there are 2 approaches that we
8383
* Utilize AssetTargetFallback
8484

8585
This would require project type changes. The project type would specify which target framework to fall back to. This likely requires fixing [5957](https://github.com/NuGet/Home/issues/5957).
86+
Note that this would *generate* a warning when managed packages are used.
8687

8788
* Introduce a `dedicated dual compatibility` framework
8889

0 commit comments

Comments
 (0)