unofficial mirror of bug-guix@gnu.org 
 help / color / mirror / code / Atom feed
* bug#37882: linux-module-build-system doesn't accept make-flags
@ 2019-10-23  8:06 Efraim Flashner
  2019-10-25 19:37 ` bug#37882: [PATCH] build-system/linux-module: Accept a #:make-flags keyword Tobias Geerinckx-Rice via Bug reports for GNU Guix
  0 siblings, 1 reply; 5+ messages in thread
From: Efraim Flashner @ 2019-10-23  8:06 UTC (permalink / raw)
  To: 37882

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

In (guix build linux-module-build-system) the build phase looks for
make-flags, but 'make-flags' doesn't exist in (guix build-system
linux-module), making it not really supported.

-- 
Efraim Flashner   <efraim@flashner.co.il>   אפרים פלשנר
GPG key = A28B F40C 3E55 1372 662D  14F7 41AA E7DC CA3D 8351
Confidentiality cannot be guaranteed on emails sent or received unencrypted

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

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

* bug#37882: [PATCH] build-system/linux-module: Accept a #:make-flags keyword.
  2019-10-23  8:06 bug#37882: linux-module-build-system doesn't accept make-flags Efraim Flashner
@ 2019-10-25 19:37 ` Tobias Geerinckx-Rice via Bug reports for GNU Guix
  2019-10-27  8:18   ` Efraim Flashner
  2019-12-04  9:28   ` Efraim Flashner
  0 siblings, 2 replies; 5+ messages in thread
From: Tobias Geerinckx-Rice via Bug reports for GNU Guix @ 2019-10-25 19:37 UTC (permalink / raw)
  To: 37882

* guix/build-system/linux-module.scm (linux-module-build): Accept a
MAKE-FLAGS argument.
<builder>: Pass it on.
---

Efraim,

Is this in any way related to the topic coming up on IRC last week, or just a coincidence?

In any case, I'd written the following simple fix but never merged it because I have yet to meet a module with a legitimate need for it.  It works, though.

Kind regards,

T G-R

 guix/build-system/linux-module.scm | 3 +++
 1 file changed, 3 insertions(+)

diff --git a/guix/build-system/linux-module.scm b/guix/build-system/linux-module.scm
index 6084d22210..dde2423434 100644
--- a/guix/build-system/linux-module.scm
+++ b/guix/build-system/linux-module.scm
@@ -1,5 +1,6 @@
 ;;; GNU Guix --- Functional package management for GNU
 ;;; Copyright © 2019 Danny Milosavljevic <dannym@scratchpost.org>
+;;; Copyright © 2019 Tobias Geerinckx-Rice <me@tobias.gr>
 ;;;
 ;;; This file is part of GNU Guix.
 ;;;
@@ -120,6 +121,7 @@
 (define* (linux-module-build store name inputs
                              #:key
                              (search-paths '())
+                             (make-flags '())
                              (tests? #t)
                              (phases '(@ (guix build linux-module-build-system)
                                          %standard-phases))
@@ -146,6 +148,7 @@
                                            search-paths)
                      #:phases ,phases
                      #:system ,system
+                     #:make-flags ,make-flags
                      #:tests? ,tests?
                      #:outputs %outputs
                      #:inputs %build-inputs)))
-- 
2.23.0

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

* bug#37882: [PATCH] build-system/linux-module: Accept a #:make-flags keyword.
  2019-10-25 19:37 ` bug#37882: [PATCH] build-system/linux-module: Accept a #:make-flags keyword Tobias Geerinckx-Rice via Bug reports for GNU Guix
@ 2019-10-27  8:18   ` Efraim Flashner
  2019-11-04  6:26     ` Danny Milosavljevic
  2019-12-04  9:28   ` Efraim Flashner
  1 sibling, 1 reply; 5+ messages in thread
From: Efraim Flashner @ 2019-10-27  8:18 UTC (permalink / raw)
  To: Tobias Geerinckx-Rice; +Cc: 37882

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

On Fri, Oct 25, 2019 at 09:37:31PM +0200, Tobias Geerinckx-Rice via Bug reports for GNU Guix wrote:
> * guix/build-system/linux-module.scm (linux-module-build): Accept a
> MAKE-FLAGS argument.
> <builder>: Pass it on.
> ---
> 
> Efraim,
> 
> Is this in any way related to the topic coming up on IRC last week, or just a coincidence?
> 
> In any case, I'd written the following simple fix but never merged it because I have yet to meet a module with a legitimate need for it.  It works, though.
> 

I thought I had a use-case for it but it turns out I'm pretty sure I
didn't need it in the end. It was more of surprise that it was in one
place and not another.

-- 
Efraim Flashner   <efraim@flashner.co.il>   אפרים פלשנר
GPG key = A28B F40C 3E55 1372 662D  14F7 41AA E7DC CA3D 8351
Confidentiality cannot be guaranteed on emails sent or received unencrypted

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

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

* bug#37882: [PATCH] build-system/linux-module: Accept a #:make-flags keyword.
  2019-10-27  8:18   ` Efraim Flashner
@ 2019-11-04  6:26     ` Danny Milosavljevic
  0 siblings, 0 replies; 5+ messages in thread
From: Danny Milosavljevic @ 2019-11-04  6:26 UTC (permalink / raw)
  To: Efraim Flashner; +Cc: 37882

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

On Sun, 27 Oct 2019 10:18:06 +0200
Efraim Flashner <efraim@flashner.co.il> wrote:

> > In any case, I'd written the following simple fix but never merged it because I have yet to meet a module with a legitimate need for it.  It works, though.
> >   
> It was more of surprise that it was in one
> place and not another.

Whoops.  LGTM anyway!

[-- Attachment #2: OpenPGP digital signature --]
[-- Type: application/pgp-signature, Size: 484 bytes --]

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

* bug#37882: [PATCH] build-system/linux-module: Accept a #:make-flags keyword.
  2019-10-25 19:37 ` bug#37882: [PATCH] build-system/linux-module: Accept a #:make-flags keyword Tobias Geerinckx-Rice via Bug reports for GNU Guix
  2019-10-27  8:18   ` Efraim Flashner
@ 2019-12-04  9:28   ` Efraim Flashner
  1 sibling, 0 replies; 5+ messages in thread
From: Efraim Flashner @ 2019-12-04  9:28 UTC (permalink / raw)
  To: Tobias Geerinckx-Rice; +Cc: 37882-done

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

I pushed this patch

-- 
Efraim Flashner   <efraim@flashner.co.il>   אפרים פלשנר
GPG key = A28B F40C 3E55 1372 662D  14F7 41AA E7DC CA3D 8351
Confidentiality cannot be guaranteed on emails sent or received unencrypted

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

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

end of thread, other threads:[~2019-12-04  9:29 UTC | newest]

Thread overview: 5+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2019-10-23  8:06 bug#37882: linux-module-build-system doesn't accept make-flags Efraim Flashner
2019-10-25 19:37 ` bug#37882: [PATCH] build-system/linux-module: Accept a #:make-flags keyword Tobias Geerinckx-Rice via Bug reports for GNU Guix
2019-10-27  8:18   ` Efraim Flashner
2019-11-04  6:26     ` Danny Milosavljevic
2019-12-04  9:28   ` Efraim Flashner

Code repositories for project(s) associated with this public inbox

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

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for read-only IMAP folder(s) and NNTP newsgroup(s).