getting syntax error in case statement in shell

Question

#!/bin/bash

branch=$1
vcva=$2
esx=$3
pxe=$4
setup=$5




#If branch is Vsphere-2015

if [ "$branch" == "vsphere2015"];then

      echo " Running Bats for Vsphere-2015 with the following details ."



if [ ! "vcva" ];then

    echo "VCVA Build is $2 "
    echo "ESX Build is $4 "
    echo "pxe info is $5 "


#If all the setups has to be run

setup=$5
case "$setup" in "all")

        echo "runnning all setups on Vsphere-2015."
        vpshere2015_primary
        vpshere2015_M1N1
        vpshere2015_M2N1                               =======> these are methods 
        vpshere2015_legacy
    ;;

I'm new to shell and after this piece of code getting

bat.sh: line 38: syntax error near unexpected token newline' 'at.sh: line 38: ;;

I want to run some functions depending upon the inputs given by the user in command line

You also need a space before the closing ] -- [ "$branch" == "vsphere2015"] should be [ "$branch" == "vsphere2015" ]. The reason: the [ command demands its last argument must be ], and arguments are separated by whitespace. — glenn jackman
– glenn jackman, Commented Sep 30, 2014 at 12:17

nu11p01n73R · Accepted Answer · 2014-09-30 11:56:44Z

1

The case satatement must end with esac

Syntax of case is

case EXPRESSION in CASE1) COMMAND-LIST;; CASE2) COMMAND-LIST;; ... CASEN) COMMAND-LIST;; esac

So here it should be

case "$setup" in "all")

        echo "runnning all setups on Vsphere-2015."
        vpshere2015_primary
        vpshere2015_M1N1
        vpshere2015_M2N1                               =======> these are methods 
        vpshere2015_legacy
    ;;

esac

here once $setup is matched with all the entire command list is excecuted

Also, it is not restricted that you should place all in quote " "

case "$setup" in all)

        echo "runnning all setups on Vsphere-2015."
        vpshere2015_primary
        vpshere2015_M1N1
        vpshere2015_M2N1                               =======> these are methods 
        vpshere2015_legacy
    ;;

esac

will also work fine

answered Sep 30, 2014 at 11:56

nu11p01n73R

26.8k3 gold badges42 silver badges52 bronze badges

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

Comments

Sergio-Nabil Khayyat Arranz · Accepted Answer · 2014-09-30 13:31:02Z

You just have to close all the if's (with "fi") and the case statement (with "esac").

I thing you also have to change the

if [ ! "vcva" ];then

into

if [ ! -z "$vcva" ] ; then

It will result something like:

#!/bin/bash

branch=$1
vcva=$2
esx=$3
pxe=$4
setup=$5

#If branch is Vsphere-2015
if [ "$branch" == "vsphere2015" ];then
  echo " Running Bats for Vsphere-2015 with the following details ."
  if [ ! -z "$vcva" ];then
    echo "VCVA Build is $2 "
    echo "ESX Build is $4 "
    echo "pxe info is $5 "

    #If all the setups has to be run
    setup=$5
    case "$setup" in 
      "all")    
        echo "runnning all setups on Vsphere-2015."
        vpshere2015_primary
        vpshere2015_M1N1
        vpshere2015_M2N1
        vpshere2015_legacy
      ;;
      *) 
        echo "default action goes here"
      ;;
    esac
  fi # close second if
fi # close first if

Collectives™ on Stack Overflow

getting syntax error in case statement in shell

2 Answers 2

Comments

Comments

Your Answer

Hot Network Questions

Collectives™ on Stack Overflow

2 Answers 2

Comments

Comments

Your Answer

Sign up or log in

Post as a guest

Related