[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [PATCH 00/10] ocaml patches
From: |
Marius Bakke |
Subject: |
Re: [PATCH 00/10] ocaml patches |
Date: |
Wed, 18 Jan 2017 23:27:06 +0100 |
User-agent: |
Notmuch/0.23.5 (https://notmuchmail.org) Emacs/25.1.1 (x86_64-unknown-linux-gnu) |
Julien Lepiller <address@hidden> writes:
> Here are the next 10 ocaml-related patches I would like to push. What do you
> think?
I don't know anything about ocaml, but the patches reads fine to me.
One thing that strikes me is the usage of native-inputs; I assume it's
because the packages are not referenced, but bear in mind that they are
built for the architecture of the build machine, and not for the target
system.
E.g. `guix build --system=i686-linux ocamlify` will add 64-bit
native-inputs even if the final product is supposed to be 32-bit. Not
sure if that will cause problems or not.
> I'm pretty sure number 9 and 10 can be improved, any idea?
Not from me, I'd say it looks fine :-)
signature.asc
Description: PGP signature
- Re: [PATCH 03/10] gnu: Add omake., (continued)
- [PATCH 06/10] gnu: Add ocaml-expect., Julien Lepiller, 2017/01/18
- [PATCH 05/10] gnu: Add ocaml-pcre., Julien Lepiller, 2017/01/18
- [PATCH 08/10] gnu: Add ocaml-oasis., Julien Lepiller, 2017/01/18
- [PATCH 07/10] gnu: Add ocaml-fileutils., Julien Lepiller, 2017/01/18
- [PATCH 10/10] gnu: Add ocaml-bin-prot., Julien Lepiller, 2017/01/18
- [PATCH 09/10] gnu: Add ocaml-js-build-tools., Julien Lepiller, 2017/01/18
- Re: [PATCH 00/10] ocaml patches,
Marius Bakke <=
- [PATCH 00/10] ocaml patches, Julien Lepiller, 2017/01/31
- [PATCH 02/10] gnu: Add ocaml-ppx-core., Julien Lepiller, 2017/01/31
- [PATCH 01/10] gnu: Add ocaml-fieldslib., Julien Lepiller, 2017/01/31
- [PATCH 03/10] gnu: Add ocaml-ppx-optcomp., Julien Lepiller, 2017/01/31
- [PATCH 04/10] gnu: Add ocaml-ppx-driver., Julien Lepiller, 2017/01/31
- [PATCH 05/10] gnu: Add ocaml-cppo., Julien Lepiller, 2017/01/31
- [PATCH 06/10] gnu: Add ocaml-ppx-deriving., Julien Lepiller, 2017/01/31
- [PATCH 07/10] gnu: Add ocaml-ppx-type-conv., Julien Lepiller, 2017/01/31
- [PATCH 08/10] gnu: Add ocaml-ppx-inline-test., Julien Lepiller, 2017/01/31