Bug 375717 - v16.12.1 beta for MS Windows: Menu "Settings/Configure/Environment/Default folders/Temporary files" default is '/tmp'
Summary: v16.12.1 beta for MS Windows: Menu "Settings/Configure/Environment/Default fo...
Status: RESOLVED FIXED
Alias: None
Product: kdenlive
Classification: Applications
Component: User Interface (show other bugs)
Version: 16.12.1
Platform: Microsoft Windows Microsoft Windows
: NOR normal
Target Milestone: ---
Assignee: Jean-Baptiste Mardelle
URL:
Keywords:
Depends on:
Blocks:
 
Reported: 2017-01-29 18:02 UTC by rbingham
Modified: 2017-04-16 04:04 UTC (History)
1 user (show)

See Also:
Latest Commit:
Version Fixed In:


Attachments
Screenshot of Configure/Default folders/Temporary files (24.92 KB, image/png)
2017-01-29 18:03 UTC, rbingham
Details

Note You need to log in before you can comment on or make changes to this bug.
Description rbingham 2017-01-29 18:02:16 UTC
See apparently related Bug 226344 for Linux environments.  For Windows this is even worse if '/tmp/' is used literally without some hidden translation magic: Kdenlive functions expecting to create or access temp files or dirs will fail. In Windows land this default should be expressed in Windows terms:

%USERPROFILE%/something where %USERPROFILE% gives the current user home dir, or

%APPDATA%/Local/Kdenlive/tmp which would give a standard practice location for a temp dir under %USERPROFILE%, etc.

See https://technet.microsoft.com/en-us/library/dd560744(v=ws.10).aspx for standard Windows env vars that can be used for constructing various reasonable default paths for Kdenlive out-of-the-box for Windows.

Note I am accepting that right-slash POSIX path notation used elsewhere in the GUI will be translated for Windows use though this will confuse Widows-only users.
Comment 1 rbingham 2017-01-29 18:03:27 UTC
Created attachment 103701 [details]
Screenshot of Configure/Default folders/Temporary files
Comment 2 Vincent PINON 2017-04-16 04:04:02 UTC
Git commit 673d0f79ed2a083dafeb74a2aefd582b31d1f3a8 by Vincent Pinon.
Committed on 16/04/2017 at 04:03.
Pushed by vpinon into branch 'Applications/17.04'.

Fix temp path on Windows

M  +4    -0    src/mainwindow.cpp

https://commits.kde.org/kdenlive/673d0f79ed2a083dafeb74a2aefd582b31d1f3a8