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

Make goss compile on windows and enable travis tests on windows. #448

Closed
wants to merge 8 commits into from

Conversation

stefan-improbable
Copy link
Contributor

This includes go modules changes from #437 . I can update this PR once go_modules are merged upstream.

stefan-improbable and others added 6 commits April 10, 2019 12:17
I've extracted the unix-only part to file_unix.go and windows part to file_windows.go. I haven't found an easy way to support owners/groups on windows, so I've left it empty.

TODO: figure out how to write and run integration tests on windows, but probably not this pr.
return "", err
}

mode := fmt.Sprintf("%04o", (fi.Mode() & 07777))
Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Note that fi.Mode has a bit different format from sys.(*syscall.Stat_t).Mode.

@mhedgpeth
Copy link

@stefan-improbable I'm interested in getting goss windows supported. How can I help? Is merging this the next step?

@stefan-improbable
Copy link
Contributor Author

@stefan-improbable I'm interested in getting goss windows supported. How can I help? Is merging this the next step?

@mhedgpeth it depends on @aelsabbahy whether he wants to merge this. We are using our fork at https://github.com/improbable/goss in the meantime, but we are hoping that we can merge our changes upstream and kill that fork.

@mhedgpeth
Copy link

@aelindeman how can I help get this merged? What are your potential concerns?

@petemounce
Copy link
Collaborator

@mhedgpeth I think you meant @aelsabbahy.

@petemounce
Copy link
Collaborator

We're now publishing Windows and Mac OS binaries at https://github.com/improbable-eng/goss/releases (but we have not ensured all features work on all platforms). We're gradually making changes to get to feature parity with Linux. You're welcome to use and give feedback.

@petemounce petemounce mentioned this pull request May 11, 2020
16 tasks
@aelsabbahy
Copy link
Member

superseded by #585

@aelsabbahy aelsabbahy closed this Jun 16, 2020
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.

None yet

4 participants