New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Update and improve Mono #23295
Update and improve Mono #23295
Conversation
, withCUPS ? true, cups | ||
, withALSA ? true, alsaLib | ||
, withLLVM ? true | ||
}: |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
The three options withCUPS
, withALSA
and withLLVM
should default to false
on Darwin (i.e., using stdenv.isLinux
or !stdenv.isDarwin
).
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Updated, thanks!
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
No. withLLVM
should be false on darwin. withX11
can be true.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I see, thank you for double-checking. Fixed, also made it so that GTK and friends aren't used on Darwin -- does that sound right?
b32da29
to
632132d
Compare
This breaks
This probably comes from an API change in the |
The patch was upstreamed, I removed it from the tree. @vbgl I'll look into this, thanks! Sadly I don't have resources to fuly run |
Bundled LLVM: 3.4svn-mono-f9b1a74368 -> 3.6-mono-2017-02-15 Remove old Mono versions. Add options to enable GTK, SVG and printing support. Build reference assemblies from source (essentially Mono 3.12.1 stripped from actual method bodies). Add headless version. Should improve support of color schemas in Windows Forms.
Sorry for a long response -- I've tried to look at it back now and then but I don't have a good idea how can I fix this. I think I understand what needs to be fixed -- we need to use NUnit 2 for Boogie instead of Mono's unit. Still, placing |
I just submitted 2 pull requests on introducing mono 4.6.2 and 4.8(.1) not knowing that there is this pull request already on improving mono: Should we consider joining the effort? I planned to submit |
@kuznero With pleasure! I'm not sure how to fix the problem with Boogie. BTW, are old Mono versions (like 4.6.2) needed? They are unused in the tree but perhaps they are useful for users in some way I don't know? |
@abbradar 4.6.2 is probably the most widely used version of the runtime as far as I know. So, supporting it would make sense in general for others to use I guess. And yes, it is done as new packages, so nothing depends on it at the moment and it should go pretty painless. Unfortunately I see some of the builds failing (4.6.2 and 5.0), but 4.8 is green. |
@kuznero Do I understand correctly that Mono is incompatible between versions (because I supposed otherwise)? Then yes, let's leave the most popular runtime be. |
@kuznero would you still support killing 4.0 and 4.4? |
@copumpkin I would keep 4.0 but discontinue 4.4 |
Any progress here? Also, anyone have plans to package 5.4? |
I'm not motivated enough to rebase this for now; this is still definitely relevant but needs moderate work. |
This looks related to #36711 |
Some parts of this are superseded by #52712 but there are some cool things in here such as:
|
Also now builds with --enable-llvmloaded. Extracted by @obadz from @abbradar's NixOS#23295 NixOS@d27768a
I think this PR may be closed. Most of the features in this PR are on master. |
Please rebase your changes. Otherwise, I'd suggest closing the PR. |
Motivation for this change
Things done
(nix.useSandbox on NixOS,
or option
build-use-sandbox
innix.conf
on non-NixOS)
nix-shell -p nox --run "nox-review wip"
./result/bin/
)I'll try to upstream
use-dllmap-everywhere.patch
. See mono/mono#4451.