micro-ROS / micro-ROS/micro_ros_espidf_component

embeddedRTPS build issue

Ouverte
#336 3 commentaires 0 réactions 0 personnes assignées Voir sur GitHub

Personne n'a encore pris cette issue.

Langage dominant
C
Étoiles
419
Forks
124
Merge moyen
21 h 35 min
PR mergées (30 j)
3

Description

`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.

Guide de contribution

Ouvrir le guide de contribution

Par où commencer

  1. Lisez l'issue en entier, puis le guide de contribution du projet.
  2. Signalez en commentaire que vous la prenez — cela évite que deux personnes fassent le même travail.
  3. Forkez le dépôt et travaillez sur une branche.
  4. Ouvrez une pull request qui référence le numéro de l'issue.

Piste de recherche

Commencez par localiser les appels à ament_cmake_target_dependencies dans micro-ROS/embeddedRTPS, micro-ROS/rmw_embeddedrtps et micro-ROS/rcl, en notant l’utilisation conditionnelle dans rcl. Vérifiez le build CI de embeddedRTPS et assurez-vous que les trois bibliothèques se construisent sans restaurer le fichier ament_cmake supprimé.

Rédigé par le modèle d'indexation à partir du texte de l'issue.

Évaluation

Stack technique
cmake
Domaine
build-system, embedded-iot
Type d'issue
Bug
Difficulté
4/5
Temps estimé
3-5 jours
Activité
Calme
Clarté
Plutôt claire
Accessibilité débutants
45/100

Recevez les nouvelles issues par e-mail

Un résumé court des issues GitHub adaptées aux débutants.