From 2f7836b1917597fdb14db102b5cdaea5deef176e Mon Sep 17 00:00:00 2001
From: Ed_ <edwardgz@gmail.com>
Date: Sun, 20 Aug 2023 13:18:09 -0400
Subject: [PATCH] Fixes to buildscript

---
 scripts/build.ci.ps1 | 34 +++++++++++++++++++---------------
 1 file changed, 19 insertions(+), 15 deletions(-)

diff --git a/scripts/build.ci.ps1 b/scripts/build.ci.ps1
index 8bcef26..993d6b3 100644
--- a/scripts/build.ci.ps1
+++ b/scripts/build.ci.ps1
@@ -2,11 +2,14 @@
 # It will most likely need a partial rewrite to segment the build process into separate script invocations based on the OS.
 # That or just rewrite it in an sh script and call it a day.
 
-Import-Module ./helpers/target_arch.psm1
-$devshell = Join-Path $PSScriptRoot 'helpers/devshell.ps1'
-
 cls
 
+Import-Module ./helpers/target_arch.psm1
+$devshell  = Join-Path $PSScriptRoot 'helpers/devshell.ps1'
+$path_root = git rev-parse --show-toplevel
+
+Push-Location $path_root
+
 #region Arguments
        $vendor       = $null
        $release      = $null
@@ -37,20 +40,12 @@ if ($IsWindows) {
     & $devshell -arch amd64
 }
 
-$path_root         = git rev-parse --show-toplevel
-$path_build        = Join-Path $path_root build
-$path_project      = Join-Path $path_root project
-$path_scripts      = Join-Path $path_root scripts
-$path_singleheader = Join-Path $path_root singleheader
-$path_test         = Join-Path $path_root test
-
-
 if ( $vendor -eq $null ) {
 	write-host "No vendor specified, assuming clang available"
 	$compiler = "clang"
 }
 
-if ( $vendor -eq $null ) {
+if ( $release -eq $null ) {
 	write-host "No build type specified, assuming debug"
 	$release = $false
 }
@@ -62,8 +57,6 @@ if ( $bootstrap -eq $false -and $singleheader -eq $false -and $test -eq $false )
 write-host "Building gencpp with $vendor"
 write-host "Build Type: $(if ($release) {"Release"} else {"Debug"} )"
 
-Push-Location $path_root
-
 function run-compiler
 {
 	param( $compiler, $unit, $compiler_args )
@@ -305,6 +298,13 @@ if ( $vendor -match "msvc" )
 #endregion Configuration
 
 #region Building
+
+$path_build        = Join-Path $path_root build
+$path_project      = Join-Path $path_root project
+$path_scripts      = Join-Path $path_root scripts
+$path_singleheader = Join-Path $path_root singleheader
+$path_test         = Join-Path $path_root test
+
 if ( $bootstrap )
 {
 	$path_build = join-path $path_project build
@@ -408,6 +408,7 @@ if ( $test )
 #endregion Building
 
 #region Formatting
+
 function format-cpp
 {
 	param( $path, $include, $exclude )
@@ -420,7 +421,10 @@ function format-cpp
 		'-verbose'
 	)
 
-	$targetFiles = @(Get-ChildItem -Recurse -Path $path -Include $include -Exclude $exclude | Select-Object -ExpandProperty FullName)
+	$targetFiles = @(
+		Get-ChildItem -Recurse -Path $path -Include $include -Exclude $exclude
+			| Select-Object -ExpandProperty FullName
+	)
 
 	$time_taken = Measure-Command {
 		clang-format $formatParams $targetFiles