unofficial mirror of guix-patches@gnu.org 
 help / color / mirror / code / Atom feed
From: "Ludovic Courtès" <ludo@gnu.org>
To: Pierre Neidhardt <mail@ambrevar.xyz>
Cc: 34758@debbugs.gnu.org
Subject: [bug#34758] [PATCH 2/4] gnu: Add vhba.
Date: Sat, 23 Mar 2019 15:52:03 +0100	[thread overview]
Message-ID: <87va09zlrg.fsf@gnu.org> (raw)
In-Reply-To: <87d0mid2l3.fsf@mimimi.i-did-not-set--mail-host-address--so-tickle-me> (Pierre Neidhardt's message of "Fri, 22 Mar 2019 22:24:56 +0100")

Hello,

Pierre Neidhardt <mail@ambrevar.xyz> skribis:

> I should have explained a little bit more: the whole patchset makes no
> sense (beside libmirage) without this kernel module.  CDEmu needs it or
> it won't run.

Oh I see, I didn’t know that.

> This raises a general, important question: how do we build kernel
> modules on guix?  At the moment, it seems that it's only possible by
> re-building the kernel.

… or maybe just unpacking the kernel source so you have the makefile
snippet around?  Perhaps that’s enough?

I’m generally dubious about out-of-tree modules.  Given the way kernel
development works, we’re often talking about modules of poor quality, or
proprietary modules.  Do you know the reason this one is not in the main
tree?

> I suggest we add a "build" output to the kernel  that can be used as an
> input to build kernel modules.  Maybe I should should an email to
> guix-devel to ask for more opinions / advice. 

Yes, sounds like a good idea.

Thanks,
Ludo’.

  parent reply	other threads:[~2019-03-23 14:59 UTC|newest]

Thread overview: 15+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2019-03-05 17:26 [bug#34758] [PATCH 1/4] gnu: Add libmirage Pierre Neidhardt
2019-03-05 17:28 ` [bug#34758] [PATCH 2/4] gnu: Add vhba Pierre Neidhardt
2019-03-05 17:28   ` [bug#34758] [PATCH 3/4] gnu: Add cdemu-daemon Pierre Neidhardt
2019-03-22 21:13     ` Ludovic Courtès
2019-03-05 17:28   ` [bug#34758] [PATCH 4/4] gnu: Add cdemu-client Pierre Neidhardt
2019-03-22 21:14     ` Ludovic Courtès
2019-03-22 21:25       ` Pierre Neidhardt
2019-03-22 21:12   ` [bug#34758] [PATCH 2/4] gnu: Add vhba Ludovic Courtès
2019-03-22 21:24     ` Pierre Neidhardt
2019-03-22 21:27       ` Pierre Neidhardt
2019-03-23 14:52       ` Ludovic Courtès [this message]
2019-03-25  8:35         ` Pierre Neidhardt
2019-03-25 21:10           ` Ludovic Courtès
     [not found] ` <handler.34758.B.15518068039570.ack@debbugs.gnu.org>
2019-03-05 17:44   ` [bug#34758] Acknowledgement ([PATCH 1/4] gnu: Add libmirage.) Pierre Neidhardt
2019-03-22 21:10 ` [bug#34758] [PATCH 1/4] gnu: Add libmirage Ludovic Courtès

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

  List information: https://guix.gnu.org/

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

  git send-email \
    --in-reply-to=87va09zlrg.fsf@gnu.org \
    --to=ludo@gnu.org \
    --cc=34758@debbugs.gnu.org \
    --cc=mail@ambrevar.xyz \
    /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 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).