micro-ROS / micro-ROS/micro_ros_espidf_component
embeddedRTPS build issue
Dieses Issue hat noch niemand übernommen.
- Vorherrschende Sprache
- C
- Sterne
- 419
- Forks
- 124
- Ø Merge
- 21 Std. 35 Min.
- Gemergte PRs (30 T.)
- 3
Beschreibung
`embeddedRTPS` failed to build in CI. This is because [`ament_cmake` recently removed `ament_cmake_target_dependencies`](https://github.com/ament/ament_cmake/pull/614). However, `micro-ROS/embeddedRTPS`, `micro-ROS/rmw_embeddedrtps`, and `micro-ROS/rcl` are still using it. The call in `micro-ROS/rcl` is wrapped with `if(NOT RCL_MICROROS)`, so it doesn't trigger during the build.
An AI provided me with a very hacky (but working) workaround — adding `ament_target_dependencies.cmake` back into `ament_cmake` during the build... But I think it's better to update the three libraries mentioned above.
Beitragsleitfaden
Erste Schritte
- Lies das ganze Issue und danach den Beitragsleitfaden des Projekts.
- Schreib ins Issue, dass du es übernimmst — das erspart doppelte Arbeit.
- Forke das Repository und arbeite in einem Branch.
- Öffne einen Pull Request, der die Issue-Nummer nennt.
Rechercherichtung
Beginnen Sie damit, die Aufrufe von ament_cmake_target_dependencies in micro-ROS/embeddedRTPS, micro-ROS/rmw_embeddedrtps und micro-ROS/rcl zu finden, und beachten Sie dabei die bedingte Verwendung in rcl. Überprüfen Sie den embeddedRTPS-CI-Build und verifizieren Sie, dass alle drei Bibliotheken erstellt werden, ohne die entfernte Datei ament_cmake wiederherzustellen.
Vom Indexierungsmodell aus dem Issue-Text verfasst.
Bewertung
- Tech-Stack
- cmake
- Bereich
- build-system, embedded-iot
- Issue-Typ
- Bug
- Schwierigkeit
- 4/5
- Geschätzter Aufwand
- 3-5 Tage
- Aktivitätsstatus
- Ruhig
- Klarheit
- Größtenteils klar
- Anfängerfreundlichkeit
- 45/100