Conversation
|
I assume this is WIP, let me if you need me to review and merge it. |
|
Hi, sorry about the delay. I add a JS binding for libCacheSim locally. You can follow the README to test it. In addition, current binding requires local building from the source. After review, I could consider to make it being a published npm package. |
|
I am getting an error when installing |
Hi, I identified the cause of the error — it was due to missing the -fPIC compiler flag. I've added it, and the issue is now resolved on my side. |
|
Awesome work! Shall we add some instructions on how to install it globally? BTW, publishing as an npm package sounds good to me! |
|
To publish as a public npm package, there are two solutions:
I prefer method 1, but what's your preference? |
|
I updated. Now, it support the local global installation. |
|
I think approach 2 might be better, because we do not expect frequent updates. We could automate the publishing process as a GitHub action. The main issue with approach 1 is that 1. it can be tricky for JavaScript developers to figure out C compilation errors; and 2. if we introduce some incompatible changes, it may cause a cascading failure.
|
Hi, to make these easier to review, I expect to merge this PR to focus on one goal: enabling js binding building from source. |
🚨 BugBot couldn't runPull requests from forked repositories are not yet supported (requestId: serverGenReqId_f0fe78a1-0978-4434-bdad-f4a2d5a79bb7). |
|
Sounds good to me! |
|





Initialization for js binding.