sillysloft / fluxbox

Fluxbox Window Manager (Mirror)
http://fluxbox.org/news/
Other
0 stars 1 forks source link

fluxbox messes up a regex when it updates the apps file #664

Open sillysloft opened 17 years ago

sillysloft commented 17 years ago

My e-mail: heilong@bluebottle.com I've found this bug in fluxbox-1.0_rc2. I have this manual entry in the ~/.fluxbox/apps file: [app] (Gecko) (title=(.*SeaMonkey|Mail.*Newsgroups)) [Dimensions] {1588 1022} [Position] (UPPERLEFT) {90 0} [end] The title regex matches both the seamonkey main window and the mail&news window. When I change some other app's "Remember..." settings, fluxbox writes the new apps file and this regex changes to [app] (Gecko) (title=\(.*SeaMonkey|Mail.*Newsgroups\)) which doesn't work, of course. I can understand that this behaviour may be intentional - for those people who forget to put the backslashes before the parentheses when they want literal parentheses, but that's very windoze-ish - to mess people's regexes to satisfy newbies. Newbies should just be pointed to a regex tutorial, or read a FAQ or smth. Please, fix this bug. Btw I think it would be very useful to have a small command-line script/utility to check if a given window matches a particular [app] line - for quicker debugging of these regexes, because it's irritating to restart fluxbox every time. If you pls point me to a list of "apps file" matching features (I know of title and role, currently), I can write this in bash myself. Thanks for this wonderful software.

Reported by: *anonymous

sillysloft commented 17 years ago

Logged In: YES user_id=1311290

I submitted this bug, forgot to log-in

Original comment by: heilong

sillysloft commented 17 years ago

Logged In: YES user_id=1538990 Originator: NO

Heh, actually it was to prevent us from screwing up ourselves: https://sourceforge.net/tracker/index.php?func=detail&aid=1327878&group\_id=35398&atid=413960

I've now fixed this properly in svn.

The other current options are name (the default), which matches the first string in `xprop WM_CLASS' and class, which matches the second string.

Original comment by: mark-t

sillysloft commented 17 years ago

Original comment by: mark-t