Re: [PATCH] NON-WORKING: new package: sent -- simple presentation tool

[Eric Bavier <ericbavier@openmailbox.org>]

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

On Sat, 13 Aug 2016 09:35:13 +0000
ng0 <ng0@we.make.ritual.n0.is> wrote:

> Alex Vong <alexvong1995@gmail.com> writes:
> 
> > Leo Famulari <leo@famulari.name> writes:
> >  
> >> On Sun, Dec 13, 2015 at 12:44:51AM +0300, KAction@gnu.org wrote:  
> >>> From: Dmitry Bogatov <KAction@gnu.org>
> >>> 
> >>> This patch does not work. I did my best during 1.5 hours, but I failed
> >>> to actually build this package.
> >>> 
> >>> Here is information I can provide:
> >>> 
> >>>   - package builds and runs flawlessly on my main Debian system
> >>>   - guix build dies on #include <ft2build.h> on
> >>>     X11/Xft/Xft.h:39
> >>>   - this file (`ft2build.h`) belongs to `freetype` and should
> >>>     be propagated with `libxft'.
> >>>   - package `st', which also uses `libxft' build find for me.  
> >>
> >> I also found this "simple" software to not be so simple to package for
> >> Guix ;)
> >>
> >> Here is my working package definition, in my "out-of-tree" repo:
> >> https://github.com/lfam/pkgs/blob/master/leo/packages/sent.scm
> >>
> >> I was waiting for a little while before submitting my package definition
> >> to let upstream work out some of the early bugs. But now that 0.2 is
> >> out, perhaps it is time. Please feel free to submit your updated package
> >> definition!
> >>  
> > It works for me! So the problem boils dpwn to the missing
> > `-I/gnu/store/...-freetype/include/freetype2' include path...  
> 
> This was never finished. Could anyone pick this up and finish it?
> 
> Thanks.

This is my take on the patch.  It builds and works fine.

Dmitry, I added a copyright line for you; does it seem alright?

`~Eric

[-- Warning: decoded text below may be mangled, UTF-8 assumed --]
[-- Attachment #2: 0001-gnu-Add-sent.patch --]
[-- Type: text/x-patch, Size: 3427 bytes --]

From fbfd29f793815d9870d0edf9044e7aa9ec7175a6 Mon Sep 17 00:00:00 2001
From: Dmitry Bogatov <KAction@gnu.org>
Date: Sun, 14 Aug 2016 01:46:58 -0500
Subject: [PATCH] gnu: Add sent.

* gnu/packages/suckless.scm (sent): New variable.

Co-authored-by: Leo Famulari <leo@famulari.name>
Co-authored-by: Eric Bavier <bavier@member.fsf.org>
---
 gnu/packages/suckless.scm | 45 +++++++++++++++++++++++++++++++++++++++++++++
 1 file changed, 45 insertions(+)

diff --git a/gnu/packages/suckless.scm b/gnu/packages/suckless.scm
index 7aab2c2..1f62e26 100644
--- a/gnu/packages/suckless.scm
+++ b/gnu/packages/suckless.scm
@@ -3,6 +3,9 @@
 ;;; Copyright © 2015 Amirouche Boubekki <amirouche@hypermove.net>
 ;;; Copyright © 2016 Al McElrath <hello@yrns.org>
 ;;; Copyright © 2016 ng0 <ng0@we.make.ritual.n0.is>
+;;; Copyright © 2015 Dmitry Bogatov <KAction@gnu.org>
+;;; Copyright © 2015 Leo Famulari <leo@famulari.name>
+;;; Copyright © 2016 Eric Bavier <bavier@member.fsf.org>
 ;;;
 ;;; This file is part of GNU Guix.
 ;;;
@@ -27,6 +30,7 @@
   #:use-module (guix build-system glib-or-gtk)
   #:use-module (gnu packages)
   #:use-module (gnu packages gnome)
+  #:use-module (gnu packages image)
   #:use-module (gnu packages xorg)
   #:use-module (gnu packages fonts)
   #:use-module (gnu packages pkg-config)
@@ -200,3 +204,44 @@ display websites and follow links.  It supports the XEmbed protocol which
 makes it possible to embed it in another application.  Furthermore, one can
 point surf to another URI by setting its XProperties.")
     (license license:x11)))
+
+(define-public sent
+  (package
+    (name "sent")
+    (version "0.2")
+    (source (origin
+              (method url-fetch)
+              (uri (string-append "http://dl.suckless.org/tools/sent-"
+                                  version ".tar.gz"))
+              (sha256
+               (base32
+                "0xhh752hwaa26k4q6wvrb9jnpbnylss2aw6z11j7l9rav7wn3fak"))))
+    (build-system gnu-build-system)
+    (arguments
+     `(#:phases (modify-phases %standard-phases
+                  (delete 'configure))  ;no configuration
+       #:tests? #f                      ;no test suite
+       #:make-flags (let ((pkg-config (lambda (flag)
+                                        (string-append
+                                         "$(shell pkg-config " flag " "
+                                         "xft fontconfig x11 libpng)"))))
+                      (list
+                       "CC=gcc"
+                       (string-append "PREFIX=" %output)
+                       (string-append "INCS=-I. " (pkg-config "--cflags"))
+                       (string-append "LIBS=" (pkg-config "--libs") " -lm")))))
+    (native-inputs
+     `(("pkg-config" ,pkg-config)))
+    (inputs
+     `(("libpng" ,libpng)
+       ("libx11" ,libx11)
+       ("libxft" ,libxft)
+       ("fontconfig" ,fontconfig)))
+    (synopsis "Plaintext presentation tool")
+    (description "Sent uses plaintext files and PNG images to create slideshow
+presentations.  Each paragraph represents a slide in the presentation.
+Especially for presentations using the Takahashi method this is very nice and
+allows you to write down the presentation for a quick lightning talk within a
+few minutes.")
+    (home-page "http://tools.suckless.org/sent")
+    (license license:x11)))
-- 
2.9.2