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

detect simultaneous runs of nodec #31

Open
raguay opened this issue May 12, 2017 · 7 comments
Open

detect simultaneous runs of nodec #31

raguay opened this issue May 12, 2017 · 7 comments

Comments

@raguay
Copy link

raguay commented May 12, 2017

Hi,

If you create the smallest script using the https://github.com/scottgonzalez/node-wordpress library to post a post to a WordPress site, it will not compile. It dies with this message:

clang: error: linker command failed with exit code 1 (use -v to see invocation)
make[1]: *** [/private/var/folders/7m/t5j9b17x4n9gkml89tck1s4c0000gn/T/nodec/node/out/Release/mksnapshot] Error 1
rm de7168347652b3d5f6d7374947b15853a841ec81.intermediate
make: *** [node] Error 2
Failed running ["make -j4"]

I've done a few other scripts that are just as small and they compile fine. But, add this library and call it it will not compile. The script runs fine using node, but if you browserify it, it no longer works. I bet that points to an issue with the library?

BTW: It took around an hour to try to compile it. That library must be very complicated.

@pmq20
Copy link
Owner

pmq20 commented May 16, 2017

@raguay Sorry for the delayed reply. I failed to reproduce this error on my local machine. I tried the Found xxx posts example on the node-wordpress readme and it works fine both before compiling and after compiling.

Could you give me more of your error log? Something is hidden before the sentence "linker command failed with exit code". I need the error message of the linker in order to diagnose. Thanks.

image

image

@raguay
Copy link
Author

raguay commented May 16, 2017 via email

@pmq20
Copy link
Owner

pmq20 commented May 17, 2017

@raguay Probably. nodec is not designed to run simultaneously at the moment, as the compile process would reuse the same set of temporary object files, which is potentially problematic. Thanks for reporting!

@raguay
Copy link
Author

raguay commented May 17, 2017 via email

@pmq20 pmq20 changed the title https://github.com/scottgonzalez/node-wordpress can't be compiled. detect simultaneous runs of nodec May 17, 2017
@pmq20
Copy link
Owner

pmq20 commented May 17, 2017

nodec should detect this and give a proper error message though. I'll add this to https://github.com/pmq20/node-compiler/blob/master/ROADMAP.md

@oneFlyingSnail
Copy link

@pmq20 Hello, I'm not sure about nodec's command. Can you help me?
image

@pmq20
Copy link
Owner

pmq20 commented May 22, 2017

@oneFlyingSnail You could try downloading https://sourceforge.net/projects/node-compiler/files/v0.9.6/nodec.exe/download and putting it inside C:\Windows to make it available every where.

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

3 participants