-
Notifications
You must be signed in to change notification settings - Fork 414
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
Quoting problem when there are several targets #701
Comments
I suggest that we use the following model, that I find slightly simpler than the current one: A variable always expand to a list of values. When a variable expands to exactly 1 value, there is no ambiguity and it can appear anywhere. When it expands to 0 or more than 2 values, it can appear:
|
I'll try to have a look later today (in the evening CET). |
I'm afraid it'll have to wait for tomorrow. |
@Chris00 do you think you'll have some time to look at this in the coming weeks? I'm thinking that it'd be nice to sort this out before we release Dune 1.0.0. |
I agree. I'll try to find the time but there are a couple of things I definitely need to finish first... |
@Chris00 I'll take over this if you don't mind. I'd like to have this fixed to support version comparisons. |
@rgrinberg Thanks — I'm doing too many things ATM... |
Fixed in master. |
Problem exposed by the test added in #699
The text was updated successfully, but these errors were encountered: