Skip to content
This repository was archived by the owner on Jun 20, 2019. It is now read-only.

Conversation

@ibuclaw
Copy link
Member

@ibuclaw ibuclaw commented Aug 6, 2018

Rather than pointing to stable, base our mirror on dmd/master.

There are four outstanding patches applied here waiting to be merged.
See: https://github.com/dlang/dmd/pulls?utf8=%E2%9C%93&q=is%3Aopen+is%3Apr+label%3AGDC

First of such weekly snapshot-style updates.

Needs changelog, and notably druntime and phobos are missing - but they could be added separately.

@ibuclaw
Copy link
Member Author

ibuclaw commented Aug 6, 2018

/home/runner/GDC/gcc/d/dmd/cond.d:109:5: error: declaration expected, not 'do'

Host compiler too old. 😄

@ibuclaw
Copy link
Member Author

ibuclaw commented Aug 6, 2018

I'm going to have to beat some dmd devs round the head and add gdc-7 onto their test system as well as gdc-8.

@wilzbach
Copy link

wilzbach commented Aug 6, 2018

FYI: We already test building DMD with gdc-8 on SemaphoreCI.

@ibuclaw ibuclaw force-pushed the dmd20180806 branch 2 times, most recently from 91e56c5 to 3d5c741 Compare August 6, 2018 21:20
@ibuclaw
Copy link
Member Author

ibuclaw commented Aug 6, 2018

@wilzbach - yeah, that's too new for me. :-)

Do we have capacity to test two versions of each compiler?

@ibuclaw ibuclaw force-pushed the dmd20180806 branch 2 times, most recently from 45c5c83 to ed59520 Compare August 7, 2018 22:52
@belka-ew
Copy link
Contributor

Can we update this to 2.081.2?

@ibuclaw
Copy link
Member Author

ibuclaw commented Aug 13, 2018

Can apply all changes to dmd/master over the last week.

@belka-ew
Copy link
Contributor

It would be great. I could merge this then into gdc-8 again and I hope we are able to fix gdc-7 till then. And then we would have a stable release in gdc-7 and gdc-8 which could remain there for some time.

@ibuclaw
Copy link
Member Author

ibuclaw commented Aug 13, 2018

Ah, I see what you mean. Actually could raise a new PR to sync dmd/stable first as that diff will be no more than a few dozen changes.

Assuming stable has been sync'd back to master in upstream, that should be fine.

@belka-ew
Copy link
Contributor

Minor releases are in the stable branch, how I could mix it up :)
I think it is a good strategy to try: master follows gcc and dmd master; and gdc-7/gdc-8 branches follow upstream stable and are a base for new releases.

@ibuclaw
Copy link
Member Author

ibuclaw commented Aug 13, 2018

Yeah, that's the direction I'd like to go in. It means that we'll have small weekly updates here. Then once every two months do a merge from master to release branches.

Minor frontend release updates get applied directly to the current gcc release branch.

@wilzbach
Copy link

Do we have capacity to test two versions of each compiler?

I think so: dlang/dmd#8565

If it gets too slow, we have to do the Buildkite migration a bit faster ;-)

jpf91
jpf91 previously requested changes Aug 19, 2018
Copy link
Contributor

@jpf91 jpf91 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

DRUNTIME_SOVERSION and PHOBOS_SOVERSION in libphobos/configure.ac need to be updated.

@ibuclaw ibuclaw mentioned this pull request Aug 19, 2018
@jpf91 jpf91 dismissed their stale review August 19, 2018 20:47

DRUNTIME_SOVERSION addressed in other PR

@ibuclaw ibuclaw changed the title Update to dmd master 20180806 Update to dmd master 20180824 Aug 24, 2018
@ibuclaw
Copy link
Member Author

ibuclaw commented Aug 24, 2018

Merged in dmd master branch in its current state as of writing.

@jpf91 - This includes the ClassKind fix.

@ibuclaw ibuclaw merged commit da7ff85 into D-Programming-GDC:master Aug 26, 2018
@ibuclaw ibuclaw deleted the dmd20180806 branch August 26, 2018 09:20
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.

4 participants