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

Revert "Run qemu in tmpfs (#1707)" #1757

Merged
merged 1 commit into from Aug 8, 2022
Merged

Revert "Run qemu in tmpfs (#1707)" #1757

merged 1 commit into from Aug 8, 2022

Conversation

ntkme
Copy link
Contributor

@ntkme ntkme commented Jul 22, 2022

This reverts commit cb74cc4.

The fix for the original issue is in dart-lang/sdk@6b04565 and has been included with dart-sdk 2.18.0-271.2.beta. I have tested it locally and verified the workaround is no longer needed.

@ntkme
Copy link
Contributor Author

ntkme commented Jul 22, 2022

Note: This is only affecting the "upcoming" linux-arm 32-bit build when dart 2.18 becomes stable.

Also see: sass/dart-sass-embedded#96

#1704 should be marked as blocked.

@nex3
Copy link
Contributor

nex3 commented Aug 2, 2022

Just to clarify: you're only expecting this to be merged once Dart 2.18 stable lands, correct?

@nex3 nex3 self-assigned this Aug 2, 2022
@nex3 nex3 mentioned this pull request Aug 2, 2022
@nex3 nex3 added the blocked Waiting on another issue to be fixed label Aug 2, 2022
@ntkme
Copy link
Contributor Author

ntkme commented Aug 3, 2022

@nex3 Same as the PR for dart-sass-embedded, this can be merged now as arm 32-bit build has not been enabled.

@nex3 nex3 merged commit ba88dd6 into sass:main Aug 8, 2022
@ntkme ntkme deleted the revert-1707-qemu-tmpfs branch August 8, 2022 20:49
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
blocked Waiting on another issue to be fixed
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants