-
Notifications
You must be signed in to change notification settings - Fork 125
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
Are snupkg files supposed to be tiny? #234
Comments
The linked snupkg is empty, which is not right. It should contain PDB files. Yes, DebugType should be "portable", which is the default when you create a new .NET Standard or .NET Core App project. |
Is my build script wrong, or should it be working? I assume it should be working if it generates artifacts and does not fail. Perhaps you have an idea what's wrong here. https://ci.appveyor.com/project/taori/amusoft-ui-wpf/builds/21730452 is built with -p:DebugType=portable and packed with --no-build, which should override the project settings unless i'm wrong. |
Why not just set And also delete this line: and add <IncludeSymbols>true</IncludeSymbols>
<SymbolPackageFormat>snupkg</SymbolPackageFormat> instead. |
I'll try that thanks - copied those settings from newtonsoft.json. Also the reason why i set it to embedded, was because i read that it cuts down on the time it takes to load symbols. Is there even any benefit of using snupkg though versus nupkg+embedded? |
We recommend snupkg because the main library package is smaller as it does not contain PDBs. |
I see. Thanks for the clarification.
|
Yes. Yes. Yes. No.
It depends on what you mean offline. Do you mean you restore online then go offline and try to debug? If you want to allow debugging without access to the repository you can set EmbedAllSources which embeds the sources to the PDB. Then you need to have the PDB available when debugging. |
Yes - I also mean it in a wlan scenario, where the connection may be flaky or internet access is suddenly restricted. Am i getting you right then, that EmbedAllSources is a switch which shall be used for snupkg too? Because judging by documentation cs files would be a filtered extension. |
EmbedAllSources will embed the (compressed) sources into the PDB, it will not add the source files to the package directly. |
Please verify this: https://ci.appveyor.com/project/taori/amusoft-ui-wpf/builds/21730035/artifacts
the snupkg seems off? Is 2kb really correct, or is there something wrong with it? There's little documentation at the moment to see how to get a proper package out of this.
#229 makes it sound like "portable" has to be the DebugType in the csproj - is that correct or does "embedded" work as well?
The text was updated successfully, but these errors were encountered: