[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [PATCH] gnu: Add mono.
From: |
Leo Famulari |
Subject: |
Re: [PATCH] gnu: Add mono. |
Date: |
Fri, 22 Jul 2016 11:55:56 -0400 |
User-agent: |
Mutt/1.6.0 (2016-04-01) |
On Fri, Jul 15, 2016 at 09:32:43PM +0200, Jan Nieuwenhuizen wrote:
> Subject: [PATCH] gnu: Add mono.
>
> * gnu/packages/mono.scm: New file.
> * gnu/packages/local.mk (GNU_SYSTEM_MODULES): Add it.
Thank you!
I assume this mono is working for you? I'm wondering if the problems
requiring skipped tests hide a larger problem or if it's benign.
Do you know if mono is supposed to work on all of our architectures? Or,
should we disable it on some of them?
> + (snippet
Did you try doing these substitutions in a build phase? The origin
snippet affects what you get from `guix build --source mono`, so we tend
to reserve it for removing non-free components, so that we don't
distribute things that we don't have a right to distribute.
> + ;; tests fail, trying to access $HOME
> + (substitute* "mcs/class/Makefile"
> + (("^include ../build/rules.make" all)
> + (string-append
> + all
> + "\nrun-test-recursive:address@hidden skipping
> tests\n")))
After moving this into a build phase, there could be a prior phase
'set-env' that just sets $HOME to '/tmp'. Can you check if that works?
> + (build-system gnu-build-system)
> + (native-inputs
> + `(("gettext" ,gnu-gettext)
> + ("glib" ,glib)
I notice many uses of glib:bin rather than the full glib output. But,
to be honest, I don't understand the distinction ;)
> + ;; these 3 tests fail
> + `(#:make-flags
> + `(,(string-append "PLATFORM_DISABLED_TESTS="
> + " appdomain-unload.exe"
> + " delegate2.exe"
> + " finally_guard.exe"
> + " remoting4.exe"))
4 tests?
> + #:parallel-tests? #f))
Parallel testing is not supported? Let's add a comment.
> + (home-page "http://mono-project.org/")
s/org/com/
- [PATCH] gnu: Add mono., Jan Nieuwenhuizen, 2016/07/15
- Re: [PATCH] gnu: Add mono., Jan Nieuwenhuizen, 2016/07/15
- Re: [PATCH] gnu: Add mono., Jan Nieuwenhuizen, 2016/07/15
- Re: [PATCH] gnu: Add mono.,
Leo Famulari <=
- Re: [PATCH] gnu: Add mono., Jan Nieuwenhuizen, 2016/07/27
- Re: [PATCH] gnu: Add mono., Danny Milosavljevic, 2016/07/29
- Re: [PATCH] gnu: Add mono., Leo Famulari, 2016/07/29
- Re: [PATCH] gnu: Add mono., Danny Milosavljevic, 2016/07/29
- Re: [PATCH] gnu: Add mono., Leo Famulari, 2016/07/29
- Re: [PATCH] gnu: Add mono., Leo Famulari, 2016/07/30