bazelbuild/bazel

Duplicate --config=foo produces a seemingly pointless warning

Open

#11 592 ouverte le 15 juin 2020

Voir sur GitHub
 (7 commentaires) (0 réactions) (0 assignés)Java (4 465 forks)batch import
P3help wantedteam-Configurabilityteam-Documentationtype: documentation (cleanup)

Métriques du dépôt

Stars
 (25 384 stars)
Métriques de merge PR
 (Merge moyen 22j 20h) (77 PRs mergées en 30 j)

Description

As of Bazel 3.2.0, if you have build --config=foo in multiple places (like .bazelrc and ~/.bazelrc), bazel build produces the following warning:

WARNING: The following configs were expanded more than once: [foo]. For repeatable flags, repeats are counted twice and may lead to unexpected behavior.

It's not clear under which scenario --config=foo may be "repeatable" or produce "unexpected behavior". To my understanding, such a thing cannot happen, and hence the warning seems like a bug. There does not appear to be any conflict in the requested flags. Could the warning be removed?

Guide contributeur

Direction de recherche
Le message d'avertissement est généré dans le code d'analyse des options de Bazel. Recherchez la chaîne 'The following configs were expanded more than once' dans le code source de Bazel (probablement dans la classe OptionsParser ou ConfigExpander). Déterminez si l'avertissement est jamais utile (par exemple, pour les drapeaux répétables) et, le cas échéant, clarifiez le ; sinon, supprimez l'avertissement. Considérez de vérifier l'issue liée pour tout commentaire des mainteneurs sur la nécessité.
Stack technique
java
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.
1-3 heures
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
Java
Accessibilité débutantScore de 1 à 100 estimant l'accessibilité de cette issue pour un premier contributeur.
65