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
next prev parent 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).