Your message dated Fri, 27 Jan 2017 11:27:03 +0100
with message-id <[email protected]>
and subject line Re: Bug#843824: kdenlive: Screen Grab has hard coded display 
setting
has caused the Debian Bug report #843824,
regarding kdenlive: Screen Grab has hard coded display setting
to be marked as done.

This means that you claim that the problem has been dealt with.
If this is not the case it is now your responsibility to reopen the
Bug report if necessary, and/or fix the problem forthwith.

(NB: If you are a system administrator and have no idea what this
message is talking about, this may indicate a serious mail system
misconfiguration somewhere. Please contact [email protected]
immediately.)


-- 
843824: http://bugs.debian.org/cgi-bin/bugreport.cgi?bug=843824
Debian Bug Tracking System
Contact [email protected] with problems
--- Begin Message ---
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 <config-kdenlive.h>
 
+#include <cstdlib>
+
 #include <KDebug>
 #include <KLocalizedString>
 #include <KStandardDirs>
@@ -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-network            4: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-svg                4:4.8.6+git64-g5dc8b2b+dfsg-3+deb8u1
ii  libqt4-xml                4:4.8.6+git64-g5dc8b2b+dfsg-3+deb8u1
ii  libqtcore4                4: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++6                4.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  dvdauthor        0.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  <none>

-- no debconf information

--- End Message ---
--- Begin Message ---
fixed #843824 16.12.1-1
thanks


Am 27.01.2017 um 10:53 schrieb Jean-Baptiste Mardelle:
> 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

Thanks for your update!

-- 

/*
Mit freundlichem Gruß / With kind regards,
 Patrick Matthäi
 GNU/Linux Debian Developer

  Blog: http://www.linux-dev.org/
E-Mail: [email protected]
        [email protected]
*/


--- End Message ---

Reply via email to