-
Notifications
You must be signed in to change notification settings - Fork 27
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
Windows support #10
Comments
That would be great! If you want to submit a PR but make sure it's backwards compatible (there is some platform detection in there) ... We are unable to develop and test this on Windows currently but would welcome any contribution for it |
Ok I was able to try some stuff out on my partners Win10 laptop and think we can make something work, but won't be very soon. @EhsanKia would you like to help / test it when we do? Will probably be a simple:
Keep your eyes peeled for now or if you make any progress yourself respond here! |
I'd be happy to help test, I can also contribute if you can give a few pointers. |
Ok we have a ticket in our JIRA board but unlikely to get there in next few weeks but will be in contact when we do. If you do develop something yourself we can help too. |
Just as an update here ... we have estimated this as quite complex since we don't have a Windows development environment, so we won't be able to get to this quickly. But we will in time - we have an environment on the way ;). |
Further update ... we had issues with a more recent version of |
Thank you for the regular updates :) |
Another one for you :) so we went down the VM route and got one working and fell down the hole of environment variables on Windows 🤦🏼 however we have decided to go with using @EhsanKia The question that raises, to install the package manager needs to be done using administrative shell, is this ok from a UX perspective? I imagine it would mean More details here: https://chocolatey.org/install |
I think that's fair if install is required. though would the package still work if op is already installed and properly setup to work in shell, with |
Right now I'm not sure it would since with other platforms it uses the bash profile. Will have to leverage the powershell PS1 profile I think to get the client working / find another library for Windows specific profiles - we shall see. |
Sorry we haven't got round to this ... some other bugs with existing platforms and our release pipeline have to take priority I am afraid. I am hoping to get round to Windows sometime in this quarter though! |
Well I definitely miss estimated this by a long shot, our previous company was acquired and a lot has got ahead of this lib. Sorry @EhsanKia ... if you want to go ahead and try it yourself go ahead, we had some great initial ideas! |
Hey @EhsanKia as you might have seen we have just released 1.0.1 which supports v2 and within the new cli there is much more windows support. We have started a |
I wonder if anyone has gotten this working on a windows system. It seems to rely on an .rcfile to work, which is not available on windows based systems.
First it complained about missing environ['HOME'], but I set that and then it complains about not finding a shell rc file. It seems like it's used to managed the OP_SESSION key, it may be nice to have a way for the user to just provide the session value manually maybe?
The text was updated successfully, but these errors were encountered: