all messages for Guix-related lists mirrored at yhetil.org
 help / color / mirror / code / Atom feed
From: Jack Hill <jackhill@jackhill.us>
To: 55253@debbugs.gnu.org
Subject: [bug#55253] [PATCH v2 1/2] gnu: nginx-rtmp-module: Use #:configure-flags from nginx.
Date: Thu, 26 May 2022 22:04:03 -0400	[thread overview]
Message-ID: <20220527020404.6603-1-jackhill@jackhill.us> (raw)
In-Reply-To: <alpine.DEB.2.21.2205262201320.11587@marsh.hcoop.net>

* gnu/packages/web.scm (nginx-rtmp-module)[arguments]: Simplify copying
of arguments from nginx and augment #:configure-flags from nginx
rather than overwriting.
---
 gnu/packages/web.scm | 9 ++++-----
 1 file changed, 4 insertions(+), 5 deletions(-)

diff --git a/gnu/packages/web.scm b/gnu/packages/web.scm
index ffcb709e82..e349a863d5 100644
--- a/gnu/packages/web.scm
+++ b/gnu/packages/web.scm
@@ -793,11 +793,10 @@ (define-public nginx-rtmp-module
        ,@(package-inputs nginx)))
     (arguments
      (substitute-keyword-arguments
-         `(#:make-flags '("modules")
-           #:modules ((guix build utils)
-                      (guix build gnu-build-system))
-           ,@(package-arguments nginx)
-           #:configure-flags '("--add-dynamic-module=."))
+         `(#:make-flags '("modules") ;Only build this module not all of nginx.
+           ,@(package-arguments nginx))
+       ((#:configure-flags flags)
+        #~(cons "--add-dynamic-module=." #$flags))
        ((#:phases phases)
         #~(modify-phases #$phases
             (add-after 'unpack 'unpack-nginx-sources
-- 
2.36.1





  reply	other threads:[~2022-05-27  2:06 UTC|newest]

Thread overview: 12+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2022-05-04  4:19 bug#55253: nginx-rtmp-module not compatible with nginx Jack Hill
2022-05-17  5:32 ` Jack Hill
2022-05-22  5:50 ` [bug#55253] [PATCH 0/2] nginx-rtmp-module use nginx's #:configure-flags and update to gexps in phases Jack Hill
2022-05-22  5:51   ` [bug#55253] [PATCH 1/2] gnu: nginx-rtmp-module: Use #:configure-flags from nginx Jack Hill
2022-05-22  5:51     ` [bug#55253] [PATCH 2/2] gnu: nginx-rtmp-module: Use gexps in phases Jack Hill
2022-05-23 12:57     ` [bug#55253] nginx-rtmp-module not compatible with nginx Ludovic Courtès
2022-05-23 14:11       ` Jack Hill
2022-05-27  2:02         ` Jack Hill
2022-05-27  2:04           ` Jack Hill [this message]
2022-05-27  2:04             ` [bug#55253] [PATCH v2 2/2] gnu: nginx-rtmp-module: Use gexps in phases Jack Hill
2022-05-27 13:01               ` Christopher Baines
2022-05-27 21:47                 ` bug#55253: " Jack Hill

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=20220527020404.6603-1-jackhill@jackhill.us \
    --to=jackhill@jackhill.us \
    --cc=55253@debbugs.gnu.org \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
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.