Videos in $XDG_VIDEOS_DIR (fallback: ~/Videos)

diff --git a/srec.py b/srec.py
index 0fc81a405c2bf92d0baf2599a7b8d61ed414143c..2639ef8c1c8ebc24fc10aee52a03bc0a045e407c 100644 (file)
--- a/srec.py
+++ b/srec.py
@@ -5,6 +5,7 @@
 # Free Software Foundation, version 3.
 
 from dataclasses import dataclass
 # Free Software Foundation, version 3.
 
 from dataclasses import dataclass

+import os

 import subprocess
 from typing import Callable
 
 import subprocess
 from typing import Callable
 


@@ -15,25 +16,36 @@ from gi.repository import Gtk  # nopep8 pylint: disable=wrong-import-position
 
 @dataclass
 class Recording:
 
 @dataclass
 class Recording:

+    filename: str

     process: subprocess.Popen[bytes]
 
 
 recording: Recording | None = None
 
 
     process: subprocess.Popen[bytes]
 
 
 recording: Recording | None = None
 
 

+def make_filename() -> str:
+    directory = os.environ.get(
+        'XDG_VIDEOS_DIR',
+        os.path.expanduser('~/Videos'))
+    os.makedirs(directory, exist_ok=True)
+    return os.path.join(directory, "screen-recording.mkv")
+
+

 def on_start_recording(_: Gtk.Button, stack: Gtk.Stack) -> None:
     global recording  # pylint: disable=global-statement
     assert recording is None
     screen_size = '1366x768'  # TODO
 def on_start_recording(_: Gtk.Button, stack: Gtk.Stack) -> None:
     global recording  # pylint: disable=global-statement
     assert recording is None
     screen_size = '1366x768'  # TODO

+    filename = make_filename()

     command = [
             'ffmpeg',
             '-video_size', screen_size,
             '-framerate', '25',
             '-f', 'x11grab', '-i', ':0.0+0,0',
             '-f', 'pulse', '-ac', '2', '-i', 'default',
     command = [
             'ffmpeg',
             '-video_size', screen_size,
             '-framerate', '25',
             '-f', 'x11grab', '-i', ':0.0+0,0',
             '-f', 'pulse', '-ac', '2', '-i', 'default',

-            'screen-recording.mkv']  # nopep8
+            filename]  # nopep8

     # pylint: disable=consider-using-with
     recording = Recording(
     # pylint: disable=consider-using-with
     recording = Recording(

+        filename=filename,

         process=subprocess.Popen(command, stdin=subprocess.PIPE))
     stack.set_visible_child_name("recording")
 
         process=subprocess.Popen(command, stdin=subprocess.PIPE))
     stack.set_visible_child_name("recording")