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

add vcmibuilder version check for innoextract #408

Merged
merged 3 commits into from Jan 28, 2018

Conversation

paulsetchko
Copy link
Contributor

add vcmibuilder check for the old innoextract is unable to unpack the latest GOG package

@ArseniyShestakov
Copy link
Member

I don't get why this printf needed, but really you should at least check if version is greater.

@paulsetchko
Copy link
Contributor Author

fixed the commented issue; do you know why is that travis takes so long to start the mac build?

@ArseniyShestakov
Copy link
Member

Travis usually takes long on Mac builds mainly because we commit a lot and they have limited Mac instances pool. Regardless it's not required for this to be merged since we don't really expose it on macOS.

@ArseniyShestakov ArseniyShestakov added the enhancement Improving beyond Heroes 3 behavior label Jan 28, 2018
@ArseniyShestakov ArseniyShestakov merged commit d1bef9d into vcmi:develop Jan 28, 2018
@ArseniyShestakov
Copy link
Member

Sorry for taking long to merge this, had to check that innoextract on 14.04(1.4) and 16.04 (1.5) in fact use same version output. Thanks for contributing!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement Improving beyond Heroes 3 behavior
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants