bazelbuild/bazel

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

Open

#21.932 aperta il 9 apr 2024

Vedi su GitHub
 (1 commento) (0 reazioni) (0 assegnatari)Java (4465 fork)batch import
P2help wantedteam-Configurabilityteam-ExternalDepstype: feature request

Metriche repository

Star
 (25.384 star)
Metriche merge PR
 (Merge medio 22g 20h) (77 PR mergiate in 30 g)

Descrizione

@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.)

Guida contributor

Direzione di ricerca
Indaga come aggiungere `exec compatible with` allo sh toolchain generato da sh configure. Esamina le definizioni di toolchain esistenti nel repository di Bazel per esempi di utilizzo di HOST CONSTRAINTS.
Tech stack
java
Dominio
build systemdevops
Tipo issue
Bug
DifficoltàQuanto dovrebbe essere impegnativa per un nuovo contributor.
2
Tempo stimatoTempo stimato per completare e verificare uno scope piccolo.
1-3 ore
Stato attivitàQuanto è probabile che la issue sia ancora attiva e reviewable.
Attiva
ChiarezzaQuanto chiaramente la issue descrive problema, scope e risultato atteso.
Chiara
Prerequisiti
BazelStarlarkGit
Adatta ai principiantiQuanto la issue è adatta a contributor alla prima esperienza.
60