forked from chocolatey-community/chocolatey-packages
-
Notifications
You must be signed in to change notification settings - Fork 0
Commit
This commit does not belong to any branch on this repository, and may belong to a fork outside of the repository.
- Loading branch information
1 parent
c92090a
commit efa2269
Showing
6 changed files
with
34 additions
and
33 deletions.
There are no files selected for viewing
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Original file line number | Diff line number | Diff line change |
---|---|---|
@@ -1,20 +1,23 @@ | ||
$ErrorActionPreference = 'Stop' | ||
|
||
|
||
$toolsPath = Split-Path -parent $MyInvocation.MyCommand.Definition | ||
$Installers = Get-ChildItem -Path $toolsPath -Filter '*.exe' | | ||
Sort-Object lastwritetime | | ||
Select-Object -Last 2 -ExpandProperty FullName | ||
$File32Name = 'Zotero-7.0.8_win32_setup.exe' | ||
$File64Name = 'Zotero-7.0.8_x64_setup.exe' | ||
|
||
$File32Path = Join-Path $toolsPath $File32Name | ||
$File64Path = Join-Path $toolsPath $File64Name | ||
|
||
$packageArgs = @{ | ||
packageName = $env:ChocolateyPackageName | ||
softwareName = "$env:ChocolateyPackageName*" | ||
softwareName = "Zotero*" | ||
fileType = 'exe' | ||
file = $Installers | Where-Object {$_ -match "win32"} | ||
file64 = $Installers | Where-Object {$_ -match "x64"} | ||
file = "$File32Path" | ||
file64 = "$File64Path" | ||
silentArgs = '/S' | ||
validExitCodes = @(0) | ||
} | ||
|
||
Install-ChocolateyInstallPackage @packageArgs | ||
|
||
Remove-Item $Installers -Force -ea 0 | ||
Remove-Item "$File32Path","$File64Path" -Force -ea 0 |
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
efa2269
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Hi teknowledgist,
I saw you had some issues getting your pull request approved.
I'm willing to help, as I really want to get Zotero 64bit in chocolatey.
I don't think I can see what issues you are having with the merge right now though.
Could you add me as a collaborator to this repo?
Thanks.
efa2269
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@ZacharyVernec I also want Zotero 64bit, and I'm rather shocked that I am the one leading the charge! 😨
If you look at the PR discussion, I had to jump through some hoops, but I believe the only remaining issue(s) is:
I am OK with Powershell code, and stumble through GitHub Desktop, but I am a dunderhead when it comes to Git; I don't know how to do those things.
If you know how, can make those changes as a collaborator, and promise not to run roughshod over the rest of my repo, I would welcome the help.
efa2269
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Yes I saw the PR discussion, that's why I contacted you. I'm willing to give rebasing a shot.
I would consider myself an intermediate Git user. I don't think I've rebased before, but I've read about it a number of times.
But don't worry, I won't do anything without being confident about the results, I'll be respectful and careful with your repo.
efa2269
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@ZacharyVernec OK. I made you a collaborator. Not sure if you got an invite or I need to follow up with that, but let's see what you can do. 😉