-
Notifications
You must be signed in to change notification settings - Fork 71
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
Cross platform post-install #266
Conversation
@ailisp do you know what is happening with our CI? It has not started... |
I also can't see reason why it doesn't. Can be github action scheduler's issue? Let me try close and reopen |
Started this time. Weird |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Good multi platform compatibility support!
|
||
const OS = await executeCommand("uname -s", false); | ||
const ARCH = await executeCommand("uname -m", false); | ||
const SUPPORTED_PLATFORMS = ["linux", "darwin"]; // Unsaported platforms: 'win32', 'aix', 'freebsd', 'openbsd', 'sunos', 'android' |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Oh, changes below are made for windows support, but there's still some part of the build process that doesn't support windows. Am I right?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
It's just part of the effort. So Windows is not on the list just yet.
Bash approach does not work on Windows