-
Notifications
You must be signed in to change notification settings - Fork 30
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
Can't build library #411
Comments
Does this happen when you try to build via the command line or an IDE? |
That was using an IDE. Here are the errors from building via CLI.
|
Thank you for reporting this. Out of curiosity, have you tried |
After Tyson suggested doing so, I ran the command and published the resulting errors in my previous post. |
Unless @TysonMN has a better idea, this looks like something is corrupted in the .nuget cache? 😕 |
Sure. Trying deleting Also, can you figure out what command produces the output seen in this comment from If neither of those work, you could try |
Output of
I'm on Windows 10 Pro 10.0.19044. Anything else? |
I wonder if you are referring to the version of the SDK, which is used to build the code, or the target version, which sets a lower bound on the version of the runtime. I don't think we need to change the target version. For the SDK version, sure. Please create a PR that updates this line and this line to the latest SDK. Then make any changes needed to get the code to build. However, that just changes the SDK used by the build on GitHub, so I don't think that will help you. |
@TysonMN Thank you. I understand better now. I'll do that once we get the build working on my machine. |
How many computers can you test this on? For example, do you have a computer at home and a computer at work? |
I have a computer at home, but even if it built on my home computer, I would be using and contributing to this library only from my work computer. |
Sure. I was just wondering if you could test this on another computer. |
@LyndonGingerich, if you change the targets to newer versions (like in PR elmish/Elmish.WPF#488), can you build then? |
Changing the targets worked after I also used NuGet to upgrade FSharp.Core. Many thanks! |
FWIW I got the same error ("Error opening binary file... Stream does not support writing.") when using VS 2022. This was on a freshly formatted Win10 machine. The error went away when I went down to VS 2019. This is a known issue. Edit: Oh hah, Tyson already linked that thread. |
Resolved by PR #416 |
First included in version |
I know I suggested to copy this PR from Elmish.WPF. However, what is the advantage of multi-targeting for this library? |
Not knowing which targets had active users, I simply upgraded each target to a version under support if possible and removed it otherwise. |
I am not talking about users. Projects targeting A project targeting any version of .NET (Core) can depend on a project that targets I think you were too aggressive when you replaced In conclusion, we can just target |
Can you make a PR with that simplification? |
I think we could target .NET Standard 1.6 (since we were before), but this is not recommended.
https://docs.microsoft.com/en-us/dotnet/standard/net-standard |
@TysonMN If I understand correctly, you would like all projects to target |
Only the |
I wish to learn more about the library and perhaps contribute to it, but I'm having significant difficulty getting started. I installed .NET Framework 4.5 and .NET 5.0, but I still get this stack of errors when I try to build the solution:
How can I resolve this?
Could we perchance upgrade Hedgehog to .NET 6 and .NET Framework 4.8 since .NET 5 and .NET Framework 4.5 have reach EOL?
The text was updated successfully, but these errors were encountered: