bazelbuild/bazel

@bazel_tools//tools/sh:local_sh_toolchain is missing any constraints to make it local

Open

#21 932 ouverte le 9 avr. 2024

Voir sur GitHub
 (1 commentaire) (0 réactions) (0 assignés)Java (4 465 forks)batch import
P2help wantedteam-Configurabilityteam-ExternalDepstype: feature request

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

@bazel_tools//tools/sh:sh_configure is a repo rule that detects and configures sh_toolchain (path to bash), by querying the local host.

The guts of the generated repo rule are: https://github.com/bazelbuild/bazel/blob/0f760395e04cb0accb912be128b4f363663e8d30/tools/sh/sh_configure.bzl#L59-L69

I think this needs at least exec_compatible_with = HOST_CONSTRAINTS or else this path will incorrectly be used on other exec platforms. (HOST_CONSTRAINTS isn't specific enough, but it's the best we've got.)

Guide contributeur

Direction de recherche
Recherchez comment ajouter `exec compatible with` au sh toolchain généré par sh configure. Consultez les definitions de toolchain existantes dans le depot Bazel pour des exemples d'utilisation de HOST CONSTRAINTS.
Stack technique
java
Domaine
build systemdevops
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.
2
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
BazelStarlarkGit
Accessibilité débutantScore de 1 à 100 estimant l'accessibilité de cette issue pour un premier contributeur.
60