all messages for Guix-related lists mirrored at yhetil.org
 help / color / mirror / code / Atom feed
* [bug#54311] [PATCH v2] gnu: tint2: Update to 17.0.2.
       [not found] <23>
@ 2022-04-03  3:01 ` Wamm K. D
  2022-05-10  2:05   ` Jaft
  2022-05-11  9:51   ` Maxime Devos
  0 siblings, 2 replies; 8+ messages in thread
From: Wamm K. D @ 2022-04-03  3:01 UTC (permalink / raw)
  To: 54311; +Cc: Wamm K. D, maximedevos

---
Here we go, Maxime; I've switched the ~fetch~ method to =git=.

I think you're absolutely right about the =fix-installation-prefix= phase not
being necessary as "etc" isn't even in the =CMakeLists.txt= file, anymore; I had
no issues building with the phase removed, as well.

Removing not–running-tests, however, did result in a "make: *** No rule to make
target 'test'.  Stop." error, however, so I'm guessing that, while the files in
that directory are used for testing, they are not incorporated as part of the
build process and there still is no target so I've left that in.

Hopefully this addresses everything and thanks a ton for looking over this!

 gnu/packages/xdisorg.scm | 24 +++++++++---------------
 1 file changed, 9 insertions(+), 15 deletions(-)

diff --git a/gnu/packages/xdisorg.scm b/gnu/packages/xdisorg.scm
index d2caee85ec..a7738aac47 100644
--- a/gnu/packages/xdisorg.scm
+++ b/gnu/packages/xdisorg.scm
@@ -54,6 +54,7 @@
 ;;; Copyright © 2021 jgart <jgart@dismail.de>
 ;;; Copyright © 2022 John Kehayias <john.kehayias@protonmail.com>
 ;;; Copyright © 2022 Jai Vetrivelan <jaivetrivelan@gmail.com>
+;;; Copyright © 2022 Wamm K. D. <jaft.r@outlook.com>
 ;;;
 ;;; This file is part of GNU Guix.
 ;;;
@@ -1790,27 +1791,20 @@ (define-public rofi-calc
 (define-public tint2
   (package
     (name "tint2")
-    (version "0.14.6")
+    (version "17.0.2")
     (source (origin
-              (method url-fetch)
-              (uri (string-append "https://gitlab.com/o9000/" name
-                                  "/repository/archive.tar.gz?ref=" version))
-              (file-name (string-append name "-" version ".tar.gz"))
+              (method git-fetch)
+              (uri (git-reference
+                    (url (string-append "https://gitlab.com/o9000/" name "/"))
+                    (commit version)))
               (sha256
                (base32
-                "1kwzwxy4myagybm3rc7dgynfgp75742n348qibn1p2an9ggyivda"))))
+                "123apmgs6x2zfv1q57dyl4mwqf0vsw5ndh5jsg6p3fvhr66l1aja"))))
     (build-system cmake-build-system)
     (arguments
-     '(#:tests? #f                      ;no test target
-       #:phases
-       (modify-phases %standard-phases
-         (add-after 'unpack 'fix-installation-prefix
-           (lambda _
-             (substitute* "CMakeLists.txt"
-               (("/etc") "${CMAKE_INSTALL_PREFIX}/etc"))
-             #t)))))
+     '(#:tests? #f))                      ;no test target
     (inputs
-     (list gtk+-2
+     (list gtk+
            imlib2
            librsvg
            libxcomposite
-- 
2.34.0





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

* [bug#54311] [PATCH v2] gnu: tint2: Update to 17.0.2.
  2022-04-03  3:01 ` [bug#54311] [PATCH v2] gnu: tint2: Update to 17.0.2 Wamm K. D
@ 2022-05-10  2:05   ` Jaft
  2022-05-11  9:51   ` Maxime Devos
  1 sibling, 0 replies; 8+ messages in thread
From: Jaft @ 2022-05-10  2:05 UTC (permalink / raw)
  To: 54311@debbugs.gnu.org; +Cc: maximedevos@telenet.be

Let me know if there's any other changes needed but just a reminder that this should be ready, I think.




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

* [bug#54311] [PATCH v2] gnu: tint2: Update to 17.0.2.
  2022-04-03  3:01 ` [bug#54311] [PATCH v2] gnu: tint2: Update to 17.0.2 Wamm K. D
  2022-05-10  2:05   ` Jaft
@ 2022-05-11  9:51   ` Maxime Devos
  2022-05-19  4:54     ` Jaft
  1 sibling, 1 reply; 8+ messages in thread
From: Maxime Devos @ 2022-05-11  9:51 UTC (permalink / raw)
  To: Wamm K. D, 54311

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

Wamm K. D schreef op za 02-04-2022 om 22:01 [-0500]:
> ---
> Here we go, Maxime; I've switched the ~fetch~ method to =git=.
> 
> I think you're absolutely right about the =fix-installation-prefix= phase not
> being necessary as "etc" isn't even in the =CMakeLists.txt= file, anymore; I had
> no issues building with the phase removed, as well.
> 
> Removing not–running-tests, however, did result in a "make: *** No rule to make
> target 'test'.  Stop." error, however, so I'm guessing that, while the files in
> that directory are used for testing, they are not incorporated as part of the
> build process and there still is no target so I've left that in.

There's 'regression.py' which has a 'main' and 'run_unit_tests'.  Maybe
'regression.py' can be run in the 'check' phase?

> 
> Hopefully this addresses everything and thanks a ton for looking over this!

Looks ok (except for 'regression.py' not being run), but for
completeneess you might want to run it in a a pure environment (./pre-
inst-env guix shell --pure tint2) and look for problems like missing
icons or configuration not working or ‘warning: configuration scheme
things / file chooser’ things.

Greetings,
Maxime.

[-- Attachment #2: This is a digitally signed message part --]
[-- Type: application/pgp-signature, Size: 260 bytes --]

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

* [bug#54311] [PATCH v2] gnu: tint2: Update to 17.0.2.
  2022-05-11  9:51   ` Maxime Devos
@ 2022-05-19  4:54     ` Jaft
  2022-05-19 13:13       ` Maxime Devos
  0 siblings, 1 reply; 8+ messages in thread
From: Jaft @ 2022-05-19  4:54 UTC (permalink / raw)
  To: 54311@debbugs.gnu.org, Maxime Devos

> On Wednesday, May 11, 2022, 04:51:19 AM CDT, Maxime Devos <maximedevos@telenet.be> wrote: 
>
>
>
>
>
> There's 'regression.py' which has a 'main' and 'run_unit_tests'.  Maybe
> 'regression.py' can be run in the 'check' phase?

Fair; would you know how to manually call it in that phase? I'm afraid I don't have any experience outside of the check phase automatically calling the designated tests files.




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

* [bug#54311] [PATCH v2] gnu: tint2: Update to 17.0.2.
  2022-05-19  4:54     ` Jaft
@ 2022-05-19 13:13       ` Maxime Devos
  2022-05-19 15:57         ` Jaft
       [not found]         ` <1643733361.385880.1652975850912@mail.yahoo.com>
  0 siblings, 2 replies; 8+ messages in thread
From: Maxime Devos @ 2022-05-19 13:13 UTC (permalink / raw)
  To: Jaft, 54311@debbugs.gnu.org

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

Jaft schreef op do 19-05-2022 om 04:54 [+0000]:
> > 
> > 
> > There's 'regression.py' which has a 'main' and 'run_unit_tests'. 
> > Maybe
> > 'regression.py' can be run in the 'check' phase?
> 
> Fair; would you know how to manually call it in that phase? I'm
> afraid I don't have any experience outside of the check phase
> automatically calling the designated tests files.

(lambda* (#:key tests? #:allow-other-keys)
  (when tests?
    (invoke "regression.py"))) ; or maybe (invoke "python3" "regression.py")

[-- Attachment #2: This is a digitally signed message part --]
[-- Type: application/pgp-signature, Size: 260 bytes --]

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

* [bug#54311] [PATCH v2] gnu: tint2: Update to 17.0.2.
  2022-05-19 13:13       ` Maxime Devos
@ 2022-05-19 15:57         ` Jaft
       [not found]         ` <1643733361.385880.1652975850912@mail.yahoo.com>
  1 sibling, 0 replies; 8+ messages in thread
From: Jaft @ 2022-05-19 15:57 UTC (permalink / raw)
  To: 54311@debbugs.gnu.org, Maxime Devos

> On Thursday, May 19, 2022, 08:14:11 AM CDT, Maxime Devos <maximedevos@telenet.be> wrote: 
>
>
>
>
>
> (lambda* (#:key tests? #:allow-other-keys)
>   (when tests?
>     (invoke "regression.py"))) ; or maybe (invoke "python3" "regression.py")

Awesome; thanks a ton, Maxime. I'll try this out.




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

* [bug#54311] [PATCH v2] gnu: tint2: Update to 17.0.2.
       [not found]         ` <1643733361.385880.1652975850912@mail.yahoo.com>
@ 2022-05-22  7:49           ` Jaft
  2022-08-04  2:17             ` bug#54311: [PATCH] " 宋文武 via Guix-patches via
  0 siblings, 1 reply; 8+ messages in thread
From: Jaft @ 2022-05-22  7:49 UTC (permalink / raw)
  To: 54311@debbugs.gnu.org, Maxime Devos

Alright; finally got a chance to try this.

Running the file ends up failing due to line 384; it tries to call a =top= command (~top -bn5 | grep 'Cpu(s)' | grep -o '[0-9\.]* id' | cut -d ' ' -f 1~) but it's not working since "Cpu(s)" isn't anywhere in the ~top -bn5~ output. The =regression.py= file was written for Python 2 so maybe the command was written for an older version of =top=? But I dunno.




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

* bug#54311: [PATCH] gnu: tint2: Update to 17.0.2.
  2022-05-22  7:49           ` Jaft
@ 2022-08-04  2:17             ` 宋文武 via Guix-patches via
  0 siblings, 0 replies; 8+ messages in thread
From: 宋文武 via Guix-patches via @ 2022-08-04  2:17 UTC (permalink / raw)
  To: Jaft; +Cc: Maxime Devos, 54311@debbugs.gnu.org

Jaft <jaft.r@outlook.com> writes:

> Alright; finally got a chance to try this.
>
> Running the file ends up failing due to line 384; it tries to call a
> =top= command (~top -bn5 | grep 'Cpu(s)' | grep -o '[0-9\.]* id' | cut
> -d ' ' -f 1~) but it's not working since "Cpu(s)" isn't anywhere in
> the ~top -bn5~ output. The =regression.py= file was written for Python
> 2 so maybe the command was written for an older version of =top=? But
> I dunno.

Thanks for investigating, pushed now!




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

end of thread, other threads:[~2022-08-04  2:18 UTC | newest]

Thread overview: 8+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
     [not found] <23>
2022-04-03  3:01 ` [bug#54311] [PATCH v2] gnu: tint2: Update to 17.0.2 Wamm K. D
2022-05-10  2:05   ` Jaft
2022-05-11  9:51   ` Maxime Devos
2022-05-19  4:54     ` Jaft
2022-05-19 13:13       ` Maxime Devos
2022-05-19 15:57         ` Jaft
     [not found]         ` <1643733361.385880.1652975850912@mail.yahoo.com>
2022-05-22  7:49           ` Jaft
2022-08-04  2:17             ` bug#54311: [PATCH] " 宋文武 via Guix-patches via

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.