Anybody dabbling in Powershell?

stevo7624

I am new to Powershell and I am trying to get an understanding of it through borrowing script and making it work for my system. I have a script that I thought would take files from a target folder and zip them into a destination folder. It is creating zip folders in my destination folder based on the names of the files in the target folders, but the zipped folders are empty. How do I get the zipped folders to contain the files from the target folder? I am obviously missing some script or haven't manipulated it enough to work for me. Was hoping to get some expert advice to help aide my journey into Powershell scripting.
Here is the script:
[HTML]
[System.Reflection.Assembly]::LoadFrom("C:\Powershell\Ionic.Zip.dll") $Target = "C:\PSTarget" $Destination = "C:\PSDest" function ZipItUp { $outputFile = [IO.FileInfo] "$Destination\$fldr.zip" $zipfile = new-object Ionic.Zip.ZipFile $selfExtractOptions = New-Object Ionic.Zip.SelfExtractorSaveOptions $selfExtractOptions.Flavor = [Ionic.Zip.SelfExtractorFlavor]::ConsoleApplication $selfExtractOptions.DefaultExtractDirectory = $outputFile.Directory.FullName $selfExtractOptions.RemoveUnpackedFilesAfterExecute = $false $e = $zipfile.AddDirectory("$Target\$fldr") $zipfile.UseZip64WhenSaving = [Ionic.Zip.Zip64Option]::Always $zipfile.SaveSelfExtractor($outputFile.FullName, $selfExtractOptions) $zipfile.Dispose(); } foreach ($fldr in Get-ChildItem $Target) { Write-Host "Zipping up $fldr to PSDest" ZipItUp }[/HTML]


Thank you in advance for any help.

Steve

undomiel

Well your formatting makes it really difficult to look through that. I don't see where you are actually adding something to the zip file though. You have $e = $zipfile.AddDirectory("$Target\$fldr") but I don't see anything being done with $e. Also I don't know if AddDirectory will work with a single file as a quick search seems to say that it zips up directories recursively. AddFile seems to be more like what you are looking for if you're wanting one zip file per a file found. Where you may want to make your loop use Get-ChildItem | where {$_.psIsContainer -eq $false} so that you skip any directories in the target directory.