-
-
Notifications
You must be signed in to change notification settings - Fork 15.5k
dmd: 2.085.1 -> 2.091.1 #82694
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
dmd: 2.085.1 -> 2.091.1 #82694
Conversation
@adisbladis @ThomasMader Please have a look. |
WIP. got a test failure here.. |
Wrt. LDC, I strongly suggest you look for alternatives to bootstrapping with v0.17: ldc-developers/ldc#3354 |
82fd1f1
to
41d23a1
Compare
41d23a1
to
3a87d97
Compare
This is OK now @adisbladis |
@GrahamcOfBorg build ldc dmd |
|
Great work @lionello, thanks. Would you mind changing the bootstrap of ldc? |
@ThomasMader Good idea. |
@lionello Why do you want to build dmdBootstrap explicitly? Isn't it enough that it builds with normal dmd? |
Doh. :) Good point. |
@ThomasMader Please have a look here: #85488 |
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.
Thanks a lot for this!
I tried it and it works for me. At least on Darwin. Haven't tried it on Linux/NixOS though. |
@@ -103,7 +108,7 @@ stdenv.mkDerivation rec { | |||
|
|||
checkPhase = '' | |||
cd dmd | |||
make -j$NIX_BUILD_CORES -C test -f Makefile PIC=1 CC=$CXX DMD=${pathToDmd} BUILD=release SHELL=$SHELL | |||
make -j$NIX_BUILD_CORES -C test -f Makefile PIC=1 DMD=${pathToDmd} BUILD=release SHELL=$SHELL |
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.
make -j$NIX_BUILD_CORES -C test -f Makefile PIC=1 DMD=${pathToDmd} BUILD=release SHELL=$SHELL | |
make -j$NIX_BUILD_CORES -C test -f Makefile PIC=1 HOST_CXX=${CXX} DMD=${pathToDmd} BUILD=release SHELL=$SHELL |
IIUC
BTW 2.091.1 is out now |
To avoid breakage on master I reverted the merge of the ldc update #85768. Can we revert that revert here? Ref #85488 (comment) |
:( |
Try to add dlang/dmd@4157298 before the patch. |
Hey @lionello @ThomasMader @worldofpeace, what's left to be done? |
@GrahamcOfBorg build dmd |
Latest patch added a path reference to |
@GrahamcOfBorg build dmd |
Done. |
@worldofpeace Passed. |
@worldofpeace I made the change you requested. Can you have a (final?) look? |
Apply patch dlang/dmd#11053
This reverts commit 31d630c.
@worldofpeace Done! |
Wooohoo! Thanks all! |
Motivation for this change
Current
dmd
in Nixpkgs usesldc
0.17.6 to bootstrapldc
1.x which will finally bootstrapdmd
. This was done since the official dmd binaries where not redistributable.This changed and as of 2.075.0 dmd binaries (and sources) are all using the Boost license. This simplifies bootstrapping so we can use the v-1 binaries to build the latest dmd sources.
Things done
sandbox
innix.conf
on non-NixOS linux)nix-shell -p nixpkgs-review --run "nixpkgs-review wip"
./result/bin/
)nix path-info -S
before and after)