git-version-bump 0.17.2 → 0.18.0.7.g2e0c3c4

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: cb5d7286ef401dcda5941cfe558c6c76a01f3643d3b64fbf4c39ef0e26df9e43
-  data.tar.gz: c52065c4822f48411b645da6c9d1e864f9439ced6fc25a2b94cec3d8d743628d
+  metadata.gz: 32795b930c87e8796ae34c50d42703165e8dc083d4fa4e6cd19763f49f71d78a
+  data.tar.gz: bdf0755443972b724acb3be0cf0b677cc5a698f52533ec9afe8c47dd35ec5553
 SHA512:
-  metadata.gz: c128a2fac14105f6703f4344658831392668ffe391460d90b404893d0d9373edc43ce5b25b7572d750f04c7fdb4c50b724b539b02095fd7b8dbe055c1b4d9fc9
-  data.tar.gz: cd902acf61793e97f98de3139d04dbea2e1e907fc8ead518e6e607283fc79ea984097e1e27b61c73facd49a2039db48091c5c56dfac276f21c8a956126c32220
+  metadata.gz: 3e97d8fd061b44c0f841c435f303930eacf1834acfb555c3b69f5ab7cfc918b678fa452e4f5ce1b8b0b780d5f01546226791a0fa45ad6985fc252d7c9cd63d5e
+  data.tar.gz: 82f01e33ed949be7ad11a544a24a44383e8f01896b67ff242c955287ea7ea51987e43c7afb9727afd026c63bce78bf0d06b3729016c8af5e4ca429d4ed832d53

data/.github/workflows/release.yml

@@ -0,0 +1,33 @@
+name: "Release to RubyGems"
+on:
+  push:
+    branch: [main]
+  release:
+    types: [created]
+  workflow_dispatch:
+
+jobs:
+  upload:
+    runs-on: ubuntu-latest
+    name: "Upload gem"
+
+    steps:
+      - uses: actions/checkout@v2
+        with:
+          fetch-depth: 0
+
+      - name: Install ruby
+        uses: ruby/setup-ruby@v1
+        with:
+          ruby-version: '2.7'
+          bundler-cache: true
+
+      - name: Workaround for https://github.com/actions/checkout/issues/290
+        run: |
+          git fetch --force --tags
+
+      - name: Do The Needful
+        env:
+          GEM_HOST_API_KEY: ${{ secrets.RUBYGEMS_API_KEY }}
+        run: |
+          bundle exec rake release:rubygem_push

data/bin/git-version-bump

@@ -1,14 +1,12 @@
 #!/usr/bin/env ruby
 
-require 'git-version-bump'
-
 if ARGV[0].nil? or
    ARGV[0].empty? or
    (ARGV.length == 1 && (ARGV[0] == "-d" || ARGV[0] == "--dry-run")) or
    ARGV[0] == '-h' or
    ARGV[0] == '--help'
 	$stderr.puts <<-EOF.gsub(/^\t\t/, '')
-		Usage: git version-bump [-n|--notes] [-d|--dry-run] <major|minor|patch|show>
+		Usage: git version-bump [-n|--notes] [-l|--lite-tags] [-d|--dry-run] <major|minor|patch|show>
 
 		'major': x.y.z -> x+1.0.0
 		'minor': x.y.z -> x.y+1.0
@@ -32,24 +30,40 @@ elsif ARGV[0] == '-h' or ARGV[0] == '--help'
 	exit 0
 end
 
-result = case ARGV[0].downcase
-	when /^maj?o?r?$/
-		"#{GVB.major_version(true) + 1}.0.0"
-	when /^min?o?r?$/
-		"#{GVB.major_version(true)}.#{GVB.minor_version(true)+1}.0"
-	when /^pa?t?c?h?$/
-		"#{GVB.major_version(true)}.#{GVB.minor_version(true)}.#{GVB.patch_version(true)+1}"
-	when /^sh?o?w?$/
-		puts GVB.version(true)
-		exit 0
-	else
-		$stderr.puts "Unknown argument: #{ARGV[0]}.  Try --help."
-		exit 1
-end
+begin
+  require 'git-version-bump'
+
+  result = case ARGV[0].downcase
+    when /^maj?o?r?$/
+      "#{GVB.major_version(true) + 1}.0.0"
+    when /^min?o?r?$/
+      "#{GVB.major_version(true)}.#{GVB.minor_version(true)+1}.0"
+    when /^pa?t?c?h?$/
+      "#{GVB.major_version(true)}.#{GVB.minor_version(true)}.#{GVB.patch_version(true)+1}"
+    when /^sh?o?w?$/
+      puts GVB.version(true)
+      exit 0
+    else
+      $stderr.puts "Unknown argument: #{ARGV[0]}.  Try --help."
+      exit 1
+  end
 
-if dry_run
-	puts result
-else
-	GVB.tag_version result, release_notes, lite_tags
-	puts "Version is now #{GVB.version(true)}."
+  if dry_run
+    puts result
+  else
+    unless GVB.tag_version result, release_notes, lite_tags
+      exit 1
+    end
+    puts "Version is now #{GVB.version(true)}."
+  end
+rescue GVB::VersionUnobtainable => ex
+  $stderr.puts "Could not obtain version information.  #{ex.message} (git available: #{GVB.git_available?.inspect})"
+  exit 1
+rescue GVB::CommandFailure => ex
+  $stderr.puts "#{ex.message} (exit status: #{ex.exitstatus})"
+  $stderr.puts "command output was:"
+  $stderr.puts "----8<----"
+  $stderr.puts ex.output
+  $stderr.puts "---->8----"
+  exit 1
 end

data/lib/git-version-bump/version.rb

@@ -1,8 +1,33 @@
 module GitVersionBump
-	VERSION = GVB.version
-	MAJOR_VERSION = GVB.major_version
-	MINOR_VERSION = GVB.minor_version
-	PATCH_VERSION = GVB.patch_version
-	INTERNAL_REVISION = GVB.internal_revision
-	DATE = GVB.date
+	def self.VERSION
+		GVB.version
+	end
+
+	def self.MAJOR_VERSION
+		GVB.major_version
+	end
+
+	def self.MINOR_VERSION
+		GVB.minor_version
+	end
+
+	def self.PATCH_VERSION
+		GVB.patch_version
+	end
+
+	def self.INTERNAL_REVISION
+		GVB.internal_revision
+	end
+
+	def self.DATE
+		GVB.date
+	end
+
+	def self.const_missing(c)
+		if self.respond_to?(c) && c =~ /\A[A-Z_]+\z/
+			public_send c
+		else
+			super
+		end
+	end
 end

data/lib/git-version-bump.rb

@@ -1,52 +1,35 @@
 require 'tempfile'
 require 'digest/sha1'
+require 'open3'
 require 'pathname'
 
 module GitVersionBump
 	class VersionUnobtainable < StandardError; end
+	class CommandFailure < StandardError
+		attr_accessor :output, :exitstatus
+
+		def initialize(m, output, exitstatus)
+			super(m)
+			@output, @exitstatus = output, exitstatus
+		end
+	end
 
 	VERSION_TAG_GLOB = 'v[0-9]*.[0-9]*.*[0-9]'
+	private_constant :VERSION_TAG_GLOB
 
 	DEVNULL = Gem.win_platform? ? "NUL" : "/dev/null"
+	private_constant :DEVNULL
 
-	def self.version(use_local_git=false, include_lite_tags=false)
-		if use_local_git
-			unless git_available?
-				raise RuntimeError,
-				      "GVB.version(use_local_git=true) called, but git isn't installed"
-			end
-
-			sq_git_dir = shell_quoted_string(Dir.pwd)
+	def self.version(use_local_dir=false, include_lite_tags=false)
+		if use_local_dir
+			repo_version(true, include_lite_tags)
 		else
-			sq_git_dir = shell_quoted_string((File.dirname(caller_file) rescue nil || Dir.pwd))
-		end
-
-		git_cmd = "git -C #{sq_git_dir} describe --dirty='.1.dirty.#{Time.now.strftime("%Y%m%d.%H%M%S")}' --match='#{VERSION_TAG_GLOB}'"
-		git_cmd << " --tags" if include_lite_tags
-
-		git_ver = `#{git_cmd} 2> #{DEVNULL}`.
-		            strip.
-		            gsub(/^v/, '').
-		            gsub('-', '.')
-
-		# If git returned success, then it gave us a described version.
-		# Success!
-		return git_ver if $? == 0
-
-		# git failed us; we're either not in a git repo or else we've never
-		# tagged anything before.
-
-		# Are we in a git repo with no tags?  If so, try to use the gemspec
-		# and if that fails then abort
-		begin
-			return gem_version(use_local_git)
-		rescue VersionUnobtainable
-			return "0.0.0.1.ENOTAG"
+			gem_version || repo_version(false, include_lite_tags)
 		end
 	end
 
-	def self.major_version(use_local_git=false, include_lite_tags=false)
-		ver = version(use_local_git, include_lite_tags)
+	def self.major_version(use_local_dir=false, include_lite_tags=false)
+		ver = version(use_local_dir, include_lite_tags)
 		v   = ver.split('.')[0]
 
 		unless v =~ /^[0-9]+$/
@@ -57,8 +40,8 @@ module GitVersionBump
 		return v.to_i
 	end
 
-	def self.minor_version(use_local_git=false, include_lite_tags=false)
-		ver = version(use_local_git, include_lite_tags)
+	def self.minor_version(use_local_dir=false, include_lite_tags=false)
+		ver = version(use_local_dir, include_lite_tags)
 		v   = ver.split('.')[1]
 
 		unless v =~ /^[0-9]+$/
@@ -69,8 +52,8 @@ module GitVersionBump
 		return v.to_i
 	end
 
-	def self.patch_version(use_local_git=false, include_lite_tags=false)
-		ver = version(use_local_git, include_lite_tags)
+	def self.patch_version(use_local_dir=false, include_lite_tags=false)
+		ver = version(use_local_dir, include_lite_tags)
 		v   = ver.split('.')[2]
 
 		unless v =~ /^[0-9]+$/
@@ -81,61 +64,33 @@ module GitVersionBump
 		return v.to_i
 	end
 
-	def self.internal_revision(use_local_git=false, include_lite_tags=false)
-		version(use_local_git, include_lite_tags).split('.', 4)[3].to_s
+	def self.internal_revision(use_local_dir=false, include_lite_tags=false)
+		version(use_local_dir, include_lite_tags).split('.', 4)[3].to_s
 	end
 
-	def self.date(use_local_git=false)
-		if use_local_git
-			unless git_available?
-				raise RuntimeError,
-				      "GVB.date(use_local_git=true), but git is not installed"
-			end
-
-			sq_git_dir = shell_quoted_string(Dir.pwd)
+	def self.date(use_local_dir=false, include_lite_tags = false)
+		if use_local_dir
+			repo_date(true, include_lite_tags)
 		else
-			sq_git_dir = shell_quoted_string((File.dirname(caller_file) rescue nil || Dir.pwd))
-		end
-
-		# Are we in a git tree?
-		system("git -C #{sq_git_dir} status > #{DEVNULL} 2>&1")
-		if $? == 0
-			# Yes, we're in git.
-
-			if dirty_tree?(sq_git_dir)
-				return Time.now.strftime("%F")
-			else
-				# Clean tree.  Date of last commit is needed.
-				return (`git -C #{sq_git_dir} show --no-show-signature --format=format:%cd --date=short`.lines.first || "").strip
-			end
-		else
-			if use_local_git
-				raise RuntimeError,
-				      "GVB.date(use_local_git=true) called from non-git location"
-			end
-
-			# Not in git; time to hit the gemspecs
-			if spec = caller_gemspec
-				return spec.date.strftime("%F")
-			end
-
-			raise RuntimeError,
-			      "GVB.date called from mysterious, non-gem location."
+			gem_date || repo_date(false, include_lite_tags)
 		end
 	end
 
 	def self.tag_version(v, release_notes = false, include_lite_tags=false)
 		if dirty_tree?
 			puts "You have uncommitted files.  Refusing to tag a dirty tree."
-		else
-			if release_notes
+			return false
+		end
+		if release_notes
+			log_file = Tempfile.new('gvb')
+
+			begin
 				# We need to find the tag before this one, so we can list all the commits
 				# between the two.  This is not a trivial operation.
-				git_cmd = "git describe --match='#{VERSION_TAG_GLOB}' --always"
-				git_cmd << ' --tags' if include_lite_tags
-				prev_tag = `#{git_cmd}`.strip.gsub(/-\d+-g[0-9a-f]+$/, '')
+				git_cmd = ["git", "describe", "--match=#{VERSION_TAG_GLOB}", "--always"]
+				git_cmd << "--tags" if include_lite_tags
 
-				log_file = Tempfile.new('gvb')
+				prev_tag = run_command(git_cmd, "getting previous release tag").strip.gsub(/-\d+-g[0-9a-f]+$/, '')
 
 				log_file.puts <<-EOF.gsub(/^\t\t\t\t\t/, '')
 
@@ -146,29 +101,30 @@ module GitVersionBump
 					# are listed below. This will become v#{v}
 					#
 				EOF
+				log_file.puts run_command(["git", "log", "--no-show-signature", "--format=# %h  %s", "#{prev_tag}..HEAD"], "getting commit range of release")
 
 				log_file.close
-				system("git log --no-show-signature --format='# %h  %s' #{prev_tag}..HEAD >>#{log_file.path}")
 
 				pre_hash = Digest::SHA1.hexdigest(File.read(log_file.path))
-				system("git config -e -f #{log_file.path}")
+				run_command(["git", "config", "-e", "-f", log_file.path], "editing release notes")
 				if Digest::SHA1.hexdigest(File.read(log_file.path)) == pre_hash
-					puts "Release notes not edited; aborting"
+					puts "Release notes not edited; not making release"
 					log_file.unlink
 					return
 				end
 
 				puts "Tagging version #{v}..."
-				system("git tag -a -F #{log_file.path} v#{v}")
+				run_command(["git", "tag", "-a", "-F", log_file.path, "v#{v}"], "tagging release with annotations")
+			ensure
 				log_file.unlink
-			else
-				# Crikey this is a lot simpler
-				system("git tag -a -m 'Version v#{v}' v#{v}")
 			end
-
-			system("git push > #{DEVNULL} 2>&1")
-			system("git push --tags > #{DEVNULL} 2>&1")
+		else
+			# Crikey this is a lot simpler
+			run_command(["git", "tag", "-a", "-m", "Version v#{v}", "v#{v}"], "tagging release")
 		end
+
+		run_command(["git", "push"], "pushing commits to the default remote repository")
+		run_command(["git", "push", "--tags"], "pushing tags to the default remote repository")
 	end
 
 	# Calculate a version number based on the date of the most recent git commit.
@@ -198,59 +154,87 @@ module GitVersionBump
 	# version.  Since hashes are hex, we're boned.  Sorry about that.  Don't
 	# make builds off a branch, basically.
 	#
-	def self.commit_date_version(use_local_git = false)
-		if use_local_git
-			unless git_available?
-				raise RuntimeError,
-				      "GVB.commit_date_version(use_local_git=true) called, but git isn't installed"
-			end
-
-			sq_git_dir = shell_quoted_string(Dir.pwd)
+	def self.commit_date_version(use_local_dir = false)
+		if use_local_dir
+			commit_date_version_string(true)
 		else
-			sq_git_dir = shell_quoted_string((File.dirname(caller_file) rescue nil || Dir.pwd))
+			gem_version || commit_date_version_string(false)
 		end
+	end
 
-		commit_dates = `git -C #{sq_git_dir} log --no-show-signature --format=%at`.
+	def self.commit_date_version_string(use_local_dir = false)
+		commit_dates = run_command(["git", "-C", git_dir(use_local_dir).to_s, "log", "--no-show-signature", "--format=%at"], "getting dates of all commits").
 		               split("\n").
 		               map { |l| Time.at(Integer(l)).strftime("%Y%m%d") }
 
-		if $? == 0
-			# We got a log; calculate our version number and we're done.
-			version_date = commit_dates.first
-			commit_count = commit_dates.select { |d| d == version_date }.length - 1
-			dirty_suffix = if dirty_tree?
-				".dirty.#{Time.now.strftime("%Y%m%d.%H%M%S")}"
-			else
-				""
-			end
-
-			return "0.#{version_date}.#{commit_count}#{dirty_suffix}"
+		version_date = commit_dates.first
+		commit_count = commit_dates.select { |d| d == version_date }.length - 1
+		dirty_suffix = if dirty_tree?
+			".dirty.#{Time.now.strftime("%Y%m%d.%H%M%S")}"
+		else
+			""
 		end
 
-		# git failed us; either we're not in a git repo or else it's a git
-		# repo that's not got any commits.
-
-		# Are we in a git repo with no commits?  If so, try to use the gemspec
-		# and if that fails then abort
-		begin
-			return gem_version(use_local_git)
-		rescue VersionUnobtainable
+		return "0.#{version_date}.#{commit_count}#{dirty_suffix}"
+	rescue CommandFailure => ex
+		p :GVB_CDVS_CMD_FAIL, ex.output if debug?
+		if ex.output =~ /fatal: your current branch .* does not have any commits yet/
 			return "0.0.0.1.ENOCOMMITS"
+		else
+			raise VersionUnobtainable, "Could not get commit date-based version from git repository at #{git_dir(use_local_dir)}"
 		end
 	end
 
-	private
-
 	def self.git_available?
-		system("git --version > #{DEVNULL} 2>&1")
-
-		$? == 0
+		try_command(["git", "--version"])
 	end
+	private_class_method :git_available?
 
-	def self.dirty_tree?(sq_git_dir='.')
+	def self.dirty_tree?(dir='.')
 		# Are we in a dirty, dirty tree?
-		! `git -C #{sq_git_dir} status --porcelain 2> #{DEVNULL}`.empty?
+		! run_command(["git", "-C", dir.to_s, "status", "--porcelain"], "checking for tree cleanliness").empty?
+	end
+	private_class_method :dirty_tree?
+
+	# Execute a command, specified as an array.
+	#
+	# On success, the full output of the command (stdout+stderr, interleaved) is returned.
+	# On error, a `CommandFailure` exception is raised.
+	#
+	def self.run_command(cmd, desc)
+		unless cmd.is_a?(Array)
+			raise ArgumentError, "Must pass command line arguments in an array"
+		end
+
+		unless cmd.all? { |s| s.is_a?(String) }
+			raise ArgumentError, "Command line arguments must be strings"
+		end
+
+		if debug?
+			p :GVB_CMD, desc, cmd
+		end
+
+		out, status = Open3.capture2e({"LC_MESSAGES" => "C"}, *cmd)
+
+		if status.exitstatus != 0
+			raise CommandFailure.new("Failed while #{desc}", out, status.exitstatus)
+		else
+			out
+		end
 	end
+	private_class_method :run_command
+
+	# Execute a command, and return whether it succeeded or failed.
+	#
+	def self.try_command(cmd)
+		begin
+			run_command(cmd, "try_command")
+			true
+		rescue CommandFailure
+			false
+		end
+	end
+	private_class_method :try_command
 
 	def self.caller_file
 		# Who called us?  Because this method gets called from other methods
@@ -260,9 +244,11 @@ module GitVersionBump
 		Pathname(
 		  caller_locations.
 		  map(&:path).
+		  tap { |c| p :CALLER_LOCATIONS, c if debug? }.
 		  find { |l| l != __FILE__ }
-		).realpath.to_s rescue nil
+		).realpath rescue nil
 	end
+	private_class_method :caller_file
 
 	def self.caller_gemspec
 		cf = caller_file or return nil
@@ -288,48 +274,75 @@ module GitVersionBump
 				end
 			end.compact!
 
-			if search_dirs.find { |d| cf.index(d) == 0 }
+			if search_dirs.find { |d| cf.to_s.index(d) == 0 }
 				return spec
 			end
 		end
 
-		raise VersionUnobtainable,
-		      "Unable to find gemspec for caller file #{cf}"
+		if debug?
+			p :GVB_NO_GEMSPEC, cf
+		end
+
+		nil
 	end
+	private_class_method :caller_gemspec
 
-	def self.gem_version(use_local_git = false)
-		if use_local_git
-			raise VersionUnobtainable,
-			      "Unable to determine version from local git repo.  This should never happen."
-		end
+	def self.gem_version
+		caller_gemspec&.version&.to_s
+	end
+	private_class_method :gem_version
 
-		if spec = caller_gemspec
-			return spec.version.to_s
-		else
-			# If we got here, something went *badly* wrong -- presumably, we
-			# weren't called from within a loaded gem, and so we've got *no*
-			# idea what's going on.  Time to bail!
-			if git_available?
-				raise VersionUnobtainable,
-				      "GVB.version(#{use_local_git.inspect}) failed, and I really don't know why."
+	def self.gem_date
+		caller_gemspec&.date&.strftime("%F")
+	end
+	private_class_method :gem_version
+
+	def self.repo_version(use_local_dir, include_lite_tags)
+		git_cmd = ["git", "-C", git_dir(use_local_dir).to_s, "describe", "--dirty=.1.dirty.#{Time.now.strftime("%Y%m%d.%H%M%S")}", "--match=#{VERSION_TAG_GLOB}"]
+		git_cmd << "--tags" if include_lite_tags
+
+		begin
+			run_command(git_cmd, "getting current version descriptor").
+			            strip.
+			            gsub(/^v/, '').
+			            gsub('-', '.')
+		rescue CommandFailure => ex
+			p :GVB_REPO_VERSION_FAILURE, ex.output if debug?
+			if ex.output =~ /fatal: No names found, cannot describe anything/
+				# aka "no tags, bro"
+				"0.0.0.1.ENOTAG"
 			else
-				raise VersionUnobtainable,
-				      "GVB.version(#{use_local_git.inspect}) failed; perhaps you need to install git?"
+				raise VersionUnobtainable, "Could not get version from gemspec or git repository at #{git_dir(use_local_dir)}"
 			end
 		end
 	end
+	private_class_method :repo_version
 
-	def self.shell_quoted_string(dir_string)
-		if Gem.win_platform?
-			return "\"#{dir_string}\""
+	def self.repo_date(use_local_dir, include_lite_tags)
+		if dirty_tree?(git_dir(use_local_dir))
+			Time.now.strftime("%F")
 		else
-			# Shell Quoted, for your convenience
-			return "'#{dir_string.gsub("'", "'\\''")}'"
+			# Clean tree.  Date of last commit is needed.
+			(run_command(["git", "-C", git_dir(use_local_dir).to_s, "show", "--no-show-signature", "--format=format:%cd", "--date=short"], "getting date of last commit").lines.first || "").strip
 		end
+	rescue CommandFailure
+		raise VersionUnobtainable, "Could not get commit date from git repository at #{git_dir(use_local_dir)}"
 	end
+	private_class_method :repo_date
 
-	private_class_method :shell_quoted_string
+	def self.git_dir(use_local_dir = false)
+		if use_local_dir
+			Dir.pwd
+		else
+			caller_file&.dirname || Dir.pwd
+		end.tap { |d| p :GVB_GIT_DIR, use_local_dir, d if debug? }
+	end
+	private_class_method :git_dir
 
+	def self.debug?
+		ENV.key?("GVB_DEBUG")
+	end
+	private_class_method :debug?
 end
 
 GVB = GitVersionBump unless defined? GVB

metadata

@@ -1,14 +1,14 @@
 --- !ruby/object:Gem::Specification
 name: git-version-bump
 version: !ruby/object:Gem::Version
-  version: 0.17.2
+  version: 0.18.0.7.g2e0c3c4
 platform: ruby
 authors:
 - Matt Palmer
 autorequire: 
 bindir: bin
 cert_chain: []
-date: 2022-04-01 00:00:00.000000000 Z
+date: 2023-01-11 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: github-release
@@ -74,6 +74,7 @@ extensions: []
 extra_rdoc_files:
 - README.md
 files:
+- ".github/workflows/release.yml"
 - ".gitignore"
 - Gemfile
 - LICENCE
@@ -98,9 +99,9 @@ required_ruby_version: !ruby/object:Gem::Requirement
       version: 2.1.0
 required_rubygems_version: !ruby/object:Gem::Requirement
   requirements:
-  - - ">="
+  - - ">"
     - !ruby/object:Gem::Version
-      version: '0'
+      version: 1.3.1
 requirements: []
 rubygems_version: 3.1.6
 signing_key: