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

Working build on Windows (SSH/custom repo) #1936

Open
wants to merge 2 commits into
base: master
Choose a base branch
from

Conversation

etinin
Copy link

@etinin etinin commented Apr 12, 2020

On VS2019 the packages won't build and, even with the mistakes fixed, without some changes it is not possible to connect through SSH to custom repos. (at least on my machine with git, ssh, etc already installed). So I figured a general solution:

Copy newest Git (used x64 over here) instead of msysgit to msysgit folder and then overwrite the SSH files with portable ones built for windows.

Fixed some minor coding mistakes.
The HOME variable shouldn't be set, else problems will arise.
Instead, a portable version of OpenSSH should be used : https://github.com/PowerShell/openssh-portable

(I guess this could be automated with a script, but I just wanted to get it working and figured out the steps, in case it interests anybody else).

Will build and produce a working "portable" package. Did not attempt to build the installer.

#1909 #1924 #1923 #1781 #1888

Alexandre de Farias added 2 commits April 12, 2020 02:26
@hbons
Copy link
Owner

hbons commented Apr 12, 2020

This is great. Thank you! :)

I'm happy to merge this, but I don't want to have binary files in the repo if it can be avoided. Can you adjust the build script to download and verify msysgit from a trusted source? This is what happens in the Mac project as well.

@etinin
Copy link
Author

etinin commented Apr 12, 2020

This is great. Thank you! :)

I'm happy to merge this, but I don't want to have binary files in the repo if it can be avoided. Can you adjust the build script to download and verify msysgit from a trusted source? This is what happens in the Mac project as well.

yes, I think that's the optimal solution! I'll try to get to it as soon as possible

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants