all messages for Guix-related lists mirrored at yhetil.org
 help / color / mirror / code / Atom feed
* [bug#73749] [PATCH] gnu: Add wtw
@ 2024-10-11 17:58 Thomas Albers Raviola via Guix-patches via
  2024-10-30 20:26 ` Sharlatan Hellseher
  0 siblings, 1 reply; 2+ messages in thread
From: Thomas Albers Raviola via Guix-patches via @ 2024-10-11 17:58 UTC (permalink / raw)
  To: 73749; +Cc: Thomas Albers Raviola

* gnu/packages/xdisorg.scm (wtw): New variable

Change-Id: Idcb580debda0d26b9e8ee6ec53d6f905415037ba
---
 gnu/packages/xdisorg.scm | 34 ++++++++++++++++++++++++++++++++++
 1 file changed, 34 insertions(+)

diff --git a/gnu/packages/xdisorg.scm b/gnu/packages/xdisorg.scm
index 21681119f1..5baaec3dc7 100644
--- a/gnu/packages/xdisorg.scm
+++ b/gnu/packages/xdisorg.scm
@@ -65,6 +65,7 @@
 ;;; Copyright © 2024 Igor Goryachev <igor@goryachev.org>
 ;;; Copyright © 2024 Ashish SHUKLA <ashish.is@lostca.se>
 ;;; Copyright © 2024 Spencer Peters <spencerpeters@protonmail.com>
+;;; Copyright © 2024 Thomas Albers Raviola <thomas@thomaslabs.org>
 ;;;
 ;;; This file is part of GNU Guix.
 ;;;
@@ -3959,3 +3960,36 @@ (define-public xxkb
 accepts events from XKB.  That means that it will work with the existing
 setup of your X Server without any modifications.")
     (license license:artistic2.0)))
+
+(define-public wtw
+  (let ((commit "a92feab7bb8e354a9136da82c9b716ffd7cdf08b")
+        (revision "0"))
+    (package
+      (name "wtw")
+      (version (git-version "0" revision commit))
+      (source
+       (origin
+         (method git-fetch)
+         (uri (git-reference
+               (url "https://codeberg.org/sewn/wtw.git")
+               (commit commit)))
+         (file-name (git-file-name name version))
+         (sha256
+          (base32 "0ni06zbrcp7pqidqfa5fr8dzkd84plf8s9y696g2rdmj8fdgynfj"))))
+      (build-system gnu-build-system)
+      (arguments
+       (list #:tests? #f
+             #:make-flags
+             #~(list
+                (string-append "CC=" #$(cc-for-target))
+                (string-append "PREFIX=" %output))
+             #:phases
+             #~(modify-phases %standard-phases
+                 (delete 'configure))))
+      (inputs (list fcft pixman wayland wayland-protocols))
+      (native-inputs (list pkg-config))
+      (synopsis "Simple text widget for wlroot compositors")
+      (description "Simple text widget for wlroot compositors; ported from stw
+and inheriting it's behavior.")
+      (home-page "https://codeberg.org/sewn/wtw")
+      (license license:expat))))
-- 
2.46.0





^ permalink raw reply related	[flat|nested] 2+ messages in thread

* [bug#73749] [PATCH] gnu: Add wtw
  2024-10-11 17:58 [bug#73749] [PATCH] gnu: Add wtw Thomas Albers Raviola via Guix-patches via
@ 2024-10-30 20:26 ` Sharlatan Hellseher
  0 siblings, 0 replies; 2+ messages in thread
From: Sharlatan Hellseher @ 2024-10-30 20:26 UTC (permalink / raw)
  To: 73749; +Cc: Thomas Albers Raviola

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


Hi Thomas,

Thank you for the patch, please see some review points.

--8<---------------cut here---------------start------------->8---
+(define-public wtw
+  (let ((commit "a92feab7bb8e354a9136da82c9b716ffd7cdf08b")
+        (revision "0"))
--8<---------------cut here---------------end--------------->8---
No release no version tag? If any open issue mentioned that in upstream
please provide, or even open a new one asking about that ;-).

--8<---------------cut here---------------start------------->8---
+      (version (git-version "0" revision commit))
--8<---------------cut here---------------end--------------->8---
"0" -> "0.0.0" if above is not resolved

--8<---------------cut here---------------start------------->8---
+       (list #:tests? #f
--8<---------------cut here---------------end--------------->8---
Please be specific why: failed (short log), no tests, require network 


--8<---------------cut here---------------start------------->8---
+                (string-append "PREFIX=" %output))
--8<---------------cut here---------------end--------------->8---
%output -> #$output


--8<---------------cut here---------------start------------->8---
+                 (delete 'configure))))
--8<---------------cut here---------------end--------------->8---
Be more specific here as well: no configure, failing etc.

--8<---------------cut here---------------start------------->8---
+      (synopsis "Simple text widget for wlroot compositors")
+      (description "Simple text widget for wlroot compositors; ported from stw
+and inheriting it's behavior.")
+      (home-page "https://codeberg.org/sewn/wtw")
--8<---------------cut here---------------end--------------->8---
Some style pick: place home-page before above synopsis, start
description from the new line.


Looking forward for the v2!

--
Oleg

[-- Attachment #2: signature.asc --]
[-- Type: application/pgp-signature, Size: 832 bytes --]

^ permalink raw reply	[flat|nested] 2+ messages in thread

end of thread, other threads:[~2024-10-30 20:29 UTC | newest]

Thread overview: 2+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2024-10-11 17:58 [bug#73749] [PATCH] gnu: Add wtw Thomas Albers Raviola via Guix-patches via
2024-10-30 20:26 ` Sharlatan Hellseher

Code repositories for project(s) associated with this external index

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

This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.