-
Notifications
You must be signed in to change notification settings - Fork 29
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
Error: 'bits/alltypes.h' "file not found" in newer images #51
Comments
Have you tried |
I initially tried |
I've found a similar issue in the main project: emscripten-core/emscripten#9205 Can you check if this is something that you can try? |
Yes that's the same issue I linked to, this comment on particular is the one that looks like it might be also related to the llvm version: emscripten-core/emscripten#9205 (comment) After that point of discussion they move onto a different issue. Fwiw I'm currently building successfully with .41 which I'm happy to stick with for now, it's basically just for demonstration use at this stage. In case you're interested, CI build script: Building micropython in the browser: |
It should be easy to debug this using Note that emscripten used to have a symlink for that, but removed it since it didn't work on windows. I'm not aware of any bugs atm in upstream. Perhaps the docker image is missing some of the necessary files? |
Sorry @andrewleech, I missed that you've posted this issue in the main topic. I've tried to reproduce this issue locally without success so far. Sample #include <stdio.h>
#include <bits/alltypes.h>
int main(int argc, char** argv) {
printf("Test Test: %d\n", argc);
fflush(stdout);
return 0;
} 1.38.41
1.38.43
latest
Tried also upstream just in case and looks that it works just fine. At least there is no sign that it shouldn't. Do you thin that you can change ci script and check with |
@andrewleech I've found: ifdef EMSCRIPTEN
CPP += -isystem $(EMSCRIPTEN)/system/include/libc -cxx-isystem $(EMSCRIPTEN)/system/include/libcxx
endif In your make file, this looks incompatible with emscripten-core/emscripten#9197 Simple changing it to: ifdef EMSCRIPTEN
CPP += -isystem $(EMSCRIPTEN)/system/include/libc
CPP += -isystem $(EMSCRIPTEN)/system/lib/libc/musl/arch/emscripten
CPP += -cxx-isystem $(EMSCRIPTEN)/system/include/libcxx
endif Fixed the problem with compilation |
I was just testing it myself and it appears to be somewhat of a false alarm; the first stage of building is using clang to pre-process a bunch of stuff. Yes, your change looks good and makes sense based on my findings, I'll submit a patch for that to the micropython project upstream. Thanks so much for your help! |
Happy to help! |
Thank you. I fix the same problem according to your way. Then I get another error that can't find wasi/api.h. I add But my questions are: |
Hi,
Thanks for your work on this image!
I'm attempting to use this for building the javascript port of micropython but on the latest images it throws a long list of
Which looks very similar to emscripten-core/emscripten#9205
I tried a couple of versions back,
trzeci/emscripten:sdk-tag-1.38.42-64bit
had the same compile error, buttrzeci/emscripten:sdk-tag-1.38.41-64bit
so this lines up with the versions in the above issue.Based on that issue, I suspect llvm may need updating?
The text was updated successfully, but these errors were encountered: