livepeer/go-livepeer

livepeer_bench: back-slashes in the file path don't work

Aperta

#1844 aperta il 15 apr 2021

Vedi su GitHubGuida contributor
 (0 commenti) (0 reazioni) (0 assegnatari)Go (223 fork)auto 404
good first issuehacktoberfesttype: bug

Metriche repository

Star
 (583 stelle)
Metriche merge PR
 (Metriche PR in attesa)

Descrizione

Describe the bug Back-slashes in the input manifest's path cause the benchmarking to fail on Windows.

To Reproduce Here's the original Discord message from a user reporting this problem.

Expected behavior Back-slashes should work out-of-the-box on Windows, or alternatives (like using forward-slashes) should be better documented.

Screenshots image

Desktop (please complete the following information):

  • OS: Windows

Additional context Golang's path library does not work well with back-slashes We can try using an alternate library when creating the segment path from the input manifest's directory: https://github.com/livepeer/go-livepeer/blob/3908252d23edd62590c61b02598c5fffc148014b/cmd/livepeer_bench/livepeer_bench.go#L106

Guida contributor

Direzione di ricerca
Esamina la gestione del percorso in cmd/livepeer bench/livepeer bench.go riga 106 e sostituisci path.Join con filepath.Join o usa una libreria di percorsi multipiattaforma.
Tech stack
go
Dominio
backend
Tipo issue
Bug
DifficoltàQuanto dovrebbe essere impegnativa per un nuovo contributor.
2
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
Go
Adatta ai principiantiQuanto la issue è adatta a contributor alla prima esperienza.
70