Skip to content
This repository has been archived by the owner on Dec 29, 2022. It is now read-only.

Refactoring PTY creation to allow support for other platforms #3

Open
FrankReh opened this issue Jan 31, 2018 · 9 comments
Open

Refactoring PTY creation to allow support for other platforms #3

FrankReh opened this issue Jan 31, 2018 · 9 comments
Assignees

Comments

@FrankReh
Copy link

I have a variation on OpenPTY that is Darwin specific. It even works for both Sierra and High Sierra. For my own use, I can live with the goterm project being in a vendor directory and using it on Linux, Darwin and soon OpenBSD platforms. But the work to support Darwin was non-trivial and I'd like to share it but I don't see how without cloning the entire goterm repo. But cloning goterm means projects like goexpect wouldn't get the new options without also having been cloned for a one line import change.

@FrankReh
Copy link
Author

Have working OpenPTY versions for OpenBSD and FreeBSD too.

@skalle
Copy link
Contributor

skalle commented Jul 4, 2018

Hey FrankReh , this sounds great! ... Happy to help out with getting this code working .. What would you suggest we'd do?

@skalle skalle self-assigned this Jul 19, 2018
@skalle
Copy link
Contributor

skalle commented Jul 19, 2018

https://github.com/google/goterm/tree/buildExperiment - Started to split the builds up there.. Adding in BSD after after sorting the Expect package out.

Don't have any Mac's around so might be a bit longer before I have a chance to get that one in.

@marco-m
Copy link
Contributor

marco-m commented Mar 6, 2019

@skalle now that Go 11 supports modules, in my understanding all the fuss about cloning the repo to change the import should not be needed any more. Would you be interested in moving goterm and goexpect to support go modules (and, to keep it simple, to support only go >= 1.11) ? If yes, I would be happy to give it a try, since I want to be able to run goexpect from macoS :-)

Or do you have another, more appropriate place to discuss this ?

I am also happy to add CircleCI builds/tests for both goterm and goexpect.

@skalle
Copy link
Contributor

skalle commented Mar 6, 2019

Hey marco-m .. I'd be more than happy for goexpect/goterm to have support for modules and MacOS.

CircleCI I've never heard of , guess I'll have to do some Googling.

About discussing, what timezone are you in? I'm in Sydney.

@marco-m
Copy link
Contributor

marco-m commented Mar 7, 2019

@skalle CircleCI is like Travis CI, free for open source, github integration. I love CI systems and among the ones offered as a service I prefer CircleCI.

Regarding the timezone, I am in Europe.

@marco-m
Copy link
Contributor

marco-m commented Apr 7, 2019

Updates:
CircleCI for goterm #4
CircleCI for goexpect google/goexpect#38

@skalle
Copy link
Contributor

skalle commented Apr 8, 2019

Great stuff Marco.

@ChinaHDJ1
Copy link

i need this feat

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

No branches or pull requests

4 participants