From f279f0a75deb801d5deed78c093825c3ebc9401c Mon Sep 17 00:00:00 2001
From: Ethan Dalool <git@voussoir.net>
Date: Sat, 8 Nov 2025 11:53:39 -0800
Subject: [PATCH] Make lock_source_file optional.

---
 voussoirkit/spinal.py | 28 ++++++++++++++++++++++++----
 1 file changed, 24 insertions(+), 4 deletions(-)

diff --git a/voussoirkit/spinal.py b/voussoirkit/spinal.py
index 0115c20..f4fe500 100644
--- a/voussoirkit/spinal.py
+++ b/voussoirkit/spinal.py
@@ -81,6 +81,7 @@ def copy_directory(
         file_progressbar=None,
         files_per_second=None,
         hash_class=None,
+        lock_source_file=True,
         overwrite=OVERWRITE_OLD,
         precalcsize=False,
         skip_symlinks=True,
@@ -160,6 +161,9 @@ def copy_directory(
     hash_class:
         Passed into each `copy_file` as `hash_class`.
 
+    lock_source_file:
+        Passed into each `copy_file` as `lock_source_file`.
+
     overwrite:
         Passed into each `copy_file` as `overwrite`.
 
@@ -304,6 +308,7 @@ def copy_directory(
             chunk_size=chunk_size,
             dry_run=dry_run,
             hash_class=hash_class,
+            lock_source_file=lock_source_file,
             overwrite=overwrite,
             progressbar=file_progressbar,
             verify_hash=verify_hash,
@@ -345,6 +350,7 @@ def copy_file(
         destination_new_root=None,
         dry_run=False,
         hash_class=None,
+        lock_source_file=True,
         overwrite=OVERWRITE_OLD,
         progressbar=None,
         verify_hash=False,
@@ -400,6 +406,10 @@ def copy_file(
         needing overwrite, this won't be set, so be prepared to handle None.
         If None, the hash will not be calculated.
 
+    lock_source_file:
+        If True, attempt to lock the source file from being modified while we
+        are copying it, to prevent corruption.
+
     overwrite:
         This option decides what to do when the destination file already exists.
         If OVERWRITE_ALL, the file will be overwritten.
@@ -499,9 +509,19 @@ def copy_file(
     if source_handle is None:
         return results
 
-    if portalocker is not None:
+    source_file_lock = None
+    if lock_source_file and portalocker is not None:
         log.loud('Locking source file.')
-        portalocker.lock(source_handle, portalocker.LockFlags.EXCLUSIVE)
+        try:
+            source_file_lock = portalocker.Lock(
+                source.absolute_path,
+                mode='rb',
+                flags=portalocker.LockFlags.EXCLUSIVE | portalocker.LockFlags.NON_BLOCKING,
+                timeout=10,
+            )
+            source_file_lock.acquire()
+        except portalocker.exceptions.LockException:
+            pass
 
     log.loud('Opening dest handle.')
     destination_handle = handlehelper(destination, 'wb')
@@ -550,8 +570,8 @@ def copy_file(
             chunk_time = time.perf_counter() - chunk_start
             chunk_size = dynamic_chunk_sizer(chunk_size, chunk_time, IDEAL_CHUNK_TIME)
 
-    if portalocker is not None:
-        portalocker.unlock(source_handle)
+    if source_file_lock is not None:
+        source_file_lock.release()
 
     progressbar.done()