-
Notifications
You must be signed in to change notification settings - Fork 3.5k
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
Support for Windows #167
Comments
@cristian-sima Have you tried compiled it on your machine? |
@montogeek Well I read the there is no support for Windows. So no |
Can you try it? |
Yes. How do I compile it? |
Check the Readme.md file :) |
There is no section on readme page about |
I was also interested in this as I'm primarily windows user, initial package installation was not successful due to dependency For a reference, I tried build on Windows 10 / msys2 (zsh) / windows-build-tools (for native node module compilation) / node v6.2 / npm v3.10.x . |
@cristian-sima : Try the notes in the I am forced to use Windows at work, and use Linux at home, so I'm looking forward to builds for those two platforms 👍 |
@avindra I have seen screenshots of builds on Linux (Ubuntu) |
We could replace |
@montogeek I'm able to run it on openSUSE Tumbleweed by following the directions in the README. The only difference I made was running |
@avindra Great! |
It builds:
the build assets currently have macos and darwin, due to harcoding of those in the packaging scripts I believe. Took a few min, but it completed the build 👍 |
Yeah, it not precisely fast . (you can omit |
There isn't a clear way to run it because this bundles a build for OS X (a However, the packager you guys use does support Windows and OS X by passing a different flag to My bet is that it should be fairly easy to get at least the Linux build rolling. |
Could you please try it? https://github.com/electron-userland/electron-packager/blob/master/docs/api.md#platform |
@montogeek : Worked with plaform=linux Produced a 41M bundle when made into a |
And does it run? |
Yup, running the Here is the file layout that it generates:
|
Success!! Now we need somebody to follow our steps and try it on Windows! |
@montogeek I've followed instructions on windows (#167 (comment)) and current major blocker is replace |
@kwonoj Could you try using https://github.com/chjj/pty.js ? |
@montogeek I can, haven't since I was worrying replacing it could possibly cause some regressions. I'll give a try soon. |
@kwonoj It's possible, check railsware/upterm@76c297a from another project, for example |
Yes I believe it's possible. my concern was breaking existing mac builds feature(if any) since there isn't any test coverage yet. Anyway I'll give it a try first before worrying out replace modules to see if I can make it work. |
Yeah, not worries, we are just exploring possible solutions, in any case a PR is more than welcome! |
I'm also interested in a windows build. Got these errors.
|
OK, this isn't lovely looking yet but.. - I've reached this : It's still based on quite much of manual wiring though. I'll try to see if I can come up with small PR for enabling preliminary support for windows build pipeline. Current goal's reuse most of scripts as-is, which'll only allows windows users who does have *nix compatible shells (cygwin, msys2..) though. |
@kwonoj can you share some artifacts from https://ci.appveyor.com/project/kwonoj/hyperterm/build/0.0.22-build_windows ? |
@vors please allow me some dumb question, are you referring |
@avindra at this moment I don't consider to update build script at those levels. I've confirmed generated artifacts allows immediate dev-build tryout without additional configuration, think that serves anyone interested to try out dev build, since current build is anyway kind of broken state to use as a package on Windows. |
Flow has added support for Windows today facebook/flow#6 (comment) I am optimistic and looking forward to see also hyperterm supporting Windows :D |
Also the Windows 10 Anniversary update drops tomorrow bringing with it Ubuntu on Windows. I would love to be able to use this as my Bash shell on Windows. |
@montogeek Funny seeing you here 😆 |
xterm.js supports windows very well, use it in hyperterm maybe easily fix this problem |
Hyperterm uses hterm under the hood, which is comparable to xterm.js, so it might indeed be feasible to swap the two. However, I don't think that would solve the compatibility problem, as the issue appears to be with child_pty, which is not a part of hterm. Though it is worth noting that xterm.js uses pty.js, which has been proposed in this thread as a possible alternative to child_pty. |
Any progressions on this issue? Can I try to install it on windows? Thank you. |
@C0deMaver1ck - it likely works fine on ubuntu node within WSL. I'm running Xenial, zsh, and Linux node with few issues on Windows. Native support on Windows flavor of node is a completely separate undertaking. |
@cchamberlain - are you running via a xserver and accesses via VNC? |
Nice. Tried to install terminator during the beta but there was too many dependency issues. |
Yo y'all please help testing it! |
Nice 💯. Thanks |
Is there a more detailed version on how to build on windows ? On Oct 31, 2016, at 2:28 AM, Cristian Sima <[email protected]mailto:[email protected]> wrote: Nice ? You are receiving this because you were mentioned. |
@jcolesio Looks like that is still being worked on. There's a quick explanation in the comments on the PR: #946 (comment) |
@jcolesio - I'd assume you can use windows-build-tools and configure python path per its instructions as an alternative to the VC++ Build Tools Technical Preview. |
From the doc I can see that
Builds for Windows and Linux are coming very soon!
The bad part is that I have Windows. Therefore, :D support -> Windows
The text was updated successfully, but these errors were encountered: