unofficial mirror of guix-devel@gnu.org 
 help / color / mirror / code / Atom feed
From: amirouche@hypermove.net
To: guix-devel@gnu.org
Subject: Re: [PATCH] Add st
Date: Tue, 09 Jun 2015 15:07:55 +0200	[thread overview]
Message-ID: <960301908817217a6148629f683a3f34@hypermove.net> (raw)
In-Reply-To: <7d59986edfeafbf10eeaaada41acee14@hypermove.net>

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

With the patch this time ^^'

On 2015-06-09 13:04, amirouche@hypermove.net wrote:
> This patch st terminal emulator from suckless.
> 
> This is my first patch!
> 
> This installs flawlessly except:
> 
> - `tic -s terminfo -s info` which fails because it try to install in
> an unknown home directory. In Gentoo they install it in
> /usr/share/terminfo [1]. A added a patch to avoid this issue. One has
> to export TERM=xterm to make it possible to use terminal applications.
> 
> - The default config.def.h use Liberation font, this is ok. However
> it's might be better to activate antialias and autohint?
> 
> 
> [1]
> https://sources.gentoo.org/cgi-bin/viewvc.cgi/gentoo-x86/x11-terms/st/st-0.5.ebuild?view=markup

[-- Warning: decoded text below may be mangled, UTF-8 assumed --]
[-- Attachment #2: 0001-gnu-add-st.patch --]
[-- Type: text/x-diff; name=0001-gnu-add-st.patch, Size: 3677 bytes --]

From 385f7129b405d136cbbb7dbffa35bf6f62cffc93 Mon Sep 17 00:00:00 2001
From: amz3 <amirouche@hypermove.net>
Date: Sun, 7 Jun 2015 19:04:28 +0200
Subject: [PATCH] gnu: add st.

* gnu/packages/dwm.scm (st): New variable.
* gnu/packages/patches/st-0.5-do-not-install-terminfo.patch: New file.
---
 gnu/packages/dwm.scm                               | 50 +++++++++++++++++++++-
 .../patches/st-0.5-do-not-install-terminfo.patch   | 13 ++++++
 2 files changed, 62 insertions(+), 1 deletion(-)
 create mode 100644 gnu/packages/patches/st-0.5-do-not-install-terminfo.patch

diff --git a/gnu/packages/dwm.scm b/gnu/packages/dwm.scm
index 98fa122..b82297f 100644
--- a/gnu/packages/dwm.scm
+++ b/gnu/packages/dwm.scm
@@ -22,7 +22,11 @@
   #:use-module (guix download)
   #:use-module (guix build-system gnu)
   #:use-module (gnu packages)
-  #:use-module (gnu packages xorg))
+  #:use-module (gnu packages xorg)
+  #:use-module (gnu packages fonts)
+  #:use-module (gnu packages ncurses)
+  #:use-module (gnu packages pkg-config)
+  #:use-module ((gnu packages fontutils) #:prefix font-utils:))
 
 (define-public dwm
   (package
@@ -139,3 +143,47 @@ numbers of user-defined menu items efficiently.")
     (description
      "Simple X session lock with trivial feedback on password entry.")
     (license x11)))
+
+
+(define-public st
+  (package
+    (name "st")
+    (version "0.5")
+    (source
+     (origin
+       (method url-fetch)
+       (uri (string-append "http://dl.suckless.org/st/st-"
+                           version ".tar.gz"))
+       (patches (list (search-patch "st-0.5-do-not-install-terminfo.patch")))
+       (sha256
+        (base32
+         "0knxpzaa86pprng6hak8hx8bw22yw22rpz1ffxjpcvqlz3xdv05f"))))
+    (build-system gnu-build-system)
+    (arguments
+     '(#:tests? #f ; no tests
+       #:make-flags (list "CC=gcc"
+                          (string-append "PREFIX=" %output))
+       #:phases
+       (alist-delete 'configure %standard-phases)))
+    (inputs
+     `(("libx11" ,libx11)
+       ("libxft" ,libxft)
+       ("libxcomposite" ,libxcomposite)
+       ("compositeproto" ,compositeproto)
+       ("libxext" ,libxext)
+       ("xextproto" ,xextproto)
+       ("libxrender" ,libxrender)
+       ("fontconfig" ,font-utils:fontconfig)
+       ("freetype" ,font-utils:freetype)
+       ("font-liberation5" ,font-liberation)))
+    (native-inputs `(("pkg-config" ,pkg-config)
+                     ("ncurses" ,ncurses)))
+    (home-page "http://st.suckless.org/")
+    (synopsis "Simple terminal emulator")
+    (description
+     "Xterm is bloated and unmaintainable.  It has over 65K lines of code and
+      emulates obscure and obsolete terminals you will never need.  The popular
+      alternative, rxvt has only 32K lines of code.  This is just too much for
+      something as simple as a terminal emulator; it’s yet another example of
+      code complexity.  st fix that.")
+    (license x11)))
diff --git a/gnu/packages/patches/st-0.5-do-not-install-terminfo.patch b/gnu/packages/patches/st-0.5-do-not-install-terminfo.patch
new file mode 100644
index 0000000..089c423
--- /dev/null
+++ b/gnu/packages/patches/st-0.5-do-not-install-terminfo.patch
@@ -0,0 +1,13 @@
+diff --git a/Makefile b/Makefile
+index 6158ab2..93cb684 100644
+--- a/Makefile
++++ b/Makefile
+@@ -48,8 +48,6 @@ install: all
+ 	@mkdir -p ${DESTDIR}${MANPREFIX}/man1
+ 	@sed "s/VERSION/${VERSION}/g" < st.1 > ${DESTDIR}${MANPREFIX}/man1/st.1
+ 	@chmod 644 ${DESTDIR}${MANPREFIX}/man1/st.1
+-	@echo Please see the README file regarding the terminfo entry of st.
+-	@tic -s st.info
+ 
+ uninstall:
+ 	@echo removing executable file from ${DESTDIR}${PREFIX}/bin
-- 
2.2.1


  reply	other threads:[~2015-06-09 13:08 UTC|newest]

Thread overview: 9+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2015-06-09 11:04 [PATCH] Add st amirouche
2015-06-09 13:07 ` amirouche [this message]
2015-06-09 14:22   ` Andreas Enge
2015-06-10  6:28     ` amirouche
2015-06-11 22:15       ` Mark H Weaver
2015-06-13 19:13         ` amirouche
2015-06-14 17:46           ` Cyril Roelandt
2015-06-15 20:33             ` Ludovic Courtès
2015-06-15 21:28           ` Mark H Weaver

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

  List information: https://guix.gnu.org/

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

  git send-email \
    --in-reply-to=960301908817217a6148629f683a3f34@hypermove.net \
    --to=amirouche@hypermove.net \
    --cc=guix-devel@gnu.org \
    /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 public inbox

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

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for read-only IMAP folder(s) and NNTP newsgroup(s).