bazelbuild/bazel

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

Open

#21.932 geöffnet am 9. Apr. 2024

Auf GitHub ansehen
 (1 Kommentar) (0 Reaktionen) (0 zugewiesene Personen)Java (4.465 Forks)batch import
P2help wantedteam-Configurabilityteam-ExternalDepstype: feature request

Repository-Metriken

Stars
 (25.384 Stars)
PR-Merge-Metriken
 (Durchschn. Merge 22T 20h) (77 gemergte PRs in 30 T)

Beschreibung

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

Contributor Guide

Research-Richtung
Untersuchen Sie, wie Sie dem von sh configure generierten sh toolchain `exec compatible with` hinzufügen können. Sehen Sie sich bestehende Toolchain Definitionen im Bazel Repository an, um Beispiele für die Verwendung von HOST CONSTRAINTS zu finden.
Tech Stack
java
Domain
build systemdevops
Issue Type
Fehler
SchwierigkeitGeschätzte Implementierungsschwierigkeit für neue Contributors, von 1 für sehr kleine Änderungen bis 5 für Expertenarbeit.
2
Geschätzte ZeitEin grober Zeitrahmen, um zu recherchieren, zu implementieren, zu testen und einen Pull Request vorzubereiten.
1-3 Stunden
AktivitätsstatusWie verfügbar das Issue gerade wirkt: frisch, aktiv, stale, blockiert oder wartet auf Maintainer-Input.
Aktiv
KlarheitWie klar das Issue die erwartete Änderung, Akzeptanzkriterien und den nächsten Schritt erklärt.
Klar
Voraussetzungen
BazelStarlarkGit
EinsteigerfreundlichkeitEin Score von 1 bis 100, der schätzt, wie zugänglich dieses Issue für First-Time Contributors ist.
60