Evolution #3

2024-02-11T23:01:57+01:00

nmorin commented

2024-02-11 23:01:57 +01:00

No description provided.

nmorin added 4 commits 2024-02-11 23:01:58 +01:00

Ajout d'une vérification sur la présence d'un tiret au premier caractère du paramètre passé. Cela permet d'éviter de considérer la prochaine option comme paramètre de l'option courante 3c6e04f845

Ajout du paramètre obligatoire "Type de disque" à l'option "-d" + contrôle du format de disque renseigné par l'utilisateur. 521d192e05

Ajout du paramètre obligatoire "Ajouter une interface réseau" afin de permettre la désactivation totale du réseau de Qemu si "n" ou "N" est spécifié 1477ef8072

Ajout de deux variables globales contenant le chemin des binaires "ip" et "bridge" afin de permettre leur exécution en dehors de "$PATH" 1d657e728b

nmorin reviewed 2024-02-11 23:02:32 +01:00

qemu-adm

						
				@@ -73,2 +75,2 @@

							echo "${2}"

							if [[ ! -z ${2} ]]; then

							echo "${2}" # Debogage

							if [[ ! -z "${2}" && "${2}" =~ ^(raw|qcow2)$ ]]; then

nmorin commented

2024-02-11 23:02:32 +01:00

Ce test dois être mis au niveau de la saisi utilisateur. Ici, le fichier sera déjà créé. A l'activation de la fonction verifMVExiste, il sera impossible à l'utilisateur de créer une nouvelle fois sa MV sans passer par une suppression.

Ce test dois être mis au niveau de la saisi utilisateur. Ici, le fichier sera déjà créé. A l'activation de la fonction `verifMVExiste`, il sera impossible à l'utilisateur de créer une nouvelle fois sa MV sans passer par une suppression.

nmorin reviewed 2024-02-11 23:07:03 +01:00

qemu-adm

						
				@@ -264,1 +275,3 @@

								commandes_a_executer+=( "nouvMV disque ${disque_nom} ${disque_format} ${disque_capacite}")

								disque_type=$( echo ${1} | awk -F: '{print $4}' )

								echo "disque_type=${disque_type}" # Debogage

								if [[ -z "${disque_type}" && "$(dirname ${1})" == "." ]]; then

nmorin commented

2024-02-11 23:07:03 +01:00

La condition && "$(dirname ${1})" == "." est a revoir dans l'ensemble des conditions de l'option `-d.

La condition `&& "$(dirname ${1})" == "."` est a revoir dans l'ensemble des conditions de l'option `-d.

This pull request can be merged automatically.

You are not authorized to merge this pull request.

View command line instructions

Checkout

From your project repository, check out a new branch and test the changes.

git fetch -u origin ycharbi:ycharbi

git checkout ycharbi

Sign in to join this conversation.

1 Participants

Notifications

Due Date

No due date set.

Dependencies

No dependencies set.

Reference: YC-NM/Qemu-adm#3