Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(45)

Side by Side Diff: tools/merge-to-branch.sh

Issue 9689062: Fix "already merging" safety check in merge-to-branch.sh (Closed) Base URL: https://v8.googlecode.com/svn/branches/bleeding_edge
Patch Set: Created 8 years, 9 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View unified diff | Download patch | Annotate | Revision Log
« no previous file with comments | « no previous file | no next file » | no next file with comments »
Toggle Intra-line Diffs ('i') | Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
OLDNEW
1 #!/bin/bash 1 #!/bin/bash
2 # Copyright 2012 the V8 project authors. All rights reserved. 2 # Copyright 2012 the V8 project authors. All rights reserved.
3 # Redistribution and use in source and binary forms, with or without 3 # Redistribution and use in source and binary forms, with or without
4 # modification, are permitted provided that the following conditions are 4 # modification, are permitted provided that the following conditions are
5 # met: 5 # met:
6 # 6 #
7 # * Redistributions of source code must retain the above copyright 7 # * Redistributions of source code must retain the above copyright
8 # notice, this list of conditions and the following disclaimer. 8 # notice, this list of conditions and the following disclaimer.
9 # * Redistributions in binary form must reproduce the above 9 # * Redistributions in binary form must reproduce the above
10 # copyright notice, this list of conditions and the following 10 # copyright notice, this list of conditions and the following
(...skipping 70 matching lines...) Expand 10 before | Expand all | Expand 10 after
81 exit 1 81 exit 1
82 ;; 82 ;;
83 esac 83 esac
84 done 84 done
85 let OPTION_COUNT=$OPTIND-1 85 let OPTION_COUNT=$OPTIND-1
86 shift $OPTION_COUNT 86 shift $OPTION_COUNT
87 87
88 ########## Regular workflow 88 ########## Regular workflow
89 89
90 # If there is a merge in progress, abort. 90 # If there is a merge in progress, abort.
91 [[ -e "$ALREADY_MERGING_SENTINEL_FILE" ]] && [[ -z "$START_STEP" ]] \ 91 [[ -e "$ALREADY_MERGING_SENTINEL_FILE" ]] && [[ $START_STEP -eq 0 ]] \
92 && die "A merge is already in progress" 92 && die "A merge is already in progress"
93 touch "$ALREADY_MERGING_SENTINEL_FILE" 93 touch "$ALREADY_MERGING_SENTINEL_FILE"
94 94
95 initial_environment_checks 95 initial_environment_checks
96 96
97 if [ $START_STEP -le $CURRENT_STEP ] ; then 97 if [ $START_STEP -le $CURRENT_STEP ] ; then
98 echo ">>> Step $CURRENT_STEP: Preparation" 98 echo ">>> Step $CURRENT_STEP: Preparation"
99 MERGE_TO_BRANCH=$1 99 MERGE_TO_BRANCH=$1
100 [[ -n "$MERGE_TO_BRANCH" ]] || die "Please specify a branch to merge to" 100 [[ -n "$MERGE_TO_BRANCH" ]] || die "Please specify a branch to merge to"
101 shift 101 shift
(...skipping 132 matching lines...) Expand 10 before | Expand all | Expand 10 after
234 https://v8.googlecode.com/svn/$TO_URL \ 234 https://v8.googlecode.com/svn/$TO_URL \
235 https://v8.googlecode.com/svn/tags/$NEWMAJOR.$NEWMINOR.$NEWBUILD.$NEWPATCH \ 235 https://v8.googlecode.com/svn/tags/$NEWMAJOR.$NEWMINOR.$NEWBUILD.$NEWPATCH \
236 -m "Tagging version $NEWMAJOR.$NEWMINOR.$NEWBUILD.$NEWPATCH" 236 -m "Tagging version $NEWMAJOR.$NEWMINOR.$NEWBUILD.$NEWPATCH"
237 fi 237 fi
238 238
239 let CURRENT_STEP+=1 239 let CURRENT_STEP+=1
240 if [ $START_STEP -le $CURRENT_STEP ] ; then 240 if [ $START_STEP -le $CURRENT_STEP ] ; then
241 echo ">>> Step $CURRENT_STEP: Cleanup." 241 echo ">>> Step $CURRENT_STEP: Cleanup."
242 common_cleanup 242 common_cleanup
243 fi 243 fi
OLDNEW
« no previous file with comments | « no previous file | no next file » | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698