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 it work on .NET 5.x as well #1057

Closed
thany opened this issue Jan 13, 2022 · 10 comments
Closed

Make it work on .NET 5.x as well #1057

thany opened this issue Jan 13, 2022 · 10 comments
Assignees
Labels
🔷 Bug 🐛 Type • An undesired situation happens. ⬜ Completed Status • Issue solved.

Comments

@thany
Copy link

thany commented Jan 13, 2022

Describe the bug
I'm not sure requiring .NET 6 is a good idea for me, as my pc is a development pc for (among others) .NET applications. I don't do .NET development myself, but I do have to compile and run stuff. Long story short, I don't want to bollocks up my installation.

On top of that, it doesn't feel right to be installing a .NET version, esp one that doesn't come with Windows, for one program.

To Reproduce
Steps to reproduce the behavior:

  1. Download the latest version
  2. Run it

Expected behavior
Works on .NET 5.x just like the previous(-ish) versions.

Screenshots
If applicable, add screenshots to help explain your problem.

Desktop (please complete the following information):

  • OS: Windows 10 21H2
  • Version: 2.35.4
@thany thany added ⬜ Pending Status • This issue stills needs to be analyzed. 🔷 Bug 🐛 Type • An undesired situation happens. labels Jan 13, 2022
@thany
Copy link
Author

thany commented Jan 13, 2022

For those encountering this issue: Latest working version for me is 2.34.1. Reverting to that for the time being. Improvements on newer version don't seem to be super important on the surface (no offense) so this will work for me until hopefully this issue can be resolved.

@NickeManarin
Copy link
Owner

If you don't want to install .NET 6 Desktop Runtime, download the Microsoft Store version, it doesn't need a .NET 6 install.

But, installing .NET 6 desktop runtime should be fine and quick.
I'm thinking about alternatives for true portable versions, but it will end up increase the file size a lot.

@thany
Copy link
Author

thany commented Jan 17, 2022

If the Store version doesn't require .NET 6, why does the portable version? 🤨

I prefer not to install anything from the Store, unless there's no other option and I cannot live without the software in question. Store problems are hopefully common knowledge...

We can also look at this from a different angle: why does it need .NET 6 in the first place? What was wrong with or lacking in .NET 5?

@NickeManarin
Copy link
Owner

Well, it still requires .NET 6, it just comes along with the downloaded files.
But with the Store version, you only download what's needed.

As for the installer and portable versions, I would need to always ship the 120MB file everytime an update was released.

Btw, I can publish in two ways the software: as a single .exe file (ScreenToGif.exe with all .NET 6 files inside) or as separated files (ScreenToGif.exe + 100s of .NET 6 files).

@thany
Copy link
Author

thany commented Jan 18, 2022

Alright, well that explain why. But you skipped my last question.

@UnicodeTreason
Copy link

As someone who just runs this as a portable .exe off a USB drive the sudden requirement to install .NET v6 was a surprise.
I will remain using v2.34.1 for now.

Use Case if curious, very handy to quickly record a screen when replicating an odd issue on a random PC.

@ElMay0r
Copy link

ElMay0r commented Jan 21, 2022

I have the same issue with last update on Win 11

@thany
Copy link
Author

thany commented Jan 23, 2022

We can also look at this from a different angle: why does it need .NET 6 in the first place? What was wrong with or lacking in .NET 5?

Can we have an answer to this please?

Repository owner locked as spam and limited conversation to collaborators Jan 23, 2022
@NickeManarin
Copy link
Owner

ScreenToGif never had a .NET 5 requirement and .NET 6 is LTS.

@NickeManarin NickeManarin added ⬜ Completed Status • Issue solved. and removed ⬜ Pending Status • This issue stills needs to be analyzed. labels Jan 31, 2022
@NickeManarin
Copy link
Owner

Next version will be distributed without the requirement for installing .NET 6 desktop runtime (except if you download the Light package).

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
🔷 Bug 🐛 Type • An undesired situation happens. ⬜ Completed Status • Issue solved.
Projects
None yet
Development

No branches or pull requests

4 participants