photein 0.0.5 → 0.0.6

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: f2e520a4d9508ba94604273c88b9f0d3bb1052aae63cef56c856595e5b0b3866
-  data.tar.gz: 14633d71731ff23c3350c861bfe33d289bfb53777d6369a6245c203de93dacde
+  metadata.gz: 5404ec50b0c5a57144fea72eff5639ca90a19a46be319cb8ee634d9f67291617
+  data.tar.gz: bce672716a27ea28f9d4203e1858affa8793a11b5ca94e3642c6d86399969680
 SHA512:
-  metadata.gz: cf796a670cc45ce45238b4e9b0e23641ce1dc21aa67008bb016e9507b0470bebdd4790620bfd5325eabdcbe833165588c32811b244b5f8271ff8fa05d5b2d473
-  data.tar.gz: deaea8ad0c1ad541c5e160ea927dbfef115902a7f25b96205e67c780e45477a0def030fe1dc4ba2063cc3c936fb7298ccb9f7f4dca70069e4e7ac4ec7fdd586f
+  metadata.gz: f11aba53e3bcdef67a150a549da7c16c45e0350b4bbdcfb4f1d1ce4c0fd778185845ef4719857e9eea4b1c15894578ae4e1b361952da3961c77fb1b0cc56c0bb
+  data.tar.gz: 2967ccf17b7d7631325ff9b18e445b052a52db8ddec3043cc0b4b15945b0b0c0bba52f0862044f554067950b578447235771abd1d97103346492ba6de821029a

data/README.md

@@ -14,21 +14,22 @@ no matter where they come from:
 ```sh
 # Before                                # After
 
-~/Pictures                              ~/Pictures
-└── _inbox                              ├── _inbox
-    ├── 1619593208911.jpeg              ├── 2020
-    ├── DCIM                            │   ├── 2020-08-01_113129.heic
-    │   └── 2021_03_26                  │   └── 2020-05-20_160209.png
-    │       ├── R0014285.MOV            └── 2021
-    │       ├── R0014286.DNG                ├── 2021-02-12_081933a.jpg
-    │       ├── R0014286.JPG                ├── 2021-02-12_081933b.jpg
-    │       ├── R0014287.DNG                ├── 2021-02-12_081939.mp4
-    │       └── R0014287.JPG                ├── 2021-03-26_161245.mp4
-    ├── IMG_20210212_081933_001.jpg         ├── 2021-03-26_161518.dng
-    ├── IMG_20210212_081933_002.jpg         ├── 2021-03-26_161518.jpg
-    ├── IMG_8953.HEIC                       ├── 2021-03-26_170304.dng
-    ├── Screenshot_20200520_160209.png      ├── 2021-03-26_170304.jpg
-    └── VID_20210212_081939.mp4             └── 2021-04-28_000008.jpg
+~                                       ~
+├── Downloads                           ├── Downloads
+│   ├── 1619593208911.jpeg              └── Pictures
+│   ├── DCIM                                ├── 2020
+│   │   └── 2021_03_26                      │   ├── 2020-08-01_113129.heic
+│   │       ├── R0014285.MOV                │   └── 2020-05-20_160209.png
+│   │       ├── R0014286.DNG                └── 2021
+│   │       ├── R0014286.JPG                    ├── 2021-02-12_081933a.jpg
+│   │       ├── R0014287.DNG                    ├── 2021-02-12_081933b.jpg
+│   │       └── R0014287.JPG                    ├── 2021-02-12_081939.mp4
+│   ├── IMG_20210212_081933_001.jpg             ├── 2021-03-26_161245.mp4
+│   ├── IMG_20210212_081933_002.jpg             ├── 2021-03-26_161518.dng
+│   ├── IMG_8953.HEIC                           ├── 2021-03-26_161518.jpg
+│   ├── Screenshot_20200520_160209.png          ├── 2021-03-26_170304.dng
+│   └── VID_20210212_081939.mp4                 ├── 2021-03-26_170304.jpg
+└── Pictures                                    └── 2021-04-28_000008.jpg
 ```
 
 Photein generates these folders & filenames
@@ -38,7 +39,7 @@ based on metadata timestamps, filename timestamps, or file creation times.
 >
 > If you use a photo management app that decides
 > where and how your photos should be stored on disk
-> (👀 looking at you, Apple Photos), Photein is not for you.
+> (looking at you, Apple Photos 👀), Photein is not for you.
 
 It can also optimize photos and videos for reduced file size.
 

data/lib/photein/image.rb

@@ -43,7 +43,12 @@ module Photein
       when '.png'
         FileUtils.cp(path, tempfile, noop: Photein::Config.dry_run)
         Photein::Logger.info "optimizing #{path}"
-        Optipng.optimize(tempfile, level: 4) unless Photein::Config.dry_run
+        begin
+          Optipng.optimize(tempfile, level: 4) unless Photein::Config.dry_run
+        rescue Errno::ENOENT
+          Photein::Logger.error('optipng is required to compress PNG images')
+          raise
+        end
       end
     end
 
@@ -51,10 +56,20 @@ module Photein
 
     def image
       @image ||= MiniMagick::Image.open(path)
+    rescue MiniMagick::Invalid => e
+      Photein::Logger.error(<<~MSG) if e.message.match?(/You must have ImageMagick/)
+        ImageMagick is required to manipulate image files
+      MSG
+      raise
     end
 
     def metadata_stamp
       MiniExiftool.new(path.to_s).date_time_original
+    rescue MiniExiftool::Error => e
+      Photein::Logger.error(<<~MSG) if e.message.match?(/exiftool: not found/)
+        exiftool is required to read timestamp metadata
+      MSG
+      raise
     end
 
     # NOTE: This may be largely unnecessary:

data/lib/photein/media_file.rb

@@ -20,6 +20,7 @@ module Photein
     end
 
     def import
+      return if corrupted?
       return if Photein::Config.interactive && denied_by_user?
       return if Photein::Config.safe && in_use?
       return if Photein::Config.optimize_for && non_optimizable_format?
@@ -44,6 +45,12 @@ module Photein
 
     private
 
+    def corrupted?(result = false)
+      return result.tap do |r|
+        Photein::Logger.error("#{path.basename}: cannot import corrupted file") if r
+      end
+    end
+
     def denied_by_user?
       $stdout.printf "Import #{path}? [y/N]"
       (STDIN.getch.downcase != 'y').tap { $stdout.puts }

data/lib/photein/version.rb

@@ -1,5 +1,5 @@
 # frozen_string_literal: true
 
 module Photein
-  VERSION = '0.0.5'
+  VERSION = '0.0.6'
 end

data/lib/photein/video.rb

@@ -53,13 +53,23 @@ module Photein
 
     private
 
+    def corrupted?
+      super(video.bitrate.nil?)
+    end
+
     def video
       @video ||= FFMPEG::Movie.new(path.to_s)
+    rescue Errno::ENOENT
+      Photein::Logger.error('ffmpeg is required to manipulate video files')
+      raise
     end
 
     def metadata_stamp
       # video timestamps are typically UTC
       MediaInfo.from(path.to_s).general.encoded_date&.getlocal
+    rescue MediaInfo::EnvironmentError
+      Photein::Logger.error('mediainfo is required to read timestamp metadata')
+      raise
     end
 
     # NOTE: This may be largely unnecessary:

metadata

@@ -1,29 +1,15 @@
 --- !ruby/object:Gem::Specification
 name: photein
 version: !ruby/object:Gem::Version
-  version: 0.0.5
+  version: 0.0.6
 platform: ruby
 authors:
 - Ryan Lue
 autorequire: 
 bindir: bin
 cert_chain: []
-date: 2021-05-28 00:00:00.000000000 Z
+date: 2021-07-23 00:00:00.000000000 Z
 dependencies:
-- !ruby/object:Gem::Dependency
-  name: debouncer
-  requirement: !ruby/object:Gem::Requirement
-    requirements:
-    - - "~>"
-      - !ruby/object:Gem::Version
-        version: '0.2'
-  type: :runtime
-  prerelease: false
-  version_requirements: !ruby/object:Gem::Requirement
-    requirements:
-    - - "~>"
-      - !ruby/object:Gem::Version
-        version: '0.2'
 - !ruby/object:Gem::Dependency
   name: mediainfo
   requirement: !ruby/object:Gem::Requirement
@@ -94,20 +80,6 @@ dependencies:
     - - "~>"
       - !ruby/object:Gem::Version
         version: '0.2'
-- !ruby/object:Gem::Dependency
-  name: rb-inotify
-  requirement: !ruby/object:Gem::Requirement
-    requirements:
-    - - "~>"
-      - !ruby/object:Gem::Version
-        version: '0.10'
-  type: :runtime
-  prerelease: false
-  version_requirements: !ruby/object:Gem::Requirement
-    requirements:
-    - - "~>"
-      - !ruby/object:Gem::Version
-        version: '0.10'
 - !ruby/object:Gem::Dependency
   name: streamio-ffmpeg
   requirement: !ruby/object:Gem::Requirement
@@ -123,19 +95,19 @@ dependencies:
       - !ruby/object:Gem::Version
         version: '3.0'
 - !ruby/object:Gem::Dependency
-  name: pry
+  name: pry-remote
   requirement: !ruby/object:Gem::Requirement
     requirements:
     - - "~>"
       - !ruby/object:Gem::Version
-        version: '0.14'
+        version: '0.1'
   type: :development
   prerelease: false
   version_requirements: !ruby/object:Gem::Requirement
     requirements:
     - - "~>"
       - !ruby/object:Gem::Version
-        version: '0.14'
+        version: '0.1'
 - !ruby/object:Gem::Dependency
   name: rspec
   requirement: !ruby/object:Gem::Requirement