Subject: bug#57164: closed (Re: bug#57164: [PATCH 1/3] gnu: Add lttng-modules.)

From: [email-masked] (GNU bug Tracking System)
Date: Mon, 15 Aug 2022 07:56:02 +0000

Your bug report

#57164: [PATCH 1/3] gnu: Add lttng-modules.

which was filed against the guix-patches package, has been closed.

The explanation is attached below, along with your original report.
If you require more details, please reply to [email-masked].

-- 
57164: https://debbugs.gnu.org/cgi/bugreport.cgi?bug=57164
GNU Bug Tracking System
Contact [email-masked] with problems

Subject: Re: bug#57164: [PATCH 1/3] gnu: Add lttng-modules.
From: Mathieu Othacehe <[email-masked]>
To: Olivier Dion <[email-masked]>
Cc: [email-masked]
Date: Mon, 15 Aug 2022 09:55:38 +0200
List-Id: <debbugs-submit.debbugs.gnu.org>

> * gnu/packages/instrumentation.scm (lttng-modules): New variable.

Pushed with the following changes:

--8<---------------cut here---------------start------------->8---
--- a/gnu/packages/instrumentation.scm
+++ b/gnu/packages/instrumentation.scm
@@ -1,5 +1,5 @@
 ;;; GNU Guix --- Functional package management for GNU
-;;; Copyright © 2021 Olivier Dion <[email-masked]>
+;;; Copyright © 2021, 2022 Olivier Dion <[email-masked]>
 ;;;
 ;;; This file is part of GNU Guix.
 ;;;
@@ -220,11 +220,11 @@ (define-public lttng-modules
                 "1vm9nnjvid7acsvgwnjyxd60ih9rmbhnfjldxip58n8x9q7d0nb1"))))
     (build-system linux-module-build-system)
     (arguments
-     `(#:tests? #f
-       #:make-flags (list "CONFIG_LTTNG=m" "CONFIG_LTTNG_CLOCK_PLUGIN_TEST=m")))
+     `(#:tests? #f ; no tests
+       #:make-flags (list "CONFIG_LTTNG=m"
+                          "CONFIG_LTTNG_CLOCK_PLUGIN_TEST=m")))
     (home-page "https://lttng.org/")
-    (synopsis "LTTng modules provide kernel tracing capability to the LTTng
-tracer toolset")
+    (synopsis "LTTng kernel modules for the LTTng tracer toolset")
     (description
      "LTTng kernel modules are Linux kernel modules which make
 LTTng kernel tracing possible.  They include essential control modules and
--8<---------------cut here---------------end--------------->8---

Thanks,

Mathieu

Subject: [PATCH 1/3] gnu: Add lttng-modules.
From: Olivier Dion <[email-masked]>
To: [email-masked]
Cc: Olivier Dion <[email-masked]>
Date: Fri, 12 Aug 2022 18:38:09 -0400
List-Id: <debbugs-submit.debbugs.gnu.org>

* gnu/packages/instrumentation.scm (lttng-modules): New variable.
---
 gnu/packages/instrumentation.scm | 25 +++++++++++++++++++++++++
 1 file changed, 25 insertions(+)

diff --git a/gnu/packages/instrumentation.scm b/gnu/packages/instrumentation.scm
index f52cf11505..8b03c285dc 100644
--- a/gnu/packages/instrumentation.scm
+++ b/gnu/packages/instrumentation.scm
@@ -50,6 +50,7 @@ (define-module (gnu packages instrumentation)
   #:use-module (guix build-system cmake)
   #:use-module (guix build-system copy)
   #:use-module (guix build-system gnu)
+  #:use-module (guix build-system linux-module)
   #:use-module (guix download)
   #:use-module (guix gexp)
   #:use-module (guix git-download)
@@ -206,6 +207,30 @@ (define-public flamegraph
 with the script @command{flamegraph.pl} and many stackcollapse scripts.")
       (license license:cddl1.0))))
 
+(define-public lttng-modules
+  (package
+    (name "lttng-modules")
+    (version "2.13.4")
+    (source (origin
+              (method url-fetch)
+              (uri (string-append "https://lttng.org/files/lttng-modules/"
+                                  "lttng-modules-" version ".tar.bz2"))
+              (sha256
+               (base32
+                "1vm9nnjvid7acsvgwnjyxd60ih9rmbhnfjldxip58n8x9q7d0nb1"))))
+    (build-system linux-module-build-system)
+    (arguments
+     `(#:phases (modify-phases %standard-phases
+                  (delete 'check))))
+    (home-page "https://lttng.org/")
+    (synopsis "LTTng modules provide kernel tracing capability to the LTTng
+tracer toolset")
+    (description
+     "LTTng kernel modules are Linux kernel modules which make
+LTTng kernel tracing possible.  They include essential control modules and
+many probes which instrument numerous interesting parts of Linux.")
+    (license (list license:lgpl2.1 license:gpl2 license:expat))))
+
 (define-public lttng-ust
   (package
     (name "lttng-ust")
-- 
2.37.1