libPyshell 0.4.0__tar.gz → 0.4.1__tar.gz

{libPyshell-0.4.0/libPyshell.egg-info → libpyshell-0.4.1}/PKG-INFO

@@ -1,6 +1,6 @@
 Metadata-Version: 2.1
 Name: libPyshell
-Version: 0.4.0
+Version: 0.4.1
 Summary: Support for writing shell scripts in Python
 Home-page: https://github.com/skogsbaer/libPyshell
 Author: Stefan Wehr
@@ -37,6 +37,10 @@ magicFiles = run(['grep', 'magic'] + files, captureStdout=splitLines, onError='i
 
 ## Changelog
 
+* 0.4.1 (2024-09-12)
+  * fix capture handling
+  * add failOnError option to rm commands
+
 * 0.4.0 (2024-03-20)
   * re-implement run in terms of subprocess.run. This fixes a bug that caused stdout to
     disappear.

{libPyshell-0.4.0 → libpyshell-0.4.1}/README.md

@@ -26,6 +26,10 @@ magicFiles = run(['grep', 'magic'] + files, captureStdout=splitLines, onError='i
 
 ## Changelog
 
+* 0.4.1 (2024-09-12)
+  * fix capture handling
+  * add failOnError option to rm commands
+
 * 0.4.0 (2024-03-20)
   * re-implement run in terms of subprocess.run. This fixes a bug that caused stdout to
     disappear.

{libPyshell-0.4.0 → libpyshell-0.4.1/libPyshell.egg-info}/PKG-INFO

@@ -1,6 +1,6 @@
 Metadata-Version: 2.1
 Name: libPyshell
-Version: 0.4.0
+Version: 0.4.1
 Summary: Support for writing shell scripts in Python
 Home-page: https://github.com/skogsbaer/libPyshell
 Author: Stefan Wehr
@@ -37,6 +37,10 @@ magicFiles = run(['grep', 'magic'] + files, captureStdout=splitLines, onError='i
 
 ## Changelog
 
+* 0.4.1 (2024-09-12)
+  * fix capture handling
+  * add failOnError option to rm commands
+
 * 0.4.0 (2024-03-20)
   * re-implement run in terms of subprocess.run. This fixes a bug that caused stdout to
     disappear.

{libPyshell-0.4.0 → libpyshell-0.4.1}/setup.py

@@ -2,7 +2,7 @@
 
 from distutils.core import setup
 
-VERSION = '0.4.0'
+VERSION = '0.4.1'
 
 with open("README.md", "r", encoding="utf-8") as fh:
     long_description = fh.read()

{libPyshell-0.4.0 → libpyshell-0.4.1}/src/__init__.py

@@ -197,7 +197,7 @@ def _handleCapture(capture: CaptureType) -> Optional[_FILE]:
         return subprocess.PIPE
     elif callable(capture):
         return subprocess.PIPE
-    elif capture is None:
+    elif capture == False:
         return None
     else:
         return capture
@@ -430,13 +430,15 @@ def removeFile(path: str):
 
 def cp(src: str, target: str):
     """
-    Copy `src` to `target`.
+    Copy `src` to `target`. Behaves like the cp shell command.
 
     * If `src` is a file and `target` is a file: overwrites `target`.
     * If `src` is a file and `target` is a dirname: places the copy in directory `target`,
       with the basename of `src.
-    * If `src` is a directory then `target` must also be a directory: copies
-      the `src` directory (*not* its content) to `target`.
+    * If `src` is a directory and `target` exists, then `target` must also be a directory:
+      copies the `src` directory (*not* its content) to `target`.
+    * If `src` is a directory and `target` does not exist: copies the `src` directory
+      and names the copy `target`.
     """
     if isFile(src):
         if isDir(target):
@@ -511,22 +513,34 @@ class workingDir:
         cd(self.old_dir)
         return False # reraise expection
 
-def rm(path: str, force: bool=False):
+def rm(path: str, force: bool=False, failOnError: bool=True):
     """
     Remove the file at `path`.
     """
     if force and not exists(path):
         return
-    os.remove(path)
+    try:
+        os.remove(path)
+    except Exception as e:
+        if failOnError:
+            raise
+        else:
+            sys.stderr.write(str(e) + '\n')
 
-def rmdir(d: str, recursive: bool=False):
+def rmdir(d: str, recursive: bool=False, failOnError: bool=True):
     """
     Remove directory `d`. Set `recursive=True` if the directory is not empty.
     """
-    if recursive:
-        shutil.rmtree(d)
-    else:
-        os.rmdir(d)
+    try:
+        if recursive:
+            shutil.rmtree(d)
+        else:
+            os.rmdir(d)
+    except Exception as e:
+        if failOnError:
+            raise
+        else:
+            sys.stderr.write(str(e) + '\n')
 
 # See https://stackoverflow.com/questions/9741351/how-to-find-exit-code-or-reason-when-atexit-callback-is-called-in-python
 class _ExitHooks(object):
@@ -581,7 +595,7 @@ def _registerAtExit(action: Any, mode: AtExitMode):
 def mkTempFile(suffix: str='', prefix: str='',
                dir:Optional[str]=None,
                deleteAtExit:AtExitMode=True):
-    """Create a temporary file.
+    """Create a temporary file name.
 
     `deleteAtExit` controls if and how the file is deleted once the shell sript terminates.
    It has one of the following values.
@@ -596,7 +610,7 @@ def mkTempFile(suffix: str='', prefix: str='',
     if deleteAtExit:
         def action():
             if isFile(f):
-                rm(f)
+                rm(f, failOnError=False)
         _registerAtExit(action, deleteAtExit)
     return f
 
@@ -610,7 +624,7 @@ def mkTempDir(suffix: str='', prefix: str='tmp',
     if deleteAtExit:
         def action():
             if isDir(d):
-                rmdir(d, True)
+                rmdir(d, recursive=True, failOnError=False)
         _registerAtExit(action, deleteAtExit)
     return d
 
@@ -646,7 +660,7 @@ class tempDir:
             return False # reraise
         if self.delete:
             if isDir(self.dir_to_delete):
-                rmdir(self.dir_to_delete, recursive=True)
+                rmdir(self.dir_to_delete, recursive=True, failOnError=False)
         return False # reraise expection
 
 def ls(d: str, *globs: str) -> list[str]: