Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

(1password) Installation failed due to change in installer #1186

Closed
mkevenaar opened this issue Jan 11, 2019 · 45 comments · Fixed by #1516
Closed

(1password) Installation failed due to change in installer #1186

mkevenaar opened this issue Jan 11, 2019 · 45 comments · Fixed by #1516

Comments

@mkevenaar
Copy link
Member

Expected Behavior

1password to be updated

Current Behavior

Failing with an error message

Installing 1PasswordSetup-7.3.657.exe...
error: Found argument '/VERYSILENT' which wasn't expected, or isn't valid in this context
USAGE:
    1PasswordSetup-7.3.657.exe [OPTIONS]
For more information try --help
ERROR:
Running ['C:\Users\[myusername]\AppData\Local\Temp\chocolatey\1password\7.3.657\1PasswordSetup-7.3.657.exe'  /VERYSILENT /NORESTART /SUPPRESSMSGBOXES /SP- /LOG="C:\Users\[myusername]\AppData\Local\Temp\chocolatey\1password.7.3.657.InnoInstall.log"] was not successful.
 Exit code was '1' See log for possible error messages..
The upgrade of 1password was NOT successful.
Error while running 'C:\ProgramData\chocolatey\lib\1password\tools\chocolateyInstall.ps1'.
 See log for details.

Possible Solution

Change silent installation parameters.

See package discussion

Steps to Reproduce (for bugs)

  1. Just install / update 1password

Context

Update of 1password is failing, resulting that one can't install the latest version of 1password

Your Environment

@mkevenaar
Copy link
Member Author

I have tried to solve this, got updates working, but new installation fails with an permission error from installer.

@ndroftheline
Copy link

same problem.

@oechsler
Copy link

image

So apparently the installer finishes correctly if I simply run it without any arguments, also there is no gui only output in powershell ...

@AdmiringWorm
Copy link
Member

@mkevenaar are you able to verify that everything works as expected when not supplying arguments to the installer?

@mkevenaar
Copy link
Member Author

@AdmiringWorm I managed to get updates working, but new install failed.

I have pushed my changes to https://github.com/mkevenaar/chocolatey-coreteampackages/tree/fix-1password.

@AdmiringWorm
Copy link
Member

@mkevenaar do you get an error during install?
If so, what kind (or does it hang).
Could be that we need to rely on autohotkey in the latter case.

@mkevenaar
Copy link
Member Author

@AdmiringWorm It was a week ago, but if I remember correctly I got an error code 5 (access denied error) I was unable to investigate at that point what was going on.

If you want I can create a PR for my changes so you can continue to work on it.

@AdmiringWorm
Copy link
Member

@mkevenaar sure, do that.

@mkevenaar
Copy link
Member Author

@AdmiringWorm done #1208

@aleksijohansson
Copy link

Any updates on this?

@AdmiringWorm
Copy link
Member

@aleksijohansson unfortunately not, we have not been able to pinpoint the issue we have with the new installer.

@fsouza
Copy link

fsouza commented May 4, 2019

@AdmiringWorm I was able to get this working with this change: #1208 (comment)

I'm no longer on Windows though, it was a one-off task, so hard to confirm it still works.

@AdmiringWorm
Copy link
Member

@mkevenaar are you able to verify that this works?
I'm unfortunately no longer in windows myself, and my vm went haywire when I tried to launch it earlier today (happens from time to time).

@mkevenaar
Copy link
Member Author

mkevenaar commented May 4, 2019

I hope I have some time soon to check this @AdmiringWorm

majkinetor added a commit that referenced this issue Jun 7, 2019
@majkinetor
Copy link
Contributor

majkinetor commented Jun 7, 2019

Should be fixed now, confirm.

@majkinetor
Copy link
Contributor

I can't work it out to force update on 1password for some reason... You will have to try it from the repo.

@AdmiringWorm
Copy link
Member

maybe uppercase stream names isn't respected by AU? or it got problems with package names starting with a number.

I also noticed it is missing the [CmdletBinding()] declaration. Could this be the problem?

@majkinetor ^^

@aleksijohansson
Copy link

I tested this again and the same issue still persists.

@majkinetor
Copy link
Contributor

@aleksijohansson it isn't released on gallery, is that how you tested it ?

@aleksijohansson
Copy link

@majkinetor Yes, I just normally tried to install 1password with chocolatey. I could test something else too, if you can provide instructions?

@majkinetor
Copy link
Contributor

Just download code cpack it and install it.

Or wait for new release. Or some admin can publish it.

/cc @gep13

@aleksijohansson
Copy link

Ok, I’ll wait for the release

@pauby
Copy link
Member

pauby commented Aug 7, 2019

New error for this:

Installing 1PasswordSetup-7.3.705.exe...                                                                                                       
error: Found argument '--log_path' which wasn't expected, or isn't valid in this context                                                       
USAGE:                                                                                                                                         
    1PasswordSetup-7.3.705.exe [FLAGS] [OPTIONS]                                                                                               
For more information try --help                                                                                                                
ERROR:                                                                                                                                         
Running ['C:\Users\Paul\AppData\Local\Temp\chocolatey\1password\7.3.705\1PasswordSetup-7.3.705.exe'  --log_path .] was not successful.         
 Exit code was '1' See log for possible error messages..                                                                                       
The upgrade of 1password was NOT successful.                                                                                                   
Error while running 'C:\ProgramData\chocolatey\lib\1password\tools\chocolateyInstall.ps1'.                                                     
 See log for details.                                                                                                                          

AdmiringWorm added a commit that referenced this issue Aug 7, 2019
As there is an issue with the current installer (see issue #1186) disabling
the OPW stream is necessary so we don't push any more broken packages.
@majkinetor
Copy link
Contributor

Well, that worked when I changed the installer. Since I don't use 1password, perhaps you can give help output.

Anyway, I am leaving this for others to take; its proprietary tool with stupid installer which I don't use. I showed how it could be done previously, perhaps something trivial could be changed such as parameter name.

Proof that it worked, i.e. that parameter existed back in time:

#1208 (comment)

@pauby
Copy link
Member

pauby commented Aug 7, 2019

Running the installer with --help gives:

USAGE:                                                                            
    1PasswordSetup-7.3.705.exe [FLAGS] [OPTIONS]                                  
                                                                                  
FLAGS:                                                                            
        --silent     Silent install.                                              
    -h, --help       Prints help information                                      
    -V, --version    Prints version information                                   
                                                                                  
OPTIONS:                                                                          
    -i, --install_location <INSTALL_LOCATION>    Optional: Custom install location

@mkevenaar
Copy link
Member Author

@pauby the Spotify package does this

@AdmiringWorm
Copy link
Member

@mkevenaar correct me if I'm wrong, but don't the spotify package run a schedule for installing spotify without admin privileges (due to requirements of the installer).

@mkevenaar
Copy link
Member Author

@AdmiringWorm it might be, I don’t know for sure

@majkinetor
Copy link
Contributor

majkinetor commented Aug 13, 2019 via email

@stale
Copy link

stale bot commented Jan 12, 2020

Is this still relevant? If so, what is blocking it? Is there anything you can do to help move it forward?

@tobyscales
Copy link

This remains an issue as of 1/13/2020, JFYI. Reading thru the GH issues it appears the problem is with 1password's installer, which attempts a second-order privilege elevation even when executed from an administrative prompt. I can confirm this happens on Win10-1909.

@stale stale bot removed the Pending closure label Jan 13, 2020
@QNimbus
Copy link

QNimbus commented Jan 17, 2020

I can confirm this issue. I was able to work around it by running :

choco install 1password -f -not-silent

@stale
Copy link

stale bot commented Jun 17, 2020

Is this still relevant? If so, what is blocking it? Is there anything you can do to help move it forward?

@mkevenaar
Copy link
Member Author

This issue still exists.

@majkinetor
Copy link
Contributor

All issues still exists when bot announces closure. I am returning pending closure as it is obvious that nobody cares enough after more then year and a half.

@virtualex-itv
Copy link

choco install 1password -f -not-silent

I too was able to install with this command but this package deserves an update as there is a newer version available than what gets installed via chocolatey

@ropearce
Copy link

This most definitely is still an issue. I use chocolatey to silently install packages (including 1Password) to my client workstations through a major name RMM. As most recent as yesterday, I was still getting the error message. If anybody wants me to test or review anything, I'm certainly happy to collaborate.

Download of 1PasswordSetup-7.3.705.exe (12.92 MB) completed.
Hashes match.
Installing 1password...
Microsoft.PowerShell.Commands.WriteErrorException
error: Found argument '--log_path' which wasn't expected, or isn't valid in this context
USAGE:
1PasswordSetup-7.3.705.exe [FLAGS] [OPTIONS]
Microsoft.PowerShell.Commands.WriteErrorException
For more information try --help
ERROR: Running ["C:\WINDOWS\TEMP\chocolatey\1password\7.3.705\1PasswordSetup-7.3.705.exe" --log_path . ] was not successful. Exit code was '1'. See log for possible error messages.
The install of 1password was NOT successful.
Error while running 'C:\ProgramData\chocolatey\lib\1password\tools\chocolateyInstall.ps1'.
See log for details.

@stale stale bot removed the Pending closure label Jul 16, 2020
@RedBaron2
Copy link
Contributor

RedBaron2 commented Jul 16, 2020

I have a fix for the issue, but requires more testing. I'll have artifacts of the updated nupkg files on my appveyor soon for anyone that wants to test.

The installer they are using is still 'raw' and they are slowly working on only adding switches that are extremely necessary.

I'll update this post shortly with the links.

Edited 07/21/2020
This is the ongoing thread from the 1password forum

@RedBaron2
Copy link
Contributor

@mkevenaar @ropearce
Thanks for the 👍
Here is the link to the zip file with my updates for the fix.
Please Test it !_! and let us know of any problems. Thanks

@majkinetor
FYI: If this works well for them. There is serious work that needs to be done to the functions that are normally called for Chocolatey Installations.

@AdmiringWorm
Copy link
Member

This issue have been fixed in the repository, thanks to @mkevenaar.

A new version is being queued to be pushed to chocolatey.org.

kai2nenobu added a commit to kai2nenobu/dotfiles that referenced this issue Feb 8, 2021
最新版の1passwordバージョン7で保管庫の管理ができそうな目処がたったので、
古い方はインストールしないように変更しました。
新しいバージョンの1passwordはchocolateyの 1password パッケージでインストール
できるはずなんだけど、パッケージの不備で現状ではインストール時にエラーが
発生する模様。

ref. chocolatey-community/chocolatey-packages#1186

とりあえずこのissueが解決するまでは手動でインストールすることにする。
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging a pull request may close this issue.