Skip to content

Conversation

andralex
Copy link
Member

@andralex andralex commented Jun 8, 2015

No description provided.

@@ -64,9 +64,9 @@ LATEST:=$(shell cd ${DMD_DIR} && \
git tag | grep '^v[0-9][0-9.]*$$' | sed 's/^v//' | sort -nr | head -n 1)
endif
ifeq (,${LATEST})
$(error Could not fetch latest version)
$(error Could not fetch latest version, place LATEST=2.xyz in the command line)
Copy link
Member

Choose a reason for hiding this comment

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

2.xyz.t ?

Copy link
Member Author

Choose a reason for hiding this comment

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

It's either one or the other. Which is more common?

Copy link
Member

Choose a reason for hiding this comment

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

Um? All recent releases are in the form 2.NNN.N. Not sure what you mean.

@CyberShadow
Copy link
Member

per title

The pull request title is not visible when browsing commits.

@andralex andralex force-pushed the better-messages-for-latest branch from 48d763b to dbc0fe7 Compare June 8, 2015 23:12
@andralex
Copy link
Member Author

andralex commented Jun 8, 2015

@CyberShadow fixed

@andralex andralex force-pushed the better-messages-for-latest branch from dbc0fe7 to 82d550b Compare June 8, 2015 23:20
CyberShadow added a commit that referenced this pull request Jun 8, 2015
Give suggestions on both error and success after fetching LATEST
@CyberShadow CyberShadow merged commit 6e3c4e8 into dlang:master Jun 8, 2015
@andralex andralex deleted the better-messages-for-latest branch December 10, 2017 20:10
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