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

feat(setup): use versioned release artifacts in Windows "xs-dev setup" #136

Merged
merged 1 commit into from
Jul 31, 2023

Conversation

andycarle
Copy link
Contributor

This update to xs-dev setup for Windows mirrors the macOS setup process by:

  1. Performing a shallow clone of the Moddable SDK repo
  2. Using the versioned release of the Moddable SDK tools rather than building from source

Like on macOS, this approach uses the versioned release tools as both the "release" and "debug" binaries in build/bin/win/release and build/bin/win/debug, respectively.

When I get a chance, I'll make a new version of #91 built on this change. But no reason to wait on that to get this smaller change in, imho.

@phoddie
Copy link
Contributor

phoddie commented Jul 25, 2023

Like on macOS, this approach uses the versioned release tools as both the "release" and "debug" binaries in build/bin/win/release and build/bin/win/debug, respectively.

FWIW – in theory it should no longer be necessary to have the debug binaries. There were some unintentional dependencies on the debug binaries but those have been cleaned up.

Copy link
Owner

@HipsterBrown HipsterBrown left a comment

Choose a reason for hiding this comment

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

Apologies for the delayed review. Awesome to see this workflow supported in Windows as well!

@HipsterBrown HipsterBrown merged commit 73e6a25 into HipsterBrown:main Jul 31, 2023
3 checks passed
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.

3 participants