-
Notifications
You must be signed in to change notification settings - Fork 2
feat(pkgs/dmd): 2.110.0 and 2.111.0 #173
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
base: main
Are you sure you want to change the base?
Conversation
Solid work as always, thanks @dukc!
Initially I thought this was a fluke, but I reran the build and got the exact same error. I tried to reproduce this on my local
I thought this caused by the TLS bug you included a patch for, so I tried change the bootstrap compiler to --- a/pkgs/dmd/supported-source-versions.json
+++ b/pkgs/dmd/supported-source-versions.json
@@ -95,6 +95,7 @@
"2.111.0": {
"dmd": "sha256-2bjsGa9nTny818dLRV6LKkes+Ycq8W+PFjwhGVrD5TM=",
"phobos": "sha256-9jySZbODoL8sULhYKj/l4+c/uiG5xY2dO240k9XQtPk=",
- "tools": "sha256-xMEHnrstL5hAkhp8+/z1I2KZWZ7eztWZnUGLTKCfbBI="
+ "tools": "sha256-xMEHnrstL5hAkhp8+/z1I2KZWZ7eztWZnUGLTKCfbBI=",
+ "host-d-compiler": "dmd-2_110_0"
}
} but then I got a different error:
Which I wasn't able to figure out. Since |
@PetarKirov yeah, I looked at the error and tried to find a relevant PR or issue, but found none. If I had a Mac myself I could try manually building 2.111.0 and running getopt unittests, but since I don't I'm not sure how I should pinpoint the problem. So yeah, I'm going to just disable Mac test for 2.111.0 for now.
Assuming you're using x86 compatibility mode for your ARM Mac. If the TLS issue is the problem, your mac should not build the other DMD versions we have either, since they use the same compiler for bootstrapping (except 2.088.1 and 2.087.1). Is this the case? It'd be strange, since I think only 1.28 and newer were affected by the TLS issue. We're using 1.25.0. |
Thanks for your Pull Request! Below you will find a summary of the cachix status of each package, for each supported platform.
|
Fixes issue #142 .
Ping @PetarKirov .