Re: [dev] Re: [wmii] wimenu custom completion

From: Connor Lane Smith <cls_AT_lubutu.com>
Date: Fri, 13 Aug 2010 17:39:12 +0100

Hey,

On 13 August 2010 02:19, Kris Maglione <maglione.k_AT_gmail.com> wrote:
> I can't speak for the dmenu developers, but new features are usually added
> only after someone's written a patch and it becomes popular.

Indeed.

On 12 August 2010 21:51, LuX <lux.onthenet_AT_free.fr> wrote:
> By the way, is it intended to add to dmenu a similar feature (and also
> the -S option of wimenu)?

An important thing to remember is that a feature does not need to be
in mainline for you to patch it in. Mainline should therefore only
include reliable code, which complex patches are not. If you want
something more exotic, you can patch it in yourself. I have made this
point before.

The '-S' flag seemed like an interesting patch to write, so I hacked
it together and you'll find it attached (diff against tip). There is
already a file completion patch available[1].

I share Anselm's opinion that if you want to execute a more complex
command, you can just open a terminal. It's important to remember that
dmenu is a menu: you select items. The fact that you can append
arbitrary arguments is just a very fortunate byproduct of its
filter-by-input design. Including custom completion etc etc is beyond
dmenu's role, which is acceptable for a patch, but should not be
included in mainline.

On 13 August 2010 11:15, LuX <lux.onthenet_AT_free.fr> wrote:
> Well, this might be today the main difference between dmenu and
> wimenu, except if A.R. Garbe has changed his mind in the mean time. As
> we say in France: « il n'y a que les imbéciles qui ne changent pas
> d'avis » (only idiots never change their mind).

That's a good point. When are you planning to change *your* mind?

(I'm the main developer of dmenu, since March. It is not only Anselm
who refuses to include this.)

Hope this helps.

cls

[1]: http://lists.suckless.org/dev/1008/5330.html

Received on Fri Aug 13 2010 - 18:39:12 CEST

This archive was generated by hypermail 2.2.0 : Fri Aug 13 2010 - 18:48:02 CEST