sindresorhus/get-windows

Silently fails under Linux if xwininfo/xprop is missing

Open

#75 aperta il 10 lug 2020

Vedi su GitHub
 (4 commenti) (0 reazioni) (0 assegnatari)JavaScript (194 fork)user submission
enhancementhelp wanted

Metriche repository

Star
 (913 star)
Metriche merge PR
 (Nessuna PR mergiata in 30 g)

Descrizione

Fails without further notice if xprop or xwininfo because they are either not installed, or not present in the PATH variable. If you ask me the promise should fail with a proper error when something fails (or throw an Error for the sync variant). The current behaviour makes debugging the root of the problem quite difficult.

https://github.com/sindresorhus/active-win/blob/430e6efc02823c1b04331a987012a519c360ab23/lib/linux.js#L109-L111

Also you might want to add that there is no Wayland support at the moment to the documentation especially considering the increasing popularity of Wayland.

I might PR this myself when I find the time.

Guida contributor

Direzione di ricerca
Esamina il codice in lib/linux.js alle righe 109-111 e aggiungi una corretta gestione degli errori quando xprop o xwininfo sono mancanti.
Tech stack
javascriptnodejs
Dominio
desktop
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
GitNode.js
Adatta ai principiantiQuanto la issue è adatta a contributor alla prima esperienza.
65