roelandjansen / pcmos386v501

PC-MOS/386 v5.01 and up, including cdrom driver sources.
GNU General Public License v3.0
419 stars 60 forks source link

Travis: Add initial build integration #102

Closed andrewbird closed 4 years ago

andrewbird commented 4 years ago

This PR adds support for PC-MOS to be built automatically on any potential PR merge and shows the maintainer that at least the build is not broken by it.

andrewbird commented 4 years ago

@roelandjansen to make this work as intended you'll need to follow the steps here https://docs.travis-ci.com/user/tutorial/#to-get-started-with-travis-ci-using-github

andrewbird commented 4 years ago

Oh here's a link to the build https://travis-ci.com/github/andrewbird/pcmos386 click on the triangle next to ./travis_build to see the build log

andrewbird commented 4 years ago

@stsp has requested some changes so I converted to a draft PR for now.

stsp commented 4 years ago

Were you able to get rid of an errors in the log? If so, could you please update an URL to the log?

andrewbird commented 4 years ago

Just retesting the error filter now that the -q issue is solved.

andrewbird commented 4 years ago

I'm happy for this to be merged (subject to your review) now that Dosemu2 has the DOSEMU_QUIET env variable to suppress some spurious startup warnings that occur in the Travis environment.

roelandjansen commented 4 years ago

nice!

andrewbird commented 4 years ago

It's great that you merged it, thanks Have you done the travis integration with github (see above) yet, as only then will each commit and PR merge get checked automatically?

roelandjansen commented 4 years ago

not yet, lack of time.