Discussion :

[romut]

Bonjour à tous,
Sur les conseils de Laurent Dardenne je poste ici ma question.

En PoSH v1, j'utilisais l'add-on PSEventing pour gérer les appels de process externes et récupérer les traces de sortie. Ca marchait nickel.

Maintenant, je suis obligé de passer à PoSH v2, qui introduit la gestion des évènements, bonne nouvelle, donc plus besoin de PSEventing.

J'ai au beaucoup de mal à trouver de la documentation sur la question, et finalement, et finalement, le créateur de PSEventing a modifié mon code pour qu'il fonctionne en PoSH v2, le voici :

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
28
29
30
31
32
33
34
35
36
37
38
39
40
41
42
43
44
45
46
47
48
49
50
51
function Launch-Process([System.Diagnostics.Process]$process, [string]$log, [int]$timeout = 0)
{
 #Connect-Event process ErrorDataReceived,OutputDataReceived
 Register-ObjectEvent -InputObject $process -EventName ErrorDataReceived > $null
 Register-ObjectEvent -InputObject $process -EventName OutputDataReceived > $null
  
 $process.Start() 
 $process.BeginErrorReadLine()
 $process.BeginOutputReadLine() 
 
 $ret = $null
 if($timeout -eq 0)
 {
 $process.WaitForExit()
 $ret = $true
 }
 else
 {
 if(-not($process.WaitForExit($timeout)))
 {
  Log-Message $log WARNING ("The process is not completed, after the specified timeout: " + $timeout)
  $ret = $false
 }
 else
 {
  $ret = $true
 }
 }
 
 #$events = @(Read-Event)
 $events = @(Wait-Event)

 foreach($event in $events)
 {
 if(-not [string]::IsNullOrEmpty($event.Args.data))
 {
  if($event.Name -eq "OutputDataReceived")
  {
  #Log-Message $log Info $event.Args.data
  Log-Message $log Info $event.SourceEventArgs.data
  }
  else
  {
  #Log-Message $log ERROR $event.Args.data
  Log-Message $log ERROR $event.SourceEventArgs.data
  }
 }
 }
 $process.Close()
 $ret
}

Déjà, y a des trucs que je ne comprends pas, comme les " > $null" à la fin des appels à Register-ObjectEvent.

Ensuite, ce code ne fonctionne pas, mon objet events (normalement rempli par

Code :

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

 $events = @(Wait-Event)

est vide.

Une idée ?

[Laurent Dardenne]

a modifié mon code

L'objectif étant la redirection des sorties d'un process.

Déjà, y a des trucs que je ne comprends pas, comme les " > $null" à la fin des appels à Register-ObjectEvent.

Cela évite d'afficher l'objet PSJob, on peut aussi utiliser

Code :

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

[void](Register-ObjectEvent ...)

Ensuite, ce code ne fonctionne pas,

Si, c'est la lecture des events générés qu'il faut revoir. J'ai associé du code via le paramètre -Action du cmdlet Register-ObjectEvent.

J'utilise cmd.exe pour la démo :

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
28
29
30
31
32
$global:log = "C:\Temp\test.log"
 
# Process object creation
$process = New-Object -TypeName System.Diagnostics.Process
$process.StartInfo.CreateNoWindow = $false
$process.StartInfo.RedirectStandardError = $true
$process.StartInfo.RedirectStandardOutput = $true
$process.StartInfo.UseShellExecute = $false
$process.StartInfo.FileName = "c:\windows\System32\CMD.EXE"
$process.StartInfo.WorkingDirectory = "C:\temp\"
$process.StartInfo.Arguments = '/k'
 
Register-ObjectEvent -InputObject $process -EventName ErrorDataReceived  -action {
 $message = "ERR: $($EventArgs.data)"  
 Write-Host $message
 $message | Out-File $global:log -Encoding ASCII -Append
}
Register-ObjectEvent -InputObject $process -EventName OutputDataReceived -action {
 $message = "Out : $($EventArgs.data)" 
 Write-Host $message
 $message | Out-File $global:log -Encoding ASCII -Append
}
 
$process.Start() 
$process.BeginErrorReadLine()
$process.BeginOutputReadLine()  
 
$process.WaitForExit()
 #saisir DIR puis exit
$process.Close()
Type $global:log

Dans le code des events, il reste à revoir la gestion de lignes vides.
Et enfin supprimer les abonnements et les job des events.

[romut]

Trop fort, je vais bien arriver à me débrouiller avec ça maintenant.

Merci pour ce gros coup de main, vraiment, ça me simplifie vraiment l'existence.

[romut]

J'ai modifié ton code comme suit :

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
28
29
30
31
32
33
34
function Log-Trace([string]$Level, [system.Diagnostics.DataReceivedEventArgs]$Trace)
{
    if($Trace -ne $null)
    {
        if(-not [string]::IsNullOrEmpty($Trace.data))
        {
            Write-Host "$Level $($Trace.data)"
        }
    }
}

# Process object creation
$process = New-Object -TypeName System.Diagnostics.Process
$process.StartInfo.CreateNoWindow = $false
$process.StartInfo.RedirectStandardError = $true
$process.StartInfo.RedirectStandardOutput = $true
$process.StartInfo.UseShellExecute = $false
$process.StartInfo.FileName = "7z.EXE"
$process.StartInfo.WorkingDirectory = ""
$process.StartInfo.Arguments = ""
 
Register-ObjectEvent -InputObject $process -EventName ErrorDataReceived  -action {
Log-Trace "ERROR" $EventArgs
}
Register-ObjectEvent -InputObject $process -EventName OutputDataReceived -action {
Log-Trace "Info" $EventArgs
}
 
$process.Start() 
$process.BeginErrorReadLine()
$process.BeginOutputReadLine()  
 
$process.WaitForExit()
$process.Close()

Ca marche pas trop mal, mais comme avec le code précédent que tu as donné, j'ai des fois des traces "WARNING: Operation is not valid due to the current state of the object.", saurais tu comment éviter ça ?

En même temps, elle ne seront pas enregistrées dans mon log, c'est donc pas trop un souci, mais je m'inquiète de la "propreté" de mon script...

Merci.

[romut]

Salut,
J'ai finalement un gros souci avec cette solution.

En fait, mon script fait partie d'un ensemble de scripts PoSH qui constitue notre plateforme de builds & tests. La société dans laquelle je travaille est un éditeur de logiciels.

Pour la GConf, nous utilisons CVS, c'est un choix historique sur lequel malheureusement je n'ai que très peu de prise, ça tiendrait qu'à moi....

Bref, mes scripts marchent très bien sur la première version de la plateforme, en PoSH v1 32 bits + PSEventing.

Pour satisfaire certains de nos besoins, nous devons évoluer, la plateforme passe donc d'un XP 32bits vers un 7 64 bits, donc PoSH v2 64 + la gestion des évènements fournie par PoSH.

Et là, récupérer les traces de CVS ne fonctionne pas bien du tout. Ca se lance, ça s'exécute correctement, je récupère les erreurs sans problème, mais impossible de récupérer les traces standard de CVS avec cette solution. Ca fonctionne très bien avec les autres logiciels bizarrement, mais pas avec CVS.

Ca fait 3 jours que je suis dessus, à faire des tests dans tous les sens et...rien à faire, impossible de récupérer les messages sortant de CVS sur "output".

T'aurais une idée des fois ? Je soupçonne un petit bug dans PoSH v2...

[Laurent Dardenne]

j'ai des fois des traces "WARNING: Operation is not valid due to the current state of the object.", saurais tu comment éviter ça ?

Il faut déjà comprendre le pourquoi de ce comportement, de mon coté je ne n'utilise pas 7zip.

En fait

Faits méconnus à l'origine, ma réponse initiale est liée au contexte présenté.

T'aurais une idée des fois ?

Pour ce pb avec CVS, si possible, teste-le en installant PS v2 32 bits sur ton OS 64 bits.
Ensuite si cela fonctionne tu peux remonter le pb dans le forum parent, qui est peut être plus approprié pour celui-ci.
Quant à mettre en cause PS c'est possible, mais comme le dit Jeffrey Snover ici :

We regularly run > 1 million tests,

Si c'est le cas, via MS-Connect, tu leur en offrira un de plus
Site sur lequel tu peux également rechercher si un pb similaire est référencé.