all messages for Guix-related lists mirrored at yhetil.org
 help / color / mirror / code / Atom feed
* [bug#50065] [PATCH] gnu: Add setroot.
@ 2021-08-15  6:46 Brice Waegeneire
  2021-08-31 14:04 ` Mathieu Othacehe
  0 siblings, 1 reply; 5+ messages in thread
From: Brice Waegeneire @ 2021-08-15  6:46 UTC (permalink / raw)
  To: 50065

* gnu/packages/xorg.scm (setroot): New variable.
---
 gnu/packages/xorg.scm | 33 +++++++++++++++++++++++++++++++++
 1 file changed, 33 insertions(+)

diff --git a/gnu/packages/xorg.scm b/gnu/packages/xorg.scm
index 6f9633985e..505af87512 100644
--- a/gnu/packages/xorg.scm
+++ b/gnu/packages/xorg.scm
@@ -6925,3 +6925,36 @@ an existing user-specified one, writes a cookie to it, and then starts the
 the server and cleaning up before returning the exit status of the command.")
     (license (list license:x11                    ; the script
                    license:gpl2+))))              ; the man page
+
+(define-public setroot
+  (package
+    (name "setroot")
+    (version "2.0.2")
+    (source (origin
+              (method git-fetch)
+              (uri (git-reference
+                    (url "https://github.com/ttzhou/setroot")
+                    (commit (string-append "v" version))))
+              (file-name (git-file-name name version))
+              (sha256
+               (base32 "0w95828v0splk7bj5kfacp4pq6wxpyamvyjmahyvn5hc3ycq21mq"))))
+    (build-system gnu-build-system)
+    (arguments
+     '(#:make-flags
+       (list (string-append "DESTDIR=" (assoc-ref %outputs "out"))
+             "PREFIX="
+             "xinerama=1")
+       #:tests? #f                      ; no tests
+       #:phases
+       (modify-phases %standard-phases
+         (delete 'configure))))
+    (inputs
+     `(("imlib2" ,imlib2)
+       ("libx11" ,libx11)
+       ("libxinerama" ,libxinerama)))
+    (home-page "https://github.com/ttzhou/setroot")
+    (synopsis "Simple X background setter inspired by imlibsetroot and feh")
+    (description "Setroot is a lightweight X background setter with feh's
+syntax without it's image viewing capabilities.  It supports multiple monitors
+and can restore previously set wallpapers and options.")
+    (license license:gpl3+)))

base-commit: e0feacfbad410ecb2f11e7ee86f18482c7413b3f
-- 
2.32.0





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

* [bug#50065] [PATCH] gnu: Add setroot.
  2021-08-15  6:46 [bug#50065] [PATCH] gnu: Add setroot Brice Waegeneire
@ 2021-08-31 14:04 ` Mathieu Othacehe
  2021-08-31 20:06   ` [bug#50065] [PATCH v2] " Brice Waegeneire
  0 siblings, 1 reply; 5+ messages in thread
From: Mathieu Othacehe @ 2021-08-31 14:04 UTC (permalink / raw)
  To: Brice Waegeneire; +Cc: 50065


Hello Brice,

> +    (source (origin

You should put origin on the next line to respect the 78 columns limit.

> +              (method git-fetch)
> +              (uri (git-reference
> +                    (url "https://github.com/ttzhou/setroot")
> +                    (commit (string-append "v" version))))
> +              (file-name (git-file-name name version))
> +              (sha256
> +               (base32 "0w95828v0splk7bj5kfacp4pq6wxpyamvyjmahyvn5hc3ycq21mq"))))
> +    (build-system gnu-build-system)
> +    (arguments
> +     '(#:make-flags
> +       (list (string-append "DESTDIR=" (assoc-ref %outputs "out"))
> +             "PREFIX="
> +             "xinerama=1")

The Makefile seems to hardcode CC to gcc which probably breaks
cross-compilation.

> +syntax without it's image viewing capabilities.  It supports multiple monitors
                    ^
                    its?

Thanks,

Mathieu




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

* [bug#50065] [PATCH v2] gnu: Add setroot.
  2021-08-31 14:04 ` Mathieu Othacehe
@ 2021-08-31 20:06   ` Brice Waegeneire
  2021-09-01  6:53     ` [bug#50065] [PATCH] " Mathieu Othacehe
  0 siblings, 1 reply; 5+ messages in thread
From: Brice Waegeneire @ 2021-08-31 20:06 UTC (permalink / raw)
  To: othacehe; +Cc: 50065

* gnu/packages/xorg.scm (setroot): New variable.
---
 gnu/packages/xorg.scm | 35 +++++++++++++++++++++++++++++++++++
 1 file changed, 35 insertions(+)

Thank you for the review Mathieu.  I fixed the three issues you pointed out.

diff --git a/gnu/packages/xorg.scm b/gnu/packages/xorg.scm
index 6f9633985e..cae0fb3f73 100644
--- a/gnu/packages/xorg.scm
+++ b/gnu/packages/xorg.scm
@@ -6925,3 +6925,38 @@ an existing user-specified one, writes a cookie to it, and then starts the
 the server and cleaning up before returning the exit status of the command.")
     (license (list license:x11                    ; the script
                    license:gpl2+))))              ; the man page
+
+(define-public setroot
+  (package
+    (name "setroot")
+    (version "2.0.2")
+    (source
+     (origin
+       (method git-fetch)
+       (uri (git-reference
+             (url "https://github.com/ttzhou/setroot")
+             (commit (string-append "v" version))))
+       (file-name (git-file-name name version))
+       (sha256
+        (base32 "0w95828v0splk7bj5kfacp4pq6wxpyamvyjmahyvn5hc3ycq21mq"))))
+    (build-system gnu-build-system)
+    (arguments
+     `(#:make-flags
+       (list (string-append "CC=" ,(cc-for-target))
+             (string-append "DESTDIR=" (assoc-ref %outputs "out"))
+             "PREFIX="
+             "xinerama=1")
+       #:tests? #f                       ; no tests
+       #:phases
+       (modify-phases %standard-phases
+         (delete 'configure))))
+    (inputs
+     `(("imlib2" ,imlib2)
+       ("libx11" ,libx11)
+       ("libxinerama" ,libxinerama)))
+    (home-page "https://github.com/ttzhou/setroot")
+    (synopsis "Simple X background setter inspired by imlibsetroot and feh")
+    (description "Setroot is a lightweight X background setter with feh's
+syntax without its image viewing capabilities.  It supports multiple monitors
+and can restore previously set wallpapers and options.")
+    (license license:gpl3+)))
-- 
2.33.0





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

* [bug#50065] [PATCH] gnu: Add setroot.
  2021-08-31 20:06   ` [bug#50065] [PATCH v2] " Brice Waegeneire
@ 2021-09-01  6:53     ` Mathieu Othacehe
  2021-09-04  7:32       ` bug#50065: " Brice Waegeneire
  0 siblings, 1 reply; 5+ messages in thread
From: Mathieu Othacehe @ 2021-09-01  6:53 UTC (permalink / raw)
  To: Brice Waegeneire; +Cc: 50065


Hey Brice,

> * gnu/packages/xorg.scm (setroot): New variable.

Looks fine, feel free to proceed :).

Mathieu




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

* bug#50065: [PATCH] gnu: Add setroot.
  2021-09-01  6:53     ` [bug#50065] [PATCH] " Mathieu Othacehe
@ 2021-09-04  7:32       ` Brice Waegeneire
  0 siblings, 0 replies; 5+ messages in thread
From: Brice Waegeneire @ 2021-09-04  7:32 UTC (permalink / raw)
  To: Mathieu Othacehe; +Cc: 50065-done

Mathieu Othacehe <othacehe@gnu.org> writes:

> Looks fine, feel free to proceed :).

Pushed as 2226d5b9590840f6fdc0d5c03b14c9d050435c4e. Thank you for the review
Mathieu.




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

end of thread, other threads:[~2021-09-04  7:34 UTC | newest]

Thread overview: 5+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2021-08-15  6:46 [bug#50065] [PATCH] gnu: Add setroot Brice Waegeneire
2021-08-31 14:04 ` Mathieu Othacehe
2021-08-31 20:06   ` [bug#50065] [PATCH v2] " Brice Waegeneire
2021-09-01  6:53     ` [bug#50065] [PATCH] " Mathieu Othacehe
2021-09-04  7:32       ` bug#50065: " Brice Waegeneire

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.