Skip to content

Conversation

@clinei
Copy link
Contributor

@clinei clinei commented Nov 18, 2015

Before,
dependency version="~>1.2.3"
caused a range violation.
Now, it says that the dependency name is missing.

The fact that dependencies cannot have multiple names is expressed, as well.

Should there be a correct usage notice here?

P.S. I get the Failed to parse package description message twice on either error, how to fix that?

s-ludwig added a commit that referenced this pull request Dec 10, 2015
[Recipe] Added error messages for SDL malformed dependency name.
@s-ludwig s-ludwig merged commit 1aa5d31 into dlang:master Dec 10, 2015
@s-ludwig
Copy link
Member

I get the Failed to parse package description message twice on either error, how to fix that?

That error message is in an inappropriate place. I'll turn it into a logDiagnostic message, as it looks like it doesn't add any value.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants