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

replace mach with mach2 #4250

Draft
wants to merge 1 commit into
base: main
Choose a base branch
from
Draft

Conversation

Arshia001
Copy link
Member

Closes #4222.

Note, since mach was also being used by region, I had to fork and patch region too. The patch can be removed once the changes are merged upstream and released.

@Arshia001 Arshia001 force-pushed the chore-replace-mach-with-mach2 branch from 132498f to 9516b24 Compare October 11, 2023 08:46
Copy link
Member

@syrusakbary syrusakbary left a comment

Choose a reason for hiding this comment

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

This is great, but we can't merge this upstream until the dependencies are in crates.io (as otherwise we can't publish wasmer to crates.io)

@Arshia001
Copy link
Member Author

Yes, I realised a few minutes too late that that this is not a wasix app 😄

Gonna have to create a PR in region and hope they accept and publish a new version.

@theduke
Copy link
Contributor

theduke commented Nov 2, 2023

@Arshia001 what's the status here?

@Arshia001
Copy link
Member Author

@theduke we still need to open a PR in region.

@Arshia001 Arshia001 marked this pull request as draft November 2, 2023 13:42
@syrusakbary
Copy link
Member

syrusakbary commented Jan 9, 2024

Solution: reach maintianer over mail, and if non-reachable we can create create region2 create with the fix

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.

Is it possible to change the mach dependency
3 participants