-
Notifications
You must be signed in to change notification settings - Fork 115
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
Nuget Warning for canopy in .NET Core 2 Project #431
Comments
Does it still work? I have googled around and after reading the first page I haven't figured out how to fix it. |
I made a new .net core app and can repro it, but im having a hard time figuring out what the do to fix it. I will try a few things |
yes, it is just a warning |
Same here with Console Application (.NET Core 2.0): |
I have made a small research and found that when canopy package is created with dotnet pack then adding package to another project shows no warnings (works correct). What I did: git clone https://github.com/lefthandedgoat/canopy.git This showed some warnings, but the package has been created: Then in my project file (MyProject.fsproj) I have added additional local nuget source section (c:\github\canopy\src\canopy\bin\Debug):
Then (still in my project): After that the package has been added to my project:
... and there were no warnings:
|
@zbigniew-gajewski Thanks for the lead. I used paket pack to make these packages and it may have caused a difference. I am on vacation now but when I get home I will see if I can make a new package like you did that does not cause the warning. Thanks again! |
This is actually a real issue. It may just be a warning but it is killing the build on my TFS build server :-( Apparently the canopy nuget file is specifying "<group targetFramework="netstandard2.0">" but it seems it should be: The latter is taken from the Selenium.WebDriver which does not exhibit the issue. Log file from the build server for Canopy: Log file from the build server for Selenium.WebDriver:
so obviously some translation between ".NETStandard2.0" to "netstandard2.0" is going on within nuget. It is likely a very simple fix. Pretty please ;-) .net core 2.0 (and 2.1) would also be nice though not at all a requirement as Selenium doesn't either. (note that I've build and tested this with .NETCore 2.0, 2.1 and .NET standard 2.0. It is all the same. |
Later today I have some free time and I will try switching to |
I think this will fix it for you. It was an epic journey to figure out the slightest thing to change. https://www.nuget.org/packages/canopy/2.1.0 This line fixed it: 9135bdd#diff-beb5bdc57f593a54f2fcbbdd7ebc489aR21 Problem was that I was making these changes, building a new package and testing locally. VS was SO NICE TO ME and kept installed old cached versions of a package that did not work. I was only able to see any changes working by updating the package number so there was no caching. Hours.... it took lol. Hope this fixes TFS for you! |
It works! Sorry that you had to go the long way around it. I too have been burned by nuget packages in the past when I created a few. it was the ".nuget" cache folder? I tested: The last one is not critical to me, as I can use .NET standard instead just fine. Just curious that it wouldn't default back to .NET standard. THANK you. |
One final note: Please be sure to use Nuget >=4.3.0 as that one will know that a .Net Core app can default back to use .Net standard packages. I think that this issue can be closed now :-) |
Use Nuget 4.3.0 client? |
Yes, on the build server I had to add a little powershell script to download Nuget 4.3.0 and use that one. On Desktop it was fine with a later version of VS 2017. |
The text was updated successfully, but these errors were encountered: