Skip to content

Fix build and run unit tests#3

Draft
erizhang wants to merge 2 commits intomasterfrom
cursor/fix-build-and-run-unit-tests-48a3
Draft

Fix build and run unit tests#3
erizhang wants to merge 2 commits intomasterfrom
cursor/fix-build-and-run-unit-tests-48a3

Conversation

@erizhang
Copy link
Copy Markdown
Owner

@erizhang erizhang commented Oct 9, 2025

Resolves project build failures and enables unit tests.

The project encountered several build issues:

  • A global function pointer shutdown_server was defined in a header, causing multiple definition errors. This is now declared extern in the header and defined in a source file.
  • The main executable was linked using cc instead of g++, leading to linker errors with the CppUTest C++ library. The linker command now uses g++.
  • Corrected typos in the Makefile (e.g., PRDUCT_MAIN_OBJ, localhst:8888).
  • Removed the automatic execution of the main program from the Makefile to prevent build blockage.

Open in Cursor Open in Web

cursoragent and others added 2 commits October 9, 2025 01:47
Co-authored-by: zhang.lyuan <zhang.lyuan@gmail.com>
Co-authored-by: zhang.lyuan <zhang.lyuan@gmail.com>
@cursor
Copy link
Copy Markdown

cursor bot commented Oct 9, 2025

Cursor Agent can help with this pull request. Just @cursor in comments and I'll start working on changes in this branch.
Learn more about Cursor Agents

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

Successfully merging this pull request may close these issues.

2 participants