-
-
Notifications
You must be signed in to change notification settings - Fork 143
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 Issue 16593 - Building "tools" produces deprecation warnings #201
Conversation
|
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Good spot - maybe we should add the building of all tools here to Travis too?
@@ -15,15 +15,15 @@ void usage() | |||
|
|||
int main(string[] args) | |||
{ | |||
if (args.length == 1) return usage(), 1; |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Holy cow a comma operator in the wild, and it's in our repo. 😆
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I had a few in ae:
CyberShadow/ae@d8ef8eb
CyberShadow/ae@99585d4
CyberShadow/ae@1674d2b
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I'd never doubt you would push the language to your advantage. :)
What's up with Travis lately? Is master broken or is it something else? |
bdcf028
to
6a25720
Compare
Yep with 2.072 we had some issues, luckily they have been fixed with #202. @CyberShadow I just rebased your PR to the upstream/master, s.t. Travis passes & we can move forward (I like the new GitHub feature). |
https://issues.dlang.org/show_bug.cgi?id=16593