Skip to content
This repository has been archived by the owner on Jul 5, 2018. It is now read-only.

Bump dmd to 2.074.0 #3

Merged
merged 2 commits into from
May 26, 2017
Merged

Bump dmd to 2.074.0 #3

merged 2 commits into from
May 26, 2017

Conversation

wilzbach
Copy link
Member

Bumps to 2.073.0 - this is interesting because e.g. compiling ~/dlang/phobos/std/algorithm/comparison.d took 1.9s on 2.071.0, whereas with 2.073.0 it takes only 0.8s

This is based on the fact that before 2.072 rdmd used to compile the main file twice, see e.g.

dlang/tools#194
dlang/tools#207

@wilzbach wilzbach closed this May 23, 2017
@wilzbach wilzbach reopened this May 23, 2017
@wilzbach wilzbach changed the title Bump dmd to 2.073.0 Bump dmd to 2.074.0 May 23, 2017
@wilzbach
Copy link
Member Author

Hmm seems like this needs more adjustment:

image

@wilzbach
Copy link
Member Author

I tried to configure a new Docker container that uses GH push events to rebuild the container, s.t. credentials don't need to be exposed here.
However, it seems that this takes a long time:

https://cloud.docker.com/swarm/dlangtour/repository/docker/dlangtour/dlang-tour-rdmd/general

@wilzbach
Copy link
Member Author

However, it seems that this takes a long time:

That's why I reverted the change and simply moved the deployment scripts to the proper "deployment" section in the travis.yml

@wilzbach
Copy link
Member Author

CC @ZombineDev

What do you think? Should I press the "magic" button, upgrade our docker image to 2.074 and re-trigger the main Travis CI build, s.t. broken examples can be found?

@PetarKirov
Copy link
Member

Yes, please do

@wilzbach wilzbach merged commit 16d5f34 into dlang-tour:master May 26, 2017
@wilzbach wilzbach deleted the bump-dmd branch May 26, 2017 11:05
@wilzbach
Copy link
Member Author

upgrade our docker image to 2.074 and re-trigger the main Travis CI build, s.t. broken examples can be found?

I just realized that there should be zero breakage, because in Travis CI we already test with the latest greatest DMD ;-)
Let's just hope that this gets properly deployed ...

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants