<div dir="auto"><div>I did submit a patch earlier which tried to resolve the same problem but inside the vlc core to which remi said it should be handled by the caller to vlc_cachedir(). Hence, I made the necessary changes and shifted it to VLSub. </div><div dir="auto">I think it should be handled by the core itself because what use is a cache dir if it doesnt already exist.</div><div dir="auto"><br></div><div dir="auto">The two loops can be factored into one. Once the above discussion is done i will make the necessary changes and send in another patch to solve this problem.</div><div dir="auto">Regards </div><div dir="auto">Blurrrb</div><div dir="auto"><br><div class="gmail_quote" dir="auto"><div dir="ltr" class="gmail_attr">On Mon, 25 Mar 2019, 23:15 Aakash Singh, <<a href="mailto:17aakashsingh1999@gmail.com">17aakashsingh1999@gmail.com</a>> wrote:<br></div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div dir="ltr"><div dir="ltr"><div>Hey everyone</div><div>Someone please review this patch. It was submitted a week ago and hasn't been reviewed yet.<br></div><div><br></div><div>Regards <br></div><div>Aakash Singh<br></div></div><br><div class="gmail_quote"><div dir="ltr" class="gmail_attr">On Tue, Mar 19, 2019 at 6:11 PM Aakash Singh <<a href="mailto:17aakashsingh1999@gmail.com" target="_blank" rel="noreferrer">17aakashsingh1999@gmail.com</a>> wrote:<br></div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex"><div dir="ltr"><div>The patch fixes creation of a cache directory for VLSub if it doesn't already exist. The patch is working correctly. If the directory already exists then vlc.io.mkdir() simply returns -1 indicating that the directory already exists and no changes were made to the folder structure.<br></div><div dir="ltr">---<br> share/lua/extensions/VLSub.lua | 17 +++++++++++++++++<br> 1 file changed, 17 insertions(+)<br><br>diff --git a/share/lua/extensions/VLSub.lua b/share/lua/extensions/VLSub.lua<br>index ee9e5f84a9..64013ae46a 100644<br>--- a/share/lua/extensions/VLSub.lua<br>+++ b/share/lua/extensions/VLSub.lua<br>@@ -1488,6 +1488,23 @@ function download_subtitles()<br> <br>   subfileName = subfileName.."."..item.SubFormat<br>   local tmp_dir = vlc.config.cachedir()<br>+  -- create the cache directory if it doens't already exist<br>+  if package.config:sub(1, 1):match("/") then<br>+    -- unix based systems<br>+    local current_dir = "/"<br>+    for dir in tmp_dir:gmatch("[^/]+") do<br>+      current_dir = current_dir.."/"..dir<br>+      vlc.io.mkdir(current_dir, "0700")<br>+    end<br>+  else<br>+    -- windows systems<br>+    local current_dir = ""<br>+    for dir in tmp_dir:gmatch("[^\\]+") do<br>+      current_dir = current_dir.."\\"..dir<br>+      vlc.io.mkdir(current_dir, "0700")<br>+    end<br>+  end<br>+<br>   local file_target_access = true<br> <br>   local tmpFileName = dump_zip(<br>-- <br>2.19.1<br><br></div></div>
</blockquote></div></div>
</blockquote></div></div></div>