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

Update travis to system with c++11 compiler #597

Merged
merged 3 commits into from
Aug 19, 2017
Merged

Conversation

elextr
Copy link
Member

@elextr elextr commented Aug 13, 2017

Test to fix Travis, do not merge unless Travis passes

@elextr
Copy link
Member Author

elextr commented Aug 13, 2017

Ok, the Travis failure is nothing to do with this change, should be ok to merge.

@eht16 add-ons seems to fail cppcheck?

@eht16
Copy link
Member

eht16 commented Aug 13, 2017

Probably fixed in 544e9f6.
Seems it is the old cppcheck version on that old Ubuntu 14.04 build box at Travis.
cppcheck 1.80 on my system doesn't complain. Anyway, the initialization won't hurt.

Maybe you could merge master into this branch and try again?

@codebrainz
Copy link
Member

This closes #596 (adding link to keep myself from getting confused).

Also to re-trigger checks
@elextr
Copy link
Member Author

elextr commented Aug 14, 2017

Ok, looks good to merge.

@codebrainz
Copy link
Member

Looks OK by inspection, after rebasing this branch on master to get 544e9f6, I would squash 82727a9 into 69a271d since the former just fixes the latter.

Seems not risky since it just messes with the CI, not any code.

@frlan frlan merged commit e5e7bf4 into geany:master Aug 19, 2017
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.

4 participants