Skip to content

Commit

Permalink
Run git reset before pulling, to avoid error on dirty master branch
Browse files Browse the repository at this point in the history
also adding line breaks for readability

Signed-off-by: RD WebDesign <[email protected]>
  • Loading branch information
rdwebdesign committed Sep 20, 2022
1 parent da6596e commit e4e3914
Showing 1 changed file with 6 additions and 2 deletions.
8 changes: 6 additions & 2 deletions automated install/basic-install.sh
Original file line number Diff line number Diff line change
Expand Up @@ -469,17 +469,21 @@ update_repo() {
pushd "${directory}" &> /dev/null || return 1
# Let the user know what's happening
printf " %b %s..." "${INFO}" "${str}"

# Stash any local commits as they conflict with our working code
git stash --all --quiet &> /dev/null || true # Okay for stash failure
git clean --quiet --force -d || true # Okay for already clean directory
# Pull the latest commits
git pull --no-rebase --quiet &> /dev/null || return $?

# Check current branch. If it is master, then reset to the latest available tag.
# In case extra commits have been added after tagging/release (i.e in case of metadata updates/README.MD tweaks)
curBranch=$(git rev-parse --abbrev-ref HEAD)
if [[ "${curBranch}" == "master" ]]; then
git reset --hard "$(git describe --abbrev=0 --tags)" || return $?
fi

# Pull the latest commits
git pull --no-rebase --quiet &> /dev/null || return $?

# Show a completion message
printf "%b %b %s\\n" "${OVER}" "${TICK}" "${str}"
# Data in the repositories is public anyway so we can make it readable by everyone (+r to keep executable permission if already set by git)
Expand Down

0 comments on commit e4e3914

Please sign in to comment.