Remove detect dep, use cross-platform pipe

onhold/after.py

@@ -10,7 +10,7 @@
 ENV_VAR = 'DING'
 
 
-def run(file: Path):
+def run(file: Optional[Path]):
   with play_after(file):
     if not is_pipeline():
       return

onhold/base.py

@@ -6,7 +6,6 @@
 #from subprocess import run
 
 from play_sounds import DEFAULT_SONG
-from detect import unix as IS_UNIX
 
 
 RC_OK = 0
@@ -21,11 +20,24 @@ def is_pipeline() -> bool:
   return not stdin.isatty()
 
 
+# python 3.8+ compatible
+#def dumb_pipe():
+#  while data := stdin.buffer.read(CHUNK):
+#    stdout.buffer.write(data)
+
+
+# python 3.6 compatible
 def dumb_pipe():
-  while data := stdin.buffer.read(CHUNK):
+  while True:
+    data = stdin.buffer.read(CHUNK)
+
+    if not data:
+      break
+
     stdout.buffer.write(data)
 
 
+
 @contextmanager
 def using_path(
   sound_path: Optional[str],

requirements.txt

@@ -2,6 +2,5 @@ click==7.1.2
 boombox==0.56
 play_sounds==0.0.5
 playsound==1.2.2
-detect==2020.7.1 
 
 PyObjc==6.2.2; sys_platform == 'darwin'

setup.py

@@ -4,7 +4,7 @@
 
 
 NAME = "onhold"
-VERSION = "0.4.1"
+VERSION = "0.5.0"
 LICENSE = "AGPL-3.0"
 
 DESC = "🔊 Play music while and after jobs complete"
@@ -23,18 +23,18 @@
 }
 
 setup(
-      name=NAME,
-      version=VERSION,
-      description=DESC,
-      long_description=README,
-      long_description_content_type="text/markdown",
-      url="https://alexdelorenzo.dev",
-      author=__author__,
-      license=LICENSE,
-      packages=[NAME],
-      zip_safe=False,
-      install_requires=REQUIREMENTS,
-      entry_points=ENTRY_POINTS,
-      python_requires='>=3.6',
-      include_package_data=True,
+  name=NAME,
+  version=VERSION,
+  description=DESC,
+  long_description=README,
+  long_description_content_type="text/markdown",
+  url="https://alexdelorenzo.dev",
+  author=__author__,
+  license=LICENSE,
+  packages=[NAME],
+  zip_safe=False,
+  install_requires=REQUIREMENTS,
+  entry_points=ENTRY_POINTS,
+  python_requires='>=3.6',
+  include_package_data=True,
 )