git-maintain 0.2.2.pre.25.gb2af5c1 → 0.2.2

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: 3d1ec5d7fc571c7dde83904b709a9bed65d9313232900e47e84258ba034315ca
-  data.tar.gz: 3beefd08e2a0c8666ccc3a91fad973475cbbc284bd54d4ad309b16e1e2a7cec0
+  metadata.gz: bde61f3ac980a295c32bbb076a99176d6d93cd82972d7ce8da69954958bfc056
+  data.tar.gz: 266b8df2a5de8795850c70a9b89dbac008d613ac688ee4e3cfd41c4a90895d01
 SHA512:
-  metadata.gz: 630b64de94f82732aa71a98002781bb23750d0a132dc59a5a00c1ef64dd5de3c07521319fa47bb9cf96e3b86dac4c7703a1be1e07e6553f9d20e6c2be46b6ab4
-  data.tar.gz: 3a0d3b47a05becc6c1f07bae3c411612e08d185baa4b41ddf312d94dc84781780e0f18bd18d80192cfcf3e0f692f57ee7ee908dda3b591fcaaab5f59ab859db2
+  metadata.gz: f62502f4308c146bbb87411e3fdb9c397b38f23b5c8e206e836b971e518d0670173af4682d17f4fd96bdac2aa02875c63fb9cf8ae093782bc114712d6cc2f764
+  data.tar.gz: 63e7f063e94f6586dbb5d2c1d747ba16851b58a17c7c095e132e520c8a56b439ac9a857731a0e6d3c22c8c0fe8db472f830fa3ea09df9cb569a07e293c5a6bf3

data/CHANGELOG

@@ -1,16 +1,7 @@
 ------------------
-0.3.0  (2018-10-09)
+0.2.3
 ------------------
 
-* Fix support for commit messages containing double quotes
-* Fix --base-version and --branch-suffix
-* Add completion support for --version
-* Fix --no-travis
-* Customizable behaviour for travis/emails in Addons
-* Allow branch action to be run on user-specified branches (instead of the stable pattern)
-* Add hpc-testing addon
-* Add git-maintain addon
-
 ------------------
 0.2.2  (2018-07-13)
 ------------------

data/README.md

@@ -27,11 +27,9 @@ The idea is to script most of the maintenance tasks so the maintainer can focus
 - Load git-maintain-completion.sh for shell completion
 
 ## Remote setup
-- the 'github' remote should be your own WIP github to test out branches before submitting to the official repo.
-  Also know as the 'Validation' repo. It can be switched to another remote by setting maintain.valid-repo=xxx in your gitconfig
+- the 'github' remote should be your own WIP github to test out branches before submitting to the official repo
 - the 'origin' remote should be the official repo in read-only mode to avoid any accidental pushes
 - the 'stable' remote should be the official repo in RW mode
-  Also know as the 'Stable' repo. It can be switched to another remote by setting maintain.stable-repo=xxx in your gitconfig
 
 ## Stealing commits
 

data/git-maintain-completion.sh

@@ -22,10 +22,6 @@ _complete_git_maintain_branch(){
 			BRANCHES=
 			__gitcomp_nl "$(git maintain list_branches)"
 			;;
-		-V|--version)
-			# Extra arg expected but not completable
-			__gitcomp_nl ""
-			;;
 	esac
 }
 
@@ -92,9 +88,6 @@ _git_maintain(){
 			 case "$prev" in
 				 *)
 					__gitcomp_nl "$OPT_LIST"
-					# Override default completion with specific branch completion (if it matches)
-					_complete_git_maintain_branch
-
 			 esac
 		 fi
 	fi

data/lib/branch.rb

@@ -52,10 +52,6 @@ module GitMaintain
             optsParser.on("-V", "--version [regexp]", Regexp, "Regexp to filter versions.") {
                 |val| opts[:version] = val}
 
-            if action != :merge
-                optsParser.on("-B", "--manual-branch <branch name>", "Work on a specific (non-stable) branch.") {
-                    |val| opts[:manual_branch] = val}
-            end
             case action
             when :cp
                 optsParser.banner += "-c <sha1> [-c <sha1> ...]"
@@ -72,7 +68,7 @@ module GitMaintain
             when :push_stable
                 optsParser.banner += "[-T]"
                 optsParser.on("-T", "--no-travis", "Ignore Travis build status and push anyway.") {
-                    |val| opts[:no_travis] = true}
+                    |val| opts[:no_travis] = val}
             end
         end
 
@@ -93,26 +89,19 @@ module GitMaintain
                 repo.stableUpdate()
             end
 
-            branchList=[]
-            if opts[:manual_branch] == nil then
-                branchList = repo.getStableList(opts[:br_suff]).map(){|br|
-                    branch = Branch::load(repo, br, travis, opts[:br_suff])
-                    case branch.is_targetted?(opts)
-                    when :too_old
-                        puts "# Skipping older v#{branch.version}"
-                        next
-                    when :no_match
-                        puts "# Skipping v#{branch.version} not matching #{opts[:version].to_s()}"
-                        next
-                    end
-                    branch
-                }.compact()
-            else
-                branchList = [ Branch::load(repo, opts[:manual_branch], travis, opts[:br_suff]) ]
-            end
-            branchList.each(){|branch|
+            repo.getStableList(opts[:br_suff]).each(){|br|
+                branch = Branch::load(repo, br, travis, opts[:br_suff])
+                case branch.is_targetted?(opts)
+                when :too_old
+                    puts "# Skipping older v#{branch.version}"
+                    next
+                when :no_match
+                    puts "# Skipping v#{branch.version} not matching #{opts[:version].to_s()}"
+                    next
+                end
+
                 puts "###############################"
-                puts "# Working on #{branch.verbose_name}"
+                puts "# Working on v#{branch.version}"
                 puts "###############################"
 
                 if NO_CHECKOUT_ACTIONS.index(action) == nil  then
@@ -126,31 +115,21 @@ module GitMaintain
             GitMaintain::checkDirectConstructor(self.class)
 
             @repo          = repo
-            @travis        = travis
             @version       = version
+            @travis        = travis
             @branch_suff   = branch_suff
 
-            if version =~ /^[0-9]+$/
-                @local_branch  = "dev/stable-v#{@version}/#{@branch_suff}"
-                @remote_branch ="stable-v#{@version}"
-                @branch_type = :std
-                @verbose_name = "v"+version
-            else
-                @remote_branch = @local_branch = version
-                @branch_type = :user_specified
-                @verbose_name = version
-            end
-
+            @local_branch  = "dev/stable-v#{@version}/#{@branch_suff}"
             @head          = @repo.runGit("rev-parse #{@local_branch}")
-            @remote_ref    = "#{@repo.stable_repo}/#{@remote_branch}"
+
+            @remote_branch ="stable-v#{@version}"
+            @remote_ref    = "#{Repo::STABLE_REPO}/#{@remote_branch}"
             @stable_head   = @repo.runGit("rev-parse #{@remote_ref}")
             @stable_base   = @repo.findStableBase(@local_branch)
-
         end
-        attr_reader :version, :local_branch, :head, :remote_branch, :remote_ref, :stable_head, :verbose_name
+        attr_reader :version, :local_branch, :head, :remote_branch, :remote_ref, :stable_head
 
         def is_targetted?(opts)
-            return true if @branch_type == :user_specified
             if @version.to_i < opts[:base_ver] then
                 return :too_old
             end
@@ -209,7 +188,7 @@ module GitMaintain
 
         # Push the branch to the validation repo
         def push(opts)
-           @repo.runGit("push #{opts[:push_force] == true ? "-f" : ""} #{@repo.valid_repo} #{@local_branch}")
+           @repo.runGit("push #{opts[:push_force] == true ? "-f" : ""} #{Repo::VALID_REPO} #{@local_branch}")
         end
 
         # Monitor the build status on Travis
@@ -237,14 +216,14 @@ module GitMaintain
 
         # Push branch to the stable repo
         def push_stable(opts)
-            if (opts[:no_travis] != true && @NO_TRAVIS != true) &&
+            if opts[:no_travis] != true &&
                @travis.checkValidState(@head) != true then
                 puts "Build is not passed on travis. Skipping push to stable"
                 return
             end
             rep = GitMaintain::checkLog(opts, @local_branch, @remote_ref, "submit")
             if rep == "y" then
-                @repo.runGit("push #{@repo.stable_repo} #{@local_branch}:#{@remote_branch}")
+                @repo.runGit("push #{Repo::STABLE_REPO} #{@local_branch}:#{@remote_branch}")
             else
                 puts "Skipping push to stable"
                 return
@@ -425,7 +404,6 @@ module GitMaintain
 
         def steal_one(opts, commit)
 		    subj=@repo.runGit("log -1 --format=\"%s\" #{commit}")
-            subj.gsub!(/"/, '\"')
 		    msg=''
 
 		    # Let's grab the mainline commit id, this is useful if the version tag

data/lib/repo.rb

@@ -1,8 +1,8 @@
 module GitMaintain
     class Repo
-        @@VALID_REPO = "github"
-        @@STABLE_REPO = "stable"
-        @@SUBMIT_BINARY="git-release"
+        VALID_REPO = "github"
+        STABLE_REPO = "stable"
+        SUBMIT_BINARY="git-release"
 
         ACTION_LIST = [
             :list_branches,
@@ -47,16 +47,10 @@ module GitMaintain
             if path == nil
                 @path = Dir.pwd()
             end
-
-            @valid_repo = runGit("config maintain.valid-repo 2> /dev/null").chomp()
-            @valid_repo = @@VALID_REPO if @valid_repo == ""
-            @stable_repo = runGit("config maintain.stable-repo 2>/dev/null").chomp()
-            @stable_repo = @@STABLE_REPO if @stable_repo == ""
-
-            @remote_valid=runGit("remote -v | egrep '^#{@valid_repo}' | grep fetch |
-                                awk '{ print $2}' | sed -e 's/.*://' -e 's/\\.git//'")
-            @remote_stable=runGit("remote -v | egrep '^#{@stable_repo}' | grep fetch |
-                                      awk '{ print $2}' | sed -e 's/.*://' -e 's/\\.git//'")
+            @remote_valid=runGit("remote -v | egrep '^#{VALID_REPO}' | grep fetch |
+                                awk '{ print $2}' | sed -e 's/.*://' -e 's/\.git//'")
+            @remote_stable=runGit("remote -v | egrep '^#{STABLE_REPO}' | grep fetch |
+                                      awk '{ print $2}' | sed -e 's/.*://' -e 's/\.git//'")
             @stable_base_patterns=
                 runGit("config --get-regexp   stable-base | egrep '^stable-base\.' | "+
                        "sed -e 's/stable-base\.//' -e 's/---/\\//g'").split("\n").inject({}){ |m, x|
@@ -65,7 +59,7 @@ module GitMaintain
                 m
                 }
         end
-        attr_reader :path, :remote_valid, :remote_stable, :valid_repo, :stable_repo
+        attr_reader :path, :remote_valid, :remote_stable
 
         def run(cmd)
             return `cd #{@path} && #{cmd}`
@@ -92,7 +86,7 @@ module GitMaintain
 
         def stableUpdate()
             puts "# Fetching stable updates..."
-            runGit("fetch #{@stable_repo}")
+            runGit("fetch #{STABLE_REPO}")
         end
         def getStableList(br_suff)
             return @stable_list if @stable_list != nil
@@ -116,8 +110,8 @@ module GitMaintain
             return @suffix_list
         end
 
-        def submitReleases(opts)
-            remote_tags=runGit("ls-remote --tags #{@stable_repo} |
+        def submitReleases()
+            remote_tags=runGit("ls-remote --tags #{STABLE_REPO} |
                                  egrep 'refs/tags/v[0-9.]*$'").split("\n").map(){
                 |x| x.gsub(/.*refs\/tags\//, '')
             }
@@ -130,49 +124,47 @@ module GitMaintain
             end
 
             puts "This will officially release these tags: #{new_tags.join(", ")}"
-            rep = GitMaintain::confirm(opts, "release them")
+            rep = GitMaintain::confirm("release them")
             if rep != 'y' then
                 raise "Aborting.."
             end
 
-            if @NOTIFY_RELEASE != false
-                mail_path=`mktemp`.chomp()
-                mail = File.open(mail_path, "w+")
-                mail.puts "From " + runGit("rev-parse HEAD") + " " + `date`.chomp()
-                mail.puts "From: " + runGit("config user.name") +
-                          " <" + runGit("config user.email") +">"
-                mail.puts "To: " + runGit("config patch.target")
-                mail.puts "Date: " + `date -R`.chomp()
-                mail.puts "Subject: [ANNOUNCE] " + File.basename(@path) + " " +
-                          (new_tags.length > 1 ?
-                               (new_tags[0 .. -2].join(", ") + " and " + new_tags[-1]) :
-                               new_tags.join(" ")) +
-                          " has been tagged/released"
-                mail.puts ""
-                mail.puts "Here's the information from the tags:"
-                new_tags.sort().each(){|tag|
-                    mail.puts `git show #{tag} --no-decorate -q | awk '!p;/^-----END PGP SIGNATURE-----/{p=1}'`
-                    mail.puts ""
-                }
-                mail.puts "It's available at the normal places:"
+            mail_path=`mktemp`.chomp()
+            mail = File.open(mail_path, "w+")
+            mail.puts "From " + runGit("rev-parse HEAD") + " " + `date`.chomp()
+            mail.puts "From: " + runGit("config user.name") +
+                      " <" + runGit("config user.email") +">"
+            mail.puts "To: " + runGit("config patch.target")
+            mail.puts "Date: " + `date -R`.chomp()
+            mail.puts "Subject: [ANNOUNCE] " + File.basename(@path) + " " +
+                      (new_tags.length > 1 ?
+                           (new_tags[0 .. -2].join(", ") + " and " + new_tags[-1]) :
+                           new_tags.join(" ")) +
+                      " has been tagged/released"
+            mail.puts ""
+            mail.puts "Here's the information from the tags:"
+            new_tags.sort().each(){|tag|
+                mail.puts `git show #{tag} --no-decorate -q | awk '!p;/^-----END PGP SIGNATURE-----/{p=1}'`
                 mail.puts ""
-                mail.puts "git://github.com/#{@remote_stable}"
-                mail.puts "https://github.com/#{@remote_stable}/releases"
-                mail.close()
+            }
+            mail.puts "It's available at the normal places:"
+            mail.puts ""
+            mail.puts "git://github.com/#{@remote_stable}"
+            mail.puts "https://github.com/#{@remote_stable}/releases"
+            mail.close()
 
-                puts runGitImap("< #{mail_path}; rm -f #{mail_path}")
-            end
+            puts runGitImap("< #{mail_path}; rm -f #{mail_path}")
 
             puts "Last chance to cancel before submitting"
-            rep= GitMaintain::confirm(opts, "submit these releases")
+            rep= GitMaintain::confirm("submit these releases")
             if rep != 'y' then
                 raise "Aborting.."
             end
-            puts `#{@@SUBMIT_BINARY}`
+            puts `#{SUBMIT_BINARY}`
         end
         def findStableBase(branch)
             @stable_base_patterns.each(){|pattern, base|
-                return base if branch =~ /#{pattern}\// || branch =~ /#{pattern}$/
+                return base if branch =~ /#{pattern}\//
             }
             raise("Could not a find a stable base for branch #{branch}")
         end
@@ -184,7 +176,7 @@ module GitMaintain
             puts getSuffixList()
         end
         def submit_release(opts)
-            submitReleases(opts)
+            submitReleases()
         end
     end
 end

metadata

@@ -1,7 +1,7 @@
 --- !ruby/object:Gem::Specification
 name: git-maintain
 version: !ruby/object:Gem::Version
-  version: 0.2.2.pre.25.gb2af5c1
+  version: 0.2.2
 platform: ruby
 authors:
 - Nicolas Morey-Chaisemartin
@@ -58,9 +58,9 @@ required_ruby_version: !ruby/object:Gem::Requirement
       version: '0'
 required_rubygems_version: !ruby/object:Gem::Requirement
   requirements:
-  - - ">"
+  - - ">="
     - !ruby/object:Gem::Version
-      version: 1.3.1
+      version: '0'
 requirements: []
 rubyforge_project: 
 rubygems_version: 2.7.7