bazelbuild/bazel

Documentation genrule example - maybe avoid $(location ..) or $(locations ..)

Open

#15.078 aperta il 18 mar 2022

Vedi su GitHub
 (0 commenti) (0 reazioni) (0 assegnatari)Java (4465 fork)batch import
P3help wantedteam-Documentationteam-Rules-Servertype: documentation (cleanup)

Metriche repository

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

Descrizione

Description of the problem / feature request:

https://docs.bazel.build/versions/main/be/general.html#genrule_examples

Elsewhere I've seen that we're not supposed to be using $(location) calls and should instead be using $(rootpath ..) or $(execpath ..)

https://github.com/bazelbuild/bazel/issues/2475#issuecomment-339318016

I just spent hours debugging why a genrule of mine didn't work, when it wasn't exactly clear from the example that

  1. We shouldn't be using $(location ..)
  2. That $(location ..) can return different things in Skylark depending on srcs vs other generated outs.

Maybe this is clear for people who've been in Bazel for years and are aware of the changes to all make-style vars.

Guida contributor

Direzione di ricerca
Il problema richiede di aggiornare gli esempi di genrule su https://docs.bazel.build/versions/main/be/general.html#genrule examples per utilizzare $(rootpath) o $(execpath) invece di $(location), facendo riferimento a una discussione precedente. Non esistono commenti o PR. La correzione comporta la modifica del file di documentazione degli esempi di genrule per sostituire le chiamate $(location) e aggiungere una nota sulla deprecazione.
Tech stack
Nessuno
Dominio
documentation
Tipo issue
Documentazione
DifficoltàQuanto dovrebbe essere impegnativa per un nuovo contributor.
1
Tempo stimatoTempo stimato per completare e verificare uno scope piccolo.
Meno di 1 ora
Stato attivitàQuanto è probabile che la issue sia ancora attiva e reviewable.
Nuova
ChiarezzaQuanto chiaramente la issue descrive problema, scope e risultato atteso.
Chiara
Prerequisiti
basic Bazel understanding
Adatta ai principiantiQuanto la issue è adatta a contributor alla prima esperienza.
85