bash add git command result to local variable

Question

I am trying to change the name of my file, adding the number of commits and the current branch name on my file before sending it to the repository.

But I get the following error:

./publish_to_artifactory.sh: line 4: local: `rev-parse': not a valid identifier'`

upload() {
  local currentBranch=git rev-parse --abbrev-ref HEAD;
  local gitNumberOfCommits=git rev-list --count HEAD;

  for file in ./../*.ipa; do
    mv $file mobile-${currentBranch}-${gitNumberOfCommits};
    echo "uploading zeos-mobile-$currentBranch-$gitNumberOfCommits";
    curl -X PUT -u $PUBLISH_USER:$PUBLISH_PASS -T mobile-$currentBranch-$gitNumberOfCommits; http://example.com/artifactory/ios-dev-local/ --fail
  done
}

upload;

What am I doing wrong?

BTW, you shouldn't put command substitutions on the same line as local at all -- doing so shadows the exit status of those substitutions with the exit status of the local command. — Charles Duffy
– Charles Duffy, Commented Jan 3, 2018 at 17:05
Instead, run local currentBranch gitNumberOfCommits as one line, and then put currentBranch=$(...) and gitNumberOfCommits=$(...) each on a separate line. See Why does local sweep the return code of a command? — Charles Duffy
– Charles Duffy, Commented Jan 3, 2018 at 17:06
Consider running your code through shellcheck.net and also fixing the various quoting bugs it identifies. — Charles Duffy
– Charles Duffy, Commented Jan 3, 2018 at 17:07

Inian · Accepted Answer · 2018-01-03 17:03:18Z

4

To assign the output of a command to a variable, you need to use command-substitution syntax in bash.

local currentBranch="$(git rev-parse --abbrev-ref HEAD)"
local gitNumberOfCommits="$(git rev-list --count HEAD)"

What you have done is stored the literal string in those local variables defined. When shell was tokenizing them to evaluate it has understood it as a variable assignment gone horribly wrong! What it understood is a local variable in the name of currentBranch set to a value git and tries to run rev-list as a command which it cannot obviously find in its list of standard paths (under $PATH)

edited Jan 3, 2018 at 17:03

answered Jan 3, 2018 at 16:45

Inian

87.1k15 gold badges167 silver badges183 bronze badges

Sign up to request clarification or add additional context in comments.

1 Comment

torek Over a year ago

Also, local a=b c means, in effect, local a=b; local c. Note that the result of expansion ($(git rev-parse ...)) won't re-expand here, so you could write local currentBranch=$(git rev-parse --abbrev-rev HEAD) even if git rev-parse might produce more than one word; but the double quotes are useful for our own sanity. :-) Finally, as Charles Duffy noted, if you want to check the exit status of a substitution, you have to separate it out from the local declaration.

Collectives™ on Stack Overflow

bash add git command result to local variable

1 Answer 1

1 Comment

Your Answer

Linked

Hot Network Questions

Collectives™ on Stack Overflow

1 Answer 1

1 Comment

Your Answer

Sign up or log in

Post as a guest

Linked

Related