-
Notifications
You must be signed in to change notification settings - Fork 171
Remove mandatory -DCMAKE_BUILD_TYPE #34
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
Comments
@chivakker : FYI |
work in progress. I will update when it is available. |
#34 Change-Id: Ib1a95807f516e3675cb1a4aab7b2433e87f0b325
Request completed. Please use latest code or tag 'igc_release_2018-12-12'. README is updated as well. |
@iwwu. I have strong suggestion to you. Please, start add "Fixes: #xxx" to commit messages if you fix github issues. For example, this commit 03dd5d5 should actually have this commit message:
There are the following reasons to that:
|
@iwwu : Another suggestion is to submit PRs on the GitHub. You should give a chance for people to review/comment/try. |
Closing the issue since the fix is already merged. |
works for me. thank you. |
Please, remove mandatory -DCMAKE_BUILD_TYPE and default it to Release inside your cmake files if it is not defined by the user.
The text was updated successfully, but these errors were encountered: