Commit b85b7e2b authored by Michal Berger's avatar Michal Berger Committed by Tomasz Zawadzki
Browse files

pkgdep/git: Don't indicate error if vagrant installation is not supported 



This would unecessarily print out usage() and call exit which wouldn't
be respected since all sources (except {,ref}spdk) are installed in the
background anyway.

Also, since std{out,err} may be bloated with bunch of messages comming
from different sources, explicitly mention Vagrant name in the echo.

Change-Id: I737f5cf08d6ae3d2d494add143d1dfaf89beeead
Signed-off-by: default avatarMichal Berger <michalx.berger@intel.com>
Reviewed-on: https://review.spdk.io/gerrit/c/spdk/spdk/+/4189


Tested-by: default avatarSPDK CI Jenkins <sys_sgci@intel.com>
Reviewed-by: default avatarJim Harris <james.r.harris@intel.com>
Reviewed-by: default avatarTomasz Zawadzki <tomasz.zawadzki@intel.com>
parent 4ea04101

test/common/config/pkgdep/git

+2 −1
Original line number Diff line number Diff line
@@ -270,7 +270,8 @@ function install_vagrant() { 
	local vagrant_plugins=(vagrant-libvirt vagrant-sshfs vagrant-cachier vagrant-proxyconf)



	if [[ $OSID != ubuntu ]]; then

		error "Currently only ubuntu is supported"

		echo "Currently, Vagrant installation is supported only on ubuntu"

		return 0

	fi



	# Install vagrant and it's plugins dependencies