Skip to content
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

Fix #620 #622

Merged
merged 6 commits into from
Aug 11, 2020
Merged

Fix #620 #622

merged 6 commits into from
Aug 11, 2020

Conversation

Hackerpilot
Copy link
Collaborator

No description provided.

libdparse: Get support for 'throw' as a function attribute
dsymbol: A bug fix that will fix issue 620
msgpack-d: Fix some deprecations and some build issues on Windows
containers: Some bug fixes.
@Hackerpilot
Copy link
Collaborator Author

Looks like I have to revert the msgpack upgrade because the ldc Windows cross-compile doesn't work.

common/dub.sdl Outdated
@@ -4,5 +4,5 @@ preGenerateCommands "rdmd \"$PACKAGE_DIR/dubhash.d\""
sourcePaths "src"
importPaths "src"

dependency "msgpack-d" version="~>1.0.0-beta.7"
dependency "msgpack-d" version="~>1.0.1"
Copy link
Member

Choose a reason for hiding this comment

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

as this is stable now (according to SemVer) we could use ~>1.0 which will be any version >=1.0.0 <2.0.0 which should not get any backwards incompatible changes

Copy link
Collaborator Author

Choose a reason for hiding this comment

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

"Should" is the correct word here. This actually needs to point at 1.0.1 only. The CI fails for any version of msgpack-d that includes this commit: msgpack/msgpack-d@a23181e

Copy link
Collaborator Author

Choose a reason for hiding this comment

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

I'm not sure if this is a problem with msgpack-d, or a problem with the release build script. I'm not very familiar with cross-compiling Windows binaries with LDC.

@WebFreak001
Copy link
Member

about 08e0b62: you only changed the minimum version, dub will still resolve to always the highest version (so both resolve to 1.0.3) - are you sure that was the issue and it wasn't just a random one?

@Hackerpilot Hackerpilot marked this pull request as ready for review July 30, 2020 10:28
@Hackerpilot
Copy link
Collaborator Author

@WebFreak001 Are you able to take another look at this?

@WebFreak001
Copy link
Member

testing it continuously over the course of the day now

@WebFreak001
Copy link
Member

oh yeah it was good

@dlang-bot dlang-bot merged commit 92db048 into master Aug 11, 2020
@Hackerpilot Hackerpilot deleted the issue-620 branch August 11, 2020 16:26
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants