From: ludo@gnu.org (Ludovic Courtès)
To: Deck Pickard <deck.r.pickard@gmail.com>
Cc: guix-devel@gnu.org
Subject: Re: [PATCH] Core sanity and taking build options from environment.
Date: Sun, 30 Nov 2014 18:55:46 +0100 [thread overview]
Message-ID: <871tokpo7x.fsf@gnu.org> (raw)
In-Reply-To: <CAJ41eezEtwQM6FKJYnt=Lk1APHmHor9OFX8B6NtqN5sHQ8oyxw@mail.gmail.com> (Deck Pickard's message of "Sun, 30 Nov 2014 18:17:36 +0100")
Deck Pickard <deck.r.pickard@gmail.com> skribis:
> On 28 Nov 2014 22:52, "Ludovic Courtès" <ludo@gnu.org> wrote:
>>
>> Deck Pickard <deck.r.pickard@gmail.com> skribis:
>>
>> > From 3693753aefc27b5a68a2b762feeebc41320e79ef Mon Sep 17 00:00:00 2001
>> > From: nebuli <nebu@kipple>
>> > Date: Wed, 26 Nov 2014 19:51:37 +0100
>> > Subject: [PATCH 1/2] guix: Default to daemon's default --cores setting
> of 1.
>>
>> I think most of the time one would prefer to use all the available cores
>> when building. WDYT?
>>
>
> Perhaps for one-off 15 minutes worth of compiles it's a OK,
I’ve always used as many cores as available, and it’s definitely
been OK, FWIW.
> but not for hours of work. And you are creating more confusion by
> having different defaults from guix-daemon.
Right, I’ve changed guix-daemon’s default to 0, which at least will
avoid confusion.
> From a4ec02e09f2bc1d4ad33e82ac9052439bdece6d0 Mon Sep 17 00:00:00 2001
> From: nebuli <nebu@kipple>
> Date: Sun, 30 Nov 2014 17:47:22 +0100
> Subject: [PATCH] guix: scripts: Add GUIX_BUILD_OPTIONS environment handling.
>
> * doc/guix.texi: Mention in the 'Invoking guix build' section.
> * guix/scripts/archive.scm: (append args (environment-build-options)
> * guix/scripts/build.scm: Ditto.
> * guix/scripts/environment.scm: Ditto.
> * guix/scripts/package.scm: Ditto.
> * guix/scripts/system.scm: Ditto.
> * guix/ui.scm (environment-build-options): New function.
> * guix/utils.scm (arguments-from-environment-variable): New function.
I tweaked guix.texi, updated test-env.in, added a test, and pushed it.
Thanks!
Ludo’.
prev parent reply other threads:[~2014-11-30 17:56 UTC|newest]
Thread overview: 5+ messages / expand[flat|nested] mbox.gz Atom feed top
[not found] <CAJ41eezW0KRVu6TBE0UeUVnNgqFSxgwML5p+FDd5h_bH3N_ghg@mail.gmail.com>
2014-11-27 23:13 ` [PATCH] Core sanity and taking build options from environment Deck Pickard
2014-11-28 21:52 ` Ludovic Courtès
2014-11-28 22:14 ` Andreas Enge
2014-11-30 17:17 ` Deck Pickard
2014-11-30 17:55 ` Ludovic Courtès [this message]
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=871tokpo7x.fsf@gnu.org \
--to=ludo@gnu.org \
--cc=deck.r.pickard@gmail.com \
--cc=guix-devel@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 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).