From: pukkamustard <pukkamustard@posteo.net>
To: 49868@debbugs.gnu.org
Cc: pukkamustard <pukkamustard@posteo.net>
Subject: [bug#49868] [PATCH 0/2] guix: dune-build-system: Add a profile parameter.
Date: Wed, 4 Aug 2021 08:04:23 +0000 [thread overview]
Message-ID: <20210804080423.3533-1-pukkamustard@posteo.net> (raw)
Hi,
This adds a profile parameter to the dune-build-system and sets it to "release" by default.
The exact effects of the "release" profile are not clearly documented in dune [1]. But it seems to be the default when installing packages with OPAM [2]. I think it should also be the default in Guix.
Some OCaml packages had set the profile to "release" manually by specifiying it in the build flags. This has been removed.
This patch will probably require a rebuild of almost all OCaml packages.
A second patch is attached that removes unnecessary arguments from the ocaml4.07-base packages. This is a more intrusive change to the package, so I've kept it in a separate commit.
-pukkamustard
[1] https://github.com/ocaml/dune/issues/3359
[2] https://dune.readthedocs.io/en/stable/faq.html#how-to-make-warnings-non-fatal
pukkamustard (2):
guix: dune-build-system: Add a profile parameter.
gnu: ocaml4.07-base: Remove unnecessary arguments.
doc/guix.texi | 7 +++++++
gnu/packages/ocaml.scm | 27 ++++++---------------------
guix/build-system/dune.scm | 3 +++
guix/build/dune-build-system.scm | 8 ++++++--
4 files changed, 22 insertions(+), 23 deletions(-)
--
2.32.0
next reply other threads:[~2021-08-04 8:05 UTC|newest]
Thread overview: 19+ messages / expand[flat|nested] mbox.gz Atom feed top
2021-08-04 8:04 pukkamustard [this message]
2021-08-04 8:08 ` [bug#49868] [PATCH 1/2] guix: dune-build-system: Add a profile parameter pukkamustard
2021-08-04 8:08 ` [bug#49868] [PATCH 2/2] gnu: ocaml4.07-base: Remove unnecessary arguments pukkamustard
2021-08-09 2:17 ` [bug#49868] [PATCH 0/2] guix: dune-build-system: Add a profile parameter Julien Lepiller
2021-08-09 7:17 ` pukkamustard
2021-08-09 7:19 ` [bug#49868] [PATCH v2 1/2] " pukkamustard
2021-08-09 7:19 ` [bug#49868] [PATCH v2 2/2] gnu: ocaml4.07-base: Remove unnecessary arguments pukkamustard
2021-08-20 22:09 ` bug#49868: [PATCH 0/2] guix: dune-build-system: Add a profile parameter Julien Lepiller
2021-08-24 6:52 ` [bug#49868] " pukkamustard
2021-08-24 7:13 ` pukkamustard
2021-08-24 7:51 ` pukkamustard
2021-09-07 9:33 ` pukkamustard
2021-09-07 11:06 ` Julien Lepiller
2021-09-07 11:46 ` pukkamustard
2021-09-07 13:03 ` Julien Lepiller
2021-09-07 18:11 ` pukkamustard
2021-09-07 22:03 ` Julien Lepiller
2021-09-08 8:24 ` pukkamustard
2021-09-08 11:45 ` Julien Lepiller
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=20210804080423.3533-1-pukkamustard@posteo.net \
--to=pukkamustard@posteo.net \
--cc=49868@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 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).