-
Notifications
You must be signed in to change notification settings - Fork 46
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
build-release fails #33
Comments
What version of node, npm and typescript do you have? |
Sorry, I should have provided that info. Here it is: |
Hello Michael, This is Michael Uman from Sigma Designs. We spoke a month ago about this issue. I am also getting errors during build, both debug and release targets. The error is similar (I will post it below). I just installed a fresh install of emscripten SDK and reinstalled all the aomanalyzer packages using the 'npm install' command. Here are the versions I am using. /home/muman2/emsdk-portable/node/4.1.1_64bit/bin/npm v2.14.4 ERROR in /mnt/muman-fw-gitroot/avs2-inpector/aomanalyzer/node_modules/@types/react/index.d.ts ERROR in ./src/components/LocalAnalyzer.tsx What is odd is that these 'ERROR's do not prevent the creation of the bundles and as far as I can tell the code runs OK. It is just that I don't like to see error messages in the build output. It is best to resolve these issues because they may point to problems in the code or logic. Thank you, |
This is most likely caused by changes in the TypeScript compiler, or some React dependency. TypeScript often produces output even if Errors appear, which is why your code still works. But I agree with you, that it should be fixed. I'll try and take a look at the issue, but I haven't had the time lately. @tdaede Have you also experienced this issue when building on Linux? |
Hi,
I'm trying to build the analyzer on a Mac. I've followed the instructions but get the following
Any idea how to fix this?
The text was updated successfully, but these errors were encountered: