Merge pull request #23 from mlocati/set-content-atomic-v2

Make Set-Content atomic and use exclusive access
mlocati · Jan 18, 2019 · 35428b1 · 35428b1
2 parents 26151e6 + 0582104
commit 35428b1
Showing 1 changed file with 21 additions and 1 deletion.
diff --git a/PhpManager/private/Set-PhpIniLine.ps1 b/PhpManager/private/Set-PhpIniLine.ps1
@@ -26,7 +26,27 @@
         if (Test-Path -Path $Path -PathType Container) {
             $Path = [System.IO.Path]::Combine($Path, 'php.ini')
         }
-        Set-Content -Path $Path -Value $Lines
+        for ($i = 1; ; $i++) {
+            try {
+                $stream = [System.IO.File]::Open($Path, 'Create', 'Write', 'Read')
+                break
+            } catch {
+                if ($i -ge 3) {
+                    throw
+                }
+            }
+        }
+        $contents = $Lines | Out-String
+        try {
+            $writer = New-Object System.IO.StreamWriter($stream)
+            try {
+                $writer.Write($contents)
+            } finally {
+                $writer.Dispose()
+            }
+        } finally {
+            $stream.Dispose()
+        }
     }
     end {
     }