all messages for Guix-related lists mirrored at yhetil.org
 help / color / mirror / code / Atom feed
From: Alex Vong <alexvong1995@gmail.com>
To: Arun Isaac <arunisaac@systemreboot.net>
Cc: 27643@debbugs.gnu.org
Subject: [bug#27643] [PATCH] gnu: Add simplescreenrecorder.
Date: Tue, 11 Jul 2017 23:22:38 +0800	[thread overview]
Message-ID: <87shi3t2v5.fsf@gmail.com> (raw)
In-Reply-To: <9a059d85.AEEANLs8YQMAAAAAAAAAAAPSfd8AAAACwQwAAAAAAAW9WABZZN3o@mailjet.com> (Arun Isaac's message of "Tue, 11 Jul 2017 19:47:02 +0530")


[-- Attachment #1.1: Type: text/plain, Size: 1160 bytes --]

Arun Isaac <arunisaac@systemreboot.net> writes:

>>>> + ("qt" ,qt-4) ; README.md: using Qt 5 causes some stability issues
>>>> +              ("ffmpeg" ,ffmpeg)
>>>> +              ("alsa-lib" ,alsa-lib)
>>>> +              ("pulseaudio" ,pulseaudio)
>>>> +              ("jack" ,jack-1)
>>>> +              ("glu" ,glu)
>>>> +              ("libx11" ,libx11)
>>>> +              ("libxfixes" ,libxfixes)
>>>> +              ("libxext" ,libxext)
>>>
>>> The package builds successfully without libx11, libxfixes and libxext
>>> included as inputs. Are you sure they are needed? Could you check again?
>>>
>>>> +              ("libxi" ,libxi)))
>>>
>> These packages are listed as Build dependencies in README[0]. What do
>> you think?
>
> I think we should not include them as dependencies. We can always
> include them later if we have problems.
>
> I did start the application built without these packages, and it started
> successfully. If you are a user of this package, you could test it more
> extensively.
>
> WDYT?

I have tested it just now. Indeed, it works without those packages. (I
recorded a 10s video.) So I remove them.

Here is the new patch:

[-- Attachment #1.2: 0001-gnu-Add-simplescreenrecorder.patch --]
[-- Type: text/x-diff, Size: 3044 bytes --]

From 3ca05bb6f3a9c6097c72681734f1fcf754cee424 Mon Sep 17 00:00:00 2001
From: Alex Vong <alexvong1995@gmail.com>
Date: Mon, 10 Jul 2017 23:25:26 +0800
Subject: [PATCH] gnu: Add simplescreenrecorder.

* gnu/packages/video.scm (simplescreenrecorder): New variable.
---
 gnu/packages/video.scm | 37 ++++++++++++++++++++++++++++++++++++-
 1 file changed, 36 insertions(+), 1 deletion(-)

diff --git a/gnu/packages/video.scm b/gnu/packages/video.scm
index 5338d375e..dc73114e4 100644
--- a/gnu/packages/video.scm
+++ b/gnu/packages/video.scm
@@ -6,7 +6,7 @@
 ;;; Copyright © 2015, 2016, 2017 Efraim Flashner <efraim@flashner.co.il>
 ;;; Copyright © 2015 Andy Patterson <ajpatter@uwaterloo.ca>
 ;;; Copyright © 2015 Ricardo Wurmus <rekado@elephly.net>
-;;; Copyright © 2015, 2016 Alex Vong <alexvong1995@gmail.com>
+;;; Copyright © 2015, 2016, 2017 Alex Vong <alexvong1995@gmail.com>
 ;;; Copyright © 2016, 2017 Alex Griffin <a@ajgrf.com>
 ;;; Copyright © 2016 Kei Kebreau <kei@openmailbox.org>
 ;;; Copyright © 2016 Dmitry Nikolaev <cameltheman@gmail.com>
@@ -1888,6 +1888,41 @@ your graphical desktop and encodes it as a video.  This is a useful tool for
 making @dfn{screencasts}.")
     (license license:gpl2+)))
 
+(define-public simplescreenrecorder
+  (package
+    (name "simplescreenrecorder")
+    (version "0.3.8")
+    (source (origin
+              (method url-fetch)
+              (uri (string-append "https://github.com/MaartenBaert/ssr/"
+                                  "archive/" version ".tar.gz"))
+              (file-name (string-append name "-" version ".tar.gz"))
+              (sha256
+               (base32
+                "0v8w35n8w772s08w7k0icynqdsdakbrcanbgx6j847bfqfsg21gg"))))
+    (build-system gnu-build-system)
+    (native-inputs `(("pkg-config" ,pkg-config)))
+    ;; Although libx11, libxfixes, libxext are listed as build dependencies in
+    ;; README. The program builds and functions properly without it.
+    ;; As a result, they are omitted. Please add them back if problems appear.
+    (inputs `(("alsa-lib" ,alsa-lib)
+              ("ffmpeg" ,ffmpeg)
+              ("glu" ,glu)
+              ("jack" ,jack-1)
+              ("libxi" ,libxi)
+              ("pulseaudio" ,pulseaudio)
+              ("qt" ,qt-4))) ; README: using Qt 5 causes some stability issues
+    ;; Using HTTPS causes part of the page displayed improperly.
+    (home-page "http://www.maartenbaert.be/simplescreenrecorder/")
+    (synopsis "Screen recorder")
+    (description "SimpleScreenRecorder is a screen recorder.  Despite the name,
+this program is actually quite complex.  It's @emph{simple} in the sense that
+it's easier to use than ffmpeg/avconv or VLC.")
+    (license (list license:gpl3+ ; most files
+                   license:zlib ; glinject/elfhacks.*
+                   license:isc ; glinject/*
+                   license:x11)))) ; build-aux/install-sh
+
 (define-public libsmpeg
   (package
     (name "libsmpeg")
-- 
2.13.2


[-- Attachment #2: signature.asc --]
[-- Type: application/pgp-signature, Size: 832 bytes --]

  parent reply	other threads:[~2017-07-11 15:24 UTC|newest]

Thread overview: 9+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2017-07-10 16:16 [bug#27643] [PATCH] gnu: Add simplescreenrecorder Alex Vong
2017-07-10 20:15 ` Arun Isaac
     [not found] ` <a885d32b.ADkAACOBA2kAAAAAAAAAAAPSfd8AAAACwQwAAAAAAAW9WABZY-Bh@mailjet.com>
2017-07-11 13:34   ` Alex Vong
2017-07-11 13:45     ` Alex Vong
2017-07-11 14:17     ` Arun Isaac
     [not found]     ` <9a059d85.AEEANLs8YQMAAAAAAAAAAAPSfd8AAAACwQwAAAAAAAW9WABZZN3o@mailjet.com>
2017-07-11 15:22       ` Alex Vong [this message]
2017-07-11 17:41         ` Arun Isaac
2017-07-12 20:32 ` bug#27643: " Arun Isaac
     [not found] ` <4ebf0597.AEUAMuQMiowAAAAAAAAAAAPSfd8AAAACwQwAAAAAAAW9WABZZod0@mailjet.com>
2017-07-13 13:59   ` [bug#27643] " Alex Vong

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=87shi3t2v5.fsf@gmail.com \
    --to=alexvong1995@gmail.com \
    --cc=27643@debbugs.gnu.org \
    --cc=arunisaac@systemreboot.net \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
Code repositories for project(s) associated with this external index

	https://git.savannah.gnu.org/cgit/guix.git

This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.