-
Notifications
You must be signed in to change notification settings - Fork 665
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 travis continuous integration system #180
Conversation
Signed-off-by: Jasmin Jessich <jasmin@anw.at> Signed-off-by: Jean-Marie Lemetayer <jeanmarie.lemetayer@gmail.com>
Signed-off-by: Jasmin Jessich <jasmin@anw.at> Signed-off-by: Jean-Marie Lemetayer <jeanmarie.lemetayer@gmail.com>
Yeay to testing! |
Add travis continuous integration system
I have seen you have already activated the travis-ci tests. Did you activate travis-ci for pushes only or for pull requests, too? |
Pull requests as well. |
The build failed. |
I have seen it. But not all selected samples. |
I passed native gdb on my machine, but then I got: grep results: ltrace configure check: @bhundven |
FYI travis-ci use Ubuntu 12.04 LTS Server Edition 64 bit virtual At the top of the travis log Maybe this can explain the first issue with GDB. 2015-09-23 1:13 GMT+02:00 Jasmin Jessich notifications@github.com:
|
@bhundven The build took 73:44 minutes, which is far too long for travis-ci. It might be, that travis-ci has a faster I will now NOT dig into the gdb ncurses problem, because it is difficult to debug (need a machine Successfull build output: |
I am building with" gcc (Ubuntu/Linaro 4.8.1-10ubuntu9) 4.8.1" and had no problem. The cross GDB runs on the host and can debug code for the target. |
I have tested on my Ubuntu server 14.04 with GCC 4.8.4 and I pass the native GDB also (it also fail during the ltrace step). I think I have found something. Here the failing trace from travis:
And here is my passing trace:
The differences are:
So, I wanted to check the ncurses configuration but the log is too short ... Maybe we have to display the whole |
Yeah, you found it! I will fix that this evening. |
Fixed with pull request #186. |
I guess we can close this now. |
For now I have selected this standard samples:
I think it's a good start. We could add the other later.