From 00090f31822d2f25ab69dcffd62c87474db0e348 Mon Sep 17 00:00:00 2001
From: Ren Tatsumoto <tatsu@autistici.org>
Date: Thu, 27 Aug 2020 10:12:30 +0300
Subject: [PATCH] audio clips and video clips have their own locations

---
 README.md     |  6 ++++--
 videoclip.lua | 16 ++++++++++------
 2 files changed, 14 insertions(+), 8 deletions(-)

diff --git a/README.md b/README.md
index 696acde..7217038 100644
--- a/README.md
+++ b/README.md
@@ -35,12 +35,14 @@ $ cd ~/.config/mpv/scripts/videoclip && git pull
 Configuration file is located at ```~/.config/mpv/script-opts/videoclip.conf```
 and should be created by the user. If a parameter is not specified
 in the config file, the default value will be used.
+mpv doesn't tolerate spaces before and after `=`.
 
 Example configuration file:
 ```
-# Absolute path to the folder where generated clips will be placed.
+# Absolute paths to the folders where generated clips will be placed.
 # `~` or `$HOME` are not supported due to mpv limitations.
-media_path=/home/user/Videos
+video_folder_path=/home/user/Videos
+audio_folder_path=/home/user/Music
 
 # Menu size
 font_size=20
diff --git a/videoclip.lua b/videoclip.lua
index 6698af3..272c79c 100644
--- a/videoclip.lua
+++ b/videoclip.lua
@@ -4,9 +4,10 @@ local mpopt = require('mp.options')
 -- Options can be changed here or in a separate config file.
 -- Config path: ~/.config/mpv/script-opts/videoclip.conf
 local config = {
-    -- absolute path
+    -- absolute paths
 	-- relative paths (e.g. ~ for home dir) do NOT work.
-    media_path = string.format('%s/Videos/', os.getenv("HOME")),
+    video_folder_path = string.format('%s/Videos/', os.getenv("HOME")),
+    audio_folder_path = string.format('%s/Music/',  os.getenv("HOME")),
 
     font_size = 24,
 
@@ -124,7 +125,7 @@ ffmpeg.execute = function(args)
 end
 
 ffmpeg.create_videoclip = function(clip_filename, video_path, track_number)
-    local clip_path = add_extension(config.media_path .. clip_filename, '.mp4')
+    local clip_path = add_extension(config.video_folder_path .. clip_filename, '.mp4')
     return ffmpeg.execute{
         '-ss', tostring(menu.timings['start']),
         '-to', tostring(menu.timings['end']),
@@ -147,7 +148,7 @@ ffmpeg.create_videoclip = function(clip_filename, video_path, track_number)
 end
 
 ffmpeg.create_audioclip = function(clip_filename, video_path, track_number)
-    local clip_path = add_extension(config.media_path .. clip_filename, '.ogg')
+    local clip_path = add_extension(config.audio_folder_path .. clip_filename, '.ogg')
 
     return ffmpeg.execute{
         '-vn',
@@ -181,17 +182,20 @@ ffmpeg.create_clip = function(clip_type)
     mp.osd_message("Please wait...", 9999)
 
     local ret
+    local location
 
     if clip_type == 'video' then
+        location = config.video_folder_path
         ret = ffmpeg.create_videoclip(clip_filename, video_path, track_number)
     elseif clip_type == 'audio' then
+        location = config.audio_folder_path
         ret = ffmpeg.create_audioclip(clip_filename, video_path, track_number)
     end
 
     if ret.status == 0 then
-        mp.osd_message(string.format("Clip saved to %s.", config.media_path), 2)
+        mp.osd_message(string.format("Clip saved to %s.", location), 2)
     else
-        mp.osd_message("Error: couldn't create the clip.", 2)
+        mp.osd_message(string.format("Error: couldn't create the clip.\nDoes %s exist?", location), 5)
     end
 end