Discussion :

[hannibal.76]

Bonjour a tous,
j'ai une méthode qui doit faire des dizaines de SVN checkout.

J'effectue ces checkout via un Runtime.exe et j'attend la fin du process externe via un Process.waitFor .

Malheureusement le waitFor ne fini jamais.
Avez vous une idée du problème ?

Je vous remercie par avance pour votre aide.

PS: Le source de la methode

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
21
22
23
24
25
26
27
 
private static void checkout(ArrayList<String> svnAllUrl, File workingDir) {
 
		if ( workingDir.exists() && workingDir.isFile() ) {
			System.err.println("-------------------------------------------------");
			System.err.println("-- Error not a directory : " + workingDir.getAbsolutePath());
			System.err.println("-------------------------------------------------");
			System.exit(-1);
		}
 
		if ( !workingDir.exists() )
			workingDir.mkdirs();
 
		Runtime runtime = Runtime.getRuntime();
 
		for ( String svnUrl : svnAllUrl ) {
			String[] cmdarray = { "svn", "checkout", svnUrl };
			try {
				System.out.println("-- Checkout: " + svnUrl);
				Process p = runtime.exec(cmdarray, null, workingDir);
				p.waitFor();
			}
			catch ( IOException | InterruptedException e ) {
				e.printStackTrace();
			}
		}
	}

[adiGuba]

Salut,


Tu ne traites pas les flux d'entrée/sorties, ce qui peut provoquer un dead-lock.
En effet il y a de forte chance que ta commande se retrouve bloqué lors d'une écriture sur sa sortie standard en attendant que ton programme traite les données... alors que ton programme attend la fin du process.

Bref les deux process attendent mutuellement...

Solution : traiter (ou fermer) les flux d'entrée/sorties.
A partir de Java 5 on privilégiera le ProcessBuilder qui permet de rediriger le flux d'erreur dans le flux standard (ce qui permet de ne traiter qu'un seul flux).
Avec Java 7 on peut même utiliser inheritIO() de ProcessBuilder pour que les flux du process soit hérité du programme Java (il devient alors inutile de les traiter).


a++

[hannibal.76]

Ah, cela explique tout.
Je pensai (un peux naïvement certes) que si je ne gérai pas les flux, ils étaient redirigé vers un /dev/null sauce java.

Donc après gestion des flux et passage sur un ProcessBuilder, aucun problème

Merci pour ton aide