Message ID | tencent_6FCA5F4BAA6AA7376B526DB3FAFEC674F609@qq.com |
---|---|
State | Accepted |
Headers | show |
Series | [bug#46018] Add dwl. | expand |
Context | Check | Description |
---|---|---|
cbaines/submitting builds | success | |
cbaines/comparison | success | View comparision |
cbaines/git branch | success | View Git branch |
cbaines/applying patch | fail | View Laminar job |
cbaines/issue | success | View issue |
Hello, Z572 <873216071@qq.com> writes: > Subject: [PATCH] gnu: Add dwl. Thank you! Some comments follow. > +(define-public dwl > + (package > + (name "dwl") > + (version "0.1") > + (source (origin > + (method git-fetch) > + (uri (git-reference > + (url "https://github.com/djpohly/dwl") > + (commit (string-append "v" version)))) > + (file-name (git-file-name name version)) > + (sha256 > + (base32 > + "18g2sx8cv54zl5iw5m9lzngrp6ra2pyp7c68qps2ava3brw9m0j2")))) > + (build-system gnu-build-system) > + (arguments > + `(#:tests? #f ; no tests > + #:make-flags > + (list > + "CC=gcc" Instead of "CC=gcc", you should use (string-append "CC=" ,(cc-for-target)) for cross-compilation. > + (string-append "PREFIX=" %output)) Here, I think it is preferable to use (string-append "PREFIX=" (assoc-ref %outputs "out")) > + #:phases > + (modify-phases %standard-phases > + (delete 'configure) ; no configure > + (replace 'install ; no install targer Typo: ";no install target" > + (lambda* (#:key outputs #:allow-other-keys) > + (let* ((out (assoc-ref outputs "out")) > + (bin (string-append out "/bin"))) > + (install-file "dwl" bin))))))) > + (native-inputs > + `(("pkg-config" ,pkg-config))) > + (inputs > + `(("wlroots" ,wlroots))) > + (home-page "https://github.com/djpohly/dwl") > + (synopsis "Dwm for Wayland") It would be nice to expound Dwm, e.g., "Dynamic window manager for Wayland" > + (description > + "A compact, hackable compositor for Wayland based on wlroots. It is This should be a complete sentence, e.g., Dwl is a compact, hackable... > +intended to fill the same space in the Wayland world that dwm does in X11, > +primarily in terms of philosophy, and secondarily in terms of functionality. > +Like dwm, dwl is: > +@itemize > +@item Easy to understand, hack on, and extend with patches > +@item One C source file (or a very small number) configurable via config.h > +@item Limited to 2000 SLOC to promote hackability > +@item Tied to as few external dependencies as possible I suggest rewrite so it doesn't include a list. For example: Like dwm, dwl is easy to understand and hack on, due to a limited size and a few external dependencies. It is configurable via @file{config.h}. Could you send an updated patch? Regards,
From fa60adfd7b2b48ea12a1ed66e7da9db94412e3af Mon Sep 17 00:00:00 2001 From: Zheng Junjie <873216071@qq.com> Date: Thu, 21 Jan 2021 20:12:13 +0800 Subject: [PATCH] gnu: Add dwl. --- gnu/packages/wm.scm | 48 +++++++++++++++++++++++++++++++++++++++++++++ 1 file changed, 48 insertions(+) diff --git a/gnu/packages/wm.scm b/gnu/packages/wm.scm index 084822f2c7..682fc69dbe 100644 --- a/gnu/packages/wm.scm +++ b/gnu/packages/wm.scm @@ -41,6 +41,7 @@ ;;; Copyright © 2020 B. Wilson <elaexuotee@wilsonb.com> ;; Copyright © 2020 Niklas Eklund <niklas.eklund@posteo.net> ;;; Copyright © 2020 Robert Smith <robertsmith@posteo.net> +;;; Copyright © 2021 Zheng Junjie <873216071@qq.com> ;;; ;;; This file is part of GNU Guix. ;;; @@ -1258,6 +1259,53 @@ It is inspired by Xmonad and dwm. Its major features include: project derived from the original Calm Window Manager.") (license license:isc))) +(define-public dwl + (package + (name "dwl") + (version "0.1") + (source (origin + (method git-fetch) + (uri (git-reference + (url "https://github.com/djpohly/dwl") + (commit (string-append "v" version)))) + (file-name (git-file-name name version)) + (sha256 + (base32 + "18g2sx8cv54zl5iw5m9lzngrp6ra2pyp7c68qps2ava3brw9m0j2")))) + (build-system gnu-build-system) + (arguments + `(#:tests? #f ; no tests + #:make-flags + (list + "CC=gcc" + (string-append "PREFIX=" %output)) + #:phases + (modify-phases %standard-phases + (delete 'configure) ; no configure + (replace 'install ; no install targer + (lambda* (#:key outputs #:allow-other-keys) + (let* ((out (assoc-ref outputs "out")) + (bin (string-append out "/bin"))) + (install-file "dwl" bin))))))) + (native-inputs + `(("pkg-config" ,pkg-config))) + (inputs + `(("wlroots" ,wlroots))) + (home-page "https://github.com/djpohly/dwl") + (synopsis "Dwm for Wayland") + (description + "A compact, hackable compositor for Wayland based on wlroots. It is +intended to fill the same space in the Wayland world that dwm does in X11, +primarily in terms of philosophy, and secondarily in terms of functionality. +Like dwm, dwl is: +@itemize +@item Easy to understand, hack on, and extend with patches +@item One C source file (or a very small number) configurable via config.h +@item Limited to 2000 SLOC to promote hackability +@item Tied to as few external dependencies as possible +@end itemize") ;; LICENSE LICENSE.dwm LICENSE.tinywl + (license (list license:gpl3+ license:expat license:cc0)))) + (define-public nitrogen (package (name "nitrogen") -- 2.30.0