msenol86/PKGBUILD

## Readme.md

      
    Raw
  

              Readme.md
            
          
    Fix deepin-screenshot clipboard bug by using xclip

On X11 clipboard content is owned by the application, Data copied to clipboard remains in the clipboard only as long as the parent application is running and is discarded when the application quits.
I have added xclip as a dependecy to solve this issue. Deepin Screenshot creates a temporary file and then sends it to the xclip.
- martyr-deepin/deepin-screenshot#8
Please see for updated version: https://aur.archlinux.org/packages/deepin-screenshot-copy-patch

  
## PKGBUILD
# Maintainer: Felix Yan <felixonmars@archlinux.org>
# Contributor: Josip Ponjavic <josipponjavic at gmail dot com>
# Contributor: Xu Fasheng <fasheng.xu[AT]gmail.com>
# Contributor: Mucahit Senol <mucahitsenol86 at gmail dot com>

pkgname=deepin-screenshot
pkgver=5.0.0
pkgrel=1
pkgdesc="Easy-to-use screenshot tool for linuxdeepin desktop environment"
arch=('x86_64')
url="https://github.com/linuxdeepin/deepin-screenshot"
license=('GPL3')
depends=('deepin-qt5integration' 'dtkwm' 'deepin-turbo' 'xclip')
makedepends=('cmake' 'qt5-tools')
groups=('deepin-extra')
source=("$pkgname-$pkgver.tar.gz::https://github.com/linuxdeepin/deepin-screenshot/archive/$pkgver.tar.gz"
	"xclip.patch")
sha512sums=('ad073961deaa78a47739af6cf8b0d0fa77a92ee6668c57f9a5bbf51ad2be1d2028681cffabdc26548be1bdbe24c7ccc7e7ff3d49b86907f794495bfbea62aa30'
            '74eadb86232153f0b652c71411aaa258f800b97066d3d9c6d9b971c4f6d55d3bb0c129b08b0c4e1448ec4db89cd1970be5733323a671b4852591ca20a2baeea8')

build() {
  cd deepin-screenshot-$pkgver
  patch src/mainwindow.cpp < $srcdir/xclip.patch
  cmake . -DCMAKE_INSTALL_PREFIX=/usr
  make
}

package() {
  cd deepin-screenshot-$pkgver
  make DESTDIR="$pkgdir" install
}

## xclip.patch
diff --git a/src/mainwindow.cpp b/src/mainwindow.cpp
index def901e..2f81456 100644
--- a/src/mainwindow.cpp
+++ b/src/mainwindow.cpp
@@ -1401,8 +1401,30 @@ bool MainWindow::saveAction(const QPixmap &pix)

     if (copyToClipboard) {
         Q_ASSERT(!screenShotPix.isNull());
-        QClipboard* cb = qApp->clipboard();
-        cb->setPixmap(screenShotPix, QClipboard::Clipboard);
+        bool tmpFileCreated = false;
+
+        if (!QFile("/usr/bin/xclip").exists()) {
+            qCritical() << "Copying to clipboard failed. Please install xclip!";
+            return false;
+        } else {
+
+            QTemporaryFile file;
+            if (file.open()) {
+                tmpFileCreated = true;
+                screenShotPix.save(file.fileName(), "PNG");
+                qDebug() << "tmp filename: " << file.fileName();
+                QString command = QString("/usr/bin/xclip -selection clipboard -t image/png -i %1").arg(file.fileName());
+                if(m_saveFileName != "") {
+                    command = QString("/usr/bin/xclip -selection clipboard -t image/png -i %1").arg(m_saveFileName); // if auto save + copy to clipboard selected, use saved file
+                }
+                qDebug() << "command: " << command;
+                QProcess::execute(command);
+            } // do not add else part to not distrubt tmp file destruction process
+        }
+        if(!tmpFileCreated) {
+            qCritical() << "Saving image into temporary file failed!";
+            return false;
+        }
     }

     return true;
	# Maintainer: Felix Yan <felixonmars@archlinux.org>
	# Contributor: Josip Ponjavic <josipponjavic at gmail dot com>
	# Contributor: Xu Fasheng <fasheng.xu[AT]gmail.com>
	# Contributor: Mucahit Senol <mucahitsenol86 at gmail dot com>

	pkgname=deepin-screenshot
	pkgver=5.0.0
	pkgrel=1
	pkgdesc="Easy-to-use screenshot tool for linuxdeepin desktop environment"
	arch=('x86_64')
	url="https://github.com/linuxdeepin/deepin-screenshot"
	license=('GPL3')
	depends=('deepin-qt5integration' 'dtkwm' 'deepin-turbo' 'xclip')
	makedepends=('cmake' 'qt5-tools')
	groups=('deepin-extra')
	source=("$pkgname-$pkgver.tar.gz::https://github.com/linuxdeepin/deepin-screenshot/archive/$pkgver.tar.gz"
	"xclip.patch")
	sha512sums=('ad073961deaa78a47739af6cf8b0d0fa77a92ee6668c57f9a5bbf51ad2be1d2028681cffabdc26548be1bdbe24c7ccc7e7ff3d49b86907f794495bfbea62aa30'
	'74eadb86232153f0b652c71411aaa258f800b97066d3d9c6d9b971c4f6d55d3bb0c129b08b0c4e1448ec4db89cd1970be5733323a671b4852591ca20a2baeea8')

	build() {
	cd deepin-screenshot-$pkgver
	patch src/mainwindow.cpp < $srcdir/xclip.patch
	cmake . -DCMAKE_INSTALL_PREFIX=/usr
	make
	}

	package() {
	cd deepin-screenshot-$pkgver
	make DESTDIR="$pkgdir" install
	}
	diff --git a/src/mainwindow.cpp b/src/mainwindow.cpp
	index def901e..2f81456 100644
	--- a/src/mainwindow.cpp
	+++ b/src/mainwindow.cpp
	@@ -1401,8 +1401,30 @@ bool MainWindow::saveAction(const QPixmap &pix)

	if (copyToClipboard) {
	Q_ASSERT(!screenShotPix.isNull());
	- QClipboard* cb = qApp->clipboard();
	- cb->setPixmap(screenShotPix, QClipboard::Clipboard);
	+ bool tmpFileCreated = false;
	+
	+ if (!QFile("/usr/bin/xclip").exists()) {
	+ qCritical() << "Copying to clipboard failed. Please install xclip!";
	+ return false;
	+ } else {
	+
	+ QTemporaryFile file;
	+ if (file.open()) {
	+ tmpFileCreated = true;
	+ screenShotPix.save(file.fileName(), "PNG");
	+ qDebug() << "tmp filename: " << file.fileName();
	+ QString command = QString("/usr/bin/xclip -selection clipboard -t image/png -i %1").arg(file.fileName());
	+ if(m_saveFileName != "") {
	+ command = QString("/usr/bin/xclip -selection clipboard -t image/png -i %1").arg(m_saveFileName); // if auto save + copy to clipboard selected, use saved file
	+ }
	+ qDebug() << "command: " << command;
	+ QProcess::execute(command);
	+ } // do not add else part to not distrubt tmp file destruction process
	+ }
	+ if(!tmpFileCreated) {
	+ qCritical() << "Saving image into temporary file failed!";
	+ return false;
	+ }
	}

	return true;