On Mon, May 15, 2023 at 06:26:41AM +0000, Klemens Nanni wrote:
> Both fvwm(1) and twm(1) have a restart menu that contains other window
> managers by default, which is useful if you want to switch around
> without restarting X and/or custom window manager config.
> 
> cwm(1) only offers to restart into itself by deafult.
> Add the other two we ship by default so users can round trip between
> them.
> 
> Feedback? OK?

Last year I mentionned that I think we should retire twm. It's really
too old and missing support for the modern window managers hints.

People still using it should switch to cwm or maybe ctwm from ports
(to keep the same configurarion system), or someone should step up to
maintain it and enhance it with exwmh support. (but this is somehow
just wasting time imho).

Otherwise ok to add this and fix the other WM menus for other window
managers (those parts of the configs are already local changes in
Xenocara)

> 
> PS:  fwvm and twm menus more programs we don't ship, e.g. "wm2", and
>   twm dies when failing to execute them (fvwm and cwm keeps running);
>   do we want to keep those default-broken entries around?
> 
> Index: conf.c
> ===================================================================
> RCS file: /cvs/xenocara/app/cwm/conf.c,v
> retrieving revision 1.255
> diff -u -p -r1.255 conf.c
> --- conf.c    26 Feb 2022 15:19:18 -0000      1.255
> +++ conf.c    15 May 2023 06:11:01 -0000
> @@ -307,6 +307,8 @@ conf_init(struct conf *c)
>       conf_cmd_add(c, "lock", "xlock");
>       conf_cmd_add(c, "term", "xterm");
>       conf_wm_add(c, "cwm", "cwm");
> +     conf_wm_add(c, "fvwm", "fvwm");
> +     conf_wm_add(c, "twm", "twm");
>  
>       c->font = xstrdup("sans-serif:pixelsize=14:bold");
>       c->wmname = xstrdup("CWM");
> 

-- 
Matthieu Herrb

Reply via email to