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

scripts/setup: Add editor mode 



This mode is meant to immitate (poorly) git's interactive mode by
allowing user to modify list of devices in the $EDITOR.

For now use it only for overriding the "in use" state of the device.
Option is hidden under '5e'.

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


Reviewed-by: default avatarKonrad Sztyber <konrad.sztyber@intel.com>
Tested-by: default avatarSPDK CI Jenkins <sys_sgci@intel.com>
Reviewed-by: default avatarJim Harris <jim.harris@gmail.com>
parent 7a1bd398

scripts/common/setup/interactive.sh

+53 −1
Original line number Diff line number Diff line
@@ -66,6 +66,7 @@ main_menu() { 
			3) fdevices 0 ;;

			4) fdevices 1 ;;

			5) odevices ;;

			5e) editor odevices ;;

			6) bdevices ;;

			q) yn "Are you sure you want to quit?" && return 1 ;;

			c | commit | config)
@@ -99,14 +100,20 @@ gdevices() { 
pdevices() {

	gdevices



	local set_marker=$1

	local use_map=()

	local -A markers=()



	use_map[0]="not used" use_map[1]="used"

	markers["not used"]=pick markers["used"]=skip



	if ((${#dev_ref[@]} == 0)); then

		echo "No devices found"

	else

		for dev in "${!dev_ref[@]}"; do

			echo "- $dev [${use_map[all_devices_d["$dev"]]}, ${drivers_d["$dev"]:-none}]"

			printf '%s- %s [%s, %s]\n' \

				"${set_marker:+${markers["${use_map[all_devices_d["$dev"]]}"]} }" \

				"$dev" "${use_map[all_devices_d["$dev"]]}" "${drivers_d["$dev"]:-none}"

		done

	fi

}
@@ -157,6 +164,51 @@ fdevices() { 
	done

}



editor() {

	local devs_list=() devs_to_modify=()

	local editor=${VISUAL:-${EDITOR:-vim}}

	local tmp_file



	type -P "$editor" > /dev/null || return



	mapfile -t devs_list < <(pdevices markers)



	tmp_file=$(mktemp -u)

	cat <<- ODEVICES > "$tmp_file" || return

		# Listing '$type' devices

		#   Devices marked as "used" (i.e. contains any data) will be skipped by default

		#   Devices marked as "not used" (i.e. does not contain data) will be picked by default



		$(printf '%s\n' "${devs_list[@]}")



		# p, pick devices

		# s, skip devices



		$([[ $editor == vi?(m) ]] && echo "# :cq[!] to not save any changes")

	ODEVICES



	"$editor" "$tmp_file" || return

	[[ -s $tmp_file ]] || return



	local action dev _type

	while read -r action _ dev _type; do

		case "${action,,}" in

			s | skip)

				[[ $_type != *"not used"* ]] && continue

				devs_to_modify+=("$dev")

				;;

			p | pick)

				[[ $_type == *"not used"* ]] && continue

				devs_to_modify+=("$dev")

				;;

		esac

	done < "$tmp_file"

	rm "$tmp_file"



	"$1" < <(printf '%s\n' "${devs_to_modify[@]}")



}



odevices() {

	local bdf