sindresorhus/get-windows

Silently fails under Linux if xwininfo/xprop is missing

Open

#75 geöffnet am 10. Juli 2020

Auf GitHub ansehen
 (4 Kommentare) (0 Reaktionen) (0 zugewiesene Personen)JavaScript (913 Stars) (194 Forks)user submission
enhancementhelp wanted

Beschreibung

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.

Contributor Guide

Tech Stack
javascriptnodejs
Domain
desktop
Issue Type
bug
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.
under 1 hour
AktivitätsstatusWie verfügbar das Issue gerade wirkt: frisch, aktiv, stale, blockiert oder wartet auf Maintainer-Input.
stale
KlarheitWie klar das Issue die erwartete Änderung, Akzeptanzkriterien und den nächsten Schritt erklärt.
clear
Voraussetzungen
JavaScriptNode.jsUnderstanding of Promises
EinsteigerfreundlichkeitEin Score von 1 bis 100, der schätzt, wie zugänglich dieses Issue für First-Time Contributors ist.
80
Research-Richtung
The issue points to a specific location in lib/linux.js (lines 109-111) where the promises fail silently when xwininfo or xprop are missing. The contributor should modify that code to catch errors during execution and reject the promise with a meaningful error message, or throw an Error for the synchronous variant. Additionally, consider updating the documentation to note the dependency on these tools and the lack of Wayland support. Testing on a Linux system without xwininfo and xprop will help verify the fix.