Skip to content

build: add node_lib_target_name to cctest deps

Rodrigo Muino Tomonari requested to merge github/fork/danbev/fix_cctest_target_dependency into master

Currently the cctest target depend on the node_core_target_name target. But it is the node_lib_target_name target that compiles the sources now which means that if a source file in src is updated the cctest executable will not be re-linked against it, but will remain unchanged. The code will still be compiled, just not linked which means that if you are debugging you'll not see the changes and also a warning will be displayed about this issue.

This commit changes the cctest target to depend on node_lib_target_name.

Checklist
  • make -j4 test (UNIX), or vcbuild test (Windows) passes
  • commit message follows commit guidelines
Affected core subsystem(s)

build, test

Merge request reports