[PATCH] gnu: ratpoison: Add multiple monitors support.

[PATCH] gnu: ratpoison: Add multiple monitors support.

[Mathieu Lirzin]

[-- Attachment #1: Type: text/plain, Size: 147 bytes --]


* gnu/packages/ratpoison.scm (ratpoison)[inputs]: Add "libxinerama".
---
 gnu/packages/ratpoison.scm | 1 +
 1 file changed, 1 insertion(+)


[-- Warning: decoded text below may be mangled, UTF-8 assumed --]
[-- Attachment #2: 0001-gnu-ratpoison-Add-multiple-monitors-support.patch --]
[-- Type: text/x-patch; name="0001-gnu-ratpoison-Add-multiple-monitors-support.patch", Size: 406 bytes --]

diff --git a/gnu/packages/ratpoison.scm b/gnu/packages/ratpoison.scm
index bde3fcc..e226d99 100644
--- a/gnu/packages/ratpoison.scm
+++ b/gnu/packages/ratpoison.scm
@@ -70,6 +70,7 @@
        ("libXtst" ,libxtst)
        ("freetype" ,freetype)
        ("fontconfig" ,fontconfig)
+       ("libxinerama" ,libxinerama)
        ("libXft" ,libxft)
        ("libXpm" ,libxpm)
        ("libXt" ,libxt)

Re: [PATCH] gnu: ratpoison: Add multiple monitors support.

[Mark H Weaver]

Mathieu Lirzin <mthl@openmailbox.org> writes:

> * gnu/packages/ratpoison.scm (ratpoison)[inputs]: Add "libxinerama".
> ---
>  gnu/packages/ratpoison.scm | 1 +
>  1 file changed, 1 insertion(+)
>
> diff --git a/gnu/packages/ratpoison.scm b/gnu/packages/ratpoison.scm
> index bde3fcc..e226d99 100644
> --- a/gnu/packages/ratpoison.scm
> +++ b/gnu/packages/ratpoison.scm
> @@ -70,6 +70,7 @@
>         ("libXtst" ,libxtst)
>         ("freetype" ,freetype)
>         ("fontconfig" ,fontconfig)
> +       ("libxinerama" ,libxinerama)
>         ("libXft" ,libxft)
>         ("libXpm" ,libxpm)
>         ("libXt" ,libxt)

Okay, please push.

   Thanks!
     Mark

Re: [PATCH] gnu: ratpoison: Add multiple monitors support.

[Ludovic Courtès]

Mathieu Lirzin <mthl@openmailbox.org> skribis:

> * gnu/packages/ratpoison.scm (ratpoison)[inputs]: Add "libxinerama".

I found out that with this change, Ratpoison no longer works on my
multi-monitor setup: When it starts, windows created from my ~/.xsession
occupy just a subset of the screen, and as soon as I try to move things
around, C-t stops responding and I have to kill it.

Removing the dependency on libxinerama solves the problem for me.

Am I missing something?  Should we revert it?

Ludo’.

Re: [PATCH] gnu: ratpoison: Add multiple monitors support.

[Mathieu Lirzin]

ludo@gnu.org (Ludovic Courtès) writes:

> Mathieu Lirzin <mthl@openmailbox.org> skribis:
>
>> * gnu/packages/ratpoison.scm (ratpoison)[inputs]: Add "libxinerama".
>
> I found out that with this change, Ratpoison no longer works on my
> multi-monitor setup: When it starts, windows created from my ~/.xsession
> occupy just a subset of the screen, and as soon as I try to move things
> around, C-t stops responding and I have to kill it.

--8<---------------cut here---------------start------------->8---
[...]
emacs &
exec ratpoison
--8<---------------cut here---------------end--------------->8---

I've tried with this in my ~/.xsession with guix on top of
Debian testing without any issue.

Can you send something so that i can try to reproduce this problem on my
machine?

> Removing the dependency on libxinerama solves the problem for me.
>
> Am I missing something?  Should we revert it?

On my setup it's the opposite.  Without this dependency, the two
monitors i have are seen by ratpoison as one "screen" So I'm not able to
use the commands described here (info "(ratpoison) Multiple Monitors")
and the display of windows is divided across the monitors.

--
Mathieu Lirzin

Re: [PATCH] gnu: ratpoison: Add multiple monitors support.

[Ludovic Courtès]

Mathieu Lirzin <mthl@openmailbox.org> skribis:

> ludo@gnu.org (Ludovic Courtès) writes:

[...]

>> Removing the dependency on libxinerama solves the problem for me.
>>
>> Am I missing something?  Should we revert it?
>
> On my setup it's the opposite.  Without this dependency, the two
> monitors i have are seen by ratpoison as one "screen" So I'm not able to
> use the commands described here (info "(ratpoison) Multiple Monitors")
> and the display of windows is divided across the monitors.

Well yes, that’s actually what I’m used to: rp sees one screen and I
simply arrange to have one frame on one screen and a second frame on the
second screen.

I’ll investigate more and see if I can reproduce with a simple case.

Thanks,
Ludo’.

Re: [PATCH] gnu: ratpoison: Add multiple monitors support.

[Mark H Weaver]

ludo@gnu.org (Ludovic Courtès) writes:

> Mathieu Lirzin <mthl@openmailbox.org> skribis:
>
>> ludo@gnu.org (Ludovic Courtès) writes:
>
> [...]
>
>>> Removing the dependency on libxinerama solves the problem for me.
>>>
>>> Am I missing something?  Should we revert it?
>>
>> On my setup it's the opposite.  Without this dependency, the two
>> monitors i have are seen by ratpoison as one "screen" So I'm not able to
>> use the commands described here (info "(ratpoison) Multiple Monitors")
>> and the display of windows is divided across the monitors.
>
> Well yes, that’s actually what I’m used to: rp sees one screen and I
> simply arrange to have one frame on one screen and a second frame on the
> second screen.

I tend to agree with Mathieu that our default ratpoison package should
include xinerama support.  Admittedly, I've not been using ratpoison
much lately, but Ludovic's method of treating multiple monitors as a
single screen would not work well in my multi-monitor use cases,
e.g. when attaching to a television to watch a video on one screen while
working on another screen, or when the two screens together do not form
a rectangle because they're not the same height.

Perhaps we should add a separate 'ratpoison-without-xinerama' package?
Or maybe there's a way to configure ratpoison at run-time to do what
Ludovic wants, even when it was compiled with xinerama support?

      Mark

Re: [PATCH] gnu: ratpoison: Add multiple monitors support.

[Ludovic Courtès]

Mathieu Lirzin <mthl@openmailbox.org> skribis:

> ludo@gnu.org (Ludovic Courtès) writes:
>
>> Mathieu Lirzin <mthl@openmailbox.org> skribis:
>>
>>> * gnu/packages/ratpoison.scm (ratpoison)[inputs]: Add "libxinerama".
>>
>> I found out that with this change, Ratpoison no longer works on my
>> multi-monitor setup: When it starts, windows created from my ~/.xsession
>> occupy just a subset of the screen, and as soon as I try to move things
>> around, C-t stops responding and I have to kill it.
>
> [...]
> emacs &
> exec ratpoison
>
> I've tried with this in my ~/.xsession with guix on top of
> Debian testing without any issue.

As an additional data point, my external screen is rotated by 90° and
has a higher resolution than the laptop’s screen.

I actually run the following sequence of commands *after* the WM has
started, which perhaps confuses it?

--8<---------------cut here---------------start------------->8---
xrandr --output "$out" --mode 1920x1200
xrandr --output "$out" --rotate left
xrandr --output "$out" --above LVDS1
--8<---------------cut here---------------end--------------->8---

Thanks,
Ludo’.

Re: [PATCH] gnu: ratpoison: Add multiple monitors support.

[Ludovic Courtès]

Mark H Weaver <mhw@netris.org> skribis:

> Perhaps we should add a separate 'ratpoison-without-xinerama' package?
> Or maybe there's a way to configure ratpoison at run-time to do what
> Ludovic wants, even when it was compiled with xinerama support?

If the problem turns out to be on my side (i.e., between keyboard and
chair), no need for a separate package.

Ludo’.