mikeplayer 1.0.3 → 1.0.8

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: 129df9ce63171d9efbe8bd1e581fdbfdf33fdadf127936089db72149ebd76a54
-  data.tar.gz: 25f8be729ae785873137a34e291c63983fcb9a5ff71ccb035e6160bbb3448007
+  metadata.gz: 98d73ca7a4c48fc9468996b9aa27ca592d544c8a2637161295625acd6fd849b2
+  data.tar.gz: 44288a081ec5d6f8e3090613b220fe63828ff239f69209a77f951edc680c8776
 SHA512:
-  metadata.gz: b1ec4ae4f8a765ffeed9fb96d7097f97392a27db66211a4bfc4bb14f7a1636b876ec40ecd84458a8ce8a53050cc160d36769a18088fa99533ce927e8ac5185c8
-  data.tar.gz: eebd1b3afce640dd29ba8e639122f56b64c7b7e0878a1b50e0f7e72cc78a4b258708e50ef8cde1e5fc1f43916b39b42444339b339ad488cdada53a387692ac6e
+  metadata.gz: bc0bea67f1ba47fcfe6f63bf7008981ac69ff4247d21e413444bbc641d02a1d245737cfba1f495f9b049562ac409e55deba85bd1a8b87d51f77f8f64eeb81708
+  data.tar.gz: 0ced60fdb31a438d80597f0bb7c4118f8662db0133fd235ea318a4ad591393e2bc85e9b8093a973dc7c5ed522874e74d1a883fb176287eac85ac3f6c44154213

data/bin/MikePlayer.rb

@@ -21,6 +21,7 @@ OptionParser.new do |opt|
   opt.on('-l', '--list', 'List songs in playlist.') { |o| options[:list] = true; }
   opt.on('-d', '--directory name', 'Directory to find mp3s.') { |o| options[:directory] = o }
   opt.on('-t', '--time minutes', 'Limit time to number of minutes.') { |o| options[:minutes] = o }
+  opt.on('-x', '--debug', 'Turn on debug.') { |o| options[:debug] = true }
 end.parse!
 
 MikePlayer::Player.new(options, ARGV).play

data/lib/mikeplayer.rb

@@ -4,28 +4,29 @@ require 'open3'
 require 'io/console'
 require 'mp3info'
 require 'mikeplayer/version'
-require 'mikeplayer/settings'
+require 'mikeplayer/display'
 require 'mikeplayer/playlist'
+require 'mikeplayer/play_thread'
+require 'mikeplayer/settings'
 require 'mikeplayer/song'
 
 module MikePlayer
   class Player
-    PAUSE_INDICATOR    = " ||".freeze
-    SLEEP_SETTING      = 0.5
+    PLAY_SLEEP         = 0.5
+    PAUSE_SLEEP        = 1.0
     STOPPED            = :stopped
     PLAYING            = :playing
     PAUSED             = :paused
+    SONG_CHANGE        = :song_change
 
     def initialize(options, *args)
       @settings  = Settings.new(options)
       @playlist  = Playlist.new(@settings.playlist)
       @minutes   = @settings.minutes
       @command   = ''
-      @pid       = nil
-      @timer_start = nil
+      @timer_start = Time.now if (@minutes > 0)
       @state     = STOPPED
-
-      check_system
+      @player    = PlayThread.new(volume: @settings.volume)
 
       if (true == @settings.list?)
         @songs.map { |song| File.basename(song) }.sort.each {|song| puts "#{File.basename(song)}"}
@@ -50,71 +51,43 @@ module MikePlayer
       puts "Mike Player v#{MikePlayer::VERSION}"
       puts "Playlist #{@playlist.info}\n"
 
-      if @playlist.finished?
+      if (0 == @playlist.size)
         puts "No songs in playlist."
         
         exit 1
       end
 
       @thread = Thread.new do
-        display_width   = 0
+        @display = Display.new
+        @song_i  = 0
 
-        while (false == @playlist.finished?)
-          song        = @playlist.current
-          @song_start = Time.now
-          info_prefix = "\r#{@playlist.current_song_info}".freeze
+        while (@song_i < @playlist.size)
+          @display.song_info = @playlist.song_info(@song_i)
 
-          stdin, stdother, thread_info = Open3.popen2e('play', '--no-show-progress', '--volume', @settings.volume, song.filename)
+          @player.play(song.filename)
 
           @state   = PLAYING
-          @pid     = thread_info.pid
-          indicator = ''
-          info_changed = false
 
-          while (true == pid_alive?)
+          while @player.playing?
             pause_if_over_time_limit
 
-            if (true == playing?)
-              indicator = "#{'>' * (playing_time % 4)}".ljust(4)
-              info_changed = true
-            elsif (true == paused?) && (PAUSE_INDICATOR != indicator)
-              indicator = PAUSE_INDICATOR
-              info_changed = true
-            end
-
-            if (true == info_changed)
-              mindicator = ""
-
-              if (0 < minutes_remaining)
-                mindicator = "(#{minutes_remaining}↓) "
-              end
-
-              print("\r" << ' '.ljust(display_width))
-
-              info  = "#{info_prefix} #{song.length_str(playing_time)} #{mindicator}#{indicator}"
+            @display.elapsed = @player.elapsed if playing?
 
-              print(info)
+            @display.display!(song.length_str(@player.elapsed), minutes_remaining)
 
-              display_width = info.size
-
-              info_changed = false
-              $stdout.flush
-            end
-
-            sleep SLEEP_SETTING
+            sleep(sleep_time)
           end
 
-          stdin.close
-          stdother.close
-
-          @pid = nil
-
-          if (true == playing?)
-            next_song unless @pid.nil?
+          if playing? && @player.stopped?
+            next_song
+          elsif paused?
+            while paused?
+              sleep(sleep_time)
+            end
           end
         end
 
-        @pid   = nil
+        @player.stop
         print("\r\n")
         exit
       end
@@ -124,30 +97,8 @@ module MikePlayer
       print("\r\n")
     end
 
-    def cmd_exist?(cmd)
-      if (true != system('command'))
-        raise "Missing 'command' command, which is used to test compatibility."
-      end
-
-      if (true != system("command -v #{cmd} >/dev/null 2>&1"))
-        return false
-      end
-
-      return true
-    end
-
     private
 
-    def check_system
-      %w(play).each do |cmd|
-        if (false == cmd_exist?(cmd))
-          raise "#{cmd} failed, do you have sox installed?"
-        end
-      end
-
-      return nil
-    end
-
     def wait_on_user
       while ('q' != @command)
         @command = STDIN.getch
@@ -158,9 +109,8 @@ module MikePlayer
           next_song
         elsif ('z' == @command)
           previous_song
-        elsif ('q' == @command) && (false == @pid.nil?)
-          stop_song
-          @thread.kill
+        elsif ('q' == @command)
+          press_stop
         elsif ('t' == @command)
           @timer_start = Time.now
         elsif (false == @timer_start.nil?) && ("#{@command.to_i}" == @command)
@@ -182,69 +132,58 @@ module MikePlayer
       return (PAUSED == @state)
     end
 
+    def changing?
+      return (SONG_CHANGE == @state)
+    end
+
+    def press_stop
+      @player.stop
+      @state = STOPPED
+    end
+
     def press_pause
-      if (true == playing?)
-        kill("STOP")
-        @pause_time = Time.now
+      debug('|')
+
+      if playing?
+        debug('>')
         @state = PAUSED
-      elsif (true == paused?)
-        kill("CONT")
-        @song_start += (Time.now - @pause_time)
-        @pause_time = nil
+        @display.paused
+        @player.pause
+      elsif paused?
+        debug('|')
         @state = PLAYING
       else
         print("Confused state #{@state}.")
       end
     end
 
-    def stop_song
-      if (true == paused?)
-        kill("CONT")
-      end
-
-      kill("INT")
-
-      sleep 0.2
-
-      if (true == pid_alive?)
-        kill("KILL")
-      end
-
-      @state = STOPPED
-      @pid   = nil
-    end
-
-    def pid_alive?(pid = @pid)
-      if (false == pid.nil?)
-        return system("ps -p #{pid} > /dev/null")
-      end
+    def next_song
+      debug('n')
 
-      return false
-    end
+      @state = SONG_CHANGE
 
-    def next_song
-      stop_song
+      @player.stop
 
-      @playlist.next
+      @song_i += 1
     end
 
     def previous_song
-      stop_song
+      debug('p')
 
-      if (playing_time < 10)
-        @playlist.previous
-      end
-    end
+      @state = SONG_CHANGE
 
-    def kill(signal)
-      if (false == @pid.nil?)
-        Process.kill(signal, @pid)
+      if (@player.elapsed < 10)
+        @song_i -= 1 if @song_i.positive?
+      else
+        debug('x')
       end
+
+      @player.stop
     end
 
     def pause_if_over_time_limit
       if (false == @timer_start.nil?) && (0 < @minutes) && (true == playing?)
-        if (0 > minutes_remaining)
+        if (minutes_remaining && 0 >= minutes_remaining)
           press_pause
           @timer_start = nil
           @minutes    = 0
@@ -252,24 +191,24 @@ module MikePlayer
       end
     end
 
-    def playing_time
-      return (Time.now - @song_start).to_i - pause_time
+    def minutes_remaining
+      return if ((0 == @minutes) || (@timer_start.nil?))
+
+      (@minutes - ((Time.now - @timer_start).to_i / 60).to_i)
     end
 
-    def pause_time
-      if (@pause_time.nil?)
-        return 0
-      else
-        return (Time.now - @pause_time).to_i
-      end
+    def sleep_time
+      return PLAY_SLEEP if playing?
+
+      PAUSE_SLEEP
     end
 
-    def minutes_remaining
-      if ((0 == @minutes) || (@timer_start.nil?))
-        return -1
-      else
-        return (@minutes - ((Time.now - @timer_start).to_i / 60).to_i)
-      end
+    def song
+      @playlist.get(@song_i)
+    end
+
+    def debug(str)
+      print(str) if @settings.debug?
     end
   end
 end

data/lib/mikeplayer/display.rb

@@ -0,0 +1,49 @@
+module MikePlayer
+  class Display
+    PAUSE_INDICATOR = '||'.freeze
+    INDICATOR_SIZE  = 4
+
+    def initialize
+      @width     = 0
+      @indicator = ''
+      @changed   = false
+    end
+
+    def song_info=(v)
+      @info_prefix = "\r#{v}".freeze
+    end
+
+    def elapsed=(v)
+      @indicator = "#{'>' * (v % INDICATOR_SIZE)}".ljust(INDICATOR_SIZE)
+      @changed   = true
+    end
+
+    def paused
+      if (false == @indicator.include?(PAUSE_INDICATOR))
+        @indicator = PAUSE_INDICATOR.ljust(INDICATOR_SIZE)
+        @changed   = true
+      end
+    end
+
+    def display!(elapsed_info, countdown = nil)
+      return unless changed?
+
+      mindicator = "(#{countdown}↓) " if countdown
+
+      print("\r" << ' '.ljust(@width))
+
+      info  = "#{@info_prefix} #{elapsed_info} #{mindicator}#{@indicator}"
+
+      print(info)
+
+      @width   = info.size
+      @changed = false
+
+      $stdout.flush
+    end
+
+    def changed?
+      true == @changed
+    end
+  end
+end

data/lib/mikeplayer/play_thread.rb

@@ -1,13 +1,121 @@
 module MikePlayer
   class PlayThread
-    def self.run(file, volume)
+    def initialize(volume: 1.0)
+      check_system
+
+      @pid     = nil
+      @volume  = volume
+      @start_t = 0
+      @elapsed = 0
+      @paused  = false
+    end
+
+    def play(file)
+      start_position = 0
+
+      if paused?
+        start_position = @elapsed
+      else
+        @elapsed = 0
+      end
+
+      start_thread(file: file, start_position: start_position)
+
+      @start_t = Time.now.to_i
+    end
+
+    def stop
+      pause
+
+      @elapsed = 0
+      @start_t = 0
+      @paused  = false
+    end
+
+    def pause
+      kill('INT')
+
+      @elapsed += Time.now.to_i - @start_t
+      @start_t  = 0
+
+      sleep 0.2
+
+      kill('KILL')
+
+      @paused = true
+    end
+
+    def kill(signal)
+      Process.kill(signal, @pid) if alive?
+    end
+
+    def alive?
+      MikePlayer::PlayThread.alive?(@pid)
+    end
+
+    def stopped?
+      false == alive?
+    end
+
+    def paused?
+      @paused && stopped?
+    end
+
+    def playing?
+      alive?
+    end
+
+    def elapsed
+      return (@elapsed + (Time.now.to_i - @start_t)) if @start_t.positive?
+
+      @elapsed
+    end
+
+    def self.alive?(pid)
+      return system("ps -p #{pid} > /dev/null") unless pid.nil?
+
+      false
+    end
+
+    def self.cmd_exist?(cmd)
+      if (true != system('command'))
+        raise "Missing 'command' command, which is used to test compatibility."
+      end
+
+      if (true != system("command -v #{cmd} >/dev/null 2>&1"))
+        return false
+      end
+
+      return true
+    end
+
+    private
+    def start_thread(file:, start_position: )
       args = [
-        :play,
+        'play',
         '--no-show-progress',
-        '--volume', volume,
-        song
+        '--volume', @volume.to_s,
+        file,
+        'trim', start_position.to_s,
       ]
-      Open3.popen2e(*args) do |stdin, out, wait_thr|
+
+      stdin, stdother, thread_info = Open3.popen2e(*args)
+
+      @pid = thread_info.pid
+
+      sleep 0.2
+
+      raise "Failed to play #{stdother.read}" unless alive?
+
+      stdin.close
+      stdother.close
+
+      self
+    end
+
+    def check_system
+      %w[play].each do |cmd|
+        raise "#{cmd} failed, do you have sox installed?" unless MikePlayer::PlayThread.cmd_exist?(cmd)
       end
     end
   end

data/lib/mikeplayer/playlist.rb

@@ -5,12 +5,11 @@ module MikePlayer
     def initialize(filename)
       @filename = filename
       @songs    = []
-      @song_i   = 0
       @length   = 0
     
       load_songs
 
-      @loaded_song_count = @songs.size
+      @loaded_song_count = self.size
     end
 
     def <<(song)
@@ -44,22 +43,8 @@ module MikePlayer
       return self
     end
 
-    def current
-      return @songs[@song_i]
-    end
-
-    def next
-      @song_i += 1
-
-      return self.current
-    end
-
-    def previous
-      @song_i -= 1
-
-      @song_i = 0 if @song_i < 0
-
-      return self.current
+    def get(i)
+      return @songs[i]
     end
 
     def shuffle!
@@ -79,17 +64,17 @@ module MikePlayer
     end
 
     def info
-      return "#{self.name} loaded #{@loaded_song_count} songs with length #{Song.as_duration_str(@length)}, added #{@songs.size - @loaded_song_count}"
+      return "#{self.name} loaded #{@loaded_song_count} and added #{self.size - @loaded_song_count} songs with length #{Song.as_duration_str(@length)}"
     end
 
-    def current_song_info
-      song_i_str = "#{@song_i + 1}".rjust(@songs.size.to_s.size)
+    def song_info(i)
+      song_i_str = "#{i + 1}".rjust(self.size.to_s.size)
 
-      return "Playing (#{song_i_str}/#{@songs.size}): #{current.info}".freeze
+      return "Playing (#{song_i_str}/#{self.size}): #{get(i).info}".freeze
     end
 
-    def finished?
-      return @song_i >= @songs.size
+    def size
+      return @songs.size
     end
 
     private

data/lib/mikeplayer/settings.rb

@@ -16,6 +16,7 @@ module MikePlayer
       @music_dir = options[:directory] || File.join(@home, DEFAULT_DIRECTORY)
       @settings_dir = options[:settings] || File.join(@home, SETTINGS_DIRECTORY)
       @minutes   = options[:minutes].to_i
+      @debug     = options[:debug]
       @random    = options[:random].to_i
 
       if (false == Dir.exist?(@settings_dir))
@@ -35,6 +36,10 @@ module MikePlayer
       return 0 < @random
     end
 
+    def debug?
+      return (true == @debug)
+    end
+
     def overwrite?
       return true == @overwrite
     end

data/lib/mikeplayer/song.rb

@@ -23,7 +23,7 @@ module MikePlayer
     end
 
     def length
-      return @mp3info.length
+      return @mp3info.length || 0
     end
 
     def length_str(elapsed_time)

data/lib/mikeplayer/version.rb

@@ -1,3 +1,3 @@
 module MikePlayer
-  VERSION = '1.0.3'.freeze
+  VERSION = '1.0.8'.freeze
 end

data/mikeplayer.gemspec

@@ -3,15 +3,15 @@ lib = File.expand_path('../lib', __FILE__)
 $LOAD_PATH.unshift(lib) unless $LOAD_PATH.include?(lib)
 require 'mikeplayer/version'
 
-Gem::Specification.new do |gem|
-  gem.name          = 'mikeplayer'
-  gem.version       = MikePlayer::VERSION
-  gem.authors       = ['Mike Crockett']
-  gem.email         = ['rubygems@mmcrockett.com']
-  gem.summary       = 'Wraps Sox\'s `play` command, allowing playslists, find, random and time limit.'
-  gem.executables   << 'MikePlayer.rb'
-  gem.description   = <<-EOF.gsub(/^\s+/, '')
-    #{gem.summary}
+Gem::Specification.new do |spec|
+  spec.name          = 'mikeplayer'
+  spec.version       = MikePlayer::VERSION
+  spec.authors       = ['Mike Crockett']
+  spec.email         = ['rubygems@mmcrockett.com']
+  spec.summary       = 'Wraps Sox\'s `play` command, allowing playslists, find, random and time limit.'
+  spec.executables   << 'MikePlayer.rb'
+  spec.description   = <<-EOF.gsub(/^\s+/, '')
+    #{spec.summary}
 
     Once a song is playing you can:
       'x' to previous
@@ -20,17 +20,20 @@ Gem::Specification.new do |gem|
       'q' to quit
       't' n to set a timer that will pause the music after n minutes
   EOF
-  gem.homepage      = 'https://github.com/mmcrockett/mikeplayer'
+  spec.homepage      = 'https://github.com/mmcrockett/mikeplayer'
 
-  gem.files         = `git ls-files`.split($/)
-  gem.executables   = gem.files.grep(%r{^bin/}).map{ |f| File.basename(f) }
-  gem.test_files    = gem.files.grep(%r{^(test|spec|features)/})
-  gem.require_paths = ['lib']
-  gem.licenses      = ['MIT']
+  spec.files = `git ls-files -z`.split("\x0").reject do |f|
+    f.match?(%r{^(test|spec|features|helpers|)/}) || f.match?(%r{^(\.[[:alnum:]]+)}) || f.match?(/console/)
+  end
 
-  gem.add_dependency 'json', '~> 1.8'
-  gem.add_dependency 'ruby-mp3info', '~> 0.8'
-  gem.add_dependency 'minitest', '~> 5'
-  gem.add_development_dependency 'rake', '~> 12'
-  gem.add_development_dependency 'byebug', '~> 11'
+  spec.executables   = spec.files.grep(%r{^bin/}).map{ |f| File.basename(f) }
+  spec.test_files    = spec.files.grep(%r{^(test|spec|features)/})
+  spec.require_paths = ['lib']
+  spec.licenses      = ['MIT']
+
+  spec.add_dependency 'json', '~> 2.5'
+  spec.add_dependency 'ruby-mp3info', '~> 0.8'
+  spec.add_dependency 'minitest', '~> 5'
+  spec.add_development_dependency 'rake', '~> 12'
+  spec.add_development_dependency 'byebug', '~> 11'
 end

metadata

@@ -1,14 +1,14 @@
 --- !ruby/object:Gem::Specification
 name: mikeplayer
 version: !ruby/object:Gem::Version
-  version: 1.0.3
+  version: 1.0.8
 platform: ruby
 authors:
 - Mike Crockett
-autorequire: 
+autorequire:
 bindir: bin
 cert_chain: []
-date: 2019-05-30 00:00:00.000000000 Z
+date: 2021-02-02 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: json
@@ -16,14 +16,14 @@ dependencies:
     requirements:
     - - "~>"
       - !ruby/object:Gem::Version
-        version: '1.8'
+        version: '2.5'
   type: :runtime
   prerelease: false
   version_requirements: !ruby/object:Gem::Requirement
     requirements:
     - - "~>"
       - !ruby/object:Gem::Version
-        version: '1.8'
+        version: '2.5'
 - !ruby/object:Gem::Dependency
   name: ruby-mp3info
   requirement: !ruby/object:Gem::Requirement
@@ -95,13 +95,13 @@ executables:
 extensions: []
 extra_rdoc_files: []
 files:
-- ".gitignore"
 - Gemfile
 - LICENSE
 - README.md
 - Rakefile
 - bin/MikePlayer.rb
 - lib/mikeplayer.rb
+- lib/mikeplayer/display.rb
 - lib/mikeplayer/executable.rb
 - lib/mikeplayer/play_thread.rb
 - lib/mikeplayer/playlist.rb
@@ -109,14 +109,11 @@ files:
 - lib/mikeplayer/song.rb
 - lib/mikeplayer/version.rb
 - mikeplayer.gemspec
-- test/mikeplayer/settings_test.rb
-- test/mikeplayer_test.rb
-- test/test_helper.rb
 homepage: https://github.com/mmcrockett/mikeplayer
 licenses:
 - MIT
 metadata: {}
-post_install_message: 
+post_install_message:
 rdoc_options: []
 require_paths:
 - lib
@@ -131,12 +128,8 @@ required_rubygems_version: !ruby/object:Gem::Requirement
     - !ruby/object:Gem::Version
       version: '0'
 requirements: []
-rubyforge_project: 
-rubygems_version: 2.7.6
-signing_key: 
+rubygems_version: 3.1.2
+signing_key:
 specification_version: 4
 summary: Wraps Sox's `play` command, allowing playslists, find, random and time limit.
-test_files:
-- test/mikeplayer/settings_test.rb
-- test/mikeplayer_test.rb
-- test/test_helper.rb
+test_files: []

data/.gitignore

@@ -1,51 +0,0 @@
-*.gem
-Gemfile.lock
-*.rbc
-/.config
-/coverage/
-/InstalledFiles
-/pkg/
-/spec/reports/
-/spec/examples.txt
-/test/tmp/
-/test/version_tmp/
-/tmp/
-
-# Used by dotenv library to load environment variables.
-# .env
-
-## Specific to RubyMotion:
-.dat*
-.repl_history
-build/
-*.bridgesupport
-build-iPhoneOS/
-build-iPhoneSimulator/
-
-## Specific to RubyMotion (use of CocoaPods):
-#
-# We recommend against adding the Pods directory to your .gitignore. However
-# you should judge for yourself, the pros and cons are mentioned at:
-# https://guides.cocoapods.org/using/using-cocoapods.html#should-i-check-the-pods-directory-into-source-control
-#
-# vendor/Pods/
-
-## Documentation cache and generated files:
-/.yardoc/
-/_yardoc/
-/doc/
-/rdoc/
-
-## Environment normalization:
-/.bundle/
-/vendor/bundle
-/lib/bundler/man/
-
-# for a library or gem, you might want to ignore these files since the code is
-# intended to run in multiple environments; otherwise, check them in:
-# Gemfile.lock
-# .ruby-version
-# .ruby-gemset
-
-# unless supporting rvm < 1.11.0 or doing something fancy, ignore this:
-.rvmrc

data/test/mikeplayer/settings_test.rb

@@ -1,17 +0,0 @@
-require 'test_helper'
-
-class SettingsTest < Minitest::Test
-  describe 'settings' do
-    let(:home) { File.join('', 'tmp') }
-    let(:sdir) { File.join(home, MikePlayer::Settings::SETTINGS_DIRECTORY) }
-
-    before do
-      Dir.delete(sdir) if Dir.exist?(sdir)
-    end
-
-    it 'creates a settings directory' do
-      MikePlayer::Settings.new(home: home)
-      assert(Dir.exist?(sdir))
-    end
-  end
-end

data/test/mikeplayer_test.rb

@@ -1,9 +0,0 @@
-require 'test_helper'
-
-class MikePlayerTest < Minitest::Test
-  describe 'mikeplayer' do
-    it 'has a version' do
-      assert(MikePlayer::VERSION)
-    end
-  end
-end

data/test/test_helper.rb

@@ -1,5 +0,0 @@
-$LOAD_PATH.unshift File.expand_path('../lib', __dir__)
-require 'mikeplayer'
-
-require 'minitest/autorun'
-require 'byebug'