screen-recorder 1.3.0 → 1.3.1

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: 223571eec67845c142f2bddbe0bb59960b8ba800ba41374158aeadab58611291
-  data.tar.gz: 7292831582ec9afb87736c0e807c061162abf75cdf1a8c5b08a0ffc40a458aa0
+  metadata.gz: 354630449024886a2588bf44a6dfad01d7837341ecb94ced49b936d6cce6fb57
+  data.tar.gz: 21140832dd0bc9f9fb51d01a9ef885b6387847c4ec1484380993613e96453ae0
 SHA512:
-  metadata.gz: f1bbe228c99deea2feb837097c7b6bc30cd00fc8051f43a2b56ad90f0dc6768ff04b87ecb45b564c56267383e1e5113c74ffdcea5c8708563a7782bca4b689fe
-  data.tar.gz: 173054cecd3169310b5a8982e9c3d5334edb5ade410436a11480c234d385bbe8b91c52a3ff25062a15e6dfc56ec7f37e52d325243cdd1a36fc423fcee870e9b6
+  metadata.gz: 58da8b78830b44b56b76408561198953b5dba59a3d5c57f0eae82052f60312ad8e6a426fc4b1bcbef9bd142fc9640ab07785aa0cba03676c5b9ce04bb99b51ea
+  data.tar.gz: 8c549968577da56770fa558a91d461240765dc32276612de7ad17bf31fe74bc29386b92f77438cbc444fe7c2d0622f33074120341810b4f9d363b1919bcf99a0

data/.gitignore

@@ -1,100 +1,100 @@
-Gemfile.lock # Part of best practice
-
-# Created by https://www.gitignore.io/api/rubymine+all
-
-### RubyMine+all ###
-# Covers JetBrains IDEs: IntelliJ, RubyMine, PhpStorm, AppCode, PyCharm, CLion, Android Studio and WebStorm
-# Reference: https://intellij-support.jetbrains.com/hc/en-us/articles/206544839
-
-/.bundle/
-/.yardoc
-/_yardoc/
-/coverage/
-/doc/
-/pkg/
-/spec/reports/
-/tmp/
-/webdrivers_bin
-*.log
-*.mkv
-*.mp4
-*.avi
-*.gif
-
-Gemfile.lock
-
-# rspec failure tracking
-.rspec_status
-
-# Generated files
-.idea/**/contentModel.xml
-
-# Sensitive or high-churn files
-.idea/**/dataSources/
-.idea/**/dataSources.ids
-.idea/**/dataSources.local.xml
-.idea/**/sqlDataSources.xml
-.idea/**/dynamic.xml
-.idea/**/uiDesigner.xml
-.idea/**/dbnavigator.xml
-
-# Gradle
-.idea/**/gradle.xml
-.idea/**/libraries
-
-# Gradle and Maven with auto-import
-# When using Gradle or Maven with auto-import, you should exclude module files,
-# since they will be recreated, and may cause churn.  Uncomment if using
-# auto-import.
-# .idea/modules.xml
-# .idea/*.iml
-# .idea/modules
-
-# CMake
-cmake-build-*/
-
-# Mongo Explorer plugin
-.idea/**/mongoSettings.xml
-
-# File-based project format
-*.iws
-
-# IntelliJ
-out/
-
-# mpeltonen/sbt-idea plugin
-.idea_modules/
-
-# JIRA plugin
-atlassian-ide-plugin.xml
-
-# Cursive Clojure plugin
-.idea/replstate.xml
-
-# Crashlytics plugin (for Android Studio and IntelliJ)
-com_crashlytics_export_strings.xml
-crashlytics.properties
-crashlytics-build.properties
-fabric.properties
-
-# Editor-based Rest Client
-.idea/httpRequests
-
-# Android studio 3.1+ serialized cache file
-.idea/caches/build_file_checksums.ser
-
-### RubyMine+all Patch ###
-# Ignores the whole .idea folder and all .iml files
-# See https://github.com/joeblau/gitignore.io/issues/186 and https://github.com/joeblau/gitignore.io/issues/360
-
-.idea/
-
-# Reason: https://github.com/joeblau/gitignore.io/issues/186#issuecomment-249601023
-
-*.iml
-modules.xml
-*.ipr
-
-.DS_Store # On mac
-
+Gemfile.lock # Part of best practice
+
+# Created by https://www.gitignore.io/api/rubymine+all
+
+### RubyMine+all ###
+# Covers JetBrains IDEs: IntelliJ, RubyMine, PhpStorm, AppCode, PyCharm, CLion, Android Studio and WebStorm
+# Reference: https://intellij-support.jetbrains.com/hc/en-us/articles/206544839
+
+/.bundle/
+/.yardoc
+/_yardoc/
+/coverage/
+/doc/
+/pkg/
+/spec/reports/
+/tmp/
+/webdrivers_bin
+*.log
+*.mkv
+*.mp4
+*.avi
+*.gif
+
+Gemfile.lock
+
+# rspec failure tracking
+.rspec_status
+
+# Generated files
+.idea/**/contentModel.xml
+
+# Sensitive or high-churn files
+.idea/**/dataSources/
+.idea/**/dataSources.ids
+.idea/**/dataSources.local.xml
+.idea/**/sqlDataSources.xml
+.idea/**/dynamic.xml
+.idea/**/uiDesigner.xml
+.idea/**/dbnavigator.xml
+
+# Gradle
+.idea/**/gradle.xml
+.idea/**/libraries
+
+# Gradle and Maven with auto-import
+# When using Gradle or Maven with auto-import, you should exclude module files,
+# since they will be recreated, and may cause churn.  Uncomment if using
+# auto-import.
+# .idea/modules.xml
+# .idea/*.iml
+# .idea/modules
+
+# CMake
+cmake-build-*/
+
+# Mongo Explorer plugin
+.idea/**/mongoSettings.xml
+
+# File-based project format
+*.iws
+
+# IntelliJ
+out/
+
+# mpeltonen/sbt-idea plugin
+.idea_modules/
+
+# JIRA plugin
+atlassian-ide-plugin.xml
+
+# Cursive Clojure plugin
+.idea/replstate.xml
+
+# Crashlytics plugin (for Android Studio and IntelliJ)
+com_crashlytics_export_strings.xml
+crashlytics.properties
+crashlytics-build.properties
+fabric.properties
+
+# Editor-based Rest Client
+.idea/httpRequests
+
+# Android studio 3.1+ serialized cache file
+.idea/caches/build_file_checksums.ser
+
+### RubyMine+all Patch ###
+# Ignores the whole .idea folder and all .iml files
+# See https://github.com/joeblau/gitignore.io/issues/186 and https://github.com/joeblau/gitignore.io/issues/360
+
+.idea/
+
+# Reason: https://github.com/joeblau/gitignore.io/issues/186#issuecomment-249601023
+
+*.iml
+modules.xml
+*.ipr
+
+.DS_Store # On mac
+
 # End of https://www.gitignore.io/api/rubymine+all

data/.rspec

@@ -1,3 +1,3 @@
---format documentation
---color
---require spec_helper
+--format documentation
+--color
+--require spec_helper

data/.rubocop.yml

@@ -32,6 +32,9 @@ Layout/AlignParameters:
 Layout/IndentFirstHashElement:
   Enabled: true
 
+Layout/TrailingBlankLines:
+  Enabled: false
+
 Metrics/LineLength:
   Max: 120
 
@@ -69,7 +72,4 @@ Style/CommentedKeyword:
   Enabled: false
 
 Style/FrozenStringLiteralComment:
-  Enabled: false
-
-TrailingBlankLines:
   Enabled: false

data/.travis.yml

@@ -16,7 +16,7 @@ matrix:
       env: RAKE_TASK=rubocop
     - rvm: 2.6.3
       dist: xenial
-      env: RAKE_TASK=spec
+      env: RAKE_TASK=spec REPORT_COVERAGE=true
     - rvm: jruby-9.2.7.0
       dist: xenial
       jdk: openjdk8

data/CHANGELOG.md

@@ -2,6 +2,10 @@
 
 This project adheres to [Semantic Versioning](https://semver.org/).
 
+### 1.3.1 (2019-10-20)
+* Reattempt `ffprobe` execution up to times if the first try raises `Errno::EAGAIN`.
+Hopefully fixes [#79](https://github.com/kapoorlakshya/screen-recorder/issues/79).
+
 ### 1.3.0 (2019-07-26)
 * Support JRuby 9.2+ ([#58](https://github.com/kapoorlakshya/screen-recorder/issues/58))
 * Add `ScreenRecorder::Window.fetch_title` as an alias for `ScreenRecorder::Titles.fetch`.

data/LICENSE.txt

@@ -1,21 +1,21 @@
-The MIT License (MIT)
-
-Copyright (c) 2018 Lakshya Kapoor
-
-Permission is hereby granted, free of charge, to any person obtaining a copy
-of this software and associated documentation files (the "Software"), to deal
-in the Software without restriction, including without limitation the rights
-to use, copy, modify, merge, publish, distribute, sublicense, and/or sell
-copies of the Software, and to permit persons to whom the Software is
-furnished to do so, subject to the following conditions:
-
-The above copyright notice and this permission notice shall be included in
-all copies or substantial portions of the Software.
-
-THE SOFTWARE IS PROVIDED "AS IS", WITHOUT WARRANTY OF ANY KIND, EXPRESS OR
-IMPLIED, INCLUDING BUT NOT LIMITED TO THE WARRANTIES OF MERCHANTABILITY,
-FITNESS FOR A PARTICULAR PURPOSE AND NONINFRINGEMENT. IN NO EVENT SHALL THE
-AUTHORS OR COPYRIGHT HOLDERS BE LIABLE FOR ANY CLAIM, DAMAGES OR OTHER
-LIABILITY, WHETHER IN AN ACTION OF CONTRACT, TORT OR OTHERWISE, ARISING FROM,
-OUT OF OR IN CONNECTION WITH THE SOFTWARE OR THE USE OR OTHER DEALINGS IN
-THE SOFTWARE.
+The MIT License (MIT)
+
+Copyright (c) 2018 Lakshya Kapoor
+
+Permission is hereby granted, free of charge, to any person obtaining a copy
+of this software and associated documentation files (the "Software"), to deal
+in the Software without restriction, including without limitation the rights
+to use, copy, modify, merge, publish, distribute, sublicense, and/or sell
+copies of the Software, and to permit persons to whom the Software is
+furnished to do so, subject to the following conditions:
+
+The above copyright notice and this permission notice shall be included in
+all copies or substantial portions of the Software.
+
+THE SOFTWARE IS PROVIDED "AS IS", WITHOUT WARRANTY OF ANY KIND, EXPRESS OR
+IMPLIED, INCLUDING BUT NOT LIMITED TO THE WARRANTIES OF MERCHANTABILITY,
+FITNESS FOR A PARTICULAR PURPOSE AND NONINFRINGEMENT. IN NO EVENT SHALL THE
+AUTHORS OR COPYRIGHT HOLDERS BE LIABLE FOR ANY CLAIM, DAMAGES OR OTHER
+LIABILITY, WHETHER IN AN ACTION OF CONTRACT, TORT OR OTHERWISE, ARISING FROM,
+OUT OF OR IN CONNECTION WITH THE SOFTWARE OR THE USE OR OTHER DEALINGS IN
+THE SOFTWARE.

data/README.md

@@ -23,7 +23,7 @@ Works on Windows, Linux, and macOS. Requires Ruby 2.0+ or JRuby 9.2+.
 
 ##### 1. Setup FFmpeg
 
-Linux and macOS instructions are [here](https://www.ffmpeg.org/download.html). 
+Linux and macOS instructions are [here](https://trac.ffmpeg.org/wiki/CompilationGuide). 
 
 For Microsoft Windows, download the *libx264* enabled binary from [here](https://ffmpeg.zeranoe.com/builds/).
 Once downloaded, add location of the `ffmpeg/bin` folder to the `PATH` 
@@ -38,13 +38,13 @@ Next, add these lines to your application's Gemfile:
 
 ```ruby
 gem 'ffi' # Windows only
-gem 'screen-recorder'
+gem 'screen-recorder', '~> 1.0'
 ```
 
 The [`ffi`](https://github.com/ffi/ffi) gem is used by the 
 [`childprocess`](https://github.com/enkessler/childprocess) gem on 
 Windows, but it does not explicitly require it. More information 
-on this [here](https://github.com/enkessler/childprocess/issues/150).
+on this [here](https://github.com/enkessler/childprocess/issues/160).
 
 
 And then execute:

data/bin/setup

@@ -1,8 +1,8 @@
-#!/usr/bin/env bash
-set -euo pipefail
-IFS=$'\n\t'
-set -vx
-
-bundle install
-
-# Do any other automated setup that you need to do here
+#!/usr/bin/env bash
+set -euo pipefail
+IFS=$'\n\t'
+set -vx
+
+bundle install
+
+# Do any other automated setup that you need to do here

data/lib/screen-recorder/common.rb

@@ -19,8 +19,8 @@ module ScreenRecorder
     #
     def start
       ScreenRecorder.logger.debug 'Starting recorder...'
-      @video     = nil # New file
-      @process   = start_ffmpeg
+      @video   = nil # New file
+      @process = start_ffmpeg
       ScreenRecorder.logger.info 'Recording...'
       @process
     end
@@ -33,7 +33,7 @@ module ScreenRecorder
       stop_ffmpeg
       ScreenRecorder.logger.debug 'Stopped ffmpeg.'
       ScreenRecorder.logger.info 'Recording complete.'
-      @video = FFMPEG::Movie.new(options.output)
+      @video = prepare_video
     end
 
     #
@@ -86,6 +86,27 @@ module ScreenRecorder
       ScreenRecorder.logger.error "Check '#{@options.log}' for more information."
     end
 
+    #
+    # Runs ffprobe on the output video file and returns
+    # a FFMPEG::Movie object.
+    #
+    def prepare_video
+      max_attempts  = 3
+      attempts_made = 0
+      delay         = 1.0
+
+      begin # Fixes #79
+        ScreenRecorder.logger.info 'Running ffprobe to prepare video (output) file.'
+        FFMPEG::Movie.new(options.output)
+      rescue Errno::EAGAIN, Errno::EACCES
+        attempts_made += 1
+        ScreenRecorder.logger.error "Failed to run ffprobe. Retrying... (#{attempts_made}/#{max_attempts})"
+        sleep(delay)
+        retry if attempts_made < max_attempts
+        raise
+      end
+    end
+
     #
     # Generates the command line arguments based on the given
     # options.

data/lib/screen-recorder/errors.rb

@@ -1,10 +1,10 @@
-module ScreenRecorder
-  # @since 1.0.0-beta5
-  module Errors
-    # @since 1.0.0-beta3
-    class ApplicationNotFound < StandardError; end
-
-    # @since 1.0.0-beta5
-    class DependencyNotFound < StandardError; end
-  end
+module ScreenRecorder
+  # @since 1.0.0-beta5
+  module Errors
+    # @since 1.0.0-beta3
+    class ApplicationNotFound < StandardError; end
+
+    # @since 1.0.0-beta5
+    class DependencyNotFound < StandardError; end
+  end
 end

data/lib/screen-recorder/version.rb

@@ -1,3 +1,3 @@
 module ScreenRecorder
-  VERSION = '1.3.0'.freeze
+  VERSION = '1.3.1'.freeze
 end

data/screen-recorder.gemspec

@@ -14,7 +14,7 @@ Gem::Specification.new do |spec|
                                'on Windows, Linux, and macOS. Primarily geared towards recording automated UI ' \
                                '(Selenium) test executions for debugging and documentation.'
   spec.license               = 'MIT'
-  # noinspection RubyStringKeysInHashInspection,RubyStringKeysInHashInspection,RubyStringKeysInHashInspection
+  # noinspection RubyStringKeysInHashInspection
   spec.metadata              = {
     'changelog_uri' => 'https://github.com/kapoorlakshya/screen-recorder/blob/master/CHANGELOG.md',
     'source_code_uri' => 'https://github.com/kapoorlakshya/screen-recorder/',
@@ -28,17 +28,17 @@ Gem::Specification.new do |spec|
 
   spec.require_paths = ['lib']
 
-  spec.add_development_dependency 'ffi' # For selenium_webdriver on Windows
-  spec.add_development_dependency 'rake', '~> 12.0'
+  spec.add_development_dependency 'ffi' # For selenium-webdriver on Windows
+  spec.add_development_dependency 'rake', '>= 12.0'
   spec.add_development_dependency 'rspec', '~> 3.0'
-  spec.add_development_dependency 'rubocop', '~> 0.73'
-  spec.add_development_dependency 'rubocop-performance', '~> 1.1.0'
-  spec.add_development_dependency 'rubocop-rspec', '~> 1.32'
+  spec.add_development_dependency 'rubocop', '~> 0.75'
+  spec.add_development_dependency 'rubocop-performance', '~> 1.0'
+  spec.add_development_dependency 'rubocop-rspec', '~> 1.0'
   spec.add_development_dependency 'simplecov', '~> 0.16'
   spec.add_development_dependency 'watir', '~> 6.0'
   spec.add_development_dependency 'webdrivers', '~> 4.0'
 
-  spec.add_runtime_dependency 'childprocess', '~> 1.0'
+  spec.add_runtime_dependency 'childprocess', '~> 3.0'
   spec.add_runtime_dependency 'os', '~> 1.0.0'
   spec.add_runtime_dependency 'streamio-ffmpeg', '~> 3.0'
 end

metadata

@@ -1,14 +1,14 @@
 --- !ruby/object:Gem::Specification
 name: screen-recorder
 version: !ruby/object:Gem::Version
-  version: 1.3.0
+  version: 1.3.1
 platform: ruby
 authors:
 - Lakshya Kapoor
 autorequire: 
 bindir: bin
 cert_chain: []
-date: 2019-07-26 00:00:00.000000000 Z
+date: 2019-10-21 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: ffi
@@ -28,14 +28,14 @@ dependencies:
   name: rake
   requirement: !ruby/object:Gem::Requirement
     requirements:
-    - - "~>"
+    - - ">="
       - !ruby/object:Gem::Version
         version: '12.0'
   type: :development
   prerelease: false
   version_requirements: !ruby/object:Gem::Requirement
     requirements:
-    - - "~>"
+    - - ">="
       - !ruby/object:Gem::Version
         version: '12.0'
 - !ruby/object:Gem::Dependency
@@ -58,42 +58,42 @@ dependencies:
     requirements:
     - - "~>"
       - !ruby/object:Gem::Version
-        version: '0.73'
+        version: '0.75'
   type: :development
   prerelease: false
   version_requirements: !ruby/object:Gem::Requirement
     requirements:
     - - "~>"
       - !ruby/object:Gem::Version
-        version: '0.73'
+        version: '0.75'
 - !ruby/object:Gem::Dependency
   name: rubocop-performance
   requirement: !ruby/object:Gem::Requirement
     requirements:
     - - "~>"
       - !ruby/object:Gem::Version
-        version: 1.1.0
+        version: '1.0'
   type: :development
   prerelease: false
   version_requirements: !ruby/object:Gem::Requirement
     requirements:
     - - "~>"
       - !ruby/object:Gem::Version
-        version: 1.1.0
+        version: '1.0'
 - !ruby/object:Gem::Dependency
   name: rubocop-rspec
   requirement: !ruby/object:Gem::Requirement
     requirements:
     - - "~>"
       - !ruby/object:Gem::Version
-        version: '1.32'
+        version: '1.0'
   type: :development
   prerelease: false
   version_requirements: !ruby/object:Gem::Requirement
     requirements:
     - - "~>"
       - !ruby/object:Gem::Version
-        version: '1.32'
+        version: '1.0'
 - !ruby/object:Gem::Dependency
   name: simplecov
   requirement: !ruby/object:Gem::Requirement
@@ -142,14 +142,14 @@ dependencies:
     requirements:
     - - "~>"
       - !ruby/object:Gem::Version
-        version: '1.0'
+        version: '3.0'
   type: :runtime
   prerelease: false
   version_requirements: !ruby/object:Gem::Requirement
     requirements:
     - - "~>"
       - !ruby/object:Gem::Version
-        version: '1.0'
+        version: '3.0'
 - !ruby/object:Gem::Dependency
   name: os
   requirement: !ruby/object:Gem::Requirement
@@ -237,7 +237,7 @@ required_rubygems_version: !ruby/object:Gem::Requirement
     - !ruby/object:Gem::Version
       version: '0'
 requirements: []
-rubygems_version: 3.0.3
+rubygems_version: 3.0.6
 signing_key: 
 specification_version: 4
 summary: Video record your computer screen using FFmpeg.