mirror of
https://github.com/NickeManarin/ScreenToGif.git
synced 2026-04-26 07:35:54 +03:00
[GH-ISSUE #454] Crash when recording in computer with different user account #374
Labels
No labels
copy cats
duplicated
future feature
pull-request
⬜ Accepted
⬜ Completed
⬜ Help Wanted 💪
⬜ In Progress
⬜ Missing Details
⬜ Pending
⬜ Waiting For Answer ⏳
🆕 feature preview
🔷 Bug 🐛
🔷 Out Of Scope
🔷 Out Of Scope
🔷 Question
🔷Enhancement
🔷Enhancement
🔷Invalid / External
🔷Knowledge Base
🔷Won't Fix
🕑 High
🕑 High
🕑 High
🕕 Medium
🕙 Low
🕛 Critical
No milestone
No project
No assignees
1 participant
Notifications
Due date
No due date set.
Dependencies
No dependencies set.
Reference
starred/ScreenToGif#374
Loading…
Add table
Add a link
Reference in a new issue
No description provided.
Delete branch "%!s()"
Deleting a branch is permanent. Although the deleted branch may continue to exist for a short time before it actually gets removed, it CANNOT be undone in most cases. Continue?
Originally created by @duarteframos on GitHub (Feb 25, 2019).
Original GitHub issue: https://github.com/NickeManarin/ScreenToGif/issues/454
This one should be easy to solve with just a new default.
I use ScreenTo Gif portably in different computers, and each has different user accounts. When recording on a new computer different from the one it was originally set up, ScreenToGif crashes with the following message.
This is because under the path set under Settings > Temporary Files > Location Folder uses an absolute path for the current user account appdata folder, which may not exist in a another computer with a different user account name.
Proposed Solution
Since environment variables are apparently already supported out of the box, I found out using
%appdata%\Local\Temp\instead of an absolute path works well.If a generic path using
%appdata%environment variable was used by default instead, it would make using ScreenToGif across different environments more robust and resilient to changes in user account names and even across different languages or operating systems that store them differently.@vatterspun commented on GitHub (Feb 26, 2019):
Portablefreeware has a workaround: under "Settings", click on "Options", then "Temporary files" and replace the path in the "Location folder" with a single period: "." Then files will be saved to the local directory.
@duarteframos commented on GitHub (Feb 26, 2019):
That work well too.
Only downside is that if you are running from a slow thumbdrive recording may lag and editing may likely also be slower.
On the plus side it will be stealthier, since temporary files won't be left behind in the host computer.
@vatterspun commented on GitHub (Feb 27, 2019):
Agreed.
I would say that even fairly cheap thumbdrives are by and large pretty darn fast nowadays, especially with bi-directional transfers in USB 3.0, but I haven't benchmarked anything on that. There might be a bottleneck on larger recordings as the images that ScreenToGif generates aren't pre-compressed unlike most video streams.
@NickeManarin commented on GitHub (Mar 9, 2019):
The problem starts with the first run of ScreenToGif.
Since there's probably no temporary folder set, the app uses this path:
C:\Users\nicke\AppData\Local\TempBut %appdata% is translated to:
C:\Users\nicke\AppData\RoamingHowever %temp% translates to:
C:\Users\nicke\AppData\Local\TempSo, I when there's no folder set as the location for the frames, %temp% will be used.
(the support for environment variables in that property needed to be manually implemented)