From mboxrd@z Thu Jan 1 00:00:00 1970 From: Mathieu Lirzin Subject: Re: [PATCH] gnu: st: Mov to terminals.scm. Date: Sun, 07 Aug 2016 15:46:33 +0200 Message-ID: <8760rcg092.fsf@gnu.org> References: <871t20ahcb.fsf@we.make.ritual.n0.is> Mime-Version: 1.0 Content-Type: text/plain; charset=utf-8 Content-Transfer-Encoding: quoted-printable Return-path: Received: from eggs.gnu.org ([2001:4830:134:3::10]:45471) by lists.gnu.org with esmtp (Exim 4.71) (envelope-from ) id 1bWOPa-0006fC-Qw for guix-devel@gnu.org; Sun, 07 Aug 2016 09:46:44 -0400 Received: from Debian-exim by eggs.gnu.org with spam-scanned (Exim 4.71) (envelope-from ) id 1bWOPW-0007xZ-JG for guix-devel@gnu.org; Sun, 07 Aug 2016 09:46:41 -0400 In-Reply-To: <871t20ahcb.fsf@we.make.ritual.n0.is> (ng0@we.make.ritual.n0.is's message of "Sun, 07 Aug 2016 12:33:56 +0000") List-Id: "Development of GNU Guix and the GNU System distribution." List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Errors-To: guix-devel-bounces+gcggd-guix-devel=m.gmane.org@gnu.org Sender: "Guix-devel" To: ng0 Cc: guix-devel@gnu.org Hi, I don't have an opinion on the move, however when moving packages across modules your have to keep track of the copyrights. In this case Amirouche Boubekki copyright should = be moved from (gnu packages suckless) to (gnu packages terminals). ng0 writes: > From 3864654025329367b6f87ccfb12a4ff789d5155a Mon Sep 17 00:00:00 2001 > From: ng0 > Date: Sun, 7 Aug 2016 12:30:06 +0000 > Subject: [PATCH] gnu: st: Move to terminals.scm. > > * gnu/packages/suckless.scm (st): Move from here ... > * gnu/packages/terminals.scm (st): ... to here. nitpick:=20=20 --8<---------------cut here---------------start------------->8--- gnu: st: Move to (gnu packages terminals). * gnu/packages/suckless.scm (st): Move to ... * gnu/packages/terminals.scm (st): ... here. --8<---------------cut here---------------end--------------->8--- > --- > gnu/packages/suckless.scm | 50 +---------------------------------------= ---- > gnu/packages/terminals.scm | 52 ++++++++++++++++++++++++++++++++++++++++= +++++- > 2 files changed, 52 insertions(+), 50 deletions(-) > > diff --git a/gnu/packages/suckless.scm b/gnu/packages/suckless.scm > index 8732a84..16e9146 100644 > --- a/gnu/packages/suckless.scm > +++ b/gnu/packages/suckless.scm > @@ -28,10 +28,8 @@ > #:use-module (gnu packages) > #:use-module (gnu packages gnome) > #:use-module (gnu packages xorg) > - #:use-module (gnu packages fonts) > #:use-module (gnu packages pkg-config) > - #:use-module (gnu packages webkit) > - #:use-module (gnu packages fontutils)) > + #:use-module (gnu packages webkit)) >=20=20 > (define-public dwm > (package > @@ -126,52 +124,6 @@ numbers of user-defined menu items efficiently.") > "Simple X session lock with trivial feedback on password entry.") > (license license:x11))) >=20=20 > -(define-public st > - (package > - (name "st") > - (version "0.6") > - (source > - (origin > - (method url-fetch) > - (uri (string-append "http://dl.suckless.org/st/st-" > - version ".tar.gz")) > - (sha256 > - (base32 > - "0avsfc1qp8zvshsfjwwrkvk411jlqy58z225bsdhjkl1qc40qcc5")))) > - (build-system gnu-build-system) > - (arguments > - '(#:tests? #f ; no tests > - #:make-flags (list "CC=3Dgcc" > - (string-append "PREFIX=3D" %output)) > - #:phases > - (modify-phases %standard-phases > - (delete 'configure) > - (add-after 'unpack 'inhibit-terminfo-install > - (lambda _ > - (substitute* "Makefile" > - (("\t@tic -s st.info") "")) > - #t))))) > - (inputs > - `(("libx11" ,libx11) > - ("libxft" ,libxft) > - ("libxcomposite" ,libxcomposite) > - ("compositeproto" ,compositeproto) > - ("libxext" ,libxext) > - ("xextproto" ,xextproto) > - ("libxrender" ,libxrender) > - ("fontconfig" ,fontconfig) > - ("freetype" ,freetype) > - ("font-liberation" ,font-liberation))) > - (native-inputs `(("pkg-config" ,pkg-config))) > - (home-page "http://st.suckless.org/") > - (synopsis "Simple terminal emulator") > - (description > - "St implements a simple and lightweight terminal emulator. It > -implements 256 colors, most VT10X escape sequences, utf8, X11 copy/paste, > -antialiased fonts (using fontconfig), fallback fonts, resizing, and line > -drawing.") > - (license license:x11))) > - > (define-public surf > (package > (name "surf") > diff --git a/gnu/packages/terminals.scm b/gnu/packages/terminals.scm > index 98f0060..02cb9fa 100644 > --- a/gnu/packages/terminals.scm > +++ b/gnu/packages/terminals.scm > @@ -2,6 +2,7 @@ > ;;; Copyright =C2=A9 2015 Efraim Flashner > ;;; Copyright =C2=A9 2016 Mckinley Olsen > ;;; Copyright =C2=A9 2016 Alex Griffin > +;;; Copyright =C2=A9 2016 ng0 > ;;; > ;;; This file is part of GNU Guix. > ;;; > @@ -36,7 +37,10 @@ > #:use-module (gnu packages wm) > #:use-module (gnu packages ncurses) > #:use-module (gnu packages gtk) > - #:use-module (gnu packages gnome)) > + #:use-module (gnu packages gnome) > + #:use-module (gnu packages xorg) > + #:use-module (gnu packages fonts) > + #:use-module (gnu packages fontutils)) >=20=20 > (define-public tilda > (package > @@ -159,3 +163,49 @@ insert mode and command mode where keybindings have = different functions.") > Forget screen recording apps and blurry video. Enjoy a lightweight, pur= ely > text-based approach to terminal recording.") > (license license:gpl3))) > + > +(define-public st > + (package > + (name "st") > + (version "0.6") > + (source > + (origin > + (method url-fetch) > + (uri (string-append "http://dl.suckless.org/st/st-" > + version ".tar.gz")) > + (sha256 > + (base32 > + "0avsfc1qp8zvshsfjwwrkvk411jlqy58z225bsdhjkl1qc40qcc5")))) > + (build-system gnu-build-system) > + (arguments > + '(#:tests? #f ; no tests > + #:make-flags (list "CC=3Dgcc" > + (string-append "PREFIX=3D" %output)) > + #:phases > + (modify-phases %standard-phases > + (delete 'configure) > + (add-after 'unpack 'inhibit-terminfo-install > + (lambda _ > + (substitute* "Makefile" > + (("\t@tic -s st.info") "")) > + #t))))) > + (inputs > + `(("libx11" ,libx11) > + ("libxft" ,libxft) > + ("libxcomposite" ,libxcomposite) > + ("compositeproto" ,compositeproto) > + ("libxext" ,libxext) > + ("xextproto" ,xextproto) > + ("libxrender" ,libxrender) > + ("fontconfig" ,fontconfig) > + ("freetype" ,freetype) > + ("font-liberation" ,font-liberation))) > + (native-inputs `(("pkg-config" ,pkg-config))) > + (home-page "http://st.suckless.org/") > + (synopsis "Simple terminal emulator") > + (description > + "St implements a simple and lightweight terminal emulator. It > +implements 256 colors, most VT10X escape sequences, utf8, X11 copy/paste, > +antialiased fonts (using fontconfig), fallback fonts, resizing, and line > +drawing.") > + (license license:x11))) > --=20 > 2.9.2 Can you send an updated patch? If nobody objects I will push this in the following days. Thanks. --=20 Mathieu Lirzin