cartographer-project/cartographer

Investigate and fix issues compiling Cartographer with gRPC support.

Open

#1 894 ouverte le 16 juil. 2022

Voir sur GitHub
 (0 commentaires) (0 réactions) (1 assigné)C++ (2 188 forks)batch import
bughelp wanted

Métriques du dépôt

Stars
 (6 665 stars)
Métriques de merge PR
 (Aucune PR mergée en 30 j)

Description

TL;DR: Best approach could be to target Ubuntu 22.04, install the deb package libgrpc++-dev and fix further build issues.

Right now Dockerfile.bionic.grpc fails when running cartographer/scripts/install_grpc.sh with output like this:

make
/bin/sh: 0: Illegal option -t
/bin/sh: 0: Illegal option -t
/bin/sh: 0: Illegal option -t
/bin/sh: 0: Illegal option -g
/bin/sh: 0: Illegal option -g

Using newer versions of Ubuntu or gRPC produces the same issue it seems. But since Ubuntu 22.04 ships with a package for gRPC, we might be able to use this instead, with no need to look into build issues of gRPC.

Using this approach still does not unbreak the full Cartographer build, but now fails building async_grpc which is part of Cartographer and has linker errors for its tests, but these issues seem unrelated to gRPC.

Guide contributeur

Direction de recherche
Mettez à jour le Dockerfile pour utiliser Ubuntu 22.04 et installez libgrpc++ dev au lieu de compiler gRPC à partir des sources. Ensuite, enquêtez sur les erreurs de l'éditeur de liens dans les tests async grpc.
Stack technique
cpp
Domaine
build system
Type d'issue
Bug
DifficultéDifficulté estimée pour un nouveau contributeur, de 1 pour un très petit changement à 5 pour un travail expert.
3
Temps estiméFourchette de temps approximative pour investiguer, implémenter, tester et préparer une pull request.
Une demi journée
Statut d'activitéDisponibilité apparente de l'issue : fraîche, active, ancienne, bloquée ou en attente d'un mainteneur.
Active
ClartéClarté avec laquelle l'issue explique le changement attendu, les critères d'acceptation et la prochaine étape.
Claire
Prérequis
C++gRPCBuild systems
Accessibilité débutantScore de 1 à 100 estimant l'accessibilité de cette issue pour un premier contributeur.
55