-
Notifications
You must be signed in to change notification settings - Fork 15
Build System Clean up! #532
Copy link
Copy link
Open
Labels
docs/toolingDocumentation/development/deployment tools which improve the developers' and users' experienceDocumentation/development/deployment tools which improve the developers' and users' experiencegood first issueGood issue for new developers looking to explore and contribute to the codebaseGood issue for new developers looking to explore and contribute to the codebasenice-to-haveGood feature/modification, but it doesn't impact the current state of the project's development.Good feature/modification, but it doesn't impact the current state of the project's development.
Milestone
Description
Metadata
Metadata
Assignees
Labels
docs/toolingDocumentation/development/deployment tools which improve the developers' and users' experienceDocumentation/development/deployment tools which improve the developers' and users' experiencegood first issueGood issue for new developers looking to explore and contribute to the codebaseGood issue for new developers looking to explore and contribute to the codebasenice-to-haveGood feature/modification, but it doesn't impact the current state of the project's development.Good feature/modification, but it doesn't impact the current state of the project's development.
Type
Fields
No fields configured for Task.
Projects
StatusShow more project fields
🤷♂️ Needs Triage
The
CMakebuild system of Odin-data needs some cleanup. Some files which have likely been renamed or deleted still have references in the code comments and the build system. Search the build system for files referenced by the CMake instructions which no longer have references in the project and remove them. Also delete/update comments in source or header files that refer to those deprecated classes/files.As an example, in
cpp/frameProcessor/src/CMakeLists.txt, we have these lines:However, the
SharedMemoryParser.cppandSocketHandler.cppfiles no longer exist in the project. A global search forSharedMemoryParserbrings up more references. These references should be removed and the relevant comments updated or removed.