On Mittwoch 20 Mai 2009, you wrote:
> On Thu, May 21, 2009 at 01:28, Andreas Klöckner <[email protected]> 
wrote:
> awesome ver?
3.3rc2

> shifty-master HEAD? (patch applies clean)
yes, HEAD

> The patch seems to be good but (before I push it) can you be a bit
> more specific as to what breaks (config snippets very helpful) if you
> run without it? I can only assume that the problem is that you're
> setting screen = 2 in somewhere in shifty.config and that breaks when
> running only on LVDS?

Yup, exactly.

Without the fix, all I can say is that windows with matches for the tags on 
the non-existent screen became inacessible--the apps are still running, but I 
could not figure out how to get the windows to show up. It was mildly 
frustrating. There was also some debug spew in .xsesion-errors, but I have no 
record of it right now.

> Btw, I've been using this sort of statement in rc.lua to force screen
> in various settings (shifty, naughty etc.) on my laptop, but I have to
> admit that I don't use multi-screen every day:
>
>    if screen.count() == 2 then LCD = 2 else LCD = 1 end
>
> Your solution might make more sense - shifty is supposed to be
> dynamic, so validation of screen could be done internally.

See it as guarding against user error--creating tags on non-existing screens 
is kind of pointless. On top of that, this fix makes dynamic dual-screen 
setups work with relative ease. What's not to like? :)

Andreas

Attachment: signature.asc
Description: This is a digitally signed message part.

Reply via email to