unable to detect DE or WM about fetch-master-6000 HOT 16 CLOSED

I will see about that.
I will lookup how neofetch does it, and implement that here.

I have used dwm and xmonad (on arch linux gui) , fm6000 detected the wms correctly
DESKTOP_SESSION and XDG_SESSION_DESKTOP were both setup correctly.

from fetch-master-6000.

xprop can be used too, but i think the best would be checking the running processes for window managers(pfetch uses this method as a fallback to xprop)

from fetch-master-6000.

yeah neofetch does that too ps and greping.
I will implement that soon.

from fetch-master-6000.

@anhsirk0 Instead of using complicated functions why not just use wmctrl if its giving direct result?

from fetch-master-6000.

@anhsirk0 Instead of using complicated functions why not just use wmctrl if its giving direct result?

It is very easy to add support for wmctrl or xprop but when we can implement it in perl, why should we prefer an external program?

from fetch-master-6000.

@anhsirk0 Instead of using complicated functions why not just use wmctrl if its giving direct result?

It is very easy to add support for wmctrl or xprop but when we can implement it in perl, why should we prefer an external program?

Exactly.

from fetch-master-6000.

Even tho its not minimal? ok doesnt matter.

from fetch-master-6000.

I updated the script. can you confirm?

from fetch-master-6000.

Even tho its not minimal? ok doesnt matter.

we don't want unneccessary dependencies and prefer to do things with the language (perl in this case) rather than using external program. This increases portability (bsd's core util and gnu core utils are not that compatible , for example uptime gnu's uptime supports uptime -p while bsd's does not. We are even avoiding to use grep or awk for the same reason.
Edit: typos

from fetch-master-6000.

I updated the script. can you confirm?

Yes it shows dwm correctly but in openbox it showing UNKNOWN

from fetch-master-6000.

added openbox to the checks too

from fetch-master-6000.

added openbox to the checks too

So this is how it gonna be? you just add the new wm to array?
Isnt there any functional way to "just detect current wm" using perl?

from fetch-master-6000.

added openbox to the checks too

So this is how it gonna be? you just add the new wm to array? Isnt there any functional way to "just detect current wm" using perl?

Sadly no
Neofetch/pfetch also does it in similar ways

from fetch-master-6000.

I added wmctrl too.

from fetch-master-6000.

Nice. Sometime we need to improvise even if its add another dependancy. but thats the beauty of perl language we can beautifully integrate unix shell with it.

from fetch-master-6000.

Well I was slightly wrong about external dependencies. Its not like fm6000 won't work without wmctrl or ps etc etc.

from fetch-master-6000.