Bug#843824: kdenlive: Screen Grab has hard coded display setting
On Tuesday, January 24, 2017 9:41:23 AM CET, Patrick Matthäi wrote: Hello JB, I think this issue is still up to date with 16.12.1, since from kdenlive-16.12.1/src/monitor/recmonitor.cpp: Thanks for your message. I just tested and the issue has been fixed, probably in 16.12.0 or even 16.08.x. The code in recmonitor.cpp is not used anymore, it is kept as archive but should be removed. The screengrab feature now displays a combobox allowing to select the monitor. Relevant code is in src/monitor/recmanager.cpp, line 65 to 75. Best regards Jean-Baptiste Mardelle 710 if (!KdenliveSettings::grab_hide_frame()) m_captureArgs << QStringLiteral("-show_region") << QStringLiteral("1"); 711 captureSize = QStringLiteral(":0.0"); 712 if (KdenliveSettings::grab_capture_type() == 0) { 713 // Full screen capture Could you apply the patch please? Am 09.11.2016 um 23:33 schrieb Daniel Keast: Package: kdenlive Version: 0.9.10-2 Severity: normal Dear Maintainer, The screen grab feature has the DISPLAY location hard coded. When I attempt to ...
Bug#843824: kdenlive: Screen Grab has hard coded display setting
Hello JB, I think this issue is still up to date with 16.12.1, since from kdenlive-16.12.1/src/monitor/recmonitor.cpp: 710 if (!KdenliveSettings::grab_hide_frame()) m_captureArgs << QStringLiteral("-show_region") << QStringLiteral("1"); 711 captureSize = QStringLiteral(":0.0"); 712 if (KdenliveSettings::grab_capture_type() == 0) { 713 // Full screen capture Could you apply the patch please? Am 09.11.2016 um 23:33 schrieb Daniel Keast: > Package: kdenlive > Version: 0.9.10-2 > Severity: normal > > Dear Maintainer, > > The screen grab feature has the DISPLAY location hard coded. When I attempt to > record my desktop I get a notification with the message "x11grab Could not > open > X display. :0.0: Input/output error". > > This patch fixes the issue for me: > > diff --git a/kdenlive-0.9.10/src/monitor/recmonitor.cpp > b/kdenlive-0.9.10/src/monitor/recmonitor.cpp > index f465d82..57b5536 100644 > --- a/kdenlive-0.9.10/src/monitor/recmonitor.cpp > +++ b/kdenlive-0.9.10/src/monitor/recmonitor.cpp > @@ -29,6 +29,8 @@ > #include "videosurface.h" > #include > > +#include > + > #include > #include > #include > @@ -731,7 +733,8 @@ void RecMonitor::slotRecord() > m_captureArgs << "-f" << "x11grab"; > if (KdenliveSettings::grab_follow_mouse()) m_captureArgs << > "-follow_mouse" << "centered"; > if (!KdenliveSettings::grab_hide_frame()) m_captureArgs << > "-show_region" << "1"; > - captureSize = ":0.0"; > + captureSize = std::genenv("DISPLAY"); > + if (captureSize == NULL) captureSize = ":0.0"; > if (KdenliveSettings::grab_capture_type() == 0) { > // Full screen capture > m_captureArgs << "-s" << QString::number(screenSize.width()) > + 'x' + QString::number(screenSize.height()); > > > -- System Information: > Debian Release: 8.6 > APT prefers stable > APT policy: (500, 'stable') > Architecture: amd64 (x86_64) > Foreign Architectures: i386 > > Kernel: Linux 3.16.0-4-amd64 (SMP w/2 CPU cores) > Locale: LANG=en_GB.utf8, LC_CTYPE=en_GB.utf8 (charmap=UTF-8) > Shell: /bin/sh linked to /bin/dash > Init: systemd (via /run/systemd/system) > > Versions of packages kdenlive depends on: > ii kde-runtime 4:4.14.2-2 > ii kdenlive-data 0.9.10-2 > ii libav-tools 6:11.8-1~deb8u1 > ii libc6 2.19-18+deb8u6 > ii libgcc1 1:4.9.2-10 > ii libgl1-mesa-glx [libgl1] 10.3.2-1+deb8u1 > ii libglu1-mesa [libglu1]9.0.0-2 > ii libkdecore5 4:4.14.2-5+deb8u1 > ii libkdeui5 4:4.14.2-5+deb8u1 > ii libkio5 4:4.14.2-5+deb8u1 > ii libknewstuff3-4 4:4.14.2-5+deb8u1 > ii libknotifyconfig4 4:4.14.2-5+deb8u1 > ii libkrossui4 4:4.14.2-5+deb8u1 > ii libmlt++3 0.9.2-2 > ii libmlt6 0.9.2-2 > ii libnepomuk4 4:4.14.2-5+deb8u1 > ii libqjson0 0.8.1-3 > ii libqt4-dbus 4:4.8.6+git64-g5dc8b2b+dfsg-3+deb8u1 > ii libqt4-network4:4.8.6+git64-g5dc8b2b+dfsg-3+deb8u1 > ii libqt4-opengl 4:4.8.6+git64-g5dc8b2b+dfsg-3+deb8u1 > ii libqt4-script 4:4.8.6+git64-g5dc8b2b+dfsg-3+deb8u1 > ii libqt4-svg4:4.8.6+git64-g5dc8b2b+dfsg-3+deb8u1 > ii libqt4-xml4:4.8.6+git64-g5dc8b2b+dfsg-3+deb8u1 > ii libqtcore44:4.8.6+git64-g5dc8b2b+dfsg-3+deb8u1 > ii libqtgui4 4:4.8.6+git64-g5dc8b2b+dfsg-3+deb8u1 > ii libsolid4 4:4.14.2-5+deb8u1 > ii libsoprano4 2.9.4+dfsg-1.1 > ii libstdc++64.9.2-10 > ii libv4l-0 1.6.0-2 > ii libx11-6 2:1.6.2-3 > ii libxau6 1:1.0.8-1 > ii libxdmcp6 1:1.1.1-1+b1 > ii libxext6 2:1.3.3-1 > ii melt 0.9.2-2 > > Versions of packages kdenlive recommends: > ii dvdauthor0.7.0-1.3 > ii dvgrab 3.5-2+b2 > ii frei0r-plugins 1.4-3 > ii genisoimage 9:1.1.11-3 > ii recordmydesktop 0.3.8.1+svn602-1+b1 > ii swh-plugins 0.4.15+1-7 > > Versions of packages kdenlive suggests: > pn khelpcenter4 > > -- no debconf information -- /* Mit freundlichem Gruß / With kind regards, Patrick Matthäi GNU/Linux Debian Developer Blog: http://www.linux-dev.org/ E-Mail: pmatth...@debian.org patr...@linux-dev.org */
Bug#843824: kdenlive: Screen Grab has hard coded display setting
Package: kdenlive Version: 0.9.10-2 Severity: normal Dear Maintainer, The screen grab feature has the DISPLAY location hard coded. When I attempt to record my desktop I get a notification with the message "x11grab Could not open X display. :0.0: Input/output error". This patch fixes the issue for me: diff --git a/kdenlive-0.9.10/src/monitor/recmonitor.cpp b/kdenlive-0.9.10/src/monitor/recmonitor.cpp index f465d82..57b5536 100644 --- a/kdenlive-0.9.10/src/monitor/recmonitor.cpp +++ b/kdenlive-0.9.10/src/monitor/recmonitor.cpp @@ -29,6 +29,8 @@ #include "videosurface.h" #include +#include + #include #include #include @@ -731,7 +733,8 @@ void RecMonitor::slotRecord() m_captureArgs << "-f" << "x11grab"; if (KdenliveSettings::grab_follow_mouse()) m_captureArgs << "-follow_mouse" << "centered"; if (!KdenliveSettings::grab_hide_frame()) m_captureArgs << "-show_region" << "1"; - captureSize = ":0.0"; + captureSize = std::genenv("DISPLAY"); + if (captureSize == NULL) captureSize = ":0.0"; if (KdenliveSettings::grab_capture_type() == 0) { // Full screen capture m_captureArgs << "-s" << QString::number(screenSize.width()) + 'x' + QString::number(screenSize.height()); -- System Information: Debian Release: 8.6 APT prefers stable APT policy: (500, 'stable') Architecture: amd64 (x86_64) Foreign Architectures: i386 Kernel: Linux 3.16.0-4-amd64 (SMP w/2 CPU cores) Locale: LANG=en_GB.utf8, LC_CTYPE=en_GB.utf8 (charmap=UTF-8) Shell: /bin/sh linked to /bin/dash Init: systemd (via /run/systemd/system) Versions of packages kdenlive depends on: ii kde-runtime 4:4.14.2-2 ii kdenlive-data 0.9.10-2 ii libav-tools 6:11.8-1~deb8u1 ii libc6 2.19-18+deb8u6 ii libgcc1 1:4.9.2-10 ii libgl1-mesa-glx [libgl1] 10.3.2-1+deb8u1 ii libglu1-mesa [libglu1]9.0.0-2 ii libkdecore5 4:4.14.2-5+deb8u1 ii libkdeui5 4:4.14.2-5+deb8u1 ii libkio5 4:4.14.2-5+deb8u1 ii libknewstuff3-4 4:4.14.2-5+deb8u1 ii libknotifyconfig4 4:4.14.2-5+deb8u1 ii libkrossui4 4:4.14.2-5+deb8u1 ii libmlt++3 0.9.2-2 ii libmlt6 0.9.2-2 ii libnepomuk4 4:4.14.2-5+deb8u1 ii libqjson0 0.8.1-3 ii libqt4-dbus 4:4.8.6+git64-g5dc8b2b+dfsg-3+deb8u1 ii libqt4-network4:4.8.6+git64-g5dc8b2b+dfsg-3+deb8u1 ii libqt4-opengl 4:4.8.6+git64-g5dc8b2b+dfsg-3+deb8u1 ii libqt4-script 4:4.8.6+git64-g5dc8b2b+dfsg-3+deb8u1 ii libqt4-svg4:4.8.6+git64-g5dc8b2b+dfsg-3+deb8u1 ii libqt4-xml4:4.8.6+git64-g5dc8b2b+dfsg-3+deb8u1 ii libqtcore44:4.8.6+git64-g5dc8b2b+dfsg-3+deb8u1 ii libqtgui4 4:4.8.6+git64-g5dc8b2b+dfsg-3+deb8u1 ii libsolid4 4:4.14.2-5+deb8u1 ii libsoprano4 2.9.4+dfsg-1.1 ii libstdc++64.9.2-10 ii libv4l-0 1.6.0-2 ii libx11-6 2:1.6.2-3 ii libxau6 1:1.0.8-1 ii libxdmcp6 1:1.1.1-1+b1 ii libxext6 2:1.3.3-1 ii melt 0.9.2-2 Versions of packages kdenlive recommends: ii dvdauthor0.7.0-1.3 ii dvgrab 3.5-2+b2 ii frei0r-plugins 1.4-3 ii genisoimage 9:1.1.11-3 ii recordmydesktop 0.3.8.1+svn602-1+b1 ii swh-plugins 0.4.15+1-7 Versions of packages kdenlive suggests: pn khelpcenter4 -- no debconf information