bazelbuild/bazel

Better bash-less Windows support in actions.run_shell: add command_bat, command_ps

Open

#15.194 aperta il 7 apr 2022

Vedi su GitHub
 (4 commenti) (0 reazioni) (0 assegnatari)Java (4465 fork)batch import
P3good first issuehelp wantedteam-Rules-APItype: feature request

Metriche repository

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

Descrizione

Description of the feature request:

For better bash-less Windows support,ctx.actions.run_shell could add command_bat (with the same behavior as genrule's cmd_bat) and command_ps (with the same behavior as genrule's cmd_ps).

This would, in particular, considerably simplify skylib's Windows support, where currently we are forced to write and execute temporary .bat files on Windows (and attempt to deal with the ancient terror of argument quoting in .bat interpreter context - take a look at https://stackoverflow.com/questions/4094699/how-does-the-windows-command-interpreter-cmd-exe-parse-scripts if you don't mind losing some points of sanity).

Pinging potentially interested parties: @brandjon @meteorcloudy @alexeagle

Guida contributor

Direzione di ricerca
Esamina l'implementazione di actions.run shell e cmd bat/cmd ps di genrule. Identifica dove aggiungere nuovi parametri e aggiornare i percorsi di codice correlati. Controlla i test esistenti per il supporto Windows.
Tech stack
javashell
Dominio
build system
Tipo issue
Funzionalità
DifficoltàQuanto dovrebbe essere impegnativa per un nuovo contributor.
3
Tempo stimatoTempo stimato per completare e verificare uno scope piccolo.
Mezza giornata
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
GitJavaBazel
Adatta ai principiantiQuanto la issue è adatta a contributor alla prima esperienza.
65