Unable to build from source

[awhitty]

Hi LaserWeb folks 👋

I've been using LaserWeb for a few days through a binary on the releases page and have been generally pleased with how well the software works! I'm still familiarizing with the project, so I hope you'll bear with me.

The Issue

I would like to contribute to the project, so I cloned the repo and ran npm install and npm run installdev. At this point, npm start fails with an error related to a dependency:

TypeError: require(...).load is not a function
    at Object.<anonymous> (<...>/LaserWeb4/node_modules/lw.comm-server/config.js:1:81)

This is due to a version mismatch between what's expected in the lw.comm-server library and what's being installed. That module's package.json specifies "dotenv": "*", and there appear to be breaking changes in dotenv (folks use require('dotenv').configure(...) these days).

Digging in a little further, I noticed that npm 5's package-lock.json was intentionally ignored in this repo's .gitignore file. A package-lock.json in this repo would have prevented that breaking change in the short run, but the issue would still have shown itself if folks worked on the lw.comm-server package independently.

I also noticed while running npm run start-app that the webpack build fails due to a missing object.omit dependency. Sure enough, src/reducers/index.js imports from object.omit, but the dependency is not listed in the package.json. I assume this was working for folks due to its presence as a transitive dependency?

Recommendations

This is as far as I have gotten so far. I have a few recommendations that I think would open this project up for more third party contributions in the future. I've ordered them from smallest scope to largest:

1. Remove package-lock.json from this repo's .gitignore file and commit a working package-lock.json to the repo

   This change at the least should enable other developers to work on this project without spending too much time fiddling with dependencies.

2. Use exact package versions for all package.jsons for this repo as well as the other lw.* packages

   It's very rare that something like "dotenv": "*" is actually a useful declaration in a package.json. Instead I recommend installing packages with the --save-exact flag to pin to the exact version. This policy, in addition to the package-lock.json change, should eliminate a class of issues related to breaking changes in dependencies and transitive dependencies.

   The tradeoff here is that folks won't always get the latest features from a new version of a package "for free". The truth is is that they never really did get them for free. Developer A might make a change to use a method introduced in version 2.6 of a package while Developer B is still using version 2.5. Developer B might pull Developer A's changes and notice that the code has broken. Developer B has no way of knowing whether Developer A wrote buggy code or if Developer B's packages are out of date compared to Developer A's local setup. Requiring developers to save the exact version of a package in the repo eliminates this ambiguity.

   Tools like Greenkeeper can reduce the friction of keeping packages up to date. In my experience, this hasn't actually been as necessary as folks would expect.

3. Introduce a CI tool to ensure this repo can always be built from source

   I noticed y'all listed "Automate Electron Builds for all platforms" on the wishlist inside the readme. Introducing a CI workflow that ensures the project can at least be built from source in a development context is a good first step in this direction. A lot of the groundwork here could be repurposed into an automated testing rig as well as an automatic build system, and both would open this project up to more contributions.

4. Consolidate all lw.* packages into a monorepo

   This suggestion might be a bit controversial, but given that this project has many associated packages that need to change in tandem, it may be easier to keep track of the history of the project and coordinate cross-cutting changes if every package were checked into the same repo. This approach works well for projects like React and Babel, and I imagine it would work well here too.

   I'm not sure if y'all are familiar with Lerna, but it's a tool that reduces the friction around managing a monorepo of multiple packages.

tl;dr

I couldn't build from source, there's a few issues precipitating that, I have some suggestions for how to eliminate those issues, and I'm happy to incorporate those suggestions if y'all are open to them. Thanks for making useful software!!

[easytarget]

I just referenced this in another issue; I'm pretty concerned that laserweb could easily become abandonware unless we can find a way to maintain it against the relatively fast moving npm dependency chains, browser requirements, evolving web standards, etc.

The first need is to get to the step 2) in your scheme and use this as an opportunity to bring dependencies up to date as far as possible. I hope that could solve the scrollbar issue, for instance. I'd be useless at that but I could help with step 3) since I am a CM/Build/Devops engineer by trade.

Critically, if we can get there then we can think of how to extend Laserweb for new controllers and maybe work on a more responsive interface.

[cprezzi]

The situation is, that all frontend devs have left the project. I'm the only dev left, but my part is the backend (lw.comm-server). I'm not realy familar with react/redux/webpack of the frontend. I would be very happy if someone could help to upgrade all dependencies and bring the whole project to a state where also unexperienced devs could compile it.

What's important:

• The frontend defs used the branch "dev-es6" for development and compiling (webpack) of the frontend only. They run lw.comm-server manually (nodejs start) to test against it.
• The backend is developed in lw.comm-server (maser).
• The electron installer is built in electron_bundler branch of lw.comm-server. To compile the exe, you first need to merge the changes of the master branch, then copy the frontend webpack files to the app folder and compile the exe. There are some scripts in package.json to do that.

[easytarget]

@cprezzi ; Thankyou for the reply, taking the time to explain things is really appreciated :-)
If I can solve some of my immediate issues (job related) I will try and get a branch building. It would be a good beginning.

I didn't realise electron was involved (new to me..) but I pretty easily got the demo running.

I hope this is not a naive question; its electron only used for building the windows executable?

• I'm fully on Linux here, Fedora on my laptop and lw.comm-server as a service on Raspian on the PI3 that serves my CNC. How critical is it for development?
• I can see that the electron build will be needed for release, but could development happen without it?

[easytarget]

@awhitty ; is there any chance you could look at getting a sensible/valid package.lock together; either for the dev-es6 branch, or start another based off that to aim for some sort of roundup release so we are at least current with dependencies and can focus on real issues again.

Not: I consider laserweb feature complete for my use. There will always be a big list of improvements it could have, but it currently fits my workflow perfectly.

[cprezzi]

@easytarget You don't need the electron branch for development. It's only needed for building the Windows and OSX executables/installers.

The frontend (LaserWeb4) and backend (lw.comm-server) are theoretically independent applications that just communicate via websockets.

For convenience, the backend nodejs webserver also serves the frontend webpack files. This means it's enough to clone lw.comm-server and run "node server.js", then access the frontend with a chrome or chromium browser on IP port 8000 (like documented in the Raspberry Pi wiki).

For frontend development, you just need LaserWeb4/dev-es6. If you also want to test the server communication, manually start lw.comm-server before testing.

[cprezzi]

I brought the LaserWeb4 dev-es6 branch and lw.comm-server master to a state where I could compile both under nodejs 10.16.3.

To get that working, you have to:

1. Clone lw.comm-server
2. Run npm install.
3. Test if node server.js works.
4. Clone LaserWeb4
5. Run npm run installdev.
6. Run npm run start-app for frontend testing. This does a webpack compile and starts a browser on localhost:8080/webpack-dev-server/. (it takes some time until webpack compile is ready!)

After updating something on the frontend:

• Run npm run bundle-dev in the LaserWeb4 folder. This creates the webpack files in the dist folder, which can then be copied over to the app folder of lw.comm-server (embeded website). Instead of manual copying you can run the script npm run update_lw4 in the lw.comm-server folder (needs bash console for ncp command).

I hope this gives you at least a start point which can be compiled!

[awhitty]

Thanks for the replies @cprezzi and @easytarget! The additional context is very helpful.

I’ll take a look at the changes you made to both repos to understand a bit more about what was going on, and I'll follow those instructions! I’m happy to PR changes to introduce a package-lock.json and pin down exact versions for packages. I’ll aim to do that tonight. I’m on pacific time, so that’ll be about 11 or 12 hours from now.

A few other things on my mind, though these topics are mostly unrelated to this specific issue:

• Is there any particular structure around contributions to these repos? I assume PRs with an approving review should suffice.
• Is there a policy on how the repos are versioned and how releases are made?
• Is there a roadmap or backlog of features that we can groom? I see some Github issues that might make sense as a starting point, and @easytarget mentioned support for more controllers and UI adjustments.

[cprezzi]

• For contributions I think PRs with approving reviews are a good solution
• For the releases: they went with the dev that used to do the release work ;)
• For the roadmap: I think the github issues are a good starting point.
• For the supported controllers: I think we already support the most common controllers. So far I tried to decouple the controllers from the frontend and handle the differences in the backend (if possible). The only exception was the "marlin gcode compiler" which was easyer to implement in the frontend.

[awhitty]

@cprezzi Thanks for answering my questions!

I'll assign PRs to you then.

Regarding roadmap, I think we should comb through the GH issues a bit and look for some common themes. Once we get a consistent build working (I think #573 gets us there), it'll be time to prioritize feature work, and that prioritization can inform which other sorts of tech debt are worth addressing sooner vs. later.

Regarding controller support, that makes sense. I'm not yet familiar with how marlin differs from grbl or what else is out there, but I trust that judgement 😛

[cprezzi]

@awhitty I've invited you to the admins of the LaserWeb oragnisation, so you can do what's needed in all our repos ;)

[awhitty]

@cprezzi By the way, this hasn't fallen off my radar! Just got back from a long vacation, so I'm picking things back up slowly.

[mfadel85]

I am not able to build the application using ubuntu, 19.10, node v13.13.0,
when executing
node src/index.js
I get this
`(node:11762) ExperimentalWarning: The ESM module loader is experimental.
file:///home/mfadel/Projects/LaserWeb4-dev-es6/src/index.js:63
return ;
^

SyntaxError: Unexpected token '<'
at Loader.moduleStrategy (internal/modules/esm/translators.js:81:18)
at async link (internal/modules/esm/module_job.js:37:21)
`

I tried another way by executing npm install

when executing
npm install
:

`> [email protected] install /home/mfadel/Projects/LaserWeb4-dev-es6/node_modules/lw.comm-server/node_modules/serialport

prebuild-install || node-gyp rebuild

prebuild-install WARN install No prebuilt binaries found (target=13.13.0 runtime=node arch=x64 platform=linux)
make: Entering directory '/home/mfadel/Projects/LaserWeb4-dev-es6/node_modules/lw.comm-server/node_modules/serialport/build'
CXX(target) Release/obj.target/serialport/src/serialport.o
In file included from ../src/./serialport.h:6,
from ../src/serialport.cpp:1:
../../../../nan/nan.h: In function ‘void Nan::AsyncQueueWorker(Nan::AsyncWorker*)’:
../../../../nan/nan.h:2298:62: warning: cast between incompatible function types from ‘void ()(uv_work_t)’ {aka ‘void ()(uv_work_s)’} to ‘uv_after_work_cb’ {aka ‘void ()(uv_work_s, int)’} [-Wcast-function-type]
2298 | , reinterpret_cast<uv_after_work_cb>(AsyncExecuteComplete)
| ^
../src/serialport.cpp: In function ‘Nan::NAN_METHOD_RETURN_TYPE Open(Nan::NAN_METHOD_ARGS_TYPE)’:
../src/serialport.cpp:41:48: error: no matching function for call to ‘v8::Value::ToString()’
41 | v8::String::Utf8Value path(info[0]->ToString());
| ^
In file included from /home/mfadel/.cache/node-gyp/13.13.0/include/node/node.h:67,
from ../../../../nan/nan.h:54,
from ../src/./serialport.h:6,
from ../src/serialport.cpp:1:
/home/mfadel/.cache/node-gyp/13.13.0/include/node/v8.h:2750:44: note: candidate: ‘v8::MaybeLocalv8::String v8::Value::ToString(v8::Localv8::Context) const’
2750 | V8_WARN_UNUSED_RESULT MaybeLocal ToString(
| ^~~~~~~~
/home/mfadel/.cache/node-gyp/13.13.0/include/node/v8.h:2750:44: note: candidate expects 1 argument, 0 provided
../src/serialport.cpp:48:53: error: no matching function for call to ‘v8::Value::ToObject()’
48 | v8::Localv8::Object options = info[1]->ToObject();
| ^
In file included from /home/mfadel/.cache/node-gyp/13.13.0/include/node/node.h:67,
from ../../../../nan/nan.h:54,
from ../src/./serialport.h:6,
from ../src/serialport.cpp:1:
/home/mfadel/.cache/node-gyp/13.13.0/include/node/v8.h:2754:44: note: candidate: ‘v8::MaybeLocalv8::Object v8::Value::ToObject(v8::Localv8::Context) const’
2754 | V8_WARN_UNUSED_RESULT MaybeLocal ToObject(
| ^~~~~~~~
/home/mfadel/.cache/node-gyp/13.13.0/include/node/v8.h:2754:44: note: candidate expects 1 argument, 0 provided
../src/serialport.cpp:78:69: warning: cast between incompatible function types from ‘void ()(uv_work_t)’ {aka ‘void ()(uv_work_s)’} to ‘uv_after_work_cb’ {aka ‘void ()(uv_work_s, int)’} [-Wcast-function-type]
78 | ueue_work(uv_default_loop(), req, EIO_Open, (uv_after_work_cb)EIO_AfterOpen);
| ^~~~~~~~~~~~~

../src/serialport.cpp: In function ‘void EIO_AfterOpen(uv_work_t*)’:
../src/serialport.cpp:95:30: warning: ‘v8::Localv8::Value Nan::Callback::Call(int, v8::Localv8::Value) const’ is deprecated [-Wdeprecated-declarations]
95 | data->callback.Call(2, argv);
| ^
In file included from ../src/./serialport.h:6,
from ../src/serialport.cpp:1:
../../../../nan/nan.h:1740:3: note: declared here
1740 | Call(int argc, v8::Localv8::Value argv[]) const {
| ^~~~
../src/serialport.cpp: In function ‘Nan::NAN_METHOD_RETURN_TYPE Update(Nan::NAN_METHOD_ARGS_TYPE)’:
../src/serialport.cpp:113:53: error: no matching function for call to ‘v8::Value::ToObject()’
113 | v8::Localv8::Object options = info[1]->ToObject();
| ^
In file included from /home/mfadel/.cache/node-gyp/13.13.0/include/node/node.h:67,
from ../../../../nan/nan.h:54,
from ../src/./serialport.h:6,
from ../src/serialport.cpp:1:
/home/mfadel/.cache/node-gyp/13.13.0/include/node/v8.h:2754:44: note: candidate: ‘v8::MaybeLocalv8::Object v8::Value::ToObject(v8::Localv8::Context) const’
2754 | V8_WARN_UNUSED_RESULT MaybeLocal ToObject(
| ^~~~~~~~
/home/mfadel/.cache/node-gyp/13.13.0/include/node/v8.h:2754:44: note: candidate expects 1 argument, 0 provided
../src/serialport.cpp:135:71: warning: cast between incompatible function types from ‘void ()(uv_work_t*)’ {aka ‘void ()(uv_work_s)’} to ‘uv_after_work_cb’ {aka ‘void ()(uv_work_s, int)’} [-Wcast-function-type]
135 | _work(uv_default_loop(), req, EIO_Update, (uv_after_work_cb)EIO_AfterUpdate);
| ^~~~~~~~~~~~~~~

../src/serialport.cpp: In function ‘void EIO_AfterUpdate(uv_work_t*)’:
../src/serialport.cpp:150:30: warning: ‘v8::Localv8::Value Nan::Callback::Call(int, v8::Localv8::Value) const’ is deprecated [-Wdeprecated-declarations]
150 | data->callback.Call(1, argv);
| ^
In file included from ../src/./serialport.h:6,
from ../src/serialport.cpp:1:
../../../../nan/nan.h:1740:3: note: declared here
1740 | Call(int argc, v8::Localv8::Value argv[]) const {
| ^~~~
../src/serialport.cpp: In function ‘Nan::NAN_METHOD_RETURN_TYPE Close(Nan::NAN_METHOD_ARGS_TYPE)’:
../src/serialport.cpp:175:70: warning: cast between incompatible function types from ‘void ()(uv_work_t*)’ {aka ‘void ()(uv_work_s)’} to ‘uv_after_work_cb’ {aka ‘void ()(uv_work_s, int)’} [-Wcast-function-type]
175 | ue_work(uv_default_loop(), req, EIO_Close, (uv_after_work_cb)EIO_AfterClose);
| ^~~~~~~~~~~~~~

../src/serialport.cpp: In function ‘void EIO_AfterClose(uv_work_t*)’:
../src/serialport.cpp:188:30: warning: ‘v8::Localv8::Value Nan::Callback::Call(int, v8::Localv8::Value) const’ is deprecated [-Wdeprecated-declarations]
188 | data->callback.Call(1, argv);
| ^
In file included from ../src/./serialport.h:6,
from ../src/serialport.cpp:1:
../../../../nan/nan.h:1740:3: note: declared here
1740 | Call(int argc, v8::Localv8::Value argv[]) const {
| ^~~~
../src/serialport.cpp: In function ‘Nan::NAN_METHOD_RETURN_TYPE Flush(Nan::NAN_METHOD_ARGS_TYPE)’:
../src/serialport.cpp:215:70: warning: cast between incompatible function types from ‘void ()(uv_work_t*)’ {aka ‘void ()(uv_work_s)’} to ‘uv_after_work_cb’ {aka ‘void ()(uv_work_s, int)’} [-Wcast-function-type]
215 | ue_work(uv_default_loop(), req, EIO_Flush, (uv_after_work_cb)EIO_AfterFlush);
| ^~~~~~~~~~~~~~

../src/serialport.cpp: In function ‘void EIO_AfterFlush(uv_work_t*)’:
../src/serialport.cpp:231:30: warning: ‘v8::Localv8::Value Nan::Callback::Call(int, v8::Localv8::Value) const’ is deprecated [-Wdeprecated-declarations]
231 | data->callback.Call(1, argv);
| ^
In file included from ../src/./serialport.h:6,
from ../src/serialport.cpp:1:
../../../../nan/nan.h:1740:3: note: declared here
1740 | Call(int argc, v8::Localv8::Value argv[]) const {
| ^~~~
../src/serialport.cpp: In function ‘Nan::NAN_METHOD_RETURN_TYPE Set(Nan::NAN_METHOD_ARGS_TYPE)’:
../src/serialport.cpp:250:53: error: no matching function for call to ‘v8::Value::ToObject()’
250 | v8::Localv8::Object options = info[1]->ToObject();
| ^
In file included from /home/mfadel/.cache/node-gyp/13.13.0/include/node/node.h:67,
from ../../../../nan/nan.h:54,
from ../src/./serialport.h:6,
from ../src/serialport.cpp:1:
/home/mfadel/.cache/node-gyp/13.13.0/include/node/v8.h:2754:44: note: candidate: ‘v8::MaybeLocalv8::Object v8::Value::ToObject(v8::Localv8::Context) const’
2754 | V8_WARN_UNUSED_RESULT MaybeLocal ToObject(
| ^~~~~~~~
/home/mfadel/.cache/node-gyp/13.13.0/include/node/v8.h:2754:44: note: candidate expects 1 argument, 0 provided
../src/serialport.cpp:270:68: warning: cast between incompatible function types from ‘void ()(uv_work_t*)’ {aka ‘void ()(uv_work_s)’} to ‘uv_after_work_cb’ {aka ‘void ()(uv_work_s, int)’} [-Wcast-function-type]
270 | _queue_work(uv_default_loop(), req, EIO_Set, (uv_after_work_cb)EIO_AfterSet);
| ^~~~~~~~~~~~

../src/serialport.cpp: In function ‘void EIO_AfterSet(uv_work_t*)’:
../src/serialport.cpp:285:30: warning: ‘v8::Localv8::Value Nan::Callback::Call(int, v8::Localv8::Value) const’ is deprecated [-Wdeprecated-declarations]
285 | data->callback.Call(1, argv);
| ^
In file included from ../src/./serialport.h:6,
from ../src/serialport.cpp:1:
../../../../nan/nan.h:1740:3: note: declared here
1740 | Call(int argc, v8::Localv8::Value argv[]) const {
| ^~~~
../src/serialport.cpp: In function ‘Nan::NAN_METHOD_RETURN_TYPE Get(Nan::NAN_METHOD_ARGS_TYPE)’:
../src/serialport.cpp:314:68: warning: cast between incompatible function types from ‘void ()(uv_work_t*)’ {aka ‘void ()(uv_work_s)’} to ‘uv_after_work_cb’ {aka ‘void ()(uv_work_s, int)’} [-Wcast-function-type]
314 | _queue_work(uv_default_loop(), req, EIO_Get, (uv_after_work_cb)EIO_AfterGet);
| ^~~~~~~~~~~~

../src/serialport.cpp: In function ‘void EIO_AfterGet(uv_work_t*)’:
../src/serialport.cpp:329:96: error: no matching function for call to ‘v8::Object::Set(v8::Localv8::String, Nan::imp::FactoryBasev8::Boolean::return_t)’
329 | >("cts").ToLocalChecked(), Nan::Newv8::Boolean(data->cts));
| ^

In file included from /home/mfadel/.cache/node-gyp/13.13.0/include/node/node.h:67,
from ../../../../nan/nan.h:54,
from ../src/./serialport.h:6,
from ../src/serialport.cpp:1:
/home/mfadel/.cache/node-gyp/13.13.0/include/node/v8.h:3547:37: note: candidate: ‘v8::Maybe v8::Object::Set(v8::Localv8::Context, v8::Localv8::Value, v8::Localv8::Value)’
3547 | V8_WARN_UNUSED_RESULT Maybe Set(Local context,
| ^~~
/home/mfadel/.cache/node-gyp/13.13.0/include/node/v8.h:3547:37: note: candidate expects 3 arguments, 2 provided
/home/mfadel/.cache/node-gyp/13.13.0/include/node/v8.h:3550:37: note: candidate: ‘v8::Maybe v8::Object::Set(v8::Localv8::Context, uint32_t, v8::Localv8::Value)’
3550 | V8_WARN_UNUSED_RESULT Maybe Set(Local context, uint32_t index,
| ^~~
/home/mfadel/.cache/node-gyp/13.13.0/include/node/v8.h:3550:37: note: candidate expects 3 arguments, 2 provided
../src/serialport.cpp:330:96: error: no matching function for call to ‘v8::Object::Set(v8::Localv8::String, Nan::imp::FactoryBasev8::Boolean::return_t)’
330 | >("dsr").ToLocalChecked(), Nan::Newv8::Boolean(data->dsr));
| ^

In file included from /home/mfadel/.cache/node-gyp/13.13.0/include/node/node.h:67,
from ../../../../nan/nan.h:54,
from ../src/./serialport.h:6,
from ../src/serialport.cpp:1:
/home/mfadel/.cache/node-gyp/13.13.0/include/node/v8.h:3547:37: note: candidate: ‘v8::Maybe v8::Object::Set(v8::Localv8::Context, v8::Localv8::Value, v8::Localv8::Value)’
3547 | V8_WARN_UNUSED_RESULT Maybe Set(Local context,
| ^~~
/home/mfadel/.cache/node-gyp/13.13.0/include/node/v8.h:3547:37: note: candidate expects 3 arguments, 2 provided
/home/mfadel/.cache/node-gyp/13.13.0/include/node/v8.h:3550:37: note: candidate: ‘v8::Maybe v8::Object::Set(v8::Localv8::Context, uint32_t, v8::Localv8::Value)’
3550 | V8_WARN_UNUSED_RESULT Maybe Set(Local context, uint32_t index,
| ^~~
/home/mfadel/.cache/node-gyp/13.13.0/include/node/v8.h:3550:37: note: candidate expects 3 arguments, 2 provided
../src/serialport.cpp:331:96: error: no matching function for call to ‘v8::Object::Set(v8::Localv8::String, Nan::imp::FactoryBasev8::Boolean::return_t)’
331 | >("dcd").ToLocalChecked(), Nan::Newv8::Boolean(data->dcd));
| ^

In file included from /home/mfadel/.cache/node-gyp/13.13.0/include/node/node.h:67,
from ../../../../nan/nan.h:54,
from ../src/./serialport.h:6,
from ../src/serialport.cpp:1:
/home/mfadel/.cache/node-gyp/13.13.0/include/node/v8.h:3547:37: note: candidate: ‘v8::Maybe v8::Object::Set(v8::Localv8::Context, v8::Localv8::Value, v8::Localv8::Value)’
3547 | V8_WARN_UNUSED_RESULT Maybe Set(Local context,
| ^~~
/home/mfadel/.cache/node-gyp/13.13.0/include/node/v8.h:3547:37: note: candidate expects 3 arguments, 2 provided
/home/mfadel/.cache/node-gyp/13.13.0/include/node/v8.h:3550:37: note: candidate: ‘v8::Maybe v8::Object::Set(v8::Localv8::Context, uint32_t, v8::Localv8::Value)’
3550 | V8_WARN_UNUSED_RESULT Maybe Set(Local context, uint32_t index,
| ^~~
/home/mfadel/.cache/node-gyp/13.13.0/include/node/v8.h:3550:37: note: candidate expects 3 arguments, 2 provided
../src/serialport.cpp:336:30: warning: ‘v8::Localv8::Value Nan::Callback::Call(int, v8::Localv8::Value) const’ is deprecated [-Wdeprecated-declarations]
336 | data->callback.Call(2, argv);
| ^
In file included from ../src/./serialport.h:6,
from ../src/serialport.cpp:1:
../../../../nan/nan.h:1740:3: note: declared here
1740 | Call(int argc, v8::Localv8::Value argv[]) const {
| ^~~~
../src/serialport.cpp: In function ‘Nan::NAN_METHOD_RETURN_TYPE GetBaudRate(Nan::NAN_METHOD_ARGS_TYPE)’:
../src/serialport.cpp:363:76: warning: cast between incompatible function types from ‘void ()(uv_work_t*)’ {aka ‘void ()(uv_work_s)’} to ‘uv_after_work_cb’ {aka ‘void ()(uv_work_s, int)’} [-Wcast-function-type]
363 | uv_default_loop(), req, EIO_GetBaudRate, (uv_after_work_cb)EIO_AfterGetBaudRate);
| ^~~~~~~~~~~~~~~~~~~~

../src/serialport.cpp: In function ‘void EIO_AfterGetBaudRate(uv_work_t*)’:
../src/serialport.cpp:378:106: error: no matching function for call to ‘v8::Object::Set(v8::Localv8::String, Nan::imp::IntegerFactoryv8::Integer::return_t)’
378 | e").ToLocalChecked(), Nan::Newv8::Integer(data->baudRate));
| ^

In file included from /home/mfadel/.cache/node-gyp/13.13.0/include/node/node.h:67,
from ../../../../nan/nan.h:54,
from ../src/./serialport.h:6,
from ../src/serialport.cpp:1:
/home/mfadel/.cache/node-gyp/13.13.0/include/node/v8.h:3547:37: note: candidate: ‘v8::Maybe v8::Object::Set(v8::Localv8::Context, v8::Localv8::Value, v8::Localv8::Value)’
3547 | V8_WARN_UNUSED_RESULT Maybe Set(Local context,
| ^~~
/home/mfadel/.cache/node-gyp/13.13.0/include/node/v8.h:3547:37: note: candidate expects 3 arguments, 2 provided
/home/mfadel/.cache/node-gyp/13.13.0/include/node/v8.h:3550:37: note: candidate: ‘v8::Maybe v8::Object::Set(v8::Localv8::Context, uint32_t, v8::Localv8::Value)’
3550 | V8_WARN_UNUSED_RESULT Maybe Set(Local context, uint32_t index,
| ^~~
/home/mfadel/.cache/node-gyp/13.13.0/include/node/v8.h:3550:37: note: candidate expects 3 arguments, 2 provided
../src/serialport.cpp:383:30: warning: ‘v8::Localv8::Value Nan::Callback::Call(int, v8::Localv8::Value) const’ is deprecated [-Wdeprecated-declarations]
383 | data->callback.Call(2, argv);
| ^
In file included from ../src/./serialport.h:6,
from ../src/serialport.cpp:1:
../../../../nan/nan.h:1740:3: note: declared here
1740 | Call(int argc, v8::Localv8::Value argv[]) const {
| ^~~~
../src/serialport.cpp: In function ‘Nan::NAN_METHOD_RETURN_TYPE Drain(Nan::NAN_METHOD_ARGS_TYPE)’:
../src/serialport.cpp:409:70: warning: cast between incompatible function types from ‘void ()(uv_work_t*)’ {aka ‘void ()(uv_work_s)’} to ‘uv_after_work_cb’ {aka ‘void ()(uv_work_s, int)’} [-Wcast-function-type]
409 | ue_work(uv_default_loop(), req, EIO_Drain, (uv_after_work_cb)EIO_AfterDrain);
| ^~~~~~~~~~~~~~

../src/serialport.cpp: In function ‘void EIO_AfterDrain(uv_work_t*)’:
../src/serialport.cpp:424:30: warning: ‘v8::Localv8::Value Nan::Callback::Call(int, v8::Localv8::Value*) const’ is deprecated [-Wdeprecated-declarations]
424 | data->callback.Call(1, argv);
| ^
In file included from ../src/./serialport.h:6,
from ../src/serialport.cpp:1:
../../../../nan/nan.h:1740:3: note: declared here
1740 | Call(int argc, v8::Localv8::Value argv[]) const {
| ^~~~
../src/serialport.cpp: At global scope:
../src/serialport.cpp:430:28: warning: unnecessary parentheses in declaration of ‘ToParityEnum’ [-Wparentheses]
430 | SerialPortParity NAN_INLINE(ToParityEnum(const v8::Localv8::String& v8str)) {
| ^
../src/serialport.cpp:449:30: warning: unnecessary parentheses in declaration of ‘ToStopBitEnum’ [-Wparentheses]
449 | SerialPortStopBits NAN_INLINE(ToStopBitEnum(double stopBits)) {
| ^
../src/serialport.cpp:460:17: error: variable or field ‘init’ declared void
460 | void init(v8::Handlev8::Object target) {
| ^~~~~~
../src/serialport.cpp:460:17: error: ‘Handle’ is not a member of ‘v8’
../src/serialport.cpp:460:34: error: expected primary-expression before ‘>’ token
460 | void init(v8::Handlev8::Object target) {
| ^
../src/serialport.cpp:460:36: error: ‘target’ was not declared in this scope
460 | void init(v8::Handlev8::Object target) {
| ^~~~~~
In file included from ../../../../nan/nan.h:54,
from ../src/./serialport.h:6,
from ../src/serialport.cpp:1:
../src/serialport.cpp:485:25: error: ‘init’ was not declared in this scope; did you mean ‘int’?
485 | NODE_MODULE(serialport, init);
| ^~~~
/home/mfadel/.cache/node-gyp/13.13.0/include/node/node.h:618:36: note: in definition of macro ‘NODE_MODULE_X’
618 | (node::addon_register_func) (regfunc),
| ^~~~~~~
../src/serialport.cpp:485:1: note: in expansion of macro ‘NODE_MODULE’
485 | NODE_MODULE(serialport, init);
| ^~~~~~~~~~~
make: *** [serialport.target.mk:114: Release/obj.target/serialport/src/serialport.o] Error 1
make: Leaving directory '/home/mfadel/Projects/LaserWeb4-dev-es6/node_modules/lw.comm-server/node_modules/serialport/build'
gyp ERR! build error
gyp ERR! stack Error: make failed with exit code: 2
gyp ERR! stack at ChildProcess.onExit (/usr/local/lib/node_modules/npm/node_modules/node-gyp/lib/build.js:194:23)
gyp ERR! stack at ChildProcess.emit (events.js:315:20)
gyp ERR! stack at Process.ChildProcess._handle.onexit (internal/child_process.js:275:12)
gyp ERR! System Linux 5.3.0-46-gecompileneric
gyp ERR! command "/usr/local/bin/node" "/usr/local/lib/node_modules/npm/node_modules/node-gyp/bin/node-gyp.js" "rebuild"
gyp ERR! cwd /home/mfadel/Projects/LaserWeb4-dev-es6/node_modules/lw.comm-server/node_modules/serialport
gyp ERR! node -v v13.13.0
gyp ERR! node-gyp -v v5.1.0
gyp ERR! not ok
npm WARN optional SKIPPING OPTIONAL DEPENDENCY: [email protected] (node_modules/fsevents):
npm WARN notsup SKIPPING OPTIONAL DEPENDENCY: Unsupported platform for [email protected]: wanted {"os":"darwin","arch":"any"} (current: {"os":"linux","arch":"x64"})

npm ERR! code ELIFECYCLE
npm ERR! errno 1
npm ERR! [email protected] install: prebuild-install || node-gyp rebuild
npm ERR! Exit status 1
npm ERR!
npm ERR! Failed at the [email protected] install script.
npm ERR! This is probably not a problem with npm. There is likely additional logging output above.

npm ERR! A complete log of this run can be found in:
npm ERR! /home/mfadel/.npm/_logs/2020-04-17T12_20_32_991Z-debug.log
`

What should I do? Where do I start from to build this application?

[mfadel85]

I tried this
npm run installdev
#590
but it didn't work,

I get

`> [email protected] installdev /home/mfadel/Projects/LaserWeb4-dev-es6

git submodule init && git submodule update --remote && npm install && npm update lw.comm-server

fatal: not a git repository (or any of the parent directories): .git
npm ERR! code ELIFECYCLE
npm ERR! errno 128
npm ERR! [email protected] installdev: git submodule init && git submodule update --remote && npm install && npm update lw.comm-server
npm ERR! Exit status 128
npm ERR!
npm ERR! Failed at the [email protected] installdev script.
npm ERR! This is probably not a problem with npm. There is likely additional logging output above.

npm ERR! A complete log of this run can be found in:
npm ERR! /home/mfadel/.npm/_logs/2020-04-17T12_24_54_785Z-debug.log
`

[mfadel85]

I tried the same using Laserweb4 4.0.994 version, got the same results,

[mfadel85]

I was able to build the software by changing node version to node 10, but now when I run the application I face another problem:

`Module not found: Error: Can't resolve '../data/lw.materials/materials' in '/home/mfadel/Projects/LaserWeb4-dev-es6/src/components'
@ ./src/components/machine-profiles.js 65:30-111
@ ./src/components/settings.js
@ ./src/components/laserweb.js
@ ./src/index.js
@ multi webpack-dev-server/client?http://0.0.0.0:8080 webpack/hot/only-dev-server babel-polyfill ./index.js

ERROR in ./src/components/about.js
Module not found: Error: Can't resolve '../data/lw.machines/machines' in '/home/mfadel/Projects/LaserWeb4-dev-es6/src/components'
@ ./src/components/about.js 61:127-190
@ ./src/components/laserweb.js
@ ./src/index.js
@ multi webpack-dev-server/client?http://0.0.0.0:8080 webpack/hot/only-dev-server babel-polyfill ./index.js

ERROR in ./src/reducers/machine-profiles.js
Module not found: Error: Can't resolve '../data/lw.machines/machines' in '/home/mfadel/Projects/LaserWeb4-dev-es6/src/reducers'
@ ./src/reducers/machine-profiles.js 24:3-69
@ ./src/reducers/index.js
@ ./src/index.js
@ multi webpack-dev-server/client?http://0.0.0.0:8080 webpack/hot/only-dev-server babel-polyfill ./index.js

ERROR in ./src/reducers/material-database.js
Module not found: Error: Can't resolve '../data/lw.materials/material-database.json' in '/home/mfadel/Projects/LaserWeb4-dev-es6/src/reducers'
@ ./src/reducers/material-database.js 44:66-120
@ ./src/reducers/index.js
@ ./src/index.js
@ multi webpack-dev-server/client?http://0.0.0.0:8080 webpack/hot/only-dev-server babel-polyfill ./index.js

ERROR in ./src/reducers/material-database.js
Module not found: Error: Can't resolve '../data/lw.materials/material-database.spec.json' in '/home/mfadel/Projects/LaserWeb4-dev-es6/src/reducers'
@ ./src/reducers/material-database.js 45:54-113
@ ./src/reducers/index.js
@ ./src/index.js
@ multi webpack-dev-server/client?http://0.0.0.0:8080 webpack/hot/only-dev-server babel-polyfill ./index.js
`

How can I solve this?

[mfadel85]

I copied the files from the https://github.com/LaserWeb/lw.machines and https://github.com/LaserWeb/lw.materials and it worked

[mfadel85]

I brought the LaserWeb4 dev-es6 branch and lw.comm-server master to a state where I could compile both under nodejs 10.16.3.

To get that working, you have to:

1. Clone lw.comm-server
2. Run npm install.
3. Test if node server.js works.
4. Clone LaserWeb4
5. Run npm run installdev.
6. Run npm run start-app for frontend testing. This does a webpack compile and starts a browser on localhost:8080/webpack-dev-server/. (it takes some time until webpack compile is ready!)

After updating something on the frontend:

• Run npm run bundle-dev in the LaserWeb4 folder. This creates the webpack files in the dist folder, which can then be copied over to the app folder of lw.comm-server (embeded website). Instead of manual copying you can run the script npm run update_lw4 in the lw.comm-server folder (needs bash console for ncp command).

I hope this gives you at least a start point which can be compiled!

I suggest adding this to the wiki like how to build from source, I couldn't found it there

[easytarget]

@mfadel85 Thankyou for posting that, I was able to get a from scratch install from source working now.

I did this on a 'vanilla' freshly installed Pi3B+ with Buster, and captured a log of it all.
I've written it all up (with markdown and notes) in a gist at:
https://gist.github.com/easytarget/fc7a6dabf893d1b461c5e66573d0725a

Appears to work well and the sidebar issues are resolved. (although I see another bug, I'll report that separately since I'd like to reproduce in chrome/windows first.)

I think this can be closed now ;-)

[cprezzi]

@easytarget Thank you alot for writing all this down and making it public! This will help new users to get it working.