envoyproxy/envoy

windows: Add build verification step to ensure path lengths

Open

#10 581 ouverte le 30 mars 2020

Voir sur GitHub
 (5 commentaires) (0 réactions) (0 assignés)C++ (5 373 forks)batch import
area/buildhelp wanted

Métriques du dépôt

Stars
 (27 997 stars)
Métriques de merge PR
 (Merge moyen 8j) (378 PRs mergées en 30 j)

Description

The Windows path length limit is 260 characters and can potentially be exceeded as seen in https://github.com/envoyproxy/envoy/pull/10560. The compiler error emitted when the max path length is exceeded does not point to the actual problem by outputting No such file or directory. It is unclear if this is the only behavior.

Ideally, there would be an explicit check for these path lengths during the Windows build and failure output that clearly indicates the problem.

Guide contributeur

Direction de recherche
Créez un script ou modifiez le système de build (par exemple CMake ou Bazel) pour vérifier la longueur des chemins de fichiers lors des builds Windows et émettre un message d'erreur clair lorsque les chemins dépassent 260 caractères. Examinez les outils ou techniques existants pour la vérification de la longueur des chemins et intégrez les dans le processus de build.
Stack technique
cpp
Domaine
backenddevops
Type d'issue
Fonctionnalité
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.
Plutôt claire
Prérequis
WindowsC++ build toolsGit
Accessibilité débutantScore de 1 à 100 estimant l'accessibilité de cette issue pour un premier contributeur.
60