[PR PATCH] xfce4-settings: use xrandr

[PR PATCH] xfce4-settings: use xrandr

[atk]

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

There is a new pull request by atk against master on the void-packages repository

https://github.com/atk/void-packages xfce4-settings-use-xrandr
https://github.com/void-linux/void-packages/pull/27486

xfce4-settings: use xrandr
xfce4-display-settings now supports to scale the display with randr, something people with multi-monitor-setup with different resolutions (like me) will find incredibly helpful - and it's one of the major improvements of this package, so I think we should ship it as well.

A patch file from https://github.com/void-linux/void-packages/pull/27486.patch is attached

[-- Warning: decoded text below may be mangled, UTF-8 assumed --]
[-- Attachment #2: github-pr-xfce4-settings-use-xrandr-27486.patch --]
[-- Type: text/x-diff, Size: 1046 bytes --]

From ad3e2594bf79701af13e28cc93676bddfe4e5260 Mon Sep 17 00:00:00 2001
From: Alex Lohr <alex.lohr@logmein.com>
Date: Sun, 27 Dec 2020 21:41:58 +0100
Subject: [PATCH] xfce4-settings: use xrandr

---
 srcpkgs/xfce4-settings/template | 4 ++--
 1 file changed, 2 insertions(+), 2 deletions(-)

diff --git a/srcpkgs/xfce4-settings/template b/srcpkgs/xfce4-settings/template
index e4d5464fd7c..cc6292c7586 100644
--- a/srcpkgs/xfce4-settings/template
+++ b/srcpkgs/xfce4-settings/template
@@ -1,12 +1,12 @@
 # Template file for 'xfce4-settings'
 pkgname=xfce4-settings
 version=4.16.0
-revision=1
+revision=2
 build_style=gnu-configure
 configure_args="--enable-sound-settings --enable-pluggable-dialogs"
 hostmakedepends="intltool pkg-config"
 makedepends="exo-devel garcon-devel libcanberra-devel libnotify-devel
- libxklavier-devel upower-devel"
+ libxklavier-devel upower-devel libXrandr-devel"
 depends="desktop-file-utils gnome-icon-theme"
 conf_files="/etc/xdg/xfce4/xfconf/xfce-perchannel-xml/xsettings.xml"
 short_desc="Xfce settings manager"

Re: xfce4-settings: use xrandr

[ericonr]

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

New comment by ericonr on void-packages repository

https://github.com/void-linux/void-packages/pull/27486#issuecomment-751522374

Comment:
The package already depends on libXrandr, so I'm not sure adding the devel package there made any difference. Maybe it would be best to use the specific configure arg, if it's indeed not been enabled?

Re: xfce4-settings: use xrandr

[atk]

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

New comment by atk on void-packages repository

https://github.com/void-linux/void-packages/pull/27486#issuecomment-751594173

Comment:
For me it made the difference that the scale control worked afterwards when it previously didn't.

Re: xfce4-settings: use xrandr

[atk]

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

New comment by atk on void-packages repository

https://github.com/void-linux/void-packages/pull/27486#issuecomment-751594173

Comment:
For me it made the difference that the scale control worked afterwards when it previously didn't. Even with the configure arg, without the -devel library, it couldn't use it.

Re: xfce4-settings: use xrandr

[ericonr]

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

New comment by ericonr on void-packages repository

https://github.com/void-linux/void-packages/pull/27486#issuecomment-751776257

Comment:
No problem. Could you add the configure arg and leave the devel package there? To be sure it won't regress.

Re: [PR PATCH] [Updated] xfce4-settings: use xrandr

[atk]

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

There is an updated pull request by atk against master on the void-packages repository

https://github.com/atk/void-packages xfce4-settings-use-xrandr
https://github.com/void-linux/void-packages/pull/27486

xfce4-settings: use xrandr
xfce4-display-settings now supports to scale the display with randr, something people with multi-monitor-setup with different resolutions (like me) will find incredibly helpful - and it's one of the major improvements of this package, so I think we should ship it as well.

A patch file from https://github.com/void-linux/void-packages/pull/27486.patch is attached

[-- Warning: decoded text below may be mangled, UTF-8 assumed --]
[-- Attachment #2: github-pr-xfce4-settings-use-xrandr-27486.patch --]
[-- Type: text/x-diff, Size: 1133 bytes --]

From 8118ab1864d57faa16bf025a803e2be6c74f6a85 Mon Sep 17 00:00:00 2001
From: Alex Lohr <alex.lohr@logmein.com>
Date: Sun, 27 Dec 2020 21:41:58 +0100
Subject: [PATCH] xfce4-settings: use xrandr

---
 srcpkgs/xfce4-settings/template | 6 +++---
 1 file changed, 3 insertions(+), 3 deletions(-)

diff --git a/srcpkgs/xfce4-settings/template b/srcpkgs/xfce4-settings/template
index e4d5464fd7c..49f78370413 100644
--- a/srcpkgs/xfce4-settings/template
+++ b/srcpkgs/xfce4-settings/template
@@ -1,12 +1,12 @@
 # Template file for 'xfce4-settings'
 pkgname=xfce4-settings
 version=4.16.0
-revision=1
+revision=2
 build_style=gnu-configure
-configure_args="--enable-sound-settings --enable-pluggable-dialogs"
+configure_args="--enable-sound-settings --enable-pluggable-dialogs --enable-xrandr"
 hostmakedepends="intltool pkg-config"
 makedepends="exo-devel garcon-devel libcanberra-devel libnotify-devel
- libxklavier-devel upower-devel"
+ libxklavier-devel upower-devel libXrandr-devel"
 depends="desktop-file-utils gnome-icon-theme"
 conf_files="/etc/xdg/xfce4/xfconf/xfce-perchannel-xml/xsettings.xml"
 short_desc="Xfce settings manager"

Re: xfce4-settings: use xrandr

[atk]

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

New comment by atk on void-packages repository

https://github.com/void-linux/void-packages/pull/27486#issuecomment-751825398

Comment:
done.

Re: [PR PATCH] [Merged]: xfce4-settings: use xrandr

[ericonr]

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

There's a merged pull request on the void-packages repository

xfce4-settings: use xrandr
https://github.com/void-linux/void-packages/pull/27486

Description:
xfce4-display-settings now supports to scale the display with randr, something people with multi-monitor-setup with different resolutions (like me) will find incredibly helpful - and it's one of the major improvements of this package, so I think we should ship it as well.