-
-
Notifications
You must be signed in to change notification settings - Fork 476
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
The Windows 32-bit and 64-bit binary don't seem to work out of the box #334
Comments
Just to add: |
Thank you very much for trying this out. |
@Zamiell @dominikschulz Yeah I've got it working just fine with |
welp, I'll try running it from source Here's one messed up thing that was relatively trivial to fix: |
It's not working for me when running from source either.
|
When debugging |
Thanks @dominikschulz Issue 1: Will type up the second issue in a minute. |
Thank you for reporting this issue. Actually I already discovered this issue while doing some Windows tests yesterday but failed to open an issue for that. Will try to fix that as soon as I get a working windows VM. |
I updated to the latest commit of gopass (as of the time of this writing).
So, looking at the file system, it seems to have created the
So it just seems to be failing on the git commit:
Not sure if that last part is expected or not? |
Indeed, it looks like the git commit is failing. This is not expected, maybe it's a setup error or we broke something in master. Will try to reproduce as soon as I've got some kind of windows test environment. |
As an aside, if a user stumbles upon this repository and they start reading the README.md file, it isn't clear if Can you add a bulleted list to the README.md file that lists the additional features that |
A further nit-pick: From the README.md file:
You don't really define what these 3 choices are or the implications of initializing a password store with either choice. (It's just telling Can't all this just be removed? README.md should probably advise the user to just do the simple |
Good point, indeed. Could you just open separate issues for these findings? This makes tracking them easier. Thanks! |
@alexmironof any tips on getting this working? Also, does anyone have any recommendations for some software similar to |
@Zamiell I can tell you which commit I am using |
Sure! |
@dominikschulz More info: I started from scratch and generated a GPG key WITHOUT a passphrase. Then, I was able to add a secret to a newly initialized password store and no longer got the git commit error from above. So I think the problem lies in that gopass fails to bring up the little window that allows the user to type in their GPG passphrase, and assumes their GPG passphrase is blank, and then proceeds to try and do the commit, and then obviously fails at that part. |
Thanks for following up on this. Actually gopass doesn't generate any popups itself, but relies on git / gpg to do that. There may be some interaction going on which prevents this. We still to properly support Windows, but so far there is no progress. |
If you still need a testing environment you should give this a shot. It is pretty similar to AppVeyor but local using Vagrant+Virtualbox. https://github.com/chocolatey/chocolatey-test-environment There are also some more updated options but I like this one because it really helps you figure out your dependencies, and it comes with Chocolatey which is probably the BEST way of distributing packages for Windows that I've come across. |
The git commit fails because git can't find the GPG executable on Windows (it's bundled with the MinGW executable which stores their keys at a different location from GPG4Win). See https://jamesmckay.net/2016/02/signing-git-commits-with-gpg-on-windows/ This command fixes it: |
This commit fixes a number of issue around GPG, improves the GPG binary detection and covers some windows cases. Fixes #5 Fixes gopasspw#334 Fixes gopasspw#418
This commit fixes a number of issue around GPG, improves the GPG binary detection and covers some windows cases. Fixes #5 Fixes gopasspw#334 Fixes gopasspw#418
Steps to reproduce:
cmd
go init
command.These steps were performed on a Windows 10 OS.
I also tried using the 32-bit binary, and I received the same error.
Any tips?
The text was updated successfully, but these errors were encountered: