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

Using from.std.* errors out when compiling to WebAssembly #4

Closed
skoppe opened this issue Aug 22, 2019 · 4 comments
Closed

Using from.std.* errors out when compiling to WebAssembly #4

skoppe opened this issue Aug 22, 2019 · 4 comments

Comments

@skoppe
Copy link
Contributor

skoppe commented Aug 22, 2019

When using from.std.xxx the from template does a import std;. The std/package.d publicly imports whole of phobos. The system.d module from phobos - which is unaware of WebAssembly - errors out with Error: static assert: "Unknown OS.".

Now, this is actually an issue with phobos. But I am just posting it here so that you are aware of the issue.

I have PR's ready for druntime and phobos to support WebAssembly, which will resolve this issue. But it is going to take some time.

@aliak00
Copy link
Owner

aliak00 commented Aug 28, 2019

Thanks for the heads up! 🙇

@aliak00
Copy link
Owner

aliak00 commented Sep 3, 2019

@skoppe what's the status on your PRs. As a work around we could have bolts not use "from" for std packages if it's causing problems.

@skoppe
Copy link
Contributor Author

skoppe commented Sep 3, 2019

The pr's aren't quite there yet. I first want to experiment with them some more.

I am still using optional 0.16.x, but I have a good workaround for the @disabled stuff, so there is no urgency.

Last 2 weeks I had to focus on some other things, but I hope to continue working on the PR's this week.

Since there aren't many people using optional in a non x86 arch, I don't mind you closing this issue.

@skoppe
Copy link
Contributor Author

skoppe commented Nov 19, 2019

Just made a pull request #5 to replace std.from with bare imports. The pr's for phobos are going to take a little longer and I really want to update spasm to the latest version.

@skoppe skoppe closed this as completed Nov 19, 2019
jll63 added a commit to jll63/bolts that referenced this issue Jul 18, 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

No branches or pull requests

2 participants